Defect #32022
closed
IssueSubtaskingTest fails with high probability
Added by Go MAEDA over 5 years ago.
Updated over 5 years ago.
Category:
Code cleanup/refactoring
Description
$ RAILS_ENV=test bin/rake db:migrate:reset
$ bin/rails test --seed 21196
.
.
.
Failure:
IssueSubtaskingTest#test_parent_total_estimated_hours_should_be_sum_of_visible_descendants [/Users/maeda/redmines/4.0-stable/test/unit/issue_subtasking_test.rb:345]:
Expected: 12
Actual: 21.0
bin/rails test test/unit/issue_subtasking_test.rb:335
- Target version set to 4.0.5
I am having been observed this issue since r18359. I think it can be fixed if r17761 is merged to 4.0-stable branch.
The test assumes that User.current
is AnonymousUser
, who cannot see other user's private issue. The error occurs when User.current
is a user whose role has permission to see see other user's private issue.
We can fix the error by backporting r17761 in order to ensure that the User.current
is always AnonymousUser
. r17761 was committed 9 days after 4.0-stable branch had been created, in order to fix test failures.
I backported r17761 to 4.0-stable and performed the same seed test.
I have confirmed that no fail occurred in IssueSubtaskingTest.
- Related to Defect #31778: Total estimated time issue query column and issue field might leak information added
- Category set to Code cleanup/refactoring
- Status changed from New to Closed
- Assignee set to Go MAEDA
- Resolution set to Fixed
- Subject changed from IssueSubtaskingTest randomly fails in 4.0-stable to IssueSubtaskingTest fails with high probability
- Status changed from Closed to Reopened
- Target version changed from 4.0.5 to 3.4.12
3.4-stable branch has the same problem. It also can be fixed by backporting r17761.
$ RAILS_ENV=test bin/rake db:migrate:reset
$ RAILS_ENV=test bin/rake test SEED=46331
.
.
.
1) Failure:
IssueSubtaskingTest#test_parent_total_estimated_hours_should_be_sum_of_visible_descendants [/Users/maeda/redmines/3.4-stable/test/unit/issue_subtasking_test.rb:334]:
Expected: 12
Actual: 21.0
- Status changed from Reopened to Closed
Also available in: Atom
PDF