diff options
author | Bruno Wagner <bwagner@riseup.net> | 2016-01-08 17:34:20 -0200 |
---|---|---|
committer | Bruno Wagner <bwagner@riseup.net> | 2016-01-08 17:34:20 -0200 |
commit | db38cc2919fbe5f52d9eb73f45fd89a84da89682 (patch) | |
tree | cccaad5e84f270961587599d3fe696b2db740072 /service/test/unit/adapter/test_mailbox_indexer_listener.py | |
parent | 0a29be8e41aa81fd1552b1122bc4ab2a3f2357cf (diff) |
Removed search engine from constant in Indexer
The search engine was being passed as a constant to
the Mail Indexer, that constrained the user agent to
one user (because the search engine is user specific).
I added the search engine as a parameter on the Mail Indexer
initialization so that we can have the Indexer working for
each user.
Diffstat (limited to 'service/test/unit/adapter/test_mailbox_indexer_listener.py')
-rw-r--r-- | service/test/unit/adapter/test_mailbox_indexer_listener.py | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/service/test/unit/adapter/test_mailbox_indexer_listener.py b/service/test/unit/adapter/test_mailbox_indexer_listener.py index 9ad3c94d..9d5f4c30 100644 --- a/service/test/unit/adapter/test_mailbox_indexer_listener.py +++ b/service/test/unit/adapter/test_mailbox_indexer_listener.py @@ -35,19 +35,17 @@ class MailboxListenerTest(unittest.TestCase): mailbox.listeners = set() when(mailbox).addListener = lambda x: mailbox.listeners.add(x) - self.assertNotIn(MailboxIndexerListener('INBOX', self.mail_store), mailbox.listeners) + self.assertNotIn(MailboxIndexerListener('INBOX', self.mail_store, mock()), mailbox.listeners) - MailboxIndexerListener.listen(self.account, 'INBOX', self.mail_store) + MailboxIndexerListener.listen(self.account, 'INBOX', self.mail_store, mock()) - self.assertIn(MailboxIndexerListener('INBOX', self.mail_store), mailbox.listeners) + self.assertIn(MailboxIndexerListener('INBOX', self.mail_store, mock()), mailbox.listeners) def test_reindex_missing_idents(self): search_engine = mock() when(search_engine).search('tag:inbox', all_mails=True).thenReturn(['ident1', 'ident2']) - MailboxIndexerListener.SEARCH_ENGINE = search_engine - - listener = MailboxIndexerListener('INBOX', self.mail_store) + listener = MailboxIndexerListener('INBOX', self.mail_store, search_engine) when(self.mail_store).get_mailbox_mail_ids('INBOX').thenReturn({'ident1', 'ident2', 'missing_ident'}) self.mail_store.used_arguments = [] self.mail_store.get_mails = lambda x: self.mail_store.used_arguments.append(x) @@ -59,7 +57,7 @@ class MailboxListenerTest(unittest.TestCase): @defer.inlineCallbacks def test_catches_exceptions_to_not_break_other_listeners(self): when(logger).error(ANY()).thenReturn(None) - listener = MailboxIndexerListener('INBOX', self.mail_store) + listener = MailboxIndexerListener('INBOX', self.mail_store, mock()) yield listener.newMessages(1, 1) |