Completion of 'Refactor ownship_of_status_message #8059'

app/models/photo.rb

@@ -67,11 +67,9 @@ def clear_empty_status_message
 
   def ownership_of_status_message
     message = StatusMessage.find_by_guid(self.status_message_guid)
-    if self.status_message_guid && message
-      self.diaspora_handle == message.diaspora_handle
-    else
-      true
-    end
+    return unless status_message_guid && message && diaspora_handle != message.diaspora_handle
+
+    errors.add(:base, "Photo must have the same owner as status message")
   end
 
   def self.diaspora_initialize(params={})

features/step_definitions/posts_steps.rb

@@ -32,7 +32,7 @@
 Given /^"([^"]*)" has a public post with text "([^"]*)" and a picture/ do |email, text|
   user = User.find_by(email: email)
   post = user.post(:status_message, text: text, public: true, to: user.aspect_ids)
-  FactoryBot.create(:photo, status_message: post)
+  FactoryBot.create(:photo, status_message: post, author: user.person)
 end
 
 Given /^there are (\d+) public posts from "([^"]*)"$/ do |n_posts, email|

spec/mailers/notifier_spec.rb

@@ -216,7 +216,7 @@
     end
 
     it "can handle status_messages without text" do
-      photo = FactoryBot.create(:photo, public: true)
+      photo = FactoryBot.create(:photo, public: true, author: alice.person)
       status = FactoryBot.create(:status_message, author: alice.person, text: nil, photos: [photo], public: true)
       like = status.likes.create!(author: bob.person)
       mail = Notifier.send_notification("liked", alice.id, like.author.id, like.id)

spec/models/status_message_spec.rb

@@ -306,7 +306,7 @@
     let(:post) { FactoryBot.create(:status_message, author: alice.person) }
 
     it "receives attached photos" do
-      photo = FactoryBot.create(:photo, status_message: post)
+      photo = FactoryBot.create(:photo, status_message: post, author: alice.person)
 
       post.receive([bob.id])
 
@@ -321,7 +321,7 @@
     end
 
     it "works with already received attached photos" do
-      photo = FactoryBot.create(:photo, status_message: post)
+      photo = FactoryBot.create(:photo, status_message: post, author: alice.person)
 
       photo.receive([bob.id])
       post.receive([bob.id])