Defect #29990
closed
Add missing fixtures for test_create_should_send_notification
Added by Marius BĂLTEANU over 5 years ago.
Updated over 5 years ago.
Category:
Code cleanup/refactoring
Description
The bellow test fails when it is run in isolation
Failure:
CommentTest#test_create_should_send_notification [/builds/marius-balteanu/redmine/test/unit/comment_test.rb:39]:
"ActionMailer::Base.deliveries.size" didn't change by 2.
Expected: 2
Actual: 1
bin/rails test test/unit/comment_test.rb:35
Attached a patch that adds the missing fixtures.
Files
- Subject changed from Add missing fixtures for test test_create_should_send_notification to Add missing fixtures for test_create_should_send_notification
- Description updated (diff)
- File add_missing_fixtures_to_documents.patch added
And one more for test/functional/documents_controller_test.rb
root@45ec3a6558b0:/work# ruby test/functional/documents_controller_test.rb
DEPRECATION WARNING: `secrets.secret_token` is deprecated in favor of `secret_key_base` and will be removed in Rails 6.0. (called from <top (required)> at /work/config/environment.rb:14)
Run options: --seed 39471
# Running:
...........F
Failure:
DocumentsControllerTest#test_create_with_one_attachment [test/functional/documents_controller_test.rb:135]:
Expected: 2
Actual: 1
bin/rails test test/functional/documents_controller_test.rb:109
...
Finished in 1.231643s, 12.1788 runs/s, 34.9127 assertions/s.
15 runs, 43 assertions, 1 failures, 0 errors, 0 skips
- Status changed from New to Confirmed
- Target version set to 4.1.0
- Copied to Defect #30054: Add missing fixtures for test_create_with_one_attachment added
Marius BALTEANU wrote:
And one more for test/functional/documents_controller_test.rb
[...]
I have created another issue #30054.
- File deleted (
add_missing_fixtures_to_documents.patch)
- Category set to Code cleanup/refactoring
- Status changed from Confirmed to Closed
- Assignee set to Go MAEDA
- Target version changed from 4.1.0 to 4.0.0
- Resolution set to Fixed
Committed. Thank you for your daily contribution.
Also available in: Atom
PDF
Updated by 
Updated by