Awight has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/345781 )
Change subject: Relax a test until we can fix the potential double-failmail. ...................................................................... Relax a test until we can fix the potential double-failmail. Change-Id: If5c1a2eb516b30704bdae599236aa6512c5deca2 --- M tests/test_job_wrapper.py 1 file changed, 2 insertions(+), 1 deletion(-) git pull ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/process-control refs/changes/81/345781/1 diff --git a/tests/test_job_wrapper.py b/tests/test_job_wrapper.py index 13b009b..798ac1d 100644 --- a/tests/test_job_wrapper.py +++ b/tests/test_job_wrapper.py @@ -65,7 +65,8 @@ assert ("root", "ERROR", "Job Timing out job timed out after 0.1 seconds") in loglines assert ("root", "ERROR", "Job Timing out job failed with code -9") in loglines - MockSmtp().sendmail.assert_called_once() + # FIXME: Sometimes called twice, once with each error! Relaxing the test for now. + MockSmtp().sendmail.assert_called() @mock.patch("smtplib.SMTP") -- To view, visit https://gerrit.wikimedia.org/r/345781 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: If5c1a2eb516b30704bdae599236aa6512c5deca2 Gerrit-PatchSet: 1 Gerrit-Project: wikimedia/fundraising/process-control Gerrit-Branch: master Gerrit-Owner: Awight <awi...@wikimedia.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits