summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKali Kaneko <kali@leap.se>2015-02-26 13:28:45 -0400
committerKali Kaneko <kali@leap.se>2015-03-03 14:27:03 -0400
commitfac71359f77d5649cf0324f7cd9bfdcf13cb120d (patch)
tree885ac5a2009bc354a44209c23b3bd5f3ee7a13bf
parent63a252fb5dadb8b57212ea6d5e09737ea22d470f (diff)
pass date explicitely, fix partial_append test
it is interpreted as "tags" otherwise.
-rw-r--r--src/leap/mail/imap/mailbox.py2
-rw-r--r--src/leap/mail/imap/tests/test_imap.py1
-rw-r--r--src/leap/mail/mail.py1
3 files changed, 2 insertions, 2 deletions
diff --git a/src/leap/mail/imap/mailbox.py b/src/leap/mail/imap/mailbox.py
index c501614..2653ae4 100644
--- a/src/leap/mail/imap/mailbox.py
+++ b/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/src/leap/mail/imap/tests/test_imap.py b/src/leap/mail/imap/tests/test_imap.py
index 67a24cd..738a674 100644
--- a/src/leap/mail/imap/tests/test_imap.py
+++ b/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/src/leap/mail/mail.py b/src/leap/mail/mail.py
index 9906ddf..584cc4a 100644
--- a/src/leap/mail/mail.py
+++ b/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)