Files
growstuff/spec/models/comment_spec.rb
google-labs-jules[bot] 8564ec7a7c Add comments to photos (#4130)
* 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>
2025-08-24 21:10:16 +09:30

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