From 8185f988723dd95b814bcc5f03ccb3da94edbae4 Mon Sep 17 00:00:00 2001
From: voidSatisfaction <lourie@naver.com>
Date: Thu, 7 Sep 2017 16:55:42 +0900
Subject: [PATCH] Feat add validation for report comment: characters under 1000
 valid (#4833)

---
 app/models/report.rb       |  2 ++
 spec/models/report_spec.rb | 14 ++++++++++++++
 2 files changed, 16 insertions(+)

diff --git a/app/models/report.rb b/app/models/report.rb
index 4d2552d30a..479aa17bb1 100644
--- a/app/models/report.rb
+++ b/app/models/report.rb
@@ -22,6 +22,8 @@ class Report < ApplicationRecord
   scope :unresolved, -> { where(action_taken: false) }
   scope :resolved,   -> { where(action_taken: true) }
 
+  validates :comment, length: { maximum: 1000 }
+
   def statuses
     Status.where(id: status_ids).includes(:account, :media_attachments, :mentions)
   end
diff --git a/spec/models/report_spec.rb b/spec/models/report_spec.rb
index 6c27238457..d40ebf6dc3 100644
--- a/spec/models/report_spec.rb
+++ b/spec/models/report_spec.rb
@@ -21,4 +21,18 @@ describe Report do
       expect(report.media_attachments).to eq [media_attachment]
     end
   end
+
+  describe 'validatiions' do
+    it 'has a valid fabricator' do
+      report = Fabricate(:report)
+      report.valid?
+      expect(report).to be_valid
+    end
+
+    it 'is invalid if comment is longer than 1000 characters' do
+      report = Fabricate.build(:report, comment: Faker::Lorem.characters(1001))
+      report.valid?
+      expect(report).to model_have_error_on_field(:comment)
+    end
+  end
 end