From: Diane Trout Date: Tue, 20 Nov 2012 22:36:49 +0000 (-0800) Subject: Correctly implement merging Notification & Manager sets X-Git-Url: http://woldlab.caltech.edu/gitweb/?p=htsworkflow.git;a=commitdiff_plain;h=98d0f2ab8362e15fcf455463a54f3ec8db311ec8 Correctly implement merging Notification & Manager sets Apparently I was off imagining functiosn that don't exist. --- diff --git a/htsworkflow/frontend/experiments/test_experiments.py b/htsworkflow/frontend/experiments/test_experiments.py index 0f24147..e6d7484 100644 --- a/htsworkflow/frontend/experiments/test_experiments.py +++ b/htsworkflow/frontend/experiments/test_experiments.py @@ -561,10 +561,11 @@ class TestEmailNotify(TestCase): response = self.client.get('/experiments/started/153/', {'send':'1','bcc':'on'}) self.assertEqual(response.status_code, 200) self.assertEqual(len(mail.outbox), 4) - bcc = set(settings.NOTIFICATION_BCC).intersect(set(settings.MANAGERS)) + bcc = set(settings.NOTIFICATION_BCC).copy() + bcc.update(set(settings.MANAGERS)) for m in mail.outbox: self.assertTrue(len(m.body) > 0) - self.assertEqual(m.bcc, bcc) + self.assertEqual(set(m.bcc), bcc) def test_email_navigation(self): """