AO3-6956 Corrected error message for duplicate comments (otwcode#5135)

* AO3-6956 Corrected error message for duplicate comments

* Updated to match suggestions

* Added test case

* Aligned with formatting guidelines

* Fixed test

* Update config/locales/models/en.yml

Co-authored-by: Bilka <[email protected]>

* Updated rspec test to only use partial matching

* Normalized translations

---------

Co-authored-by: Bilka <[email protected]>

Author: 2 people

app/models/comment.rb

@@ -71,7 +71,7 @@ def check_for_spam
   validates :comment_content, uniqueness: {
     scope: [:commentable_id, :commentable_type, :name, :email, :pseud_id],
     unless: :is_deleted?,
-    message: ts("^This comment has already been left on this work. (It may not appear right away for performance reasons.)")
+    message: :duplicate_comment
   }
 
   scope :ordered_by_date, -> { order('created_at DESC') }

config/locales/models/en.yml

@@ -90,6 +90,8 @@ en:
               required: can't be blank
         comment:
           attributes:
+            comment_content:
+              duplicate_comment: "^You've already left this comment here. (It may not appear right away for performance reasons.)"
             commentable:
               format: "%{message}"
               guest_replies_off: Sorry, this user doesn't allow non-Archive users to reply to their comments.

spec/models/comment_spec.rb

@@ -94,17 +94,18 @@
     end
   end
 
-  context "with an existing comment from the same user" do
+  context "with an existing comment from the same user" do 
     let(:first_comment) { create(:comment) }
 
     let(:second_comment) do
       attributes = %w[pseud_id commentable_id commentable_type comment_content name email]
       Comment.new(first_comment.attributes.slice(*attributes))
     end
 
-    it "should be invalid if exactly duplicated" do
+    it "should be invalid if exactly duplicated" do 
       expect(second_comment.valid?).to be_falsy
       expect(second_comment.errors.attribute_names).to include(:comment_content)
+      expect(second_comment.errors.full_messages.first).to include("You've already")
     end
 
     it "should not be invalid if in the process of being deleted" do