diff options
author | Duda Dornelles <ddornell@thoughtworks.com> | 2014-10-27 17:52:40 +0100 |
---|---|---|
committer | Duda Dornelles <ddornell@thoughtworks.com> | 2014-10-27 17:52:40 +0100 |
commit | 69dfbb0377c3378edabf3a78353907223f2d2f17 (patch) | |
tree | cb8654681c250fd53efb1e5d914761566c702178 | |
parent | 4919c62e298e934771a436606c48daff08fc1381 (diff) |
Fixing pep8 violations
-rw-r--r-- | service/test/unit/controllers/mails_controller_test.py | 20 |
1 files changed, 7 insertions, 13 deletions
diff --git a/service/test/unit/controllers/mails_controller_test.py b/service/test/unit/controllers/mails_controller_test.py index af1e5c95..a64207e9 100644 --- a/service/test/unit/controllers/mails_controller_test.py +++ b/service/test/unit/controllers/mails_controller_test.py @@ -19,9 +19,7 @@ from mockito import * from pixelated.controllers.mails_controller import MailsController - class TestMailsController(unittest.TestCase): - def setUp(self): self.mail_service = mock() self.search_engine = mock() @@ -48,7 +46,8 @@ class TestMailsController(unittest.TestCase): result = self.mails_controller.send_mail(self.input_mail) self.assertEqual(result.status_code, 200) - self.assertEqual(result.data, '{"status": [], "body": "email body", "ident": 1, "tags": [], "header": {"to": "b@b.b", "from": "a@a.a"}, "security_casing": {}}') + self.assertEqual(result.data, + '{"status": [], "body": "email body", "ident": 1, "tags": [], "header": {"to": "b@b.b", "from": "a@a.a"}, "security_casing": {}}') def test_sending_mail_return_error_message_when_send_fails(self): self.mail_service.send = self._send_that_throws_exception @@ -56,7 +55,8 @@ class TestMailsController(unittest.TestCase): result = self.mails_controller.send_mail(self.input_mail) self.assertEqual(result.status_code, 422) - self.assertEqual(result.data, '{"message": "email sending failed\\nmore information of error\\n123\\nthere was a code before this"}') + self.assertEqual(result.data, + '{"message": "email sending failed\\nmore information of error\\n123\\nthere was a code before this"}') def test_fetching_mail_gets_mail_from_mail_service(self): mail = mock() @@ -90,7 +90,7 @@ class TestMailsController(unittest.TestCase): def test_delete_permanently_when_mail_in_trash(self): mail = mock() - mail.mailbox_name ='TRASH' + mail.mailbox_name = 'TRASH' when(self.mail_service).mail(1).thenReturn(mail) self.mails_controller.delete_mail(1) @@ -98,7 +98,7 @@ class TestMailsController(unittest.TestCase): def test_move_message_to_trash(self): mail = mock() - mail.mailbox_name ='INBOX' + mail.mailbox_name = 'INBOX' when(self.mail_service).mail(1).thenReturn(mail) when(self.mails_controller).delete_mail(1).thenReturn(mail) when(self.search_engine).index_mail(mail) @@ -107,16 +107,10 @@ class TestMailsController(unittest.TestCase): def _successfuly_send_mail(self, ident, mail): sent_mail = mock() - mail.mailbox_name ='TRASH' + mail.mailbox_name = 'TRASH' sent_mail.as_dict = lambda: self.input_mail.json return sent_mail def _send_that_throws_exception(self, ident, mail): raise Exception('email sending failed', 'more information of error', 123, 'there was a code before this') - - - - - - |