From 98d0f2ab8362e15fcf455463a54f3ec8db311ec8 Mon Sep 17 00:00:00 2001 From: Diane Trout Date: Tue, 20 Nov 2012 14:36:49 -0800 Subject: [PATCH] Correctly implement merging Notification & Manager sets Apparently I was off imagining functiosn that don't exist. --- htsworkflow/frontend/experiments/test_experiments.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) 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): """ -- 2.30.2