test/controllers/diary_comments_controller_test.rb

   1 require "test_helper"
   2
   3 class DiaryCommentsControllerTest < ActionDispatch::IntegrationTest
   4   def setup
   5     super
   6     # Create the default language for diary entries
   7     create(:language, :code => "en")
   8   end
   9
  10   def test_routes
  11     assert_routing(
  12       { :path => "/user/username/diary_comments", :method => :get },
  13       { :controller => "diary_comments", :action => "index", :display_name => "username" }
  14     )
  15     assert_routing(
  16       { :path => "/user/username/diary/1/comments", :method => :post },
  17       { :controller => "diary_comments", :action => "create", :display_name => "username", :id => "1" }
  18     )
  19     assert_routing(
  20       { :path => "/diary_comments/2/hide", :method => :post },
  21       { :controller => "diary_comments", :action => "hide", :comment => "2" }
  22     )
  23     assert_routing(
  24       { :path => "/diary_comments/2/unhide", :method => :post },
  25       { :controller => "diary_comments", :action => "unhide", :comment => "2" }
  26     )
  27
  28     get "/user/username/diary/comments/1"
  29     assert_redirected_to "/user/username/diary_comments"
  30
  31     get "/user/username/diary/comments"
  32     assert_redirected_to "/user/username/diary_comments"
  33   end
  34
  35   def test_index
  36     user = create(:user)
  37     other_user = create(:user)
  38     suspended_user = create(:user, :suspended)
  39     deleted_user = create(:user, :deleted)
  40
  41     # Test a user with no comments
  42     get user_diary_comments_path(user)
  43     assert_response :success
  44     assert_template :index
  45     assert_select "h4", :html => "No diary comments"
  46
  47     # Test a user with a comment
  48     create(:diary_comment, :user => other_user)
  49
  50     get user_diary_comments_path(other_user)
  51     assert_response :success
  52     assert_template :index
  53     assert_dom "a[href='#{user_path(other_user)}']", :text => other_user.display_name
  54     assert_select "table.table-striped tbody" do
  55       assert_select "tr", :count => 1
  56     end
  57
  58     # Test a suspended user
  59     get user_diary_comments_path(suspended_user)
  60     assert_response :not_found
  61
  62     # Test a deleted user
  63     get user_diary_comments_path(deleted_user)
  64     assert_response :not_found
  65   end
  66
  67   def test_index_invalid_paged
  68     user = create(:user)
  69
  70     %w[-1 0 fred].each do |id|
  71       get user_diary_comments_path(user, :before => id)
  72       assert_redirected_to :controller => :errors, :action => :bad_request
  73
  74       get user_diary_comments_path(user, :after => id)
  75       assert_redirected_to :controller => :errors, :action => :bad_request
  76     end
  77   end
  78
  79   def test_create
  80     user = create(:user)
  81     other_user = create(:user)
  82     entry = create(:diary_entry, :user => user)
  83     create(:diary_entry_subscription, :diary_entry => entry, :user => user)
  84
  85     # Make sure that you are denied when you are not logged in
  86     post comment_diary_entry_path(entry.user, entry)
  87     assert_response :forbidden
  88
  89     session_for(other_user)
  90
  91     # Verify that you get a not found error, when you pass a bogus id
  92     post comment_diary_entry_path(entry.user, :id => 9999)
  93     assert_response :not_found
  94     assert_select "div.content-heading", :count => 1 do
  95       assert_select "h1", :text => "No entry with the id: 9999", :count => 1
  96     end
  97
  98     # Now try an invalid comment with an empty body
  99     assert_no_difference "ActionMailer::Base.deliveries.size" do
 100       assert_no_difference "DiaryComment.count" do
 101         assert_no_difference "entry.subscribers.count" do
 102           perform_enqueued_jobs do
 103             post comment_diary_entry_path(entry.user, entry, :diary_comment => { :body => "" })
 104           end
 105         end
 106       end
 107     end
 108     assert_response :success
 109     assert_template :new
 110     assert_match(/img-src \* data:;/, @response.headers["Content-Security-Policy-Report-Only"])
 111
 112     # Now try again with the right id
 113     assert_difference "ActionMailer::Base.deliveries.size", entry.subscribers.count do
 114       assert_difference "DiaryComment.count", 1 do
 115         assert_difference "entry.subscribers.count", 1 do
 116           perform_enqueued_jobs do
 117             post comment_diary_entry_path(entry.user, entry, :diary_comment => { :body => "New comment" })
 118           end
 119         end
 120       end
 121     end
 122     comment = DiaryComment.last
 123     assert_redirected_to diary_entry_path(entry.user, entry, :anchor => "comment#{comment.id}")
 124     email = ActionMailer::Base.deliveries.first
 125     assert_equal [user.email], email.to
 126     assert_equal "[OpenStreetMap] #{other_user.display_name} commented on a diary entry", email.subject
 127     assert_match(/New comment/, email.text_part.decoded)
 128     assert_match(/New comment/, email.html_part.decoded)
 129     ActionMailer::Base.deliveries.clear
 130     assert_equal entry.id, comment.diary_entry_id
 131     assert_equal other_user.id, comment.user_id
 132     assert_equal "New comment", comment.body
 133
 134     # Now show the diary entry, and check the new comment is present
 135     get diary_entry_path(entry.user, entry)
 136     assert_response :success
 137     assert_select ".diary-comment", :count => 1 do
 138       assert_select "#comment#{comment.id}", :count => 1 do
 139         assert_select "a[href='/user/#{ERB::Util.u(other_user.display_name)}']", :text => other_user.display_name, :count => 1
 140       end
 141       assert_select ".richtext", :text => /New comment/, :count => 1
 142     end
 143   end
 144
 145   def test_create_spammy
 146     user = create(:user)
 147     other_user = create(:user)
 148     entry = create(:diary_entry, :user => user)
 149     create(:diary_entry_subscription, :diary_entry => entry, :user => user)
 150
 151     session_for(other_user)
 152
 153     # Generate some spammy content
 154     spammy_text = 1.upto(50).map { |n| "http://example.com/spam#{n}" }.join(" ")
 155
 156     # Try creating a spammy comment
 157     assert_difference "ActionMailer::Base.deliveries.size", 1 do
 158       assert_difference "DiaryComment.count", 1 do
 159         perform_enqueued_jobs do
 160           post comment_diary_entry_path(entry.user, entry, :diary_comment => { :body => spammy_text })
 161         end
 162       end
 163     end
 164     comment = DiaryComment.last
 165     assert_redirected_to diary_entry_path(entry.user, entry, :anchor => "comment#{comment.id}")
 166     email = ActionMailer::Base.deliveries.first
 167     assert_equal [user.email], email.to
 168     assert_equal "[OpenStreetMap] #{other_user.display_name} commented on a diary entry", email.subject
 169     assert_match %r{http://example.com/spam}, email.text_part.decoded
 170     assert_match %r{http://example.com/spam}, email.html_part.decoded
 171     ActionMailer::Base.deliveries.clear
 172     assert_equal entry.id, comment.diary_entry_id
 173     assert_equal other_user.id, comment.user_id
 174     assert_equal spammy_text, comment.body
 175     assert_equal "suspended", User.find(other_user.id).status
 176
 177     # Follow the redirect
 178     get diary_entries_path(:display_name => user.display_name)
 179     assert_redirected_to :controller => :users, :action => :suspended
 180
 181     # Now show the diary entry, and check the new comment is not present
 182     get diary_entry_path(entry.user, entry)
 183     assert_response :success
 184     assert_select ".diary-comment", :count => 0
 185   end
 186
 187   def test_hide
 188     user = create(:user)
 189     diary_entry = create(:diary_entry, :user => user)
 190     diary_comment = create(:diary_comment, :diary_entry => diary_entry)
 191
 192     # Try without logging in
 193     post hide_diary_comment_path(diary_comment)
 194     assert_response :forbidden
 195     assert DiaryComment.find(diary_comment.id).visible
 196
 197     # Now try as a normal user
 198     session_for(user)
 199     post hide_diary_comment_path(diary_comment)
 200     assert_redirected_to :controller => :errors, :action => :forbidden
 201     assert DiaryComment.find(diary_comment.id).visible
 202
 203     # Try as a moderator
 204     session_for(create(:moderator_user))
 205     post hide_diary_comment_path(diary_comment)
 206     assert_redirected_to diary_entry_path(user, diary_entry)
 207     assert_not DiaryComment.find(diary_comment.id).visible
 208
 209     # Reset
 210     diary_comment.reload.update(:visible => true)
 211
 212     # Finally try as an administrator
 213     session_for(create(:administrator_user))
 214     post hide_diary_comment_path(diary_comment)
 215     assert_redirected_to diary_entry_path(user, diary_entry)
 216     assert_not DiaryComment.find(diary_comment.id).visible
 217   end
 218
 219   def test_unhide
 220     user = create(:user)
 221     diary_entry = create(:diary_entry, :user => user)
 222     diary_comment = create(:diary_comment, :diary_entry => diary_entry, :visible => false)
 223
 224     # Try without logging in
 225     post unhide_diary_comment_path(diary_comment)
 226     assert_response :forbidden
 227     assert_not DiaryComment.find(diary_comment.id).visible
 228
 229     # Now try as a normal user
 230     session_for(user)
 231     post unhide_diary_comment_path(diary_comment)
 232     assert_redirected_to :controller => :errors, :action => :forbidden
 233     assert_not DiaryComment.find(diary_comment.id).visible
 234
 235     # Now try as a moderator
 236     session_for(create(:moderator_user))
 237     post unhide_diary_comment_path(diary_comment)
 238     assert_redirected_to diary_entry_path(user, diary_entry)
 239     assert DiaryComment.find(diary_comment.id).visible
 240
 241     # Reset
 242     diary_comment.reload.update(:visible => true)
 243
 244     # Finally try as an administrator
 245     session_for(create(:administrator_user))
 246     post unhide_diary_comment_path(diary_comment)
 247     assert_redirected_to diary_entry_path(user, diary_entry)
 248     assert DiaryComment.find(diary_comment.id).visible
 249   end
 250 end