mirror of
https://github.com/Growstuff/growstuff.git
synced 2025-12-30 21:17:50 -05:00
* Add comments to photos Extend the photo show page to support comments by logged in users. - Make the Comment model polymorphic. - Update the Photo and Post models to have comments. - Update the comments controller to handle the polymorphic association. - Update the photo show page to display comments and a comment form. - Create a reusable comments partial. * Add migration * Fix tests * Fix tests * Slightly fix tests * Fix variables * Add field * Refactor slightly * Refactor slightly * Refactor slightly * Refactor * Photos respond to this as well * Refactor to polymorphic_url * Rename * Wrong relationship * Refactor and fix tests * Fix relationships * Fix rendering * Fix tests * Fix model tests * Fix test * Fix test * Fix test * Fix test * Fix controller spec * Fix view tests --------- Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com> Co-authored-by: Daniel O'Connor <daniel.oconnor@gmail.com>
57 lines
1.6 KiB
Ruby
57 lines
1.6 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
require 'rails_helper'
|
|
|
|
describe Comment do
|
|
context "basic" do
|
|
let(:comment) { FactoryBot.create(:comment) }
|
|
|
|
it "belongs to a post" do
|
|
comment.commentable.should be_an_instance_of Post
|
|
end
|
|
|
|
it "belongs to an author" do
|
|
comment.author.should be_an_instance_of Member
|
|
end
|
|
end
|
|
|
|
context "notifications" do
|
|
it "sends a notification when a comment is posted" do
|
|
expect do
|
|
FactoryBot.create(:comment)
|
|
end.to change(Notification, :count).by(1)
|
|
end
|
|
|
|
it "sets the notification fields" do
|
|
@c = FactoryBot.create(:comment)
|
|
@n = Notification.first
|
|
@n.sender.should eq @c.author
|
|
@n.recipient.should eq @c.commentable.author
|
|
@n.subject.should include 'commented on'
|
|
@n.body.should eq @c.body
|
|
@n.notifiable.should eq @c.commentable # polymorphic association, this is a Post.
|
|
end
|
|
|
|
it "doesn't send notifications to yourself" do
|
|
@m = FactoryBot.create(:member)
|
|
@p = FactoryBot.create(:post, author: @m)
|
|
expect do
|
|
FactoryBot.create(:comment, commentable: @p, author: @m)
|
|
end.not_to change(Notification, :count)
|
|
end
|
|
end
|
|
|
|
context "ordering" do
|
|
before do
|
|
@m = FactoryBot.create(:member)
|
|
@p = FactoryBot.create(:post, author: @m)
|
|
@c1 = FactoryBot.create(:comment, commentable: @p, author: @m)
|
|
@c2 = FactoryBot.create(:comment, commentable: @p, author: @m)
|
|
end
|
|
|
|
it 'has a scope for ASC order for displaying on post page' do
|
|
described_class.post_order.should eq [@c1, @c2]
|
|
end
|
|
end
|
|
end
|