adds tests and listener for indexing mails created outside the user agent, see #103

3 files changed, 53 insertions, 0 deletions

diff --git a/service/pixelated/adapter/listener.py b/service/pixelated/adapter/listener.py
new file mode 100644
index 00000000..ca264f31
--- /dev/null
+++ b/service/pixelated/adapter/listener.py
@@ -0,0 +1,46 @@
+#
+# Copyright (c) 2014 ThoughtWorks, Inc.
+#
+# Pixelated is free software: you can redistribute it and/or modify
+# it under the terms of the GNU Affero General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+#
+# Pixelated is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PCULAR PURPOSE.  See the
+# GNU Affero General Public License for more details.
+#
+# You should have received a copy of the GNU Affero General Public License
+# along with Pixelated. If not, see <http://www.gnu.org/licenses/>.
+from pixelated.adapter.soledad_querier import SoledadQuerier
+
+
+class MailboxListener(object):
+    """ Listens for new mails, keeping the index updated """
+
+    SEARCH_ENGINE = None
+
+    @classmethod
+    def listen(cls, account, mailbox_name):
+        listener = MailboxListener(mailbox_name)
+        if listener not in account.getMailbox(mailbox_name).listeners:
+            account.getMailbox(mailbox_name).addListener(listener)
+
+    def __init__(self, mailbox_name):
+        self.mailbox_name = mailbox_name
+        self.querier = SoledadQuerier.get_instance()
+
+    def newMessages(self, exists, recent):
+        indexed_idents = set(self.SEARCH_ENGINE.search('tag:' + self.mailbox_name.lower()))
+        soledad_idents = self.querier.get_idents_by_mailbox(self.mailbox_name)
+
+        missing_idents = soledad_idents.difference(indexed_idents)
+
+        self.SEARCH_ENGINE.index_mails(self.querier.mails(missing_idents))
+
+    def __eq__(self, other):
+        return other and other.mailbox_name == self.mailbox_name
+
+    def __hash__(self):
+        return self.mailbox_name.__hash__()
diff --git a/service/pixelated/adapter/pixelated_mailboxes.py b/service/pixelated/adapter/pixelated_mailboxes.py
index c87b7ab3..1025f58e 100644
--- a/service/pixelated/adapter/pixelated_mailboxes.py
+++ b/service/pixelated/adapter/pixelated_mailboxes.py
@@ -15,6 +15,7 @@
 # along with Pixelated. If not, see <http://www.gnu.org/licenses/>.
 from pixelated.adapter.pixelated_mailbox import PixelatedMailbox
 from pixelated.adapter.soledad_querier import SoledadQuerier
+from pixelated.adapter.listener import MailboxListener
 
 
 class PixelatedMailBoxes():
@@ -22,11 +23,14 @@ class PixelatedMailBoxes():
     def __init__(self, account):
         self.account = account
         self.querier = SoledadQuerier.get_instance()
+        for mailbox_name in account.mailboxes:
+            MailboxListener.listen(self.account, mailbox_name)
 
     def _create_or_get(self, mailbox_name):
         mailbox_name = mailbox_name.upper()
         if mailbox_name not in self.account.mailboxes:
             self.account.addMailbox(mailbox_name)
+        MailboxListener.listen(self.account, mailbox_name)
         return PixelatedMailbox.create(mailbox_name)
 
     def inbox(self):
diff --git a/service/pixelated/adapter/soledad_querier.py b/service/pixelated/adapter/soledad_querier.py
index 43b9891e..cb3db66a 100644
--- a/service/pixelated/adapter/soledad_querier.py
+++ b/service/pixelated/adapter/soledad_querier.py
@@ -88,6 +88,9 @@ class SoledadQuerier:
         self.soledad.delete_doc(_mail.hdoc)
         self.soledad.delete_doc(_mail.fdoc)
 
+    def get_idents_by_mailbox(self, mailbox_name):
+        return set(doc.content['chash'] for doc in self.soledad.get_from_index('by-type-and-mbox-and-deleted', 'flags', mailbox_name, '0'))
+
     def _next_uid_for_mailbox(self, mailbox_name):
         mails = self.all_mails_by_mailbox(mailbox_name)
         mails.sort(key=lambda x: x.uid, reverse=True)