From 50bc463c2250089240519e386a053bf8e3e7f04e Mon Sep 17 00:00:00 2001 From: Kali Kaneko Date: Thu, 26 Feb 2015 13:28:45 -0400 Subject: pass date explicitely, fix partial_append test it is interpreted as "tags" otherwise. --- mail/src/leap/mail/imap/mailbox.py | 2 +- mail/src/leap/mail/imap/tests/test_imap.py | 1 + mail/src/leap/mail/mail.py | 1 - 3 files changed, 2 insertions(+), 2 deletions(-) diff --git a/mail/src/leap/mail/imap/mailbox.py b/mail/src/leap/mail/imap/mailbox.py index c5016146..2653ae49 100644 --- a/mail/src/leap/mail/imap/mailbox.py +++ b/mail/src/leap/mail/imap/mailbox.py @@ -352,7 +352,7 @@ class IMAPMailbox(object): # extension or patching imap server to support pipelining. # TODO add notify_new as a callback here... - return self.collection.add_msg(message, flags, date, + return self.collection.add_msg(message, flags, date=date, notify_just_mdoc=True) def notify_new(self, *args): diff --git a/mail/src/leap/mail/imap/tests/test_imap.py b/mail/src/leap/mail/imap/tests/test_imap.py index 67a24cde..738a6740 100644 --- a/mail/src/leap/mail/imap/tests/test_imap.py +++ b/mail/src/leap/mail/imap/tests/test_imap.py @@ -1044,6 +1044,7 @@ class LEAPIMAP4ServerTestCase(IMAP4HelperMixin): self._cbTestPartialAppend, infile) def _cbTestPartialAppend(self, fetched, infile): + fetched = list(fetched) self.assertTrue(len(fetched) == 1) self.assertTrue(len(fetched[0]) == 2) uid, msg = fetched[0] diff --git a/mail/src/leap/mail/mail.py b/mail/src/leap/mail/mail.py index 9906ddfc..584cc4a3 100644 --- a/mail/src/leap/mail/mail.py +++ b/mail/src/leap/mail/mail.py @@ -540,7 +540,6 @@ class MessageCollection(object): """ # TODO watch out if the use of this method in IMAP COPY/APPEND is # passing the right date. - # XXX mdoc ref is a leaky abstraction here. generalize. leap_assert_type(flags, tuple) -- cgit v1.2.3