diff options
| author | Victor Shyba <victor.shyba@gmail.com> | 2015-11-23 17:47:03 -0300 |
|---|---|---|
| committer | Victor Shyba <victor.shyba@gmail.com> | 2015-11-23 17:47:03 -0300 |
| commit | 958a916cb3ebb6d101cbdbd75473b1b11f30a3e5 (patch) | |
| tree | 615f8a9edf27d06dfccca7c6b9e1cabbfb559786 /service/test | |
| parent | e68f6670f824564ce352f91fe6bca3ec4850606e (diff) | |
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
Diffstat (limited to 'service/test')
| -rw-r--r-- | service/test/integration/test_leap_mailstore.py | 6 |
1 files changed, 5 insertions, 1 deletions
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 |