[REF] test_mail: rename test models to ease understanding

PURPOSE

Have a cleaner test_mail addons

SPECIFICATIONS

Keep only mail-related tests, move odoobot in test mail full, send "update
notification" tests in mail (specific to mail). Merge some test files to
lessen number of files, perform light file renaming.

Split test mail models file to prepare some cleaning in those models and tests.

LINKS

Prepares Task ID 2238597 (clean notification models)
Prepares Task ID 2083854 (improve mass mailing technical flows)
PR #49891

1 files changed, 34 insertions, 34 deletions

diff --git a/addons/test_mail/tests/test_performance.py b/addons/test_mail/tests/test_performance.py
index 1d8ab7729c9..c6c4d8b0527 100644
--- a/addons/test_mail/tests/test_performance.py
+++ b/addons/test_mail/tests/test_performance.py
@@ -54,7 +54,7 @@ class TestMailPerformance(BaseMailPerformance):
             'name': 'Azure Interior',
             'email': 'azure.Interior24@example.com',
         })
-        self.env['test_performance.mail'].create([
+        self.env['mail.performance.thread'].create([
             {
                 'name': 'Object 0',
                 'value': 0,
@@ -82,7 +82,7 @@ class TestMailPerformance(BaseMailPerformance):
     @warmup
     def test_read_mail(self):
         """ Read records inheriting from 'mail.thread'. """
-        records = self.env['test_performance.mail'].search([])
+        records = self.env['mail.performance.thread'].search([])
         self.assertEqual(len(records), 5)
 
         with self.assertQueryCount(__system__=2, demo=2):
@@ -104,7 +104,7 @@ class TestMailPerformance(BaseMailPerformance):
     @warmup
     def test_write_mail(self):
         """ Write records inheriting from 'mail.thread' (no recomputation). """
-        records = self.env['test_performance.mail'].search([])
+        records = self.env['mail.performance.thread'].search([])
         self.assertEqual(len(records), 5)
 
         with self.assertQueryCount(__system__=2, demo=2):
@@ -114,7 +114,7 @@ class TestMailPerformance(BaseMailPerformance):
     @warmup
     def test_write_mail_with_recomputation(self):
         """ Write records inheriting from 'mail.thread' (with recomputation). """
-        records = self.env['test_performance.mail'].search([])
+        records = self.env['mail.performance.thread'].search([])
         self.assertEqual(len(records), 5)
 
         with self.assertQueryCount(__system__=2, demo=2):
@@ -124,7 +124,7 @@ class TestMailPerformance(BaseMailPerformance):
     @warmup
     def test_write_mail_with_tracking(self):
         """ Write records inheriting from 'mail.thread' (with field tracking). """
-        record = self.env['test_performance.mail'].create({
+        record = self.env['mail.performance.thread'].create({
             'name': 'Test',
             'track': 'Y',
             'value': 40,
@@ -138,7 +138,7 @@ class TestMailPerformance(BaseMailPerformance):
     @warmup
     def test_create_mail(self):
         """ Create records inheriting from 'mail.thread' (without field tracking). """
-        model = self.env['test_performance.mail']
+        model = self.env['mail.performance.thread']
 
         with self.assertQueryCount(__system__=2, demo=2):
             model.with_context(tracking_disable=True).create({'name': 'X'})
@@ -148,7 +148,7 @@ class TestMailPerformance(BaseMailPerformance):
     def test_create_mail_with_tracking(self):
         """ Create records inheriting from 'mail.thread' (with field tracking). """
         with self.assertQueryCount(__system__=7, demo=7):
-            self.env['test_performance.mail'].create({'name': 'X'})
+            self.env['mail.performance.thread'].create({'name': 'X'})
 
     @users('__system__', 'emp')
     @warmup
@@ -188,7 +188,7 @@ class TestMailAPIPerformance(BaseMailPerformance):
         self.env.ref('mail.mt_activities').write({'default': True})
 
     def _create_test_records(self):
-        self.test_record_full = self.env['mail.test.full'].with_context(self._quick_create_ctx).create({
+        self.test_record_full = self.env['mail.test.ticket'].with_context(self._quick_create_ctx).create({
             'name': 'TestRecord',
             'customer_id': self.customer.id,
             'user_id': self.user_test.id,
@@ -196,7 +196,7 @@ class TestMailAPIPerformance(BaseMailPerformance):
         })
         self.test_template_full = self.env['mail.template'].create({
             'name': 'TestTemplate',
-            'model_id': self.env['ir.model']._get('mail.test.full').id,
+            'model_id': self.env['ir.model']._get('mail.test.ticket').id,
             'subject': 'About ${object.name}',
             'body_html': '<p>Hello ${object.name}</p>',
             'email_from': '${object.user_id.email_formatted | safe}',
@@ -258,7 +258,7 @@ class TestMailAPIPerformance(BaseMailPerformance):
     @mute_logger('odoo.tests', 'odoo.addons.mail.models.mail_mail')
     def test_mail_composer(self):
         self._create_test_records()
-        test_record = self.env['mail.test.full'].browse(self.test_record_full.id)
+        test_record = self.env['mail.test.ticket'].browse(self.test_record_full.id)
         customer_id = self.customer.id
         with self.assertQueryCount(__system__=4, emp=4):
             composer = self.env['mail.compose.message'].with_context({
@@ -278,7 +278,7 @@ class TestMailAPIPerformance(BaseMailPerformance):
     @mute_logger('odoo.tests', 'odoo.addons.mail.models.mail_mail', 'odoo.models.unlink')
     def test_mail_composer_w_template(self):
         self._create_test_records()
-        test_record = self.env['mail.test.full'].browse(self.test_record_full.id)
+        test_record = self.env['mail.test.ticket'].browse(self.test_record_full.id)
         test_template = self.env['mail.template'].browse(self.test_template_full.id)
         # TODO FIXME non deterministic, last check 25 Mar 2020. Runbot at most +7 compared to local.
         with self.assertQueryCount(__system__=28, emp=29):
@@ -403,7 +403,7 @@ class TestMailAPIPerformance(BaseMailPerformance):
     @users('__system__', 'emp')
     @warmup
     def test_message_track(self):
-        record = self.env['mail.test.tracking'].create({'name': 'Zizizatestname'})
+        record = self.env['mail.performance.tracking'].create({'name': 'Zizizatestname'})
         with self.assertQueryCount(__system__=3, emp=3):
             record.write({'name': 'Zizizanewtestname'})
             record.flush()
@@ -447,7 +447,7 @@ class TestMailComplexPerformance(BaseMailPerformance):
             'name': 'Test Customer',
             'email': 'test@example.com'
         })
-        self.umbrella = self.env['mail.test'].with_context(mail_create_nosubscribe=True).create({
+        self.umbrella = self.env['mail.test.container'].with_context(mail_create_nosubscribe=True).create({
             'name': 'Test Umbrella',
             'customer_id': self.customer.id,
             'alias_name': 'test-alias',
@@ -458,7 +458,7 @@ class TestMailComplexPerformance(BaseMailPerformance):
             self.partners |= Partners.create({'name': 'Test %s' % x, 'email': 'test%s@example.com' % x})
         self.umbrella.message_subscribe(self.partners.ids, subtype_ids=[
             self.env.ref('mail.mt_comment').id,
-            self.env.ref('test_mail.st_mail_test_child_full').id]
+            self.env.ref('test_mail.st_mail_test_container_child_full').id]
         )
         # `test_complex_mail_mail_send`
         self.umbrella.flush()
@@ -472,7 +472,7 @@ class TestMailComplexPerformance(BaseMailPerformance):
             'body': '<p>Test</p>',
             'author_id': self.env.user.partner_id.id,
             'email_from': self.env.user.partner_id.email,
-            'model': 'mail.test',
+            'model': 'mail.test.container',
             'res_id': self.umbrella.id,
         })
         mail = self.env['mail.mail'].sudo().create({
@@ -509,7 +509,7 @@ class TestMailComplexPerformance(BaseMailPerformance):
     def test_complex_message_post_template(self):
         self.umbrella.message_subscribe(self.user_portal.partner_id.ids)
         record = self.umbrella.with_user(self.env.user)
-        template_id = self.env.ref('test_mail.mail_test_tpl').id
+        template_id = self.env.ref('test_mail.mail_test_container_tpl').id
 
         with self.assertQueryCount(__system__=78, emp=80):
             record.message_post_with_template(template_id, message_type='comment', composition_mode='comment')
@@ -523,9 +523,9 @@ class TestMailComplexPerformance(BaseMailPerformance):
     def test_complex_message_subscribe(self):
         pids = self.partners.ids
         cids = self.channel.ids
-        subtypes = self.env.ref('mail.mt_comment') | self.env.ref('test_mail.st_mail_test_full_umbrella_upd')
+        subtypes = self.env.ref('mail.mt_comment') | self.env.ref('test_mail.st_mail_test_ticket_container_upd')
         subtype_ids = subtypes.ids
-        rec = self.env['mail.test.full'].create({
+        rec = self.env['mail.test.ticket'].create({
             'name': 'Test',
             'umbrella_id': False,
             'customer_id': False,
@@ -574,7 +574,7 @@ class TestMailComplexPerformance(BaseMailPerformance):
     @warmup
     def test_complex_tracking_assignation(self):
         """ Assignation performance test on already-created record """
-        rec = self.env['mail.test.full'].create({
+        rec = self.env['mail.test.ticket'].create({
             'name': 'Test',
             'umbrella_id': self.umbrella.id,
             'customer_id': self.customer.id,
@@ -589,7 +589,7 @@ class TestMailComplexPerformance(BaseMailPerformance):
         self.assertEqual(rec1.message_ids[0].subtype_id, self.env.ref('mail.mt_note'))
         self.assertEqual(rec1.message_ids[0].notified_partner_ids, self.env['res.partner'])
         # creation message
-        self.assertEqual(rec1.message_ids[1].subtype_id, self.env.ref('test_mail.st_mail_test_full_umbrella_upd'))
+        self.assertEqual(rec1.message_ids[1].subtype_id, self.env.ref('test_mail.st_mail_test_ticket_container_upd'))
         self.assertEqual(rec1.message_ids[1].notified_partner_ids, self.partners)
         self.assertEqual(len(rec1.message_ids), 2)
 
@@ -603,7 +603,7 @@ class TestMailComplexPerformance(BaseMailPerformance):
         user_id = self.user_portal.id
 
         with self.assertQueryCount(__system__=116, emp=117):
-            rec = self.env['mail.test.full'].create({
+            rec = self.env['mail.test.ticket'].create({
                 'name': 'Test',
                 'umbrella_id': umbrella_id,
                 'customer_id': customer_id,
@@ -613,7 +613,7 @@ class TestMailComplexPerformance(BaseMailPerformance):
         rec1 = rec.with_context(active_test=False)      # to see inactive records
         self.assertEqual(rec1.message_partner_ids, self.partners | self.env.user.partner_id | self.user_portal.partner_id)
         # creation message
-        self.assertEqual(rec1.message_ids[0].subtype_id, self.env.ref('test_mail.st_mail_test_full_umbrella_upd'))
+        self.assertEqual(rec1.message_ids[0].subtype_id, self.env.ref('test_mail.st_mail_test_ticket_container_upd'))
         self.assertEqual(rec1.message_ids[0].notified_partner_ids, self.partners | self.user_portal.partner_id)
         self.assertEqual(len(rec1.message_ids), 1)
 
@@ -622,7 +622,7 @@ class TestMailComplexPerformance(BaseMailPerformance):
     @warmup
     def test_complex_tracking_subscription_subtype(self):
         """ Write performance test involving auto subscription, tracking with subtype """
-        rec = self.env['mail.test.full'].create({
+        rec = self.env['mail.test.ticket'].create({
             'name': 'Test',
             'umbrella_id': False,
             'customer_id': False,
@@ -639,7 +639,7 @@ class TestMailComplexPerformance(BaseMailPerformance):
 
         self.assertEqual(rec1.message_partner_ids, self.partners | self.env.user.partner_id | self.user_portal.partner_id)
         # write tracking message
-        self.assertEqual(rec1.message_ids[0].subtype_id, self.env.ref('test_mail.st_mail_test_full_umbrella_upd'))
+        self.assertEqual(rec1.message_ids[0].subtype_id, self.env.ref('test_mail.st_mail_test_ticket_container_upd'))
         self.assertEqual(rec1.message_ids[0].notified_partner_ids, self.partners | self.user_portal.partner_id)
         # creation message
         self.assertEqual(rec1.message_ids[1].subtype_id, self.env.ref('mail.mt_note'))
@@ -653,13 +653,13 @@ class TestMailComplexPerformance(BaseMailPerformance):
         """ Write performance test involving auto subscription, tracking with subtype and template send """
         umbrella_id = self.umbrella.id
         customer_id = self.customer.id
-        umbrella2 = self.env['mail.test'].with_context(mail_create_nosubscribe=True).create({
+        umbrella2 = self.env['mail.test.container'].with_context(mail_create_nosubscribe=True).create({
             'name': 'Test Umbrella 2',
             'customer_id': False,
             'alias_name': False,
         })
 
-        rec = self.env['mail.test.full'].create({
+        rec = self.env['mail.test.ticket'].create({
             'name': 'Test',
             'umbrella_id': umbrella2.id,
             'customer_id': False,
@@ -677,10 +677,10 @@ class TestMailComplexPerformance(BaseMailPerformance):
 
         self.assertEqual(rec1.message_partner_ids, self.partners | self.env.user.partner_id | self.user_portal.partner_id)
         # write tracking message
-        self.assertEqual(rec1.message_ids[0].subtype_id, self.env.ref('test_mail.st_mail_test_full_umbrella_upd'))
+        self.assertEqual(rec1.message_ids[0].subtype_id, self.env.ref('test_mail.st_mail_test_ticket_container_upd'))
         self.assertEqual(rec1.message_ids[0].notified_partner_ids, self.partners | self.user_portal.partner_id)
         # creation message
-        self.assertEqual(rec1.message_ids[1].subtype_id, self.env.ref('test_mail.st_mail_test_full_umbrella_upd'))
+        self.assertEqual(rec1.message_ids[1].subtype_id, self.env.ref('test_mail.st_mail_test_ticket_container_upd'))
         self.assertEqual(rec1.message_ids[1].notified_partner_ids, self.user_portal.partner_id)
         self.assertEqual(len(rec1.message_ids), 2)
 
@@ -691,12 +691,12 @@ class TestMailComplexPerformance(BaseMailPerformance):
         """ Write performance test involving assignation, tracking with template """
         customer_id = self.customer.id
         self.assertTrue(self.env.registry.ready, "We need to simulate that registery is ready")
-        rec = self.env['mail.test.full'].create({
+        rec = self.env['mail.test.ticket'].create({
             'name': 'Test',
             'umbrella_id': self.umbrella.id,
             'customer_id': False,
             'user_id': self.user_portal.id,
-            'mail_template': self.env.ref('test_mail.mail_test_full_tracking_tpl').id,
+            'mail_template': self.env.ref('test_mail.mail_test_ticket_tracking_tpl').id,
         })
         rec1 = rec.with_context(active_test=False)      # to see inactive records
         self.assertEqual(rec1.message_partner_ids, self.partners | self.env.user.partner_id | self.user_portal.partner_id)
@@ -715,7 +715,7 @@ class TestMailComplexPerformance(BaseMailPerformance):
         self.assertEqual(rec1.message_ids[1].subtype_id, self.env.ref('mail.mt_note'))
         self.assertEqual(rec1.message_ids[1].notified_partner_ids, self.env['res.partner'])
         # creation message
-        self.assertEqual(rec1.message_ids[2].subtype_id, self.env.ref('test_mail.st_mail_test_full_umbrella_upd'))
+        self.assertEqual(rec1.message_ids[2].subtype_id, self.env.ref('test_mail.st_mail_test_ticket_container_upd'))
         self.assertEqual(rec1.message_ids[2].notified_partner_ids, self.partners | self.user_portal.partner_id)
         self.assertEqual(len(rec1.message_ids), 3)
 
@@ -737,7 +737,7 @@ class TestMailComplexPerformance(BaseMailPerformance):
             'body': '<p>Test 0</p>',
             'author_id': self.partners[0].id,
             'email_from': self.partners[0].email,
-            'model': 'mail.test',
+            'model': 'mail.test.container',
             'res_id': self.umbrella.id,
             'subtype_id': self.env['ir.model.data'].xmlid_to_res_id('mail.mt_comment'),
             'attachment_ids': [
@@ -781,7 +781,7 @@ class TestMailComplexPerformance(BaseMailPerformance):
             'body': '<p>Test 1</p>',
             'author_id': self.partners[1].id,
             'email_from': self.partners[1].email,
-            'model': 'mail.test',
+            'model': 'mail.test.container',
             'res_id': self.umbrella.id,
             'subtype_id': self.env['ir.model.data'].xmlid_to_res_id('mail.mt_note'),
             'attachment_ids': [
@@ -844,7 +844,7 @@ class TestMailHeavyPerformancePost(BaseMailPerformance):
             'name': 'customer',
             'email': 'customer@example.com',
         })
-        self.record = self.env['mail.test'].with_context(mail_create_nosubscribe=True).create({
+        self.record = self.env['mail.test.container'].with_context(mail_create_nosubscribe=True).create({
             'name': 'Test record',
             'customer_id': self.customer.id,
             'alias_name': 'test-alias',