Correctly implement merging Notification & Manager sets
authorDiane Trout <diane@caltech.edu>
Tue, 20 Nov 2012 22:36:49 +0000 (14:36 -0800)
committerDiane Trout <diane@caltech.edu>
Tue, 20 Nov 2012 22:36:49 +0000 (14:36 -0800)
Apparently I was off imagining functiosn that don't exist.

htsworkflow/frontend/experiments/test_experiments.py

index 0f241476e78ff7938145337bb64409accfa69a04..e6d7484151149b4f7f7ce147cbacf046c28bb9d9 100644 (file)
@@ -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):
         """