diff options
author | Caio Carrara <ccarrara@thoughtworks.com> | 2016-05-11 16:24:46 -0300 |
---|---|---|
committer | Caio Carrara <ccarrara@thoughtworks.com> | 2016-05-11 16:24:46 -0300 |
commit | c5f92efdd19464b876e56750f85a6d8ba4d7b558 (patch) | |
tree | 27153897785261460f7bd9fa3a7c588eb1d5137a /service/test/unit/test_welcome_mail.py | |
parent | a23d3bebc0c879682f7d70b2a8451cc98b2d70ab (diff) |
Remove dispatcher references from service
Since we're not using dispatcher anymore, it doesn't make sense to keep
dispatcher references and program flows that depend on it. For example,
it doesn't make sense keep the flag --organization-mode, so it was
removed as well.
Diffstat (limited to 'service/test/unit/test_welcome_mail.py')
-rw-r--r-- | service/test/unit/test_welcome_mail.py | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/service/test/unit/test_welcome_mail.py b/service/test/unit/test_welcome_mail.py index 829740d3..fe2b3005 100644 --- a/service/test/unit/test_welcome_mail.py +++ b/service/test/unit/test_welcome_mail.py @@ -15,6 +15,7 @@ # along with Pixelated. If not, see <http://www.gnu.org/licenses/>. import os +import re import unittest from mockito import verify, mock from mockito.matchers import Matcher @@ -61,13 +62,12 @@ class WelcomeMailCapture(Matcher): def _format(self, mail): splitter = '\n' - arr = mail.split(splitter) - arr = self._remove_variable_value(arr) + mail_lines = mail.split(splitter) + mail_lines = self._remove_boundaries(mail_lines) + return splitter.join(mail_lines) - return splitter.join(arr) - - def _remove_variable_value(self, arr): - arr.pop(0) - arr.pop(6) - arr.pop(44) - return arr + def _remove_boundaries(self, mail_lines): + # boundary example --===============5031169581469213585==-- + boundary_regex = re.compile("^(.*)(\={15})(\w*)(\={2})(.*)$") + boundaries = filter(boundary_regex.match, mail_lines) + return [line for line in mail_lines if line not in boundaries] |