]> git.openstreetmap.org Git - rails.git/blob - test/controllers/user_blocks_controller_test.rb
Remove /user_blocks/new route
[rails.git] / 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 the blocks_on action
624   def test_blocks_on
625     blocked_user = create(:user)
626     unblocked_user = create(:user)
627     normal_user = create(:user)
628     active_block = create(:user_block, :user => blocked_user)
629     revoked_block = create(:user_block, :revoked, :user => blocked_user)
630     expired_block = create(:user_block, :expired, :user => unblocked_user)
631
632     # Asking for a list of blocks with a bogus user name should fail
633     get user_blocks_on_path("non_existent_user")
634     assert_response :not_found
635     assert_template "users/no_such_user"
636     assert_select "h1", "The user non_existent_user does not exist"
637
638     # Check the list of blocks for a user that has never been blocked
639     get user_blocks_on_path(normal_user)
640     assert_response :success
641     assert_select "table#block_list", false
642     assert_select "p", "#{normal_user.display_name} has not been blocked yet."
643
644     # Check the list of blocks for a user that is currently blocked
645     get user_blocks_on_path(blocked_user)
646     assert_response :success
647     assert_select "h1 a[href='#{user_path blocked_user}']", :text => blocked_user.display_name
648     assert_select "table#block_list tbody", :count => 1 do
649       assert_select "tr", 2
650       assert_select "a[href='#{user_block_path(active_block)}']", 1
651       assert_select "a[href='#{user_block_path(revoked_block)}']", 1
652     end
653
654     # Check the list of blocks for a user that has previously been blocked
655     get user_blocks_on_path(unblocked_user)
656     assert_response :success
657     assert_select "h1 a[href='#{user_path unblocked_user}']", :text => unblocked_user.display_name
658     assert_select "table#block_list tbody", :count => 1 do
659       assert_select "tr", 1
660       assert_select "a[href='#{user_block_path(expired_block)}']", 1
661     end
662   end
663
664   ##
665   # test the blocks_on action with multiple pages
666   def test_blocks_on_paged
667     user = create(:user)
668     user_blocks = create_list(:user_block, 50, :user => user).reverse
669     next_path = user_blocks_on_path(user)
670
671     get next_path
672     assert_response :success
673     check_user_blocks_table user_blocks[0...20]
674     check_no_page_link "Newer Blocks"
675     next_path = check_page_link "Older Blocks"
676
677     get next_path
678     assert_response :success
679     check_user_blocks_table user_blocks[20...40]
680     check_page_link "Newer Blocks"
681     next_path = check_page_link "Older Blocks"
682
683     get next_path
684     assert_response :success
685     check_user_blocks_table user_blocks[40...50]
686     check_page_link "Newer Blocks"
687     check_no_page_link "Older Blocks"
688   end
689
690   ##
691   # test the blocks_on action with invalid pages
692   def test_blocks_on_invalid_paged
693     user = create(:user)
694
695     %w[-1 0 fred].each do |id|
696       get user_blocks_on_path(user, :before => id)
697       assert_redirected_to :controller => :errors, :action => :bad_request
698
699       get user_blocks_on_path(user, :after => id)
700       assert_redirected_to :controller => :errors, :action => :bad_request
701     end
702   end
703
704   ##
705   # test the blocks_by action
706   def test_blocks_by
707     moderator_user = create(:moderator_user)
708     second_moderator_user = create(:moderator_user)
709     normal_user = create(:user)
710     active_block = create(:user_block, :creator => moderator_user)
711     expired_block = create(:user_block, :expired, :creator => second_moderator_user)
712     revoked_block = create(:user_block, :revoked, :creator => second_moderator_user)
713
714     # Asking for a list of blocks with a bogus user name should fail
715     get user_blocks_by_path("non_existent_user")
716     assert_response :not_found
717     assert_template "users/no_such_user"
718     assert_select "h1", "The user non_existent_user does not exist"
719
720     # Check the list of blocks given by one moderator
721     get user_blocks_by_path(moderator_user)
722     assert_response :success
723     assert_select "h1 a[href='#{user_path moderator_user}']", :text => moderator_user.display_name
724     assert_select "table#block_list tbody", :count => 1 do
725       assert_select "tr", 1
726       assert_select "a[href='#{user_block_path(active_block)}']", 1
727     end
728
729     # Check the list of blocks given by a different moderator
730     get user_blocks_by_path(second_moderator_user)
731     assert_response :success
732     assert_select "h1 a[href='#{user_path second_moderator_user}']", :text => second_moderator_user.display_name
733     assert_select "table#block_list tbody", :count => 1 do
734       assert_select "tr", 2
735       assert_select "a[href='#{user_block_path(expired_block)}']", 1
736       assert_select "a[href='#{user_block_path(revoked_block)}']", 1
737     end
738
739     # Check the list of blocks (not) given by a normal user
740     get user_blocks_by_path(normal_user)
741     assert_response :success
742     assert_select "table#block_list", false
743     assert_select "p", "#{normal_user.display_name} has not made any blocks yet."
744   end
745
746   ##
747   # test the blocks_by action with multiple pages
748   def test_blocks_by_paged
749     user = create(:moderator_user)
750     user_blocks = create_list(:user_block, 50, :creator => user).reverse
751     next_path = user_blocks_by_path(user)
752
753     get next_path
754     assert_response :success
755     check_user_blocks_table user_blocks[0...20]
756     check_no_page_link "Newer Blocks"
757     next_path = check_page_link "Older Blocks"
758
759     get next_path
760     assert_response :success
761     check_user_blocks_table user_blocks[20...40]
762     check_page_link "Newer Blocks"
763     next_path = check_page_link "Older Blocks"
764
765     get next_path
766     assert_response :success
767     check_user_blocks_table user_blocks[40...50]
768     check_page_link "Newer Blocks"
769     check_no_page_link "Older Blocks"
770   end
771
772   ##
773   # test the blocks_by action with invalid pages
774   def test_blocks_by_invalid_paged
775     user = create(:moderator_user)
776
777     %w[-1 0 fred].each do |id|
778       get user_blocks_by_path(user, :before => id)
779       assert_redirected_to :controller => :errors, :action => :bad_request
780
781       get user_blocks_by_path(user, :after => id)
782       assert_redirected_to :controller => :errors, :action => :bad_request
783     end
784   end
785
786   private
787
788   def check_block_buttons(block, edit: 0, revoke: 0)
789     [user_blocks_path, user_block_path(block)].each do |path|
790       get path
791       assert_response :success
792       assert_select "a[href='#{edit_user_block_path block}']", :count => edit
793       assert_select "a[href='#{revoke_user_block_path block}']", :count => revoke
794     end
795   end
796
797   def check_inactive_block_updates(block)
798     put user_block_path(block,
799                         :user_block_period => "0",
800                         :user_block => { :needs_view => false, :reason => "Updated Reason" })
801     assert_redirected_to user_block_path(block)
802     assert_equal "Block updated.", flash[:notice]
803     block.reload
804     assert_not_predicate block, :active?
805     assert_equal "Updated Reason", block.reason
806
807     put user_block_path(block,
808                         :user_block_period => "0",
809                         :user_block => { :needs_view => true, :reason => "Updated Reason Needs View" })
810     assert_response :success
811     assert_equal "This block is inactive and cannot be reactivated.", flash[:error]
812     block.reload
813     assert_not_predicate block, :active?
814     assert_equal "Updated Reason", block.reason
815
816     put user_block_path(block,
817                         :user_block_period => "1",
818                         :user_block => { :needs_view => false, :reason => "Updated Reason Duration Extended" })
819     assert_response :success
820     assert_equal "This block is inactive and cannot be reactivated.", flash[:error]
821     block.reload
822     assert_not_predicate block, :active?
823     assert_equal "Updated Reason", block.reason
824
825     put user_block_path(block,
826                         :user_block_period => "0",
827                         :user_block => { :needs_view => false, :reason => "Updated Reason Again" })
828     assert_redirected_to user_block_path(block)
829     assert_equal "Block updated.", flash[:notice]
830     block.reload
831     assert_not_predicate block, :active?
832     assert_equal "Updated Reason Again", block.reason
833   end
834
835   def check_user_blocks_table(user_blocks)
836     assert_dom "table#block_list tbody tr" do |rows|
837       assert_equal user_blocks.count, rows.count, "unexpected number of rows in user blocks table"
838       rows.zip(user_blocks).map do |row, user_block|
839         assert_dom row, "a[href='#{user_block_path user_block}']", 1
840       end
841     end
842   end
843
844   def check_no_page_link(name)
845     assert_select "a.page-link", { :text => /#{Regexp.quote(name)}/, :count => 0 }, "unexpected #{name} page link"
846   end
847
848   def check_page_link(name)
849     assert_select "a.page-link", { :text => /#{Regexp.quote(name)}/ }, "missing #{name} page link" do |buttons|
850       return buttons.first.attributes["href"].value
851     end
852   end
853 end