bulk mark as read #114

4 files changed, 30 insertions, 0 deletions

diff --git a/service/pixelated/config/app_factory.py b/service/pixelated/config/app_factory.py
index 6c9c07fe..ed752118 100644
--- a/service/pixelated/config/app_factory.py
+++ b/service/pixelated/config/app_factory.py
@@ -36,6 +36,7 @@ def _setup_routes(app, home_controller, mails_controller, tags_controller, featu
     app.add_url_rule('/mail/<mail_id>/read', methods=['POST'], view_func=mails_controller.mark_mail_as_read)
     app.add_url_rule('/mail/<mail_id>/unread', methods=['POST'], view_func=mails_controller.mark_mail_as_unread)
     app.add_url_rule('/mails/unread', methods=['POST'], view_func=mails_controller.mark_many_mail_unread)
+    app.add_url_rule('/mails/read', methods=['POST'], view_func=mails_controller.mark_many_mail_read)
     app.add_url_rule('/mail/<mail_id>', methods=['GET'], view_func=mails_controller.mail)
     app.add_url_rule('/mail/<mail_id>', methods=['DELETE'], view_func=mails_controller.delete_mail)
     app.add_url_rule('/mails', methods=['DELETE'], view_func=mails_controller.delete_mails)
diff --git a/service/pixelated/controllers/mails_controller.py b/service/pixelated/controllers/mails_controller.py
index b850d24d..ba8cbba1 100644
--- a/service/pixelated/controllers/mails_controller.py
+++ b/service/pixelated/controllers/mails_controller.py
@@ -65,6 +65,13 @@ class MailsController:
             self._search_engine.index_mail(mail)
         return ""
 
+    def mark_many_mail_read(self):
+        idents = json.loads(request.form['idents'])
+        for ident in idents:
+            mail = self._mail_service.mark_as_read(ident)
+            self._search_engine.index_mail(mail)
+        return ""
+
     def delete_mail(self, mail_id):
         trashed_mail = self._mail_service.delete_mail(mail_id)
         self._search_engine.index_mail(trashed_mail)
diff --git a/service/test/integration/mark_as_read_unread_test.py b/service/test/integration/mark_as_read_unread_test.py
index 94c23565..3f83153d 100644
--- a/service/test/integration/mark_as_read_unread_test.py
+++ b/service/test/integration/mark_as_read_unread_test.py
@@ -61,3 +61,22 @@ class MarkAsReadUnreadTest(unittest.TestCase, SoledadTestBase):
 
         self.assertNotIn('read', mails[0].status)
         self.assertNotIn('read', mails[1].status)
+
+    def test_mark_many_mails_as_read(self):
+        input_mail = MailBuilder().build_input_mail()
+        input_mail2 = MailBuilder().build_input_mail()
+
+        self.add_mail_to_inbox(input_mail)
+        self.add_mail_to_inbox(input_mail2)
+
+        mails = self.get_mails_by_tag('inbox')
+
+        self.assertNotIn('read', mails[0].status)
+        self.assertNotIn('read', mails[1].status)
+
+        self.mark_many_as_read([input_mail.ident, input_mail2.ident])
+
+        mails = self.get_mails_by_tag('inbox')
+
+        self.assertIn('read', mails[0].status)
+        self.assertIn('read', mails[1].status)
diff --git a/service/test/support/integration_helper.py b/service/test/support/integration_helper.py
index 47bc8f11..f6b4249e 100644
--- a/service/test/support/integration_helper.py
+++ b/service/test/support/integration_helper.py
@@ -211,6 +211,9 @@ class SoledadTestBase:
     def mark_many_as_unread(self, idents):
         self.client.post('/mails/unread', data={'idents': json.dumps(idents)})
 
+    def mark_many_as_read(self, idents):
+        self.client.post('/mails/read', data={'idents': json.dumps(idents)})
+
     def add_mail_to_inbox(self, input_mail):
         mail = self.mailboxes.inbox().add(input_mail)
         mail.update_tags(input_mail.tags)