Defect #28931
closed
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
- 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.
- Status changed from New to Closed
Also available in: Atom
PDF