diff --git a/addons/test_mail/tests/test_mail_followers.py b/addons/test_mail/tests/test_mail_followers.py index ac64cd377fd..5670869f928 100644 --- a/addons/test_mail/tests/test_mail_followers.py +++ b/addons/test_mail/tests/test_mail_followers.py @@ -3,8 +3,8 @@ from odoo.addons.test_mail.tests.common import TestMailCommon from odoo.exceptions import AccessError -from odoo.tests import tagged -from odoo.tests import users +from odoo.tests import tagged, users +from odoo.tools import mute_logger @tagged('mail_followers') @@ -145,6 +145,7 @@ class BaseFollowersTest(TestMailCommon): self.assertEqual(document.message_follower_ids.partner_id, self.partner_portal | customer) @users('employee') + @mute_logger('odoo.models.unlink') def test_followers_inverse_message_partner(self): test_record = self.test_record.with_env(self.env) partner0, partner1, partner2, partner3 = self.env['res.partner'].create( @@ -183,6 +184,7 @@ class BaseFollowersTest(TestMailCommon): test_record.write({'message_partner_ids': [(4, partner0.id), (4, partner1.id)]}) self.assertEqual(test_record.message_follower_ids.partner_id, partner1) + @mute_logger('odoo.addons.base.models.ir_model', 'odoo.models') def test_followers_inverse_message_partner_access_rights(self): """ Make sure we're not bypassing security checks by setting a partner instead of a follower """ @@ -301,6 +303,7 @@ class AdvancedFollowersTest(TestMailCommon): """ Creator of records are automatically added as followers """ self.assertEqual(self.test_track.message_partner_ids, self.user_employee.partner_id) + @mute_logger('odoo.models.unlink') def test_auto_subscribe_inactive(self): """ Test inactive are not added as followers in automated subscription """ self.test_track.user_id = False @@ -317,6 +320,14 @@ class AdvancedFollowersTest(TestMailCommon): self.assertEqual(self.test_track.message_partner_ids, self.user_employee.partner_id) self.assertEqual(self.test_track.message_follower_ids.partner_id, self.user_employee.partner_id) + new_record = self.env['mail.test.track'].with_user(self.user_admin).create({ + 'name': 'Test', + }) + self.assertFalse(new_record.message_partner_ids, + 'Filters out inactive partners') + self.assertEqual(new_record.message_follower_ids.partner_id, self.partner_admin, + 'TODO: should not subscribe inactive partner') + def test_auto_subscribe_post(self): """ People posting a message are automatically added as followers """ self.test_track.with_user(self.user_admin).message_post(body='Coucou hibou', message_type='comment') @@ -340,6 +351,7 @@ class AdvancedFollowersTest(TestMailCommon): }) self.assertEqual(sub.message_partner_ids, (self.user_employee.partner_id | self.user_admin.partner_id)) + @mute_logger('odoo.models.unlink') def test_auto_subscribe_defaults(self): """ Test auto subscription based on an container record. This mimics the behavior of addons like project and task where subscribing to @@ -573,6 +585,7 @@ class RecipientsNotificationTest(TestMailCommon): partner_to_users={self.common_partner.id: self.user_2}) @users('employee') + @mute_logger('odoo.models.unlink') def test_notification_unlink(self): """ Check that we unlink the created user_notification after unlinked the related document. """