diff --git a/opentech/apply/activity/messaging.py b/opentech/apply/activity/messaging.py index 1af108b16ea6e1bf6b3fe7b7e6a97f3c6dbebdf3..3c5670c57d0f185cebec3c0f0e383dc37648455d 100644 --- a/opentech/apply/activity/messaging.py +++ b/opentech/apply/activity/messaging.py @@ -532,7 +532,6 @@ class EmailAdapter(AdapterBase): } for reviewer, submissions in reviewers_to_message.items() ] - def reviewers(self, submission): return [ reviewer.email diff --git a/opentech/apply/activity/tests/test_messaging.py b/opentech/apply/activity/tests/test_messaging.py index e2c8b6318234ba997d35a0d04d96a0b259e23e23..c7e1d854021a069fa7ae5264592e0979571c5f5e 100644 --- a/opentech/apply/activity/tests/test_messaging.py +++ b/opentech/apply/activity/tests/test_messaging.py @@ -247,13 +247,13 @@ class TestActivityAdapter(TestCase): def test_internal_transition_kwarg_for_invisible_transition(self): submission = ApplicationSubmissionFactory(status='post_review_discussion') - kwargs = self.adapter.extra_kwargs(MESSAGES.TRANSITION, submission=submission) + kwargs = self.adapter.extra_kwargs(MESSAGES.TRANSITION, submission=submission, submissions=None) self.assertEqual(kwargs['visibility'], INTERNAL) def test_public_transition_kwargs(self): submission = ApplicationSubmissionFactory() - kwargs = self.adapter.extra_kwargs(MESSAGES.TRANSITION, submission=submission) + kwargs = self.adapter.extra_kwargs(MESSAGES.TRANSITION, submission=submission, submissions=None) self.assertNotIn('visibility', kwargs)