diff options
| author | Kali Kaneko <kali@leap.se> | 2015-03-02 14:34:40 -0400 |
|---|---|---|
| committer | Kali Kaneko <kali@leap.se> | 2015-03-03 14:30:18 -0400 |
| commit | da63b59cf49a3578f069095b83921ae8901787d3 (patch) | |
| tree | 96547817c2022b8c69f8427974312622335a433f | |
| parent | 82c96eeadb2a80e4aa7fcc48782edcfad2f8a54d (diff) | |
[bug] Fix IllegalMailboxCreate
We're raising the exception now, not a Failure.
Releases: 0.9.0
| -rw-r--r-- | mail/src/leap/mail/imap/tests/test_imap.py | 15 |
1 files changed, 1 insertions, 14 deletions
diff --git a/mail/src/leap/mail/imap/tests/test_imap.py b/mail/src/leap/mail/imap/tests/test_imap.py index fbe02d40..a94cea7c 100644 --- a/mail/src/leap/mail/imap/tests/test_imap.py +++ b/mail/src/leap/mail/imap/tests/test_imap.py @@ -1025,7 +1025,6 @@ class LEAPIMAP4ServerTestCase(IMAP4HelperMixin): self.assertItemsEqual(self.results, [1, 3]) -# TODO -------- Fix this testcase class AccountTestCase(IMAP4HelperMixin): """ Test the Account. @@ -1037,19 +1036,7 @@ class AccountTestCase(IMAP4HelperMixin): return self.server.theAccount.addMailbox('one') def test_illegalMailboxCreate(self): - # FIXME --- account.addMailbox needs to raise a failure, - # not the direct exception. - self.stashed = None - - def stash(result): - self.stashed = result - - d = self._create_empty_mailbox() - d.addBoth(stash) - d.addCallback(lambda _: self.failUnless(isinstance(self.stashed, - failure.Failure))) - return d - #self.assertRaises(AssertionError, self._create_empty_mailbox) + self.assertRaises(AssertionError, self._create_empty_mailbox) class IMAP4ServerSearchTestCase(IMAP4HelperMixin): |