summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDuda Dornelles <ddornell@thoughtworks.com>2014-10-27 17:52:40 +0100
committerDuda Dornelles <ddornell@thoughtworks.com>2014-10-27 17:52:40 +0100
commit69dfbb0377c3378edabf3a78353907223f2d2f17 (patch)
treecb8654681c250fd53efb1e5d914761566c702178
parent4919c62e298e934771a436606c48daff08fc1381 (diff)
Fixing pep8 violations
-rw-r--r--service/test/unit/controllers/mails_controller_test.py20
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')
-
-
-
-
-
-