Merge remote-tracking branch 'upstream/pull/4297'

[rails.git] / test / models / issue_test.rb

diff --git a/test/models/issue_test.rb b/test/models/issue_test.rb
index 7ee700124c7448a84a6bc02fa08a59f66069a6cd..f166282c1dcbf1bad0aec59d1eafd40f34b2f31b 100644 (file)
--- a/test/models/issue_test.rb
+++ b/test/models/issue_test.rb
@@ -1,24 +1,40 @@
 require "test_helper"
 
 class IssueTest < ActiveSupport::TestCase
+  def test_assigned_role
+    issue = create(:issue)
+
+    assert_predicate issue, :valid?
+    issue.assigned_role = "bogus"
+    assert_not issue.valid?
+  end
+
   def test_reported_user
-    note = create(:note_comment, :author => create(:user)).note
-    user = create(:user)
     create(:language, :code => "en")
+    user = create(:user)
+    note = create(:note_comment, :author => create(:user)).note
+    anonymous_note = create(:note_comment, :author => nil).note
     diary_entry = create(:diary_entry)
-    issue = Issue.new
+    diary_comment = create(:diary_comment, :diary_entry => diary_entry)
 
-    issue.reportable = user
+    issue = build(:issue, :reportable => user, :assigned_role => "administrator")
     issue.save!
     assert_equal issue.reported_user, user
 
-    # FIXME: doesn't handle anonymous notes
-    issue.reportable = note
+    issue = build(:issue, :reportable => note, :assigned_role => "administrator")
     issue.save!
     assert_equal issue.reported_user, note.author
 
-    issue.reportable = diary_entry
+    issue = build(:issue, :reportable => anonymous_note, :assigned_role => "administrator")
+    issue.save!
+    assert_nil issue.reported_user
+
+    issue = build(:issue, :reportable => diary_entry, :assigned_role => "administrator")
     issue.save!
     assert_equal issue.reported_user, diary_entry.user
+
+    issue = build(:issue, :reportable => diary_comment, :assigned_role => "administrator")
+    issue.save!
+    assert_equal issue.reported_user, diary_comment.user
   end
 end