Files
growstuff/spec/models/post_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

183 lines
5.7 KiB
Ruby

# frozen_string_literal: true
require 'rails_helper'
describe Post do
let(:member) { FactoryBot.create(:member, login_name: 'whinacooper') }
it_behaves_like "it is likeable"
it "has a slug" do
post = FactoryBot.create(:post, author: member, subject: 'A Post')
time = post.created_at
datestr = time.strftime("%Y%m%d")
# 2 digit day and month, full-length years
# Counting digits using Math.log is not precise enough!
datestr.size.should eq(4 + time.year.to_s.size)
post.slug.should eq("#{member.login_name}-#{datestr}-a-post")
end
it "has many comments" do
post = FactoryBot.create(:post, author: member)
FactoryBot.create(:comment, commentable: post)
FactoryBot.create(:comment, commentable: post)
post.comments.size.should == 2
end
it "supports counting comments" do
post = FactoryBot.create(:post, author: member)
FactoryBot.create(:comment, commentable: post)
FactoryBot.create(:comment, commentable: post)
post.comment_count.should == 2
end
it "destroys comments when deleted" do
post = FactoryBot.create(:post, author: member)
FactoryBot.create(:comment, commentable: post)
FactoryBot.create(:comment, commentable: post)
post.comments.size.should eq(2)
all = Comment.count
post.destroy
Comment.count.should eq(all - 2)
end
it "belongs to a forum" do
post = FactoryBot.create(:forum_post)
post.forum.should be_an_instance_of Forum
end
it "doesn't allow a nil subject" do
post = FactoryBot.build(:post, subject: nil)
post.should_not be_valid
end
it "doesn't allow a blank subject" do
post = FactoryBot.build(:post, subject: "")
post.should_not be_valid
end
it "doesn't allow a subject with only spaces" do
post = FactoryBot.build(:post, subject: " ")
post.should_not be_valid
end
context "recent activity" do
before do
Time.stub(now: Time.zone.now)
end
let!(:post) { FactoryBot.create(:post, created_at: 1.day.ago) }
it "sets recent activity to post time" do
post.recent_activity.to_i.should eq post.created_at.to_i
end
it "sets recent activity to comment time" do
comment = FactoryBot.create(:comment, commentable: post,
created_at: 1.hour.ago)
post.recent_activity.to_i.should eq comment.created_at.to_i
end
it "shiny new post is recently active" do
# create a shiny new post
post2 = FactoryBot.create(:post, created_at: 1.minute.ago)
described_class.recently_active.first.should eq post2
described_class.recently_active.second.should eq post
end
it "new comment on old post is recently active" do
# now comment on an older post
post2 = FactoryBot.create(:post, created_at: 1.minute.ago)
FactoryBot.create(:comment, commentable: post, created_at: 1.second.ago)
described_class.recently_active.first.should eq post
described_class.recently_active.second.should eq post2
end
end
context "notifications" do
let(:member2) { FactoryBot.create(:member) }
it "sends a notification when a member is mentioned using @-syntax" do
expect do
FactoryBot.create(:post, author: member, body: "Hey @#{member2}")
end.to change(Notification, :count).by(1)
end
it "sends a notification when a member is mentioned using [](member) syntax" do
expect do
FactoryBot.create(:post, author: member, body: "Hey [#{member2}](member)")
end.to change(Notification, :count).by(1)
end
it "sets the notification field" do
p = FactoryBot.create(:post, author: member, body: "Hey @#{member2}")
n = Notification.first
n.sender.should eq member
n.recipient.should eq member2
n.subject.should match(/mentioned you in their post/)
n.body.should eq p.body
end
it "sends notifications to all members mentioned" do
member3 = FactoryBot.create(:member)
expect do
FactoryBot.create(:post, author: member, body: "Hey @#{member2} & @#{member3}")
end.to change(Notification, :count).by(2)
end
it "doesn't send notifications if you mention yourself" do
expect do
FactoryBot.create(:post, author: member, body: "@#{member}")
end.not_to change(Notification, :count)
end
end
context "crop-post association" do
let!(:tomato) { FactoryBot.create(:tomato) }
let!(:maize) { FactoryBot.create(:maize) }
let!(:chard) { FactoryBot.create(:chard) }
let!(:post) { FactoryBot.create(:post, body: "[maize](crop)[tomato](crop)[tomato](crop)") }
it "is generated" do
expect(tomato.posts).to eq [post]
expect(maize.posts).to eq [post]
end
it "does not duplicate" do
expect(post.crops).to match_array([tomato, maize])
end
it "is updated when post was modified" do
post.update(body: "[chard](crop)")
expect(post.crops).to eq [chard]
expect(chard.posts).to eq [post]
expect(tomato.posts).to eq []
expect(maize.posts).to eq []
end
describe "destroying the post" do
before do
post.destroy
end
it "deletes the association" do
expect(Crop.find(tomato.id).posts).to eq []
expect(Crop.find(maize.id).posts).to eq []
end
it "does not delete the crops" do
expect(Crop.find(tomato.id)).not_to be_nil
expect(Crop.find(maize.id)).not_to be_nil
end
end
end
it 'excludes deleted members' do
post = FactoryBot.create(:post, author: member)
expect(described_class.joins(:author).all).to include(post)
member.destroy
expect(described_class.joins(:author).all).not_to include(post)
end
end