From 63a252fb5dadb8b57212ea6d5e09737ea22d470f Mon Sep 17 00:00:00 2001 From: Kali Kaneko Date: Thu, 26 Feb 2015 13:10:18 -0400 Subject: fix rename_mailbox implementation, make test pass --- src/leap/mail/mail.py | 10 +++------- src/leap/mail/tests/test_mail.py | 6 +++--- 2 files changed, 6 insertions(+), 10 deletions(-) diff --git a/src/leap/mail/mail.py b/src/leap/mail/mail.py index 37ab829..9906ddf 100644 --- a/src/leap/mail/mail.py +++ b/src/leap/mail/mail.py @@ -816,16 +816,12 @@ class Account(object): return d def rename_mailbox(self, oldname, newname): - # TODO incomplete/wrong!!! - # Should rename also ALL of the document ids that are pointing - # to the old mailbox!!! - - # TODO part-docs identifiers should have the UID_validity of the - # mailbox embedded, instead of the name! (so they can survive a rename) def _rename_mbox(wrapper): wrapper.mbox = newname - return wrapper, wrapper.update(self.store) + d = wrapper.update(self.store) + d.addCallback(lambda result: wrapper) + return d d = self.adaptor.get_or_create_mbox(self.store, oldname) d.addCallback(_rename_mbox) diff --git a/src/leap/mail/tests/test_mail.py b/src/leap/mail/tests/test_mail.py index 24dcc8b..d326ca8 100644 --- a/src/leap/mail/tests/test_mail.py +++ b/src/leap/mail/tests/test_mail.py @@ -342,9 +342,9 @@ class AccountTestCase(SoledadTestMixin): def test_rename_mailbox(self): acc = self.get_account() - d = acc.callWhenReady(lambda _: acc.add_mailbox("TestMailbox")) - d = acc.callWhenReady(lambda _: acc.rename_mailbox( - "TestMailbox", "RenamedMailbox")) + d = acc.callWhenReady(lambda _: acc.add_mailbox("OriginalMailbox")) + d.addCallback(lambda _: acc.rename_mailbox( + "OriginalMailbox", "RenamedMailbox")) d.addCallback(lambda _: acc.list_all_mailbox_names()) d.addCallback(self._test_rename_mailbox_cb) return d -- cgit v1.2.3