Actions
Defect #28931
closedUnreachable code in QueriesControllerTest#test_bulk_copy_to_another_project
Start date:
Due date:
% Done:
0%
Estimated time:
Resolution:
Fixed
Affected version:
Description
this test probably never acually worked
Issue.order('id DESC').limit(issues.size)
issues.size was zero in rails < 5.2, it tests nothing
after this change it fails
https://github.com/rails/rails/commit/06f45435da941d713afefd3140421d1ced7abbdc
so I removed it
Files
Related issues
Updated by Go MAEDA over 6 years ago
- Related to Patch #28933: Migrate to Rails 5.2 added
Updated by Go MAEDA over 6 years ago
I think it is better to modify the test to work rather than simply deleting it.
Index: test/functional/issues_controller_test.rb
===================================================================
--- test/functional/issues_controller_test.rb (revision 17367)
+++ test/functional/issues_controller_test.rb (working copy)
@@ -5967,10 +5967,11 @@
def test_bulk_copy_to_another_project
@request.session[:user_id] = 2
- assert_difference 'Issue.count', 2 do
+ issue_ids = [1, 2]
+ assert_difference 'Issue.count', issue_ids.size do
assert_no_difference 'Project.find(1).issues.count' do
post :bulk_update, :params => {
- :ids => [1, 2],
+ :ids => issue_ids,
:issue => {
:project_id => '2'
},
@@ -5980,7 +5981,7 @@
end
assert_redirected_to '/projects/ecookbook/issues'
- copies = Issue.order('id DESC').limit(issues.size)
+ copies = Issue.order('id DESC').limit(issue_ids.size)
copies.each do |copy|
assert_equal 2, copy.project_id
end
Updated by Go MAEDA over 6 years ago
- Tracker changed from Patch to Defect
- Subject changed from [Rails 5.2] fix spec test_bulk_copy_to_another_project to Unreachable code in QueriesControllerTest#test_bulk_copy_to_another_project
- Assignee set to Go MAEDA
- Target version set to 4.0.0
- Resolution set to Fixed
Committed in r17372. Thank you for reporting this issue.
Updated by Go MAEDA over 6 years ago
- Related to Feature #23630: Migrate to Rails 5.2 added
Updated by Go MAEDA over 6 years ago
- Related to deleted (Patch #28933: Migrate to Rails 5.2)
Actions