Actions
Patch #31065
closed
Fix tests that has incorrect verification target
Start date:
Due date:
% Done:
0%
Estimated time:
Description
- test_issue_edit_subject_should_include_status_changes_if_setting_is_enabled
- test_issue_edit_subject_should_not_include_status_changes_if_setting_is_disabled
These two tests check Mailer.deliver_issue_add.
I think those tests should check Mailer.deliver_issue_edit as described in the title.
This patch fixes those tests.
Files
Related issues
Updated by Go MAEDA about 5 years ago
- Related to Feature #13111: New setting to include the status changes in issue mail notifications subject added
Updated by Go MAEDA about 5 years ago
Those two tests are added by #13111 for upcoming 4.1.0.
Updated by Go MAEDA almost 5 years ago
- Assignee set to Marius BĂLTEANU
Marius, could you check the attached fix?
Updated by Marius BĂLTEANU almost 5 years ago
- Assignee changed from Marius BĂLTEANU to Go MAEDA
Mizuki, you're right, the tests should check Mailer.deliver_issue_edit, thanks for catching and fixing this.
Go, it can be committed as part of #13111.
Updated by Go MAEDA almost 5 years ago
- Status changed from New to Closed
Committed the fix. Thank you all for working on this issue.
Actions