From 958a916cb3ebb6d101cbdbd75473b1b11f30a3e5 Mon Sep 17 00:00:00 2001 From: Victor Shyba Date: Mon, 23 Nov 2015 17:47:03 -0300 Subject: Making explicit the goal for test that checks double deletion It had no assert, making it confusing. With a failure clause on the except handler we show that its not supposed to raise errors --Issue #512 --- service/test/integration/test_leap_mailstore.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'service/test/integration') diff --git a/service/test/integration/test_leap_mailstore.py b/service/test/integration/test_leap_mailstore.py index f72a99be..af5ba8a0 100644 --- a/service/test/integration/test_leap_mailstore.py +++ b/service/test/integration/test_leap_mailstore.py @@ -199,4 +199,8 @@ class LeapMailStoreTest(SoledadTestBase): mail = yield self.mail_store.add_mail('INBOX', mail.as_string()) yield self.mail_store.delete_mail(mail.ident) - yield self.mail_store.delete_mail(mail.ident) + try: + yield self.mail_store.delete_mail(mail.ident) + except Exception as e: + self.fail("Deleting a deleted mail should be ok, but raised an error") + raise e -- cgit v1.2.3