summaryrefslogtreecommitdiff
path: root/service
diff options
context:
space:
mode:
authorVictor Shyba <victor.shyba@gmail.com>2015-11-23 17:47:03 -0300
committerVictor Shyba <victor.shyba@gmail.com>2015-11-23 17:47:03 -0300
commit958a916cb3ebb6d101cbdbd75473b1b11f30a3e5 (patch)
tree615f8a9edf27d06dfccca7c6b9e1cabbfb559786 /service
parente68f6670f824564ce352f91fe6bca3ec4850606e (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')
-rw-r--r--service/test/integration/test_leap_mailstore.py6
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