test/controllers/user_blocks_controller_test.rb

   1 require "test_helper"
   2
   3 class UserBlocksControllerTest < ActionDispatch::IntegrationTest
   4   ##
   5   # test all routes which lead to this controller
   6   def test_routes
   7     assert_routing(
   8       { :path => "/blocks/new/username", :method => :get },
   9       { :controller => "user_blocks", :action => "new", :display_name => "username" }
  10     )
  11
  12     assert_routing(
  13       { :path => "/user_blocks", :method => :get },
  14       { :controller => "user_blocks", :action => "index" }
  15     )
  16     assert_routing(
  17       { :path => "/user_blocks", :method => :post },
  18       { :controller => "user_blocks", :action => "create" }
  19     )
  20     assert_routing(
  21       { :path => "/user_blocks/1", :method => :get },
  22       { :controller => "user_blocks", :action => "show", :id => "1" }
  23     )
  24     assert_routing(
  25       { :path => "/user_blocks/1/edit", :method => :get },
  26       { :controller => "user_blocks", :action => "edit", :id => "1" }
  27     )
  28     assert_routing(
  29       { :path => "/user_blocks/1", :method => :put },
  30       { :controller => "user_blocks", :action => "update", :id => "1" }
  31     )
  32     assert_routing(
  33       { :path => "/user_blocks/1", :method => :delete },
  34       { :controller => "user_blocks", :action => "destroy", :id => "1" }
  35     )
  36     assert_routing(
  37       { :path => "/blocks/1/revoke", :method => :get },
  38       { :controller => "user_blocks", :action => "revoke", :id => "1" }
  39     )
  40     assert_routing(
  41       { :path => "/blocks/1/revoke", :method => :post },
  42       { :controller => "user_blocks", :action => "revoke", :id => "1" }
  43     )
  44
  45     assert_routing(
  46       { :path => "/user/username/blocks", :method => :get },
  47       { :controller => "user_blocks", :action => "blocks_on", :display_name => "username" }
  48     )
  49     assert_routing(
  50       { :path => "/user/username/blocks_by", :method => :get },
  51       { :controller => "user_blocks", :action => "blocks_by", :display_name => "username" }
  52     )
  53     assert_routing(
  54       { :path => "/user/username/blocks/revoke_all", :method => :get },
  55       { :controller => "user_blocks", :action => "revoke_all", :display_name => "username" }
  56     )
  57     assert_routing(
  58       { :path => "/user/username/blocks/revoke_all", :method => :post },
  59       { :controller => "user_blocks", :action => "revoke_all", :display_name => "username" }
  60     )
  61   end
  62
  63   ##
  64   # test the index action
  65   def test_index
  66     revoked_block = create(:user_block, :revoked)
  67
  68     get user_blocks_path
  69     assert_response :success
  70     assert_select "table#block_list tbody tr", :count => 1 do
  71       assert_select "a[href='#{user_path revoked_block.user}']", :text => revoked_block.user.display_name
  72       assert_select "a[href='#{user_path revoked_block.creator}']", :text => revoked_block.creator.display_name
  73       assert_select "a[href='#{user_path revoked_block.revoker}']", :text => revoked_block.revoker.display_name
  74     end
  75
  76     active_block = create(:user_block)
  77     expired_block = create(:user_block, :expired)
  78
  79     get user_blocks_path
  80     assert_response :success
  81     assert_select "table#block_list tbody", :count => 1 do
  82       assert_select "tr", 3
  83       assert_select "a[href='#{user_block_path(active_block)}']", 1
  84       assert_select "a[href='#{user_block_path(expired_block)}']", 1
  85       assert_select "a[href='#{user_block_path(revoked_block)}']", 1
  86     end
  87   end
  88
  89   ##
  90   # test the index action with multiple pages
  91   def test_index_paged
  92     user_blocks = create_list(:user_block, 50).reverse
  93     next_path = user_blocks_path
  94
  95     get next_path
  96     assert_response :success
  97     check_user_blocks_table user_blocks[0...20]
  98     check_no_page_link "Newer Blocks"
  99     next_path = check_page_link "Older Blocks"
 100
 101     get next_path
 102     assert_response :success
 103     check_user_blocks_table user_blocks[20...40]
 104     check_page_link "Newer Blocks"
 105     next_path = check_page_link "Older Blocks"
 106
 107     get next_path
 108     assert_response :success
 109     check_user_blocks_table user_blocks[40...50]
 110     check_page_link "Newer Blocks"
 111     check_no_page_link "Older Blocks"
 112   end
 113
 114   ##
 115   # test the index action with invalid pages
 116   def test_index_invalid_paged
 117     %w[-1 0 fred].each do |id|
 118       get user_blocks_path(:before => id)
 119       assert_redirected_to :controller => :errors, :action => :bad_request
 120
 121       get user_blocks_path(:after => id)
 122       assert_redirected_to :controller => :errors, :action => :bad_request
 123     end
 124   end
 125
 126   ##
 127   # test the show action
 128   def test_show
 129     active_block = create(:user_block, :needs_view)
 130     expired_block = create(:user_block, :expired)
 131     revoked_block = create(:user_block, :revoked)
 132
 133     # Viewing a block should fail when a bogus ID is given
 134     get user_block_path(:id => 99999)
 135     assert_response :not_found
 136     assert_template "not_found"
 137     assert_select "p", "Sorry, the user block with ID 99999 could not be found."
 138
 139     # Viewing an expired block should work
 140     get user_block_path(:id => expired_block)
 141     assert_response :success
 142     assert_select "h1 a[href='#{user_path expired_block.user}']", :text => expired_block.user.display_name
 143     assert_select "h1 a[href='#{user_path expired_block.creator}']", :text => expired_block.creator.display_name
 144
 145     # Viewing a revoked block should work
 146     get user_block_path(:id => revoked_block)
 147     assert_response :success
 148     assert_select "h1 a[href='#{user_path revoked_block.user}']", :text => revoked_block.user.display_name
 149     assert_select "h1 a[href='#{user_path revoked_block.creator}']", :text => revoked_block.creator.display_name
 150     assert_select "a[href='#{user_path revoked_block.revoker}']", :text => revoked_block.revoker.display_name
 151
 152     # Viewing an active block should work, but shouldn't mark it as seen
 153     get user_block_path(:id => active_block)
 154     assert_response :success
 155     assert_select "h1 a[href='#{user_path active_block.user}']", :text => active_block.user.display_name
 156     assert_select "h1 a[href='#{user_path active_block.creator}']", :text => active_block.creator.display_name
 157     assert UserBlock.find(active_block.id).needs_view
 158
 159     # Login as the blocked user
 160     session_for(active_block.user)
 161
 162     # Now viewing it should mark it as seen
 163     get user_block_path(:id => active_block)
 164     assert_response :success
 165     assert_not UserBlock.find(active_block.id).needs_view
 166   end
 167
 168   ##
 169   # test edit/revoke link for active blocks
 170   def test_active_block_buttons
 171     creator_user = create(:moderator_user)
 172     other_moderator_user = create(:moderator_user)
 173     block = create(:user_block, :creator => creator_user)
 174
 175     session_for(other_moderator_user)
 176     check_block_buttons block, :revoke => 1
 177
 178     session_for(creator_user)
 179     check_block_buttons block, :edit => 1, :revoke => 1
 180   end
 181
 182   ##
 183   # test the edit link for expired blocks
 184   def test_expired_block_buttons
 185     creator_user = create(:moderator_user)
 186     other_moderator_user = create(:moderator_user)
 187     block = create(:user_block, :expired, :creator => creator_user)
 188
 189     session_for(other_moderator_user)
 190     check_block_buttons block
 191
 192     session_for(creator_user)
 193     check_block_buttons block, :edit => 1
 194   end
 195
 196   ##
 197   # test the edit link for revoked blocks
 198   def test_revoked_block_buttons
 199     creator_user = create(:moderator_user)
 200     revoker_user = create(:moderator_user)
 201     other_moderator_user = create(:moderator_user)
 202     block = create(:user_block, :revoked, :creator => creator_user, :revoker => revoker_user)
 203
 204     session_for(other_moderator_user)
 205     check_block_buttons block
 206
 207     session_for(creator_user)
 208     check_block_buttons block, :edit => 1
 209
 210     session_for(revoker_user)
 211     check_block_buttons block, :edit => 1
 212   end
 213
 214   ##
 215   # test the new action
 216   def test_new
 217     target_user = create(:user)
 218
 219     # Check that the block creation page requires us to login
 220     get new_user_block_path(target_user)
 221     assert_redirected_to login_path(:referer => new_user_block_path(target_user))
 222
 223     # Login as a normal user
 224     session_for(create(:user))
 225
 226     # Check that normal users can't load the block creation page
 227     get new_user_block_path(target_user)
 228     assert_redirected_to :controller => "errors", :action => "forbidden"
 229
 230     # Login as a moderator
 231     session_for(create(:moderator_user))
 232
 233     # Check that the block creation page loads for moderators
 234     get new_user_block_path(target_user)
 235     assert_response :success
 236     assert_select "h1 a[href='#{user_path target_user}']", :text => target_user.display_name
 237     assert_select "form#new_user_block", :count => 1 do
 238       assert_select "textarea#user_block_reason", :count => 1
 239       assert_select "select#user_block_period", :count => 1
 240       assert_select "input#user_block_needs_view[type='checkbox']", :count => 1
 241       assert_select "input#display_name[type='hidden']", :count => 1
 242       assert_select "input[type='submit'][value='Create block']", :count => 1
 243     end
 244
 245     # We should get an error if the user doesn't exist
 246     get new_user_block_path(:display_name => "non_existent_user")
 247     assert_response :not_found
 248     assert_template "users/no_such_user"
 249     assert_select "h1", "The user non_existent_user does not exist"
 250   end
 251
 252   ##
 253   # test the edit action
 254   def test_edit
 255     active_block = create(:user_block)
 256
 257     # Check that the block edit page requires us to login
 258     get edit_user_block_path(:id => active_block)
 259     assert_redirected_to login_path(:referer => edit_user_block_path(active_block))
 260
 261     # Login as a normal user
 262     session_for(create(:user))
 263
 264     # Check that normal users can't load the block edit page
 265     get edit_user_block_path(:id => active_block)
 266     assert_redirected_to :controller => "errors", :action => "forbidden"
 267
 268     # Login as a moderator
 269     session_for(create(:moderator_user))
 270
 271     # Check that the block edit page loads for moderators
 272     get edit_user_block_path(:id => active_block)
 273     assert_response :success
 274     assert_select "h1 a[href='#{user_path active_block.user}']", :text => active_block.user.display_name
 275     assert_select "form#edit_user_block_#{active_block.id}", :count => 1 do
 276       assert_select "textarea#user_block_reason", :count => 1
 277       assert_select "select#user_block_period", :count => 1
 278       assert_select "input#user_block_needs_view[type='checkbox']", :count => 1
 279       assert_select "input[type='submit'][value='Update block']", :count => 1
 280     end
 281
 282     # We should get an error if the user doesn't exist
 283     get edit_user_block_path(:id => 99999)
 284     assert_response :not_found
 285     assert_template "not_found"
 286     assert_select "p", "Sorry, the user block with ID 99999 could not be found."
 287   end
 288
 289   ##
 290   # test the edit action when the remaining block duration doesn't match the available select options
 291   def test_edit_duration
 292     moderator_user = create(:moderator_user)
 293
 294     freeze_time do
 295       active_block = create(:user_block, :creator => moderator_user, :ends_at => Time.now.utc + 96.hours)
 296
 297       session_for(moderator_user)
 298       get edit_user_block_path(active_block)
 299
 300       assert_select "form#edit_user_block_#{active_block.id}", :count => 1 do
 301         assert_select "select#user_block_period", :count => 1 do
 302           assert_select "option[value='96'][selected]", :count => 1
 303         end
 304       end
 305
 306       travel 2.hours
 307       get edit_user_block_path(active_block)
 308
 309       assert_select "form#edit_user_block_#{active_block.id}", :count => 1 do
 310         assert_select "select#user_block_period", :count => 1 do
 311           assert_select "option[value='96'][selected]", :count => 1
 312         end
 313       end
 314     end
 315   end
 316
 317   ##
 318   # test the create action
 319   def test_create
 320     target_user = create(:user)
 321     moderator_user = create(:moderator_user)
 322
 323     # Not logged in yet, so creating a block should fail
 324     post user_blocks_path
 325     assert_response :forbidden
 326
 327     # Login as a normal user
 328     session_for(create(:user))
 329
 330     # Check that normal users can't create blocks
 331     post user_blocks_path
 332     assert_redirected_to :controller => "errors", :action => "forbidden"
 333
 334     # Login as a moderator
 335     session_for(moderator_user)
 336
 337     # A bogus block period should result in an error
 338     assert_no_difference "UserBlock.count" do
 339       post user_blocks_path(:display_name => target_user.display_name,
 340                             :user_block_period => "99")
 341     end
 342     assert_redirected_to new_user_block_path(target_user)
 343     assert_equal "The blocking period must be one of the values selectable in the drop-down list.", flash[:error]
 344
 345     # Check that creating a block works
 346     assert_difference "UserBlock.count", 1 do
 347       post user_blocks_path(:display_name => target_user.display_name,
 348                             :user_block_period => "12",
 349                             :user_block => { :needs_view => false, :reason => "Vandalism" })
 350     end
 351     b = UserBlock.last
 352     assert_redirected_to user_block_path(:id => b.id)
 353     assert_equal "Created a block on user #{target_user.display_name}.", flash[:notice]
 354     assert_in_delta Time.now.utc, b.created_at, 1
 355     assert_in_delta Time.now.utc, b.updated_at, 1
 356     assert_in_delta Time.now.utc + 12.hours, b.ends_at, 1
 357     assert_not b.needs_view
 358     assert_equal "Vandalism", b.reason
 359     assert_equal "markdown", b.reason_format
 360     assert_equal moderator_user.id, b.creator_id
 361
 362     # We should get an error if no user is specified
 363     post user_blocks_path
 364     assert_response :not_found
 365     assert_template "users/no_such_user"
 366     assert_select "h1", "The user  does not exist"
 367
 368     # We should get an error if the user doesn't exist
 369     post user_blocks_path(:display_name => "non_existent_user")
 370     assert_response :not_found
 371     assert_template "users/no_such_user"
 372     assert_select "h1", "The user non_existent_user does not exist"
 373   end
 374
 375   ##
 376   # test the duration of a created block
 377   def test_create_duration
 378     target_user = create(:user)
 379     moderator_user = create(:moderator_user)
 380
 381     session_for(moderator_user)
 382     post user_blocks_path(:display_name => target_user.display_name,
 383                           :user_block_period => "336",
 384                           :user_block => { :needs_view => false, :reason => "Vandalism" })
 385
 386     block = UserBlock.last
 387     assert_equal 1209600, block.ends_at - block.created_at
 388   end
 389
 390   ##
 391   # test the update action
 392   def test_update
 393     moderator_user = create(:moderator_user)
 394     second_moderator_user = create(:moderator_user)
 395     active_block = create(:user_block, :creator => moderator_user)
 396
 397     # Not logged in yet, so updating a block should fail
 398     put user_block_path(:id => active_block)
 399     assert_response :forbidden
 400
 401     # Login as a normal user
 402     session_for(create(:user))
 403
 404     # Check that normal users can't update blocks
 405     put user_block_path(:id => active_block)
 406     assert_redirected_to :controller => "errors", :action => "forbidden"
 407
 408     # Login as the wrong moderator
 409     session_for(second_moderator_user)
 410
 411     # Check that only the person who created a block can update it
 412     assert_no_difference "UserBlock.count" do
 413       put user_block_path(:id => active_block,
 414                           :user_block_period => "12",
 415                           :user_block => { :needs_view => true, :reason => "Vandalism" })
 416     end
 417     assert_redirected_to edit_user_block_path(active_block)
 418     assert_equal "Only the moderator who created this block can edit it.", flash[:error]
 419
 420     # Login as the correct moderator
 421     session_for(moderator_user)
 422
 423     # A bogus block period should result in an error
 424     assert_no_difference "UserBlock.count" do
 425       put user_block_path(:id => active_block, :user_block_period => "99")
 426     end
 427     assert_redirected_to edit_user_block_path(active_block)
 428     assert_equal "The blocking period must be one of the values selectable in the drop-down list.", flash[:error]
 429
 430     # Check that updating a block works
 431     assert_no_difference "UserBlock.count" do
 432       put user_block_path(:id => active_block,
 433                           :user_block_period => "12",
 434                           :user_block => { :needs_view => true, :reason => "Vandalism" })
 435     end
 436     assert_redirected_to user_block_path(active_block)
 437     assert_equal "Block updated.", flash[:notice]
 438     b = UserBlock.find(active_block.id)
 439     assert_in_delta Time.now.utc, b.updated_at, 1
 440     assert b.needs_view
 441     assert_equal "Vandalism", b.reason
 442
 443     # We should get an error if the block doesn't exist
 444     put user_block_path(:id => 99999)
 445     assert_response :not_found
 446     assert_template "not_found"
 447     assert_select "p", "Sorry, the user block with ID 99999 could not be found."
 448   end
 449
 450   ##
 451   # test the update action on expired blocks
 452   def test_update_expired
 453     creator_user = create(:moderator_user)
 454     other_moderator_user = create(:moderator_user)
 455     block = create(:user_block, :expired, :creator => creator_user, :reason => "Original Reason")
 456
 457     session_for(other_moderator_user)
 458     put user_block_path(block,
 459                         :user_block_period => "0",
 460                         :user_block => { :needs_view => false, :reason => "Updated Reason" })
 461     assert_redirected_to edit_user_block_path(block)
 462     assert_equal "Only the moderator who created this block can edit it.", flash[:error]
 463     block.reload
 464     assert_not block.active?
 465     assert_equal "Original Reason", block.reason
 466
 467     session_for(creator_user)
 468     check_inactive_block_updates(block)
 469   end
 470
 471   ##
 472   # test the update action on revoked blocks
 473   def test_update_revoked
 474     creator_user = create(:moderator_user)
 475     revoker_user = create(:moderator_user)
 476     other_moderator_user = create(:moderator_user)
 477     block = create(:user_block, :revoked, :creator => creator_user, :revoker => revoker_user, :reason => "Original Reason")
 478
 479     session_for(other_moderator_user)
 480     put user_block_path(block,
 481                         :user_block_period => "0",
 482                         :user_block => { :needs_view => false, :reason => "Updated Reason" })
 483     assert_redirected_to edit_user_block_path(block)
 484     assert_equal "Only the moderators who created or revoked this block can edit it.", flash[:error]
 485     block.reload
 486     assert_not_predicate block, :active?
 487     assert_equal "Original Reason", block.reason
 488
 489     session_for(creator_user)
 490     check_inactive_block_updates(block)
 491
 492     session_for(revoker_user)
 493     check_inactive_block_updates(block)
 494   end
 495
 496   ##
 497   # test the revoke action
 498   def test_revoke
 499     active_block = create(:user_block)
 500
 501     # Check that the block revoke page requires us to login
 502     get revoke_user_block_path(:id => active_block)
 503     assert_redirected_to login_path(:referer => revoke_user_block_path(:id => active_block))
 504
 505     # Login as a normal user
 506     session_for(create(:user))
 507
 508     # Check that normal users can't load the block revoke page
 509     get revoke_user_block_path(:id => active_block)
 510     assert_redirected_to :controller => "errors", :action => "forbidden"
 511
 512     # Login as a moderator
 513     session_for(create(:moderator_user))
 514
 515     # Check that the block revoke page loads for moderators
 516     get revoke_user_block_path(:id => active_block)
 517     assert_response :success
 518     assert_template "revoke"
 519     assert_select "h1 a[href='#{user_path active_block.user}']", :text => active_block.user.display_name
 520     assert_select "form", :count => 1 do
 521       assert_select "input#confirm[type='checkbox']", :count => 1
 522       assert_select "input[type='submit'][value='Revoke!']", :count => 1
 523     end
 524
 525     # Check that revoking a block using GET should fail
 526     get revoke_user_block_path(:id => active_block, :confirm => true)
 527     assert_response :success
 528     assert_template "revoke"
 529     b = UserBlock.find(active_block.id)
 530     assert_operator b.ends_at - Time.now.utc, :>, 100
 531
 532     # Check that revoking a block works using POST
 533     post revoke_user_block_path(:id => active_block, :confirm => true)
 534     assert_redirected_to user_block_path(active_block)
 535     b = UserBlock.find(active_block.id)
 536     assert_in_delta Time.now.utc, b.ends_at, 1
 537
 538     # We should get an error if the block doesn't exist
 539     get revoke_user_block_path(:id => 99999)
 540     assert_response :not_found
 541     assert_template "not_found"
 542     assert_select "p", "Sorry, the user block with ID 99999 could not be found."
 543   end
 544
 545   ##
 546   # test the revoke all page
 547   def test_revoke_all_page
 548     blocked_user = create(:user)
 549     create(:user_block, :user => blocked_user)
 550
 551     # Asking for the revoke all blocks page with a bogus user name should fail
 552     get user_blocks_on_path("non_existent_user")
 553     assert_response :not_found
 554
 555     # Check that the revoke all blocks page requires us to login
 556     get revoke_all_user_blocks_path(blocked_user)
 557     assert_redirected_to login_path(:referer => revoke_all_user_blocks_path(blocked_user))
 558
 559     # Login as a normal user
 560     session_for(create(:user))
 561
 562     # Check that normal users can't load the revoke all blocks page
 563     get revoke_all_user_blocks_path(blocked_user)
 564     assert_redirected_to :controller => "errors", :action => "forbidden"
 565
 566     # Login as a moderator
 567     session_for(create(:moderator_user))
 568
 569     # Check that the revoke all blocks page loads for moderators
 570     get revoke_all_user_blocks_path(blocked_user)
 571     assert_response :success
 572     assert_select "h1 a[href='#{user_path blocked_user}']", :text => blocked_user.display_name
 573   end
 574
 575   ##
 576   # test the revoke all action
 577   def test_revoke_all_action
 578     blocked_user = create(:user)
 579     active_block1 = create(:user_block, :user => blocked_user)
 580     active_block2 = create(:user_block, :user => blocked_user)
 581     expired_block1 = create(:user_block, :expired, :user => blocked_user)
 582     blocks = [active_block1, active_block2, expired_block1]
 583     moderator_user = create(:moderator_user)
 584
 585     assert_predicate active_block1, :active?
 586     assert_predicate active_block2, :active?
 587     assert_not_predicate expired_block1, :active?
 588
 589     # Login as a normal user
 590     session_for(create(:user))
 591
 592     # Check that normal users can't load the block revoke page
 593     get revoke_all_user_blocks_path(:blocked_user)
 594     assert_redirected_to :controller => "errors", :action => "forbidden"
 595
 596     # Login as a moderator
 597     session_for(moderator_user)
 598
 599     # Check that revoking blocks using GET should fail
 600     get revoke_all_user_blocks_path(blocked_user, :confirm => true)
 601     assert_response :success
 602     assert_template "revoke_all"
 603
 604     blocks.each(&:reload)
 605     assert_predicate active_block1, :active?
 606     assert_predicate active_block2, :active?
 607     assert_not_predicate expired_block1, :active?
 608
 609     # Check that revoking blocks works using POST
 610     post revoke_all_user_blocks_path(blocked_user, :confirm => true)
 611     assert_redirected_to user_blocks_on_path(blocked_user)
 612
 613     blocks.each(&:reload)
 614     assert_not_predicate active_block1, :active?
 615     assert_not_predicate active_block2, :active?
 616     assert_not_predicate expired_block1, :active?
 617     assert_equal moderator_user, active_block1.revoker
 618     assert_equal moderator_user, active_block2.revoker
 619     assert_not_equal moderator_user, expired_block1.revoker
 620   end
 621
 622   ##
 623   # test changes to end/deactivation dates
 624   def test_dates_when_viewed_before_end
 625     blocked_user = create(:user)
 626     moderator_user = create(:moderator_user)
 627
 628     freeze_time do
 629       session_for(moderator_user)
 630       assert_difference "UserBlock.count", 1 do
 631         post user_blocks_path(:display_name => blocked_user.display_name,
 632                               :user_block_period => "48",
 633                               :user_block => { :needs_view => true, :reason => "Testing deactivates_at" })
 634       end
 635       block = UserBlock.last
 636       assert_equal Time.now.utc + 2.days, block.ends_at
 637       assert_nil block.deactivates_at
 638
 639       travel 1.day
 640       session_for(blocked_user)
 641       get user_block_path(block)
 642       block.reload
 643       assert_equal Time.now.utc + 1.day, block.ends_at
 644       assert_equal Time.now.utc + 1.day, block.deactivates_at
 645     end
 646   end
 647
 648   def test_dates_when_viewed_after_end
 649     blocked_user = create(:user)
 650     moderator_user = create(:moderator_user)
 651
 652     freeze_time do
 653       session_for(moderator_user)
 654       assert_difference "UserBlock.count", 1 do
 655         post user_blocks_path(:display_name => blocked_user.display_name,
 656                               :user_block_period => "24",
 657                               :user_block => { :needs_view => true, :reason => "Testing deactivates_at" })
 658       end
 659       block = UserBlock.last
 660       assert_equal Time.now.utc + 1.day, block.ends_at
 661       assert_nil block.deactivates_at
 662
 663       travel 2.days
 664       session_for(blocked_user)
 665       get user_block_path(block)
 666       block.reload
 667       assert_equal Time.now.utc - 1.day, block.ends_at
 668       assert_equal Time.now.utc, block.deactivates_at
 669     end
 670   end
 671
 672   def test_dates_when_edited_before_end
 673     blocked_user = create(:user)
 674     moderator_user = create(:moderator_user)
 675
 676     freeze_time do
 677       session_for(moderator_user)
 678       assert_difference "UserBlock.count", 1 do
 679         post user_blocks_path(:display_name => blocked_user.display_name,
 680                               :user_block_period => "48",
 681                               :user_block => { :needs_view => false, :reason => "Testing deactivates_at" })
 682       end
 683       block = UserBlock.last
 684       assert_equal Time.now.utc + 2.days, block.ends_at
 685       assert_equal Time.now.utc + 2.days, block.deactivates_at
 686
 687       travel 1.day
 688       put user_block_path(block,
 689                           :user_block_period => "48",
 690                           :user_block => { :needs_view => false, :reason => "Testing deactivates_at updated" })
 691       block.reload
 692       assert_equal Time.now.utc + 2.days, block.ends_at
 693       assert_equal Time.now.utc + 2.days, block.deactivates_at
 694     end
 695   end
 696
 697   def test_dates_when_edited_after_end
 698     blocked_user = create(:user)
 699     moderator_user = create(:moderator_user)
 700
 701     freeze_time do
 702       session_for(moderator_user)
 703       assert_difference "UserBlock.count", 1 do
 704         post user_blocks_path(:display_name => blocked_user.display_name,
 705                               :user_block_period => "24",
 706                               :user_block => { :needs_view => false, :reason => "Testing deactivates_at" })
 707       end
 708       block = UserBlock.last
 709       assert_equal Time.now.utc + 1.day, block.ends_at
 710       assert_equal Time.now.utc + 1.day, block.deactivates_at
 711
 712       travel 2.days
 713       put user_block_path(block,
 714                           :user_block_period => "0",
 715                           :user_block => { :needs_view => false, :reason => "Testing deactivates_at updated" })
 716       block.reload
 717       assert_equal Time.now.utc - 1.day, block.ends_at
 718       assert_equal Time.now.utc - 1.day, block.deactivates_at
 719     end
 720   end
 721
 722   ##
 723   # test updates on legacy records without correctly initialized deactivates_at
 724   def test_update_legacy_deactivates_at
 725     blocked_user = create(:user)
 726     moderator_user = create(:moderator_user)
 727
 728     freeze_time do
 729       block = UserBlock.new :user => blocked_user,
 730                             :creator => moderator_user,
 731                             :reason => "because",
 732                             :ends_at => Time.now.utc + 1.day,
 733                             :needs_view => false
 734
 735       assert_difference "UserBlock.count", 1 do
 736         block.save :validate => false
 737       end
 738
 739       travel 2.days
 740       session_for(moderator_user)
 741       put user_block_path(block,
 742                           :user_block_period => "0",
 743                           :user_block => { :needs_view => false, :reason => "Testing legacy block update" })
 744       block.reload
 745       assert_equal Time.now.utc - 1.day, block.ends_at
 746       assert_equal Time.now.utc - 1.day, block.deactivates_at
 747     end
 748   end
 749
 750   ##
 751   # test the blocks_on action
 752   def test_blocks_on
 753     blocked_user = create(:user)
 754     unblocked_user = create(:user)
 755     normal_user = create(:user)
 756     active_block = create(:user_block, :user => blocked_user)
 757     revoked_block = create(:user_block, :revoked, :user => blocked_user)
 758     expired_block = create(:user_block, :expired, :user => unblocked_user)
 759
 760     # Asking for a list of blocks with a bogus user name should fail
 761     get user_blocks_on_path("non_existent_user")
 762     assert_response :not_found
 763     assert_template "users/no_such_user"
 764     assert_select "h1", "The user non_existent_user does not exist"
 765
 766     # Check the list of blocks for a user that has never been blocked
 767     get user_blocks_on_path(normal_user)
 768     assert_response :success
 769     assert_select "table#block_list", false
 770     assert_select "p", "#{normal_user.display_name} has not been blocked yet."
 771
 772     # Check the list of blocks for a user that is currently blocked
 773     get user_blocks_on_path(blocked_user)
 774     assert_response :success
 775     assert_select "h1 a[href='#{user_path blocked_user}']", :text => blocked_user.display_name
 776     assert_select "table#block_list tbody", :count => 1 do
 777       assert_select "tr", 2
 778       assert_select "a[href='#{user_block_path(active_block)}']", 1
 779       assert_select "a[href='#{user_block_path(revoked_block)}']", 1
 780     end
 781
 782     # Check the list of blocks for a user that has previously been blocked
 783     get user_blocks_on_path(unblocked_user)
 784     assert_response :success
 785     assert_select "h1 a[href='#{user_path unblocked_user}']", :text => unblocked_user.display_name
 786     assert_select "table#block_list tbody", :count => 1 do
 787       assert_select "tr", 1
 788       assert_select "a[href='#{user_block_path(expired_block)}']", 1
 789     end
 790   end
 791
 792   ##
 793   # test the blocks_on action with multiple pages
 794   def test_blocks_on_paged
 795     user = create(:user)
 796     user_blocks = create_list(:user_block, 50, :user => user).reverse
 797     next_path = user_blocks_on_path(user)
 798
 799     get next_path
 800     assert_response :success
 801     check_user_blocks_table user_blocks[0...20]
 802     check_no_page_link "Newer Blocks"
 803     next_path = check_page_link "Older Blocks"
 804
 805     get next_path
 806     assert_response :success
 807     check_user_blocks_table user_blocks[20...40]
 808     check_page_link "Newer Blocks"
 809     next_path = check_page_link "Older Blocks"
 810
 811     get next_path
 812     assert_response :success
 813     check_user_blocks_table user_blocks[40...50]
 814     check_page_link "Newer Blocks"
 815     check_no_page_link "Older Blocks"
 816   end
 817
 818   ##
 819   # test the blocks_on action with invalid pages
 820   def test_blocks_on_invalid_paged
 821     user = create(:user)
 822
 823     %w[-1 0 fred].each do |id|
 824       get user_blocks_on_path(user, :before => id)
 825       assert_redirected_to :controller => :errors, :action => :bad_request
 826
 827       get user_blocks_on_path(user, :after => id)
 828       assert_redirected_to :controller => :errors, :action => :bad_request
 829     end
 830   end
 831
 832   ##
 833   # test the blocks_by action
 834   def test_blocks_by
 835     moderator_user = create(:moderator_user)
 836     second_moderator_user = create(:moderator_user)
 837     normal_user = create(:user)
 838     active_block = create(:user_block, :creator => moderator_user)
 839     expired_block = create(:user_block, :expired, :creator => second_moderator_user)
 840     revoked_block = create(:user_block, :revoked, :creator => second_moderator_user)
 841
 842     # Asking for a list of blocks with a bogus user name should fail
 843     get user_blocks_by_path("non_existent_user")
 844     assert_response :not_found
 845     assert_template "users/no_such_user"
 846     assert_select "h1", "The user non_existent_user does not exist"
 847
 848     # Check the list of blocks given by one moderator
 849     get user_blocks_by_path(moderator_user)
 850     assert_response :success
 851     assert_select "h1 a[href='#{user_path moderator_user}']", :text => moderator_user.display_name
 852     assert_select "table#block_list tbody", :count => 1 do
 853       assert_select "tr", 1
 854       assert_select "a[href='#{user_block_path(active_block)}']", 1
 855     end
 856
 857     # Check the list of blocks given by a different moderator
 858     get user_blocks_by_path(second_moderator_user)
 859     assert_response :success
 860     assert_select "h1 a[href='#{user_path second_moderator_user}']", :text => second_moderator_user.display_name
 861     assert_select "table#block_list tbody", :count => 1 do
 862       assert_select "tr", 2
 863       assert_select "a[href='#{user_block_path(expired_block)}']", 1
 864       assert_select "a[href='#{user_block_path(revoked_block)}']", 1
 865     end
 866
 867     # Check the list of blocks (not) given by a normal user
 868     get user_blocks_by_path(normal_user)
 869     assert_response :success
 870     assert_select "table#block_list", false
 871     assert_select "p", "#{normal_user.display_name} has not made any blocks yet."
 872   end
 873
 874   ##
 875   # test the blocks_by action with multiple pages
 876   def test_blocks_by_paged
 877     user = create(:moderator_user)
 878     user_blocks = create_list(:user_block, 50, :creator => user).reverse
 879     next_path = user_blocks_by_path(user)
 880
 881     get next_path
 882     assert_response :success
 883     check_user_blocks_table user_blocks[0...20]
 884     check_no_page_link "Newer Blocks"
 885     next_path = check_page_link "Older Blocks"
 886
 887     get next_path
 888     assert_response :success
 889     check_user_blocks_table user_blocks[20...40]
 890     check_page_link "Newer Blocks"
 891     next_path = check_page_link "Older Blocks"
 892
 893     get next_path
 894     assert_response :success
 895     check_user_blocks_table user_blocks[40...50]
 896     check_page_link "Newer Blocks"
 897     check_no_page_link "Older Blocks"
 898   end
 899
 900   ##
 901   # test the blocks_by action with invalid pages
 902   def test_blocks_by_invalid_paged
 903     user = create(:moderator_user)
 904
 905     %w[-1 0 fred].each do |id|
 906       get user_blocks_by_path(user, :before => id)
 907       assert_redirected_to :controller => :errors, :action => :bad_request
 908
 909       get user_blocks_by_path(user, :after => id)
 910       assert_redirected_to :controller => :errors, :action => :bad_request
 911     end
 912   end
 913
 914   private
 915
 916   def check_block_buttons(block, edit: 0, revoke: 0)
 917     [user_blocks_path, user_block_path(block)].each do |path|
 918       get path
 919       assert_response :success
 920       assert_select "a[href='#{edit_user_block_path block}']", :count => edit
 921       assert_select "a[href='#{revoke_user_block_path block}']", :count => revoke
 922     end
 923   end
 924
 925   def check_inactive_block_updates(block)
 926     original_ends_at = block.ends_at
 927
 928     put user_block_path(block,
 929                         :user_block_period => "0",
 930                         :user_block => { :needs_view => false, :reason => "Updated Reason" })
 931     assert_redirected_to user_block_path(block)
 932     assert_equal "Block updated.", flash[:notice]
 933     block.reload
 934     assert_not_predicate block, :active?
 935     assert_equal "Updated Reason", block.reason
 936     assert_equal original_ends_at, block.ends_at
 937
 938     put user_block_path(block,
 939                         :user_block_period => "0",
 940                         :user_block => { :needs_view => true, :reason => "Updated Reason Needs View" })
 941     assert_response :success
 942     assert_equal "This block is inactive and cannot be reactivated.", flash[:error]
 943     block.reload
 944     assert_not_predicate block, :active?
 945     assert_equal "Updated Reason", block.reason
 946     assert_equal original_ends_at, block.ends_at
 947
 948     put user_block_path(block,
 949                         :user_block_period => "1",
 950                         :user_block => { :needs_view => false, :reason => "Updated Reason Duration Extended" })
 951     assert_response :success
 952     assert_equal "This block is inactive and cannot be reactivated.", flash[:error]
 953     block.reload
 954     assert_not_predicate block, :active?
 955     assert_equal "Updated Reason", block.reason
 956     assert_equal original_ends_at, block.ends_at
 957
 958     put user_block_path(block,
 959                         :user_block_period => "0",
 960                         :user_block => { :needs_view => false, :reason => "Updated Reason Again" })
 961     assert_redirected_to user_block_path(block)
 962     assert_equal "Block updated.", flash[:notice]
 963     block.reload
 964     assert_not_predicate block, :active?
 965     assert_equal "Updated Reason Again", block.reason
 966     assert_equal original_ends_at, block.ends_at
 967   end
 968
 969   def check_user_blocks_table(user_blocks)
 970     assert_dom "table#block_list tbody tr" do |rows|
 971       assert_equal user_blocks.count, rows.count, "unexpected number of rows in user blocks table"
 972       rows.zip(user_blocks).map do |row, user_block|
 973         assert_dom row, "a[href='#{user_block_path user_block}']", 1
 974       end
 975     end
 976   end
 977
 978   def check_no_page_link(name)
 979     assert_select "a.page-link", { :text => /#{Regexp.quote(name)}/, :count => 0 }, "unexpected #{name} page link"
 980   end
 981
 982   def check_page_link(name)
 983     assert_select "a.page-link", { :text => /#{Regexp.quote(name)}/ }, "missing #{name} page link" do |buttons|
 984       return buttons.first.attributes["href"].value
 985     end
 986   end
 987 end