[MOV] (test_)mail: move tests into their dedicated module

Purpose: avoid having useless tables for test models in a
production environment.

1 files changed, 276 insertions, 0 deletions

diff --git a/addons/test_mail/tests/test_mail_message.py b/addons/test_mail/tests/test_mail_message.py
new file mode 100644
index 00000000000..1fe8924dda6
--- /dev/null
+++ b/addons/test_mail/tests/test_mail_message.py
@@ -0,0 +1,276 @@
+# -*- coding: utf-8 -*-
+# Part of Odoo. See LICENSE file for full copyright and licensing details.
+import base64
+
+from odoo.addons.test_mail.tests import common
+from odoo.exceptions import AccessError, except_orm
+from odoo.tools import mute_logger
+
+
+class TestMessageValues(common.BaseFunctionalTest, common.MockEmails):
+
+    @classmethod
+    def setUpClass(cls):
+        super(TestMessageValues, cls).setUpClass()
+
+        cls.alias_record = cls.env['mail.test'].with_context(cls._quick_create_ctx).create({
+            'name': 'Pigs',
+            'alias_name': 'pigs',
+            'alias_contact': 'followers',
+        })
+
+        cls.Message = cls.env['mail.message'].sudo(cls.user_employee)
+
+    def test_mail_message_values_basic(self):
+        self.env['ir.config_parameter'].search([('key', '=', 'mail.catchall.domain')]).unlink()
+
+        msg = self.Message.create({
+            'reply_to': 'test.reply@example.com',
+            'email_from': 'test.from@example.com',
+        })
+        self.assertIn('-private', msg.message_id, 'mail_message: message_id for a void message should be a "private" one')
+        self.assertEqual(msg.reply_to, 'test.reply@example.com')
+        self.assertEqual(msg.email_from, 'test.from@example.com')
+
+    def test_mail_message_values_default(self):
+        self.env['ir.config_parameter'].search([('key', '=', 'mail.catchall.domain')]).unlink()
+
+        msg = self.Message.create({})
+        self.assertIn('-private', msg.message_id, 'mail_message: message_id for a void message should be a "private" one')
+        self.assertEqual(msg.reply_to, '%s <%s>' % (self.user_employee.name, self.user_employee.email))
+        self.assertEqual(msg.email_from, '%s <%s>' % (self.user_employee.name, self.user_employee.email))
+
+    @mute_logger('odoo.models.unlink')
+    def test_mail_message_values_alias(self):
+        alias_domain = 'example.com'
+        self.env['ir.config_parameter'].set_param('mail.catchall.domain', alias_domain)
+        self.env['ir.config_parameter'].search([('key', '=', 'mail.catchall.alias')]).unlink()
+
+        msg = self.Message.create({})
+        self.assertIn('-private', msg.message_id, 'mail_message: message_id for a void message should be a "private" one')
+        self.assertEqual(msg.reply_to, '%s <%s>' % (self.user_employee.name, self.user_employee.email))
+        self.assertEqual(msg.email_from, '%s <%s>' % (self.user_employee.name, self.user_employee.email))
+
+    def test_mail_message_values_alias_catchall(self):
+        alias_domain = 'example.com'
+        alias_catchall = 'pokemon'
+        self.env['ir.config_parameter'].set_param('mail.catchall.domain', alias_domain)
+        self.env['ir.config_parameter'].set_param('mail.catchall.alias', alias_catchall)
+
+        msg = self.Message.create({})
+        self.assertIn('-private', msg.message_id, 'mail_message: message_id for a void message should be a "private" one')
+        self.assertEqual(msg.reply_to, '%s <%s@%s>' % (self.env.user.company_id.name, alias_catchall, alias_domain))
+        self.assertEqual(msg.email_from, '%s <%s>' % (self.user_employee.name, self.user_employee.email))
+
+    def test_mail_message_values_document_no_alias(self):
+        self.env['ir.config_parameter'].search([('key', '=', 'mail.catchall.domain')]).unlink()
+
+        msg = self.Message.create({
+            'model': 'mail.test',
+            'res_id': self.alias_record.id
+        })
+        self.assertIn('-openerp-%d-mail.test' % self.alias_record.id, msg.message_id)
+        self.assertEqual(msg.reply_to, '%s <%s>' % (self.user_employee.name, self.user_employee.email))
+        self.assertEqual(msg.email_from, '%s <%s>' % (self.user_employee.name, self.user_employee.email))
+
+    @mute_logger('odoo.models.unlink')
+    def test_mail_message_values_document_alias(self):
+        alias_domain = 'example.com'
+        self.env['ir.config_parameter'].set_param('mail.catchall.domain', alias_domain)
+        self.env['ir.config_parameter'].search([('key', '=', 'mail.catchall.alias')]).unlink()
+
+        msg = self.Message.create({
+            'model': 'mail.test',
+            'res_id': self.alias_record.id
+        })
+        self.assertIn('-openerp-%d-mail.test' % self.alias_record.id, msg.message_id)
+        self.assertEqual(msg.reply_to, '%s %s <%s@%s>' % (self.env.user.company_id.name, self.alias_record.name, self.alias_record.alias_name, alias_domain))
+        self.assertEqual(msg.email_from, '%s <%s>' % (self.user_employee.name, self.user_employee.email))
+
+    def test_mail_message_values_document_alias_catchall(self):
+        alias_domain = 'example.com'
+        alias_catchall = 'pokemon'
+        self.env['ir.config_parameter'].set_param('mail.catchall.domain', alias_domain)
+        self.env['ir.config_parameter'].set_param('mail.catchall.alias', alias_catchall)
+
+        msg = self.Message.create({
+            'model': 'mail.test',
+            'res_id': self.alias_record.id
+        })
+        self.assertIn('-openerp-%d-mail.test' % self.alias_record.id, msg.message_id)
+        self.assertEqual(msg.reply_to, '%s %s <%s@%s>' % (self.env.user.company_id.name, self.alias_record.name, self.alias_record.alias_name, alias_domain))
+        self.assertEqual(msg.email_from, '%s <%s>' % (self.user_employee.name, self.user_employee.email))
+
+    def test_mail_message_values_no_auto_thread(self):
+        msg = self.Message.create({
+            'model': 'mail.test',
+            'res_id': self.alias_record.id,
+            'no_auto_thread': True,
+        })
+        self.assertIn('reply_to', msg.message_id)
+        self.assertNotIn('mail.test', msg.message_id)
+        self.assertNotIn('-%d-' % self.alias_record.id, msg.message_id)
+
+
+class TestMessageAccess(common.BaseFunctionalTest, common.MockEmails):
+
+    @classmethod
+    def setUpClass(cls):
+        super(TestMessageAccess, cls).setUpClass()
+
+        Users = cls.env['res.users'].with_context(cls._quick_create_user_ctx)
+        cls.user_public = Users.create({
+            'name': 'Bert Tartignole',
+            'login': 'bert',
+            'email': 'b.t@example.com',
+            'groups_id': [(6, 0, [cls.env.ref('base.group_public').id])]})
+        cls.user_portal = Users.create({
+            'name': 'Chell Gladys',
+            'login': 'chell',
+            'email': 'chell@gladys.portal',
+            'groups_id': [(6, 0, [cls.env.ref('base.group_portal').id])]})
+
+        Channel = cls.env['mail.channel'].with_context(cls._quick_create_ctx)
+        # Pigs: base group for tests
+        cls.group_pigs = Channel.create({
+            'name': 'Pigs',
+            'public': 'groups',
+            'group_public_id': cls.env.ref('base.group_user').id})
+        # Jobs: public group
+        cls.group_public = Channel.create({
+            'name': 'Jobs',
+            'description': 'NotFalse',
+            'public': 'public'})
+        # Private: private gtroup
+        cls.group_private = Channel.create({
+            'name': 'Private',
+            'public': 'private'})
+        cls.message = cls.env['mail.message'].create({
+            'body': 'My Body',
+            'model': 'mail.channel',
+            'res_id': cls.group_private.id,
+        })
+
+    @mute_logger('odoo.addons.mail.models.mail_mail')
+    def test_mail_message_access_search(self):
+        # Data: various author_ids, partner_ids, documents
+        msg1 = self.env['mail.message'].create({
+            'subject': '_Test', 'body': 'A', 'subtype_id': self.ref('mail.mt_comment')})
+        msg2 = self.env['mail.message'].create({
+            'subject': '_Test', 'body': 'A+B', 'subtype_id': self.ref('mail.mt_comment'),
+            'partner_ids': [(6, 0, [self.user_public.partner_id.id])]})
+        msg3 = self.env['mail.message'].create({
+            'subject': '_Test', 'body': 'A Pigs', 'subtype_id': False,
+            'model': 'mail.channel', 'res_id': self.group_pigs.id})
+        msg4 = self.env['mail.message'].create({
+            'subject': '_Test', 'body': 'A+P Pigs', 'subtype_id': self.ref('mail.mt_comment'),
+            'model': 'mail.channel', 'res_id': self.group_pigs.id,
+            'partner_ids': [(6, 0, [self.user_public.partner_id.id])]})
+        msg5 = self.env['mail.message'].create({
+            'subject': '_Test', 'body': 'A+E Pigs', 'subtype_id': self.ref('mail.mt_comment'),
+            'model': 'mail.channel', 'res_id': self.group_pigs.id,
+            'partner_ids': [(6, 0, [self.user_employee.partner_id.id])]})
+        msg6 = self.env['mail.message'].create({
+            'subject': '_Test', 'body': 'A Birds', 'subtype_id': self.ref('mail.mt_comment'),
+            'model': 'mail.channel', 'res_id': self.group_private.id})
+        msg7 = self.env['mail.message'].sudo(self.user_employee).create({
+            'subject': '_Test', 'body': 'B', 'subtype_id': self.ref('mail.mt_comment')})
+        msg8 = self.env['mail.message'].sudo(self.user_employee).create({
+            'subject': '_Test', 'body': 'B+E', 'subtype_id': self.ref('mail.mt_comment'),
+            'partner_ids': [(6, 0, [self.user_employee.partner_id.id])]})
+
+        # Test: Public: 2 messages (recipient)
+        messages = self.env['mail.message'].sudo(self.user_public).search([('subject', 'like', '_Test')])
+        self.assertEqual(messages, msg2 | msg4)
+
+        # Test: Employee: 3 messages on Pigs Raoul can read (employee can read group with default values)
+        messages = self.env['mail.message'].sudo(self.user_employee).search([('subject', 'like', '_Test'), ('body', 'ilike', 'A')])
+        self.assertEqual(messages, msg3 | msg4 | msg5)
+
+        # Test: Raoul: 3 messages on Pigs Raoul can read (employee can read group with default values), 0 on Birds (private group) + 2 messages as author
+        messages = self.env['mail.message'].sudo(self.user_employee).search([('subject', 'like', '_Test')])
+        self.assertEqual(messages, msg3 | msg4 | msg5 | msg7 | msg8)
+
+        # Test: Admin: all messages
+        messages = self.env['mail.message'].search([('subject', 'like', '_Test')])
+        self.assertEqual(messages, msg1 | msg2 | msg3 | msg4 | msg5 | msg6 | msg7 | msg8)
+
+        # Test: Portal: 0 (no access to groups, not recipient)
+        messages = self.env['mail.message'].sudo(self.user_portal).search([('subject', 'like', '_Test')])
+        self.assertFalse(messages)
+
+        # Test: Portal: 2 messages (public group with a subtype)
+        self.group_pigs.write({'public': 'public'})
+        messages = self.env['mail.message'].sudo(self.user_portal).search([('subject', 'like', '_Test')])
+        self.assertEqual(messages, msg4 | msg5)
+
+    @mute_logger('odoo.addons.base.models.ir_model', 'odoo.models')
+    def test_mail_message_access_read_crash(self):
+        # TODO: Change the except_orm to Warning ( Because here it's call check_access_rule
+        # which still generate exception in except_orm.So we need to change all
+        # except_orm to warning in mail module.)
+        with self.assertRaises(except_orm):
+            self.message.sudo(self.user_employee).read()
+
+    @mute_logger('odoo.models')
+    def test_mail_message_access_read_crash_portal(self):
+        with self.assertRaises(except_orm):
+            self.message.sudo(self.user_portal).read(['body', 'message_type', 'subtype_id'])
+
+    def test_mail_message_access_read_ok_portal(self):
+        self.message.write({'subtype_id': self.ref('mail.mt_comment'), 'res_id': self.group_public.id})
+        self.message.sudo(self.user_portal).read(['body', 'message_type', 'subtype_id'])
+
+    def test_mail_message_access_read_notification(self):
+        attachment = self.env['ir.attachment'].create({
+            'datas': base64.b64encode(b'My attachment'),
+            'name': 'doc.txt',
+            'datas_fname': 'doc.txt'})
+        # attach the attachment to the message
+        self.message.write({'attachment_ids': [(4, attachment.id)]})
+        self.message.write({'partner_ids': [(4, self.user_employee.partner_id.id)]})
+        self.message.sudo(self.user_employee).read()
+        # Test: Bert has access to attachment, ok because he can read message
+        attachment.sudo(self.user_employee).read(['name', 'datas'])
+
+    def test_mail_message_access_read_author(self):
+        self.message.write({'author_id': self.user_employee.partner_id.id})
+        self.message.sudo(self.user_employee).read()
+
+    def test_mail_message_access_read_doc(self):
+        self.message.write({'model': 'mail.channel', 'res_id': self.group_public.id})
+        # Test: Bert reads the message, ok because linked to a doc he is allowed to read
+        self.message.sudo(self.user_employee).read()
+
+    @mute_logger('odoo.addons.base.models.ir_model')
+    def test_mail_message_access_create_crash_public(self):
+        # Do: Bert creates a message on Pigs -> ko, no creation rights
+        with self.assertRaises(AccessError):
+            self.env['mail.message'].sudo(self.user_public).create({'model': 'mail.channel', 'res_id': self.group_pigs.id, 'body': 'Test'})
+
+        # Do: Bert create a message on Jobs -> ko, no creation rights
+        with self.assertRaises(AccessError):
+            self.env['mail.message'].sudo(self.user_public).create({'model': 'mail.channel', 'res_id': self.group_public.id, 'body': 'Test'})
+
+    @mute_logger('odoo.models')
+    def test_mail_message_access_create_crash(self):
+        # Do: Bert create a private message -> ko, no creation rights
+        with self.assertRaises(except_orm):
+            self.env['mail.message'].sudo(self.user_employee).create({'model': 'mail.channel', 'res_id': self.group_private.id, 'body': 'Test'})
+
+    @mute_logger('odoo.models')
+    def test_mail_message_access_create_doc(self):
+        # TODO Change the except_orm to Warning
+        Message = self.env['mail.message'].sudo(self.user_employee)
+        # Do: Raoul creates a message on Jobs -> ok, write access to the related document
+        Message.create({'model': 'mail.channel', 'res_id': self.group_public.id, 'body': 'Test'})
+        # Do: Raoul creates a message on Priv -> ko, no write access to the related document
+        with self.assertRaises(except_orm):
+            Message.create({'model': 'mail.channel', 'res_id': self.group_private.id, 'body': 'Test'})
+
+    def test_mail_message_access_create_private(self):
+        self.env['mail.message'].sudo(self.user_employee).create({'body': 'Test'})
+
+    def test_mail_message_access_create_reply(self):
+        self.message.write({'partner_ids': [(4, self.user_employee.partner_id.id)]})
+        self.env['mail.message'].sudo(self.user_employee).create({'model': 'mail.channel', 'res_id': self.group_private.id, 'body': 'Test', 'parent_id': self.message.id})