From 4ea673bfd84546b13a831772fd0bd9db66b9eca7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Thibault=20Delavall=C3=A9e?= <tde@odoo.com>
Date: Mon, 8 Nov 2021 14:38:58 +0000
Subject: [PATCH] [IMP] digest: add tests about tone down and unsubscribe

Purpose is to ensure behavior of digests and prepare future fixes and
improvements. Tone down and unsubscribe links are currently not tested.
This commit fixes that by adding relevant tests.

Task-2641394 (Digest emails sending improvement)
Task-2582128 (Digest onbarding and usage improvement)

X-original-commit: f340859a3508ebb369b7116f64c81d8f03774e61
Part-of: odoo/odoo#79877
---
 addons/digest/tests/test_digest.py | 96 +++++++++++++++++++++++++++++-
 1 file changed, 95 insertions(+), 1 deletion(-)

diff --git a/addons/digest/tests/test_digest.py b/addons/digest/tests/test_digest.py
index 30cf7ea75eb..193d0b6c95c 100644
--- a/addons/digest/tests/test_digest.py
+++ b/addons/digest/tests/test_digest.py
@@ -6,9 +6,12 @@ import random
 
 from dateutil.relativedelta import relativedelta
 from lxml import html
+from werkzeug.urls import url_encode
 
-from odoo import fields
+from odoo import fields, SUPERUSER_ID
+from odoo.addons.base.tests.common import HttpCaseWithUserDemo
 from odoo.addons.mail.tests import common as mail_test
+from odoo.tests import tagged
 from odoo.tests.common import users
 
 
@@ -109,3 +112,94 @@ class TestDigest(mail_test.MailCommon):
             "check the user was subscribed as action_subscribe will silently "
             "ignore subs of non-employees"
         )
+
+    @users('admin')
+    def test_digest_tone_down(self):
+        digest = self.env['digest.digest'].browse(self.test_digest.ids)
+        digest._action_subscribe_users(self.user_employee)
+
+        # initial data
+        self.assertEqual(digest.periodicity, 'daily')
+
+        # no logs for employee -> should tone down periodicity
+        digest.flush()
+        with self.mock_mail_gateway():
+            digest.action_send()
+
+        self.assertEqual(digest.periodicity, 'weekly')
+
+    @users('admin')
+    def test_digest_tone_down_wlogs(self):
+        digest = self.env['digest.digest'].browse(self.test_digest.ids)
+        digest._action_subscribe_users(self.user_employee)
+
+        # initial data
+        self.assertEqual(digest.periodicity, 'daily')
+
+        # logs for employee -> should not tone down
+        self.env['res.users.log'].with_user(SUPERUSER_ID).create({'create_uid': self.user_employee.id})
+        digest.flush()
+        with self.mock_mail_gateway():
+            digest.action_send()
+
+        self.assertEqual(digest.periodicity, 'daily')
+
+
+@tagged('-at_install', 'post_install')
+class TestUnsubscribe(HttpCaseWithUserDemo):
+
+    def setUp(self):
+        super(TestUnsubscribe, self).setUp()
+
+        self.test_digest = self.env['digest.digest'].create({
+            'kpi_mail_message_total': True,
+            'kpi_res_users_connected': True,
+            'name': "My Digest",
+            'periodicity': 'daily',
+            'user_ids': self.user_demo.ids,
+        })
+        self.test_digest._action_subscribe_users(self.user_demo)
+        self.base_url = self.test_digest.get_base_url()
+
+    @users('demo')
+    def test_unsubscribe_classic(self):
+        self.assertIn(self.user_demo, self.test_digest.user_ids)
+        self.authenticate(self.env.user.login, self.env.user.login)
+
+        response = self._url_unsubscribe()
+        self.assertEqual(response.status_code, 200)
+        self.assertNotIn(self.user_demo, self.test_digest.user_ids)
+
+    @users('demo')
+    def test_unsubscribe_issues(self):
+        """ Test when not being member """
+        self.test_digest.write({'user_ids': [(3, self.user_demo.id)]})
+        self.assertNotIn(self.user_demo, self.test_digest.user_ids)
+
+        # unsubscribe
+        self.authenticate(self.env.user.login, self.env.user.login)
+        response = self._url_unsubscribe()
+        self.assertEqual(response.status_code, 200)
+        self.assertNotIn(self.user_demo, self.test_digest.user_ids)
+
+    def test_unsubscribe_public(self):
+        """ Check public users are redirected when trying to catch unsubscribe
+        route. """
+        self.authenticate(None, None)
+
+        response = self._url_unsubscribe()
+        self.assertIn('web/login?redirect', response.url)
+
+    def _url_unsubscribe(self, token=None, user_id=None):
+        url_params = {}
+        if token is not None:
+            url_params['token'] = token
+        if user_id is not None:
+            url_params['user_id'] = user_id
+
+        url = "%s/digest/%s/unsubscribe?%s" % (
+            self.base_url,
+            self.test_digest.id,
+            url_encode(url_params)
+        )
+        return self.url_open(url)