Use each_with_object

[rails.git] / test / system / report_note_test.rb

diff --git a/test/system/report_note_test.rb b/test/system/report_note_test.rb
index c5a70b488fc8807c599916edfe8a2b80a801d207..e45a4e702bbb1a1735e2a3c13e9324de560c6d52 100644 (file)
--- a/test/system/report_note_test.rb
+++ b/test/system/report_note_test.rb
@@ -6,7 +6,7 @@ class ReportNoteTest < ApplicationSystemTestCase
     visit browse_note_path(note)
     assert page.has_content?(note.comments.first.body)
 
-    assert !page.has_content?(I18n.t("browse.note.report"))
+    assert_not page.has_content?(I18n.t("browse.note.report"))
   end
 
   def test_can_report_anonymous_notes
@@ -16,16 +16,18 @@ class ReportNoteTest < ApplicationSystemTestCase
 
     click_on I18n.t("browse.note.report")
     assert page.has_content? "Report"
-    assert page.has_content? I18n.t("issues.new.disclaimer.intro")
+    assert page.has_content? I18n.t("reports.new.disclaimer.intro")
 
-    choose I18n.t("reports.categories.note.spam")
+    choose I18n.t("reports.new.categories.note.spam_label")
     fill_in "report_details", :with => "This is spam"
-    click_on "Create Report"
+    assert_difference "Issue.count", 1 do
+      click_on "Create Report"
+    end
 
-    assert page.has_content? "Your report has been registered sucessfully"
+    assert page.has_content? "Your report has been registered successfully"
 
-    assert_equal 1, Issue.count
-    assert Issue.last.reportable == note
+    assert_equal note, Issue.last.reportable
+    assert_equal "moderator", Issue.last.assigned_role
   end
 
   def test_can_report_notes_with_author
@@ -35,15 +37,17 @@ class ReportNoteTest < ApplicationSystemTestCase
 
     click_on I18n.t("browse.note.report")
     assert page.has_content? "Report"
-    assert page.has_content? I18n.t("issues.new.disclaimer.intro")
+    assert page.has_content? I18n.t("reports.new.disclaimer.intro")
 
-    choose I18n.t("reports.categories.note.spam")
+    choose I18n.t("reports.new.categories.note.spam_label")
     fill_in "report_details", :with => "This is spam"
-    click_on "Create Report"
+    assert_difference "Issue.count", 1 do
+      click_on "Create Report"
+    end
 
-    assert page.has_content? "Your report has been registered sucessfully"
+    assert page.has_content? "Your report has been registered successfully"
 
-    assert_equal 1, Issue.count
-    assert Issue.last.reportable == note
+    assert_equal note, Issue.last.reportable
+    assert_equal "moderator", Issue.last.assigned_role
   end
 end