@@ -579,10 +579,13 @@ def test_update_all_with_failure
579
579
580
580
def test_download_all_with_valid_container
581
581
@request . session [ :user_id ] = 2
582
- get :download_all , :params => {
582
+ get (
583
+ :download_all ,
584
+ :params => {
583
585
:object_type => 'issues' ,
584
586
:object_id => '2'
585
587
}
588
+ )
586
589
assert_response 200
587
590
assert_equal response . headers [ 'Content-Type' ] , 'application/zip'
588
591
assert_match /issue-2-attachments.zip/ , response . headers [ 'Content-Disposition' ]
@@ -591,31 +594,40 @@ def test_download_all_with_valid_container
591
594
592
595
def test_download_all_with_invalid_container
593
596
@request . session [ :user_id ] = 2
594
- get :download_all , :params => {
597
+ get (
598
+ :download_all ,
599
+ :params => {
595
600
:object_type => 'issues' ,
596
601
:object_id => '999'
597
602
}
603
+ )
598
604
assert_response 404
599
605
end
600
606
601
607
def test_download_all_without_readable_attachments
602
608
@request . session [ :user_id ] = 2
603
- get :download_all , :params => {
609
+ get (
610
+ :download_all ,
611
+ :params => {
604
612
:object_type => 'issues' ,
605
613
:object_id => '1'
606
614
}
615
+ )
607
616
assert_equal Issue . find ( 1 ) . attachments , [ ]
608
617
assert_response 404
609
618
end
610
619
611
620
def test_download_all_with_maximum_bulk_download_size_larger_than_attachments
612
621
with_settings :bulk_download_max_size => 0 do
613
622
@request . session [ :user_id ] = 2
614
- get :download_all , :params => {
623
+ get (
624
+ :download_all ,
625
+ :params => {
615
626
:object_type => 'issues' ,
616
627
:object_id => '2' ,
617
628
:back_url => '/issues/2'
618
- }
629
+ }
630
+ )
619
631
assert_redirected_to '/issues/2'
620
632
assert_equal flash [ :error ] , 'These attachments cannot be bulk downloaded because the total file size exceeds the maximum allowed size (0 Bytes)'
621
633
end
0 commit comments