From e8afeee6c893104048149b9c2f7e79da1fea0cf4 Mon Sep 17 00:00:00 2001 From: Neissi Torres Lima Date: Fri, 5 Sep 2014 11:43:45 -0300 Subject: Neissi/Duda - make sure that complex recipients (e.g. 'Foo Bar ) are showing properly --- service/pixelated/adapter/pixelated_mail.py | 2 +- service/test/adapter/pixelated_mail_test.py | 13 +++++++------ 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/service/pixelated/adapter/pixelated_mail.py b/service/pixelated/adapter/pixelated_mail.py index 40e4f71f..cd34fe46 100644 --- a/service/pixelated/adapter/pixelated_mail.py +++ b/service/pixelated/adapter/pixelated_mail.py @@ -56,7 +56,7 @@ class PixelatedMail: def _split_recipients(self, header_type, temporary_headers): if(temporary_headers.get(header_type) is not None): recipients = temporary_headers[header_type].split(',') - temporary_headers[header_type] = map(lambda x: x.replace(' ', ''), recipients) + temporary_headers[header_type] = map(lambda x: x.lstrip(), recipients) def _extract_headers(self): temporary_headers = {} diff --git a/service/test/adapter/pixelated_mail_test.py b/service/test/adapter/pixelated_mail_test.py index 66aa3590..7de7c96a 100644 --- a/service/test/adapter/pixelated_mail_test.py +++ b/service/test/adapter/pixelated_mail_test.py @@ -90,17 +90,18 @@ class TestPixelatedMail(unittest.TestCase): def test_extract_headers_should_break_header_in_multiple_recipients(self): headers = test_helper.DEFAULT_HEADERS.copy() - headers['to'] = "nlima@example.com, ddornelles@example.com" - headers['bcc'] = "nlima@example.com, ddornelles@example.com" - headers['cc'] = "nlima@example.com, ddornelles@example.com" + + headers['to'] = "nlima@example.com, Duda Dornelles " + headers['bcc'] = "ddornelles@example.com, Neissi Lima " + headers['cc'] = "nlima@example.com, Duda Dornelles " leap_mail = test_helper.leap_mail(headers=headers) pixelated_mail = PixelatedMail.from_leap_mail(leap_mail) - self.assertEquals(pixelated_mail.headers['to'], ["nlima@example.com", "ddornelles@example.com"]) - self.assertEquals(pixelated_mail.headers['bcc'], ["nlima@example.com", "ddornelles@example.com"]) - self.assertEquals(pixelated_mail.headers['cc'], ["nlima@example.com", "ddornelles@example.com"]) + self.assertEquals(pixelated_mail.headers['to'], ["nlima@example.com", "Duda Dornelles "]) + self.assertEquals(pixelated_mail.headers['bcc'], ["ddornelles@example.com", "Neissi Lima "]) + self.assertEquals(pixelated_mail.headers['cc'], ["nlima@example.com", "Duda Dornelles "]) def test_mark_as_read(self): mail = PixelatedMail.from_dict(self.mail_dict) -- cgit v1.2.3