From 2fb15ab3f6ed53434803791df9a38879e44b1195 Mon Sep 17 00:00:00 2001 From: Vicente Luchi Date: Sat, 24 Jan 2015 17:01:40 -0200 Subject: #184 - not creating empty tags anymore --- service/test/integration/test_tags.py | 10 ++++++++++ service/test/support/integration/app_test_client.py | 2 +- service/test/support/integration/soledad_test_base.py | 10 ++++------ 3 files changed, 15 insertions(+), 7 deletions(-) (limited to 'service/test') diff --git a/service/test/integration/test_tags.py b/service/test/integration/test_tags.py index 6072392c..b32e89c7 100644 --- a/service/test/integration/test_tags.py +++ b/service/test/integration/test_tags.py @@ -42,6 +42,16 @@ class TagsTest(SoledadTestBase): mails = self.get_mails_by_tag('important') self.assertEquals('Mail with tags', mails[0].subject) + def test_empty_tags_are_not_allowed(self): + mail = MailBuilder().with_subject('Mail with tags').build_input_mail() + self.client.add_mail_to_inbox(mail) + + self.post_tags(mail.ident, self._tags_json(['tag1', ''])) + + mail = self.get_mail(mail.ident) + + self.assertEquals(mail['tags'], ['tag1']) + def test_addition_of_reserved_tags_is_not_allowed(self): mail = MailBuilder().with_subject('Mail with tags').build_input_mail() self.client.add_mail_to_inbox(mail) diff --git a/service/test/support/integration/app_test_client.py b/service/test/support/integration/app_test_client.py index 860b9c40..e8f79d2b 100644 --- a/service/test/support/integration/app_test_client.py +++ b/service/test/support/integration/app_test_client.py @@ -106,7 +106,7 @@ class AppTestClient: time.sleep(1) return lambda: process.terminate() - def get(self, path, get_args, as_json=True): + def get(self, path, get_args='', as_json=True): request = request_mock(path) request.args = get_args return self._render(request, as_json) diff --git a/service/test/support/integration/soledad_test_base.py b/service/test/support/integration/soledad_test_base.py index 5892de60..f7693ad4 100644 --- a/service/test/support/integration/soledad_test_base.py +++ b/service/test/support/integration/soledad_test_base.py @@ -25,12 +25,6 @@ class SoledadTestBase(unittest.TestCase): DEFERRED_TIMEOUT = 120 DEFERRED_TIMEOUT_LONG = 300 - @classmethod - def setUpClass(cls): - from nose.twistedtools import threaded_reactor - - threaded_reactor() - def setUp(self): self.client = AppTestClient() @@ -66,6 +60,10 @@ class SoledadTestBase(unittest.TestCase): res, req = self.client.get('/tags', kwargs) return res + def get_mail(self, mail_ident): + res, req = self.client.get('/mail/%s' % mail_ident) + return res + def delete_mail(self, mail_ident): res, req = self.client.delete("/mail/%s" % mail_ident) return req -- cgit v1.2.3