Fix comment notification subject for posts without text

Fixes #7854
This commit is contained in:
Benjamin Neff 2018-09-05 01:32:36 +02:00
parent 9b3f90dc90
commit 2eb17e109b
No known key found for this signature in database
GPG key ID: 971464C3F1A90194
2 changed files with 17 additions and 1 deletions

View file

@ -62,7 +62,11 @@ class StatusMessage < Post
end
def comment_email_subject
if message.present?
message.title
elsif photos.present?
I18n.t("posts.show.photos_by", count: photos.size, author: author_name)
end
end
def first_photo_url(*args)

View file

@ -170,6 +170,18 @@ describe StatusMessage, type: :model do
end
end
describe "#comment_email_subject" do
it "delegates to message.title if the post have a text" do
expect(status.comment_email_subject).to eq(status.message.title)
end
it "includes the photos count if there are photos without text" do
photo = FactoryGirl.build(:photo, public: true)
status = FactoryGirl.build(:status_message, author: photo.author, text: nil, photos: [photo], public: true)
expect(status.comment_email_subject).to eq(I18n.t("posts.show.photos_by", count: 1, author: status.author_name))
end
end
describe "tags" do
it_should_behave_like "it is taggable" do
let(:object) { build(:status_message) }