[MOV] mail: reorganize content of tests, separate tests from models and data

Purpose of this commit is to reorganize the mail tests. Containing

 * move test models into mail/tests/models/;
 * move data into mail/tests/data even if it is python code, as it is
   likely that some additional data will appear soon;
 * split test_mail_features into test for discuss app and test related
   to message post and composer, to better identify test target;
 * split and remove test_portal, those tests are now integrated in the
   other tests;

There is no change in this commit. Only some renaming and file move or
split.

mergeup: remove / split test portal

1 files changed, 335 insertions, 0 deletions

diff --git a/addons/mail/tests/test_message_compose.py b/addons/mail/tests/test_message_compose.py
new file mode 100644
index 00000000000..791ead4ef20
--- /dev/null
+++ b/addons/mail/tests/test_message_compose.py
@@ -0,0 +1,335 @@
+# -*- coding: utf-8 -*-
+# Part of Odoo. See LICENSE file for full copyright and licensing details.
+import base64
+
+from odoo.addons.mail.tests.common import TestMail
+from odoo.addons.mail.tests.test_mail_gateway import MAIL_TEMPLATE_PLAINTEXT
+from odoo.tools import mute_logger
+
+
+class TestMessagePost(TestMail):
+
+    @mute_logger('odoo.addons.mail.models.mail_mail')
+    def test_post_no_subscribe_author(self):
+        original = self.test_pigs.message_follower_ids
+        self.test_pigs.sudo(self.user_employee).with_context({'mail_create_nosubscribe': True}).message_post(
+            body='Test Body', message_type='comment', subtype='mt_comment')
+        self.assertEqual(self.test_pigs.message_follower_ids.mapped('partner_id'), original.mapped('partner_id'))
+        self.assertEqual(self.test_pigs.message_follower_ids.mapped('channel_id'), original.mapped('channel_id'))
+
+    # TODO : the author of a message post on mail.test should not be added as follower
+
+    # @mute_logger('odoo.addons.mail.models.mail_mail')
+    # def test_post_subscribe_author(self):
+    #     original = self.test_pigs.message_follower_ids
+    #     self.test_pigs.sudo(self.user_employee).message_post(
+    #         body='Test Body', message_type='comment', subtype='mt_comment')
+    #     self.assertEqual(self.test_pigs.message_follower_ids.mapped('partner_id'), original.mapped('partner_id') | self.user_employee.partner_id)
+    #     self.assertEqual(self.test_pigs.message_follower_ids.mapped('channel_id'), original.mapped('channel_id'))
+
+    @mute_logger('odoo.addons.mail.models.mail_mail')
+    def test_post_no_subscribe_recipients(self):
+        original = self.test_pigs.message_follower_ids
+        self.test_pigs.sudo(self.user_employee).with_context({'mail_create_nosubscribe': True}).message_post(
+            body='Test Body', message_type='comment', subtype='mt_comment', partner_ids=[(4, self.partner_1.id), (4, self.partner_2.id)])
+        self.assertEqual(self.test_pigs.message_follower_ids.mapped('partner_id'), original.mapped('partner_id'))
+        self.assertEqual(self.test_pigs.message_follower_ids.mapped('channel_id'), original.mapped('channel_id'))
+
+    @mute_logger('odoo.addons.mail.models.mail_mail')
+    def test_post_subscribe_recipients(self):
+        original = self.test_pigs.message_follower_ids
+        self.test_pigs.sudo(self.user_employee).with_context({'mail_create_nosubscribe': True, 'mail_post_autofollow': True}).message_post(
+            body='Test Body', message_type='comment', subtype='mt_comment', partner_ids=[(4, self.partner_1.id), (4, self.partner_2.id)])
+        self.assertEqual(self.test_pigs.message_follower_ids.mapped('partner_id'), original.mapped('partner_id') | self.partner_1 | self.partner_2)
+        self.assertEqual(self.test_pigs.message_follower_ids.mapped('channel_id'), original.mapped('channel_id'))
+
+    @mute_logger('odoo.addons.mail.models.mail_mail')
+    def test_post_subscribe_recipients_partial(self):
+        original = self.test_pigs.message_follower_ids
+        self.test_pigs.sudo(self.user_employee).with_context({'mail_create_nosubscribe': True, 'mail_post_autofollow': True, 'mail_post_autofollow_partner_ids': [self.partner_2.id]}).message_post(
+            body='Test Body', message_type='comment', subtype='mt_comment', partner_ids=[(4, self.partner_1.id), (4, self.partner_2.id)])
+        self.assertEqual(self.test_pigs.message_follower_ids.mapped('partner_id'), original.mapped('partner_id') | self.partner_2)
+        self.assertEqual(self.test_pigs.message_follower_ids.mapped('channel_id'), original.mapped('channel_id'))
+
+    @mute_logger('odoo.addons.mail.models.mail_mail')
+    def test_post_notifications(self):
+        _body, _body_alt = '<p>Test Body</p>', 'Test Body'
+        _subject = 'Test Subject'
+        _attachments = [
+            ('List1', b'My first attachment'),
+            ('List2', b'My second attachment')
+        ]
+        _attach_1 = self.env['ir.attachment'].sudo(self.user_employee).create({
+            'name': 'Attach1', 'datas_fname': 'Attach1',
+            'datas': 'bWlncmF0aW9uIHRlc3Q=',
+            'res_model': 'mail.compose.message', 'res_id': 0})
+        _attach_2 = self.env['ir.attachment'].sudo(self.user_employee).create({
+            'name': 'Attach2', 'datas_fname': 'Attach2',
+            'datas': 'bWlncmF0aW9uIHRlc3Q=',
+            'res_model': 'mail.compose.message', 'res_id': 0})
+
+        self.user_admin.write({'notification_type': 'email'})
+
+        # subscribe second employee to the group to test notifications
+        self.test_pigs.message_subscribe_users(user_ids=[self.env.user.id])
+
+        # use aliases
+        _domain = 'schlouby.fr'
+        _catchall = 'test_catchall'
+        self.env['ir.config_parameter'].set_param('mail.catchall.domain', _domain)
+        self.env['ir.config_parameter'].set_param('mail.catchall.alias', _catchall)
+
+        msg = self.test_pigs.sudo(self.user_employee).message_post(
+            body=_body, subject=_subject, partner_ids=[self.partner_1.id, self.partner_2.id],
+            attachment_ids=[_attach_1.id, _attach_2.id], attachments=_attachments,
+            message_type='comment', subtype='mt_comment')
+
+        # message content
+        self.assertEqual(msg.subject, _subject)
+        self.assertEqual(msg.body, _body)
+        self.assertEqual(msg.partner_ids, self.partner_1 | self.partner_2)
+        self.assertEqual(msg.needaction_partner_ids, self.env.user.partner_id | self.partner_1 | self.partner_2)
+        self.assertEqual(msg.channel_ids, self.env['mail.channel'])
+
+        # attachments
+        self.assertEqual(set(msg.attachment_ids.mapped('res_model')), set(['mail.test']),
+                         'message_post: all atttachments should be linked to the mail.test model')
+        self.assertEqual(set(msg.attachment_ids.mapped('res_id')), set([self.test_pigs.id]),
+                         'message_post: all atttachments should be linked to the pigs group')
+        self.assertEqual(set([base64.b64decode(x) for x in msg.attachment_ids.mapped('datas')]),
+                         set([b'migration test', _attachments[0][1], _attachments[1][1]]))
+        self.assertTrue(set([_attach_1.id, _attach_2.id]).issubset(msg.attachment_ids.ids),
+                        'message_post: mail.message attachments duplicated')
+        # notifications
+        self.assertFalse(self.env['mail.mail'].search([('mail_message_id', '=', msg.message_id)]),
+                         'message_post: mail.mail notifications should have been auto-deleted')
+
+        # notification emails: followers + recipients - author (user_employee)
+        self.assertEqual(set(m['email_from'] for m in self._mails),
+                         set(['%s <%s>' % (self.user_employee.name, self.user_employee.email)]),
+                         'message_post: notification email wrong email_from: should use sender email')
+        self.assertEqual(set(m['email_to'][0] for m in self._mails),
+                         set(['%s <%s>' % (self.partner_1.name, self.partner_1.email),
+                              '%s <%s>' % (self.partner_2.name, self.partner_2.email),
+                              '%s <%s>' % (self.env.user.name, self.env.user.email)]))
+        self.assertFalse(any(len(m['email_to']) != 1 for m in self._mails),
+                         'message_post: notification email should be sent to one partner at a time')
+        self.assertEqual(set(m['reply_to'] for m in self._mails),
+                         set(['%s %s <%s@%s>' % (self.env.user.company_id.name, self.test_pigs.name, self.test_pigs.alias_name, _domain)]),
+                         'message_post: notification email should use group aliases and data for reply to')
+        self.assertTrue(all(_subject in m['subject'] for m in self._mails))
+        self.assertTrue(all(_body in m['body'] for m in self._mails))
+        self.assertTrue(all(_body_alt in m['body'] for m in self._mails))
+        self.assertFalse(any(m['references'] for m in self._mails))
+
+    @mute_logger('odoo.addons.mail.models.mail_mail')
+    def test_post_answer(self):
+        _body = '<p>Test Body</p>'
+        _subject = 'Test Subject'
+
+        # use aliases
+        _domain = 'schlouby.fr'
+        _catchall = 'test_catchall'
+        self.env['ir.config_parameter'].set_param('mail.catchall.domain', _domain)
+        self.env['ir.config_parameter'].set_param('mail.catchall.alias', _catchall)
+
+        parent_msg = self.test_pigs.sudo(self.user_employee).message_post(
+            body=_body, subject=_subject,
+            message_type='comment', subtype='mt_comment')
+
+        self.assertEqual(parent_msg.partner_ids, self.env['res.partner'])
+
+        msg = self.test_pigs.sudo(self.user_employee).message_post(
+            body=_body, subject=_subject, partner_ids=[self.partner_1.id],
+            message_type='comment', subtype='mt_comment', parent_id=parent_msg.id)
+
+        self.assertEqual(msg.parent_id.id, parent_msg.id)
+        self.assertEqual(msg.partner_ids, self.partner_1)
+        # self.assertEqual(parent_msg.partner_ids, self.partner_1)  # TDE FIXME: to check
+        self.assertTrue(all('openerp-%d-mail.test' % self.test_pigs.id in m['references'] for m in self._mails))
+        new_msg = self.test_pigs.sudo(self.user_employee).message_post(
+            body=_body, subject=_subject,
+            message_type='comment', subtype='mt_comment', parent_id=msg.id)
+
+        self.assertEqual(new_msg.parent_id.id, parent_msg.id, 'message_post: flatten error')
+        self.assertFalse(new_msg.partner_ids)
+
+    def test_post_portal(self):
+        self.test_pigs.message_subscribe((self.partner_1 | self.user_employee.partner_id).ids)
+        new_msg = self.test_pigs.sudo(self.user_portal).message_post(
+            body='<p>Test</p>', subject='Subject',
+            message_type='comment', subtype='mt_comment')
+        self.assertEqual(new_msg.sudo().needaction_partner_ids, (self.partner_1 | self.user_employee.partner_id))
+
+        self.assertEqual(
+            set(m['email_to'][0] for m in self._mails),
+            set(['%s <%s>' % (self.partner_1.name, self.partner_1.email),
+                 '%s <%s>' % (self.user_employee.name, self.user_employee.email)]))
+
+    @mute_logger('odoo.addons.mail.models.mail_mail')
+    def test_post_internal(self):
+        self.test_pigs.message_subscribe_users([self.user_admin.id])
+        msg = self.test_pigs.sudo(self.user_employee).message_post(
+            body='My Body', subject='My Subject',
+            message_type='comment', subtype='mt_note')
+        self.assertEqual(msg.partner_ids, self.env['res.partner'])
+        self.assertEqual(msg.needaction_partner_ids, self.env['res.partner'])
+        self.assertEqual(self.test_pigs.message_ids, msg)
+
+        self.format_and_process(
+            MAIL_TEMPLATE_PLAINTEXT,
+            email_from=self.user_admin.email,
+            msg_id='<1198923581.41972151344608186800.JavaMail.diff1@agrolait.com>',
+            to='not_my_businesss@example.com',
+            extra='In-Reply-To:\r\n\t%s\n' % msg.message_id)
+        reply = self.test_pigs.message_ids - msg
+        self.assertTrue(reply)
+        self.assertEqual(reply.subtype_id, self.env.ref('mail.mt_note'))
+        self.assertEqual(reply.needaction_partner_ids, self.user_employee.partner_id)
+
+    @mute_logger('odoo.addons.mail.models.mail_mail')
+    def test_message_compose(self):
+        composer = self.env['mail.compose.message'].with_context({
+            'default_composition_mode': 'comment',
+            'default_model': 'mail.test',
+            'default_res_id': self.test_pigs.id,
+        }).sudo(self.user_employee).create({
+            'body': '<p>Test Body</p>',
+            'partner_ids': [(4, self.partner_1.id), (4, self.partner_2.id)]
+        })
+        self.assertEqual(composer.composition_mode,  'comment')
+        self.assertEqual(composer.model, 'mail.test')
+        self.assertEqual(composer.subject, 'Re: %s' % self.test_pigs.name)
+        self.assertEqual(composer.record_name, self.test_pigs.name)
+
+        composer.send_mail()
+        message = self.test_pigs.message_ids[0]
+
+        composer = self.env['mail.compose.message'].with_context({
+            'default_composition_mode': 'comment',
+            'default_res_id': self.test_pigs.id,
+            'default_parent_id': message.id
+        }).sudo(self.user_employee).create({})
+
+        self.assertEqual(composer.model, 'mail.test')
+        self.assertEqual(composer.res_id, self.test_pigs.id)
+        self.assertEqual(composer.parent_id, message)
+        self.assertEqual(composer.subject, 'Re: %s' % self.test_pigs.name)
+
+        # TODO: test attachments ?
+
+    def test_mail_compose_access_rights(self):
+        test_channel = self.env['mail.channel'].create({
+            'name': 'Pigs',
+            'public': 'groups',
+            'group_public_id': self.env.ref('base.group_portal').id})
+        port_msg = test_channel.message_post(body='Message')
+
+        # Do: Chell comments Pigs, ok because can write on it (public group)
+        test_channel.sudo(self.user_portal).message_post(body='I love Pigs', message_type='comment', subtype='mail.mt_comment')
+        # Do: Chell creates a mail.compose.message record on Pigs, because he uses the wizard
+        compose = self.env['mail.compose.message'].with_context({
+            'default_composition_mode': 'comment',
+            'default_model': 'mail.channel',
+            'default_res_id': test_channel.id
+        }).sudo(self.user_portal).create({
+            'subject': 'Subject',
+            'body': 'Body text',
+            'partner_ids': []})
+        compose.send_mail()
+
+        # Do: Chell replies to a Pigs message using the composer
+        compose = self.env['mail.compose.message'].with_context({
+            'default_composition_mode': 'comment',
+            'default_parent_id': port_msg.id
+        }).sudo(self.user_portal).create({
+            'subject': 'Subject',
+            'body': 'Body text'})
+        compose.send_mail()
+
+    @mute_logger('odoo.addons.mail.models.mail_mail')
+    def test_message_compose_mass_mail(self):
+        composer = self.env['mail.compose.message'].with_context({
+            'default_composition_mode': 'mass_mail',
+            'default_model': 'mail.test',
+            'default_res_id': False,
+            'active_ids': [self.test_pigs.id, self.test_public.id]
+        }).sudo(self.user_employee).create({
+            'subject': 'Testing ${object.name}',
+            'body': '<p>${object.description}</p>',
+            'partner_ids': [(4, self.partner_1.id), (4, self.partner_2.id)]
+        })
+        composer.with_context({
+            'default_res_id': -1,
+            'active_ids': [self.test_pigs.id, self.test_public.id]
+        }).send_mail()
+
+        # check mail_mail
+        mails = self.env['mail.mail'].search([('subject', 'ilike', 'Testing')])
+        for mail in mails:
+            self.assertEqual(mail.recipient_ids, self.partner_1 | self.partner_2,
+                             'compose wizard: mail_mail mass mailing: mail.mail in mass mail incorrect recipients')
+
+        # check message on test_pigs
+        message1 = self.test_pigs.message_ids[0]
+        self.assertEqual(message1.subject, 'Testing %s' % self.test_pigs.name)
+        self.assertEqual(message1.body, '<p>%s</p>' % self.test_pigs.description)
+
+        # check message on test_public
+        message1 = self.test_public.message_ids[0]
+        self.assertEqual(message1.subject, 'Testing %s' % self.test_public.name)
+        self.assertEqual(message1.body, '<p>%s</p>' % self.test_public.description)
+
+        # # Test: Pigs and Bird did receive their message
+        # # check logged messages
+        # message1 = test_pigs.message_ids[0]
+        # message2 = group_bird.message_ids[0]
+        # # Test: Pigs and Bird did receive their message
+        # messages = self.MailMessage.search([], limit=2)
+        # mail = self.MailMail.search([('mail_message_id', '=', message2.id)], limit=1)
+        # self.assertTrue(mail, 'message_send: mail.mail message should have in processing mail queue')
+        # #check mass mail state...
+        # mails = self.MailMail.search([('state', '=', 'exception')])
+        # self.assertNotIn(mail.id, mails.ids, 'compose wizard: Mail sending Failed!!')
+        # self.assertIn(message1.id, messages.ids, 'compose wizard: Pigs did not receive its mass mailing message')
+        # self.assertIn(message2.id, messages.ids, 'compose wizard: Bird did not receive its mass mailing message')
+
+        # check followers ?
+        # Test: mail.test followers: author not added as follower in mass mail mode
+        # self.assertEqual(set(test_pigs.message_follower_ids.ids), set([self.partner_admin_id, p_b.id, p_c.id, p_d.id]),
+        #                 'compose wizard: mail_post_autofollow and mail_create_nosubscribe context keys not correctly taken into account')
+        # self.assertEqual(set(group_bird.message_follower_ids.ids), set([self.partner_admin_id]),
+        #                 'compose wizard: mail_post_autofollow and mail_create_nosubscribe context keys not correctly taken into account')
+
+    @mute_logger('odoo.addons.mail.models.mail_mail')
+    def test_message_compose_mass_mail_active_domain(self):
+        self.env['mail.compose.message'].with_context({
+            'default_composition_mode': 'mass_mail',
+            'default_model': 'mail.test',
+            'default_use_active_domain': True,
+            'active_ids': [self.test_pigs.id],
+            'active_domain': [('name', 'in', ['%s' % self.test_pigs.name, '%s' % self.test_public.name])],
+        }).sudo(self.user_employee).create({
+            'subject': 'From Composer Test',
+            'body': '${object.description}',
+        }).send_mail()
+
+        self.assertEqual(self.test_pigs.message_ids[0].subject, 'From Composer Test')
+        self.assertEqual(self.test_public.message_ids[0].subject, 'From Composer Test')
+
+
+    @mute_logger('odoo.addons.mail.models.mail_mail')
+    def test_message_compose_mass_mail_no_active_domain(self):
+        self.env['mail.compose.message'].with_context({
+            'default_composition_mode': 'mass_mail',
+            'default_model': 'mail.test',
+            'default_use_active_domain': False,
+            'active_ids': [self.test_pigs.id],
+            'active_domain': [('name', 'in', ['%s' % self.test_pigs.name, '%s' % self.test_public.name])],
+        }).sudo(self.user_employee).create({
+            'subject': 'From Composer Test',
+            'body': '${object.description}',
+        }).send_mail()
+
+        self.assertEqual(self.test_pigs.message_ids[0].subject, 'From Composer Test')
+        self.assertFalse(self.test_public.message_ids.ids)