[ADD] test_base_automation: Move tests and related models

Purpose
=======

This module contains tests related to base automation. It makes no
sense as they have no business value.

Specification
=============

Move all the tests to a separate module as it contains models used only
to perform tests independently to functional aspects of other models.

5 files changed, 0 insertions, 357 deletions

diff --git a/addons/base_automation/__init__.py b/addons/base_automation/__init__.py
index 56990e4d8ee..dc5e6b693d1 100644
--- a/addons/base_automation/__init__.py
+++ b/addons/base_automation/__init__.py
@@ -2,4 +2,3 @@
 # Part of Odoo. See LICENSE file for full copyright and licensing details.
 
 from . import models
-from .tests import test_models
diff --git a/addons/base_automation/security/ir.model.access.csv b/addons/base_automation/security/ir.model.access.csv
index 2dc7103a91b..6b1d50350d0 100644
--- a/addons/base_automation/security/ir.model.access.csv
+++ b/addons/base_automation/security/ir.model.access.csv
@@ -1,5 +1,3 @@
 id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink

 access_base_automation,base.automation,model_base_automation,,1,0,0,0

 access_base_automation_config,base.automation config,model_base_automation,base.group_system,1,1,1,1

-access_base_automation_lead_test,access_base_automation_lead_test,model_base_automation_lead_test,base.group_system,1,1,1,1

-access_base_automation_line_test,access_base_automation_line_test,model_base_automation_line_test,base.group_system,1,1,1,1
\ No newline at end of file
diff --git a/addons/base_automation/tests/__init__.py b/addons/base_automation/tests/__init__.py
deleted file mode 100644
index b0ebb370682..00000000000
--- a/addons/base_automation/tests/__init__.py
+++ /dev/null
@@ -1,5 +0,0 @@
-# -*- coding: utf-8 -*-
-# Part of Odoo. See LICENSE file for full copyright and licensing details.
-
-from . import test_models
-from . import test_base_automation
diff --git a/addons/base_automation/tests/test_base_automation.py b/addons/base_automation/tests/test_base_automation.py
deleted file mode 100644
index c768bc7a141..00000000000
--- a/addons/base_automation/tests/test_base_automation.py
+++ /dev/null
@@ -1,297 +0,0 @@
-# # -*- coding: utf-8 -*-
-# # Part of Odoo. See LICENSE file for full copyright and licensing details.
-from unittest.mock import patch
-
-from odoo.addons.base.tests.common import TransactionCaseWithUserDemo
-from odoo.tests import tagged
-
-
-@tagged('post_install', '-at_install')
-class BaseAutomationTest(TransactionCaseWithUserDemo):
-
-    def setUp(self):
-        super(BaseAutomationTest, self).setUp()
-        self.user_root = self.env.ref('base.user_root')
-        self.user_admin = self.env.ref('base.user_admin')
-
-        self.test_mail_template_automation = self.env['mail.template'].create({
-            'name': 'Template Automation',
-            'model_id': self.env.ref('base_automation.model_base_automation_lead_test').id,
-            'body_html': """<div>Email automation</div>""",
-        })
-
-        self.res_partner_1 = self.env['res.partner'].create({'name': 'My Partner'})
-        self.env['base.automation'].create([
-            {
-                'name': 'Base Automation: test rule on create',
-                'model_id': self.env.ref('base_automation.model_base_automation_lead_test').id,
-                'state': 'code',
-                'code': "records.write({'user_id': %s})" % (self.user_demo.id),
-                'trigger': 'on_create',
-                'active': True,
-                'filter_domain': "[('state', '=', 'draft')]",
-            }, {
-                'name': 'Base Automation: test rule on write',
-                'model_id': self.env.ref('base_automation.model_base_automation_lead_test').id,
-                'state': 'code',
-                'code': "records.write({'user_id': %s})" % (self.user_demo.id),
-                'trigger': 'on_write',
-                'active': True,
-                'filter_domain': "[('state', '=', 'done')]",
-                'filter_pre_domain': "[('state', '=', 'open')]",
-            }, {
-                'name': 'Base Automation: test rule on recompute',
-                'model_id': self.env.ref('base_automation.model_base_automation_lead_test').id,
-                'state': 'code',
-                'code': "records.write({'user_id': %s})" % (self.user_demo.id),
-                'trigger': 'on_write',
-                'active': True,
-                'filter_domain': "[('employee', '=', True)]",
-            }, {
-                'name': 'Base Automation: test recursive rule',
-                'model_id': self.env.ref('base_automation.model_base_automation_lead_test').id,
-                'state': 'code',
-                'code': """
-record = model.browse(env.context['active_id'])
-if 'partner_id' in env.context['old_values'][record.id]:
-    record.write({'state': 'draft'})""",
-                'trigger': 'on_write',
-                'active': True,
-            }, {
-                'name': 'Base Automation: test rule on secondary model',
-                'model_id': self.env.ref('base_automation.model_base_automation_line_test').id,
-                'state': 'code',
-                'code': "records.write({'user_id': %s})" % (self.user_demo.id),
-                'trigger': 'on_create',
-                'active': True,
-            }, {
-                'name': 'Base Automation: test rule on write check context',
-                'model_id': self.env.ref('base_automation.model_base_automation_lead_test').id,
-                'state': 'code',
-                'code': """
-record = model.browse(env.context['active_id'])
-if 'user_id' in env.context['old_values'][record.id]:
-    record.write({'is_assigned_to_admin': (record.user_id.id == 1)})""",
-                'trigger': 'on_write',
-                'active': True,
-            }, {
-                'name': 'Base Automation: test rule with trigger',
-                'model_id': self.env.ref('base_automation.model_base_automation_lead_test').id,
-                'trigger_field_ids': [(4, self.env.ref('base_automation.field_base_automation_lead_test__state').id)],
-                'state': 'code',
-                'code': """
-record = model.browse(env.context['active_id'])
-record['name'] = record.name + 'X'""",
-                'trigger': 'on_write',
-                'active': True,
-            }, {
-                'name': 'Base Automation: test send an email',
-                'model_id': self.env.ref('base_automation.model_base_automation_lead_test').id,
-                'template_id': self.test_mail_template_automation.id,
-                'trigger_field_ids': [(4, self.env.ref('base_automation.field_base_automation_lead_test__deadline').id)],
-                'state': 'email',
-                'code': """
-record = model.browse(env.context['active_id'])
-record['name'] = record.name + 'X'""",
-                'trigger': 'on_write',
-                'active': True,
-                'filter_domain': "[('deadline', '!=', False)]",
-                'filter_pre_domain': "[('deadline', '=', False)]",
-            }
-        ])
-
-    def tearDown(self):
-        super().tearDown()
-        self.env['base.automation']._unregister_hook()
-
-    def create_lead(self, **kwargs):
-        vals = {
-            'name': "Lead Test",
-            'user_id': self.user_root.id,
-        }
-        vals.update(kwargs)
-        return self.env['base.automation.lead.test'].create(vals)
-
-    def test_00_check_to_state_open_pre(self):
-        """
-        Check that a new record (with state = open) doesn't change its responsible
-        when there is a precondition filter which check that the state is open.
-        """
-        lead = self.create_lead(state='open')
-        self.assertEqual(lead.state, 'open')
-        self.assertEqual(lead.user_id, self.user_root, "Responsible should not change on creation of Lead with state 'open'.")
-
-    def test_01_check_to_state_draft_post(self):
-        """
-        Check that a new record changes its responsible when there is a postcondition
-        filter which check that the state is draft.
-        """
-        lead = self.create_lead()
-        self.assertEqual(lead.state, 'draft', "Lead state should be 'draft'")
-        self.assertEqual(lead.user_id, self.user_demo, "Responsible should be change on creation of Lead with state 'draft'.")
-
-    def test_02_check_from_draft_to_done_with_steps(self):
-        """
-        A new record is created and goes from states 'open' to 'done' via the
-        other states (open, pending and cancel). We have a rule with:
-         - precondition: the record is in "open"
-         - postcondition: that the record is "done".
-        If the state goes from 'open' to 'done' the responsible is changed.
-        If those two conditions aren't verified, the responsible remains the same.
-        """
-        lead = self.create_lead(state='open')
-        self.assertEqual(lead.state, 'open', "Lead state should be 'open'")
-        self.assertEqual(lead.user_id, self.user_root, "Responsible should not change on creation of Lead with state 'open'.")
-        # change state to pending and check that responsible has not changed
-        lead.write({'state': 'pending'})
-        self.assertEqual(lead.state, 'pending', "Lead state should be 'pending'")
-        self.assertEqual(lead.user_id, self.user_root, "Responsible should not change on creation of Lead with state from 'draft' to 'open'.")
-        # change state to done and check that responsible has not changed
-        lead.write({'state': 'done'})
-        self.assertEqual(lead.state, 'done', "Lead state should be 'done'")
-        self.assertEqual(lead.user_id, self.user_root, "Responsible should not chang on creation of Lead with state from 'pending' to 'done'.")
-
-    def test_03_check_from_draft_to_done_without_steps(self):
-        """
-        A new record is created and goes from states 'open' to 'done' via the
-        other states (open, pending and cancel). We have a rule with:
-         - precondition: the record is in "open"
-         - postcondition: that the record is "done".
-        If the state goes from 'open' to 'done' the responsible is changed.
-        If those two conditions aren't verified, the responsible remains the same.
-        """
-        lead = self.create_lead(state='open')
-        self.assertEqual(lead.state, 'open', "Lead state should be 'open'")
-        self.assertEqual(lead.user_id, self.user_root, "Responsible should not change on creation of Lead with state 'open'.")
-        # change state to done and check that responsible has changed
-        lead.write({'state': 'done'})
-        self.assertEqual(lead.state, 'done', "Lead state should be 'done'")
-        self.assertEqual(lead.user_id, self.user_demo, "Responsible should be change on write of Lead with state from 'open' to 'done'.")
-
-    def test_10_recomputed_field(self):
-        """
-        Check that a rule is executed whenever a field is recomputed after a
-        change on another model.
-        """
-        partner = self.res_partner_1
-        partner.write({'employee': False})
-        lead = self.create_lead(state='open', partner_id=partner.id)
-        self.assertFalse(lead.employee, "Customer field should updated to False")
-        self.assertEqual(lead.user_id, self.user_root, "Responsible should not change on creation of Lead with state from 'draft' to 'open'.")
-        # change partner, recompute on lead should trigger the rule
-        partner.write({'employee': True})
-        lead.flush()
-        self.assertTrue(lead.employee, "Customer field should updated to True")
-        self.assertEqual(lead.user_id, self.user_demo, "Responsible should be change on write of Lead when Customer becomes True.")
-
-    def test_11_recomputed_field(self):
-        """
-        Check that a rule is executed whenever a field is recomputed and the
-        context contains the target field
-        """
-        partner = self.res_partner_1
-        lead = self.create_lead(state='draft', partner_id=partner.id)
-        self.assertFalse(lead.deadline, 'There should not be a deadline defined')
-        # change priority and user; this triggers deadline recomputation, and
-        # the server action should set the boolean field to True
-        lead.write({'priority': True, 'user_id': self.user_root.id})
-        self.assertTrue(lead.deadline, 'Deadline should be defined')
-        self.assertTrue(lead.is_assigned_to_admin, 'Lead should be assigned to admin')
-
-    def test_11b_recomputed_field(self):
-        mail_automation = self.env['base.automation'].search([('name', '=', 'Base Automation: test send an email')])
-        send_mail_count = 0
-
-        def _patched_get_actions(*args, **kwargs):
-            obj = args[0]
-            if '__action_done' not in obj._context:
-                obj = obj.with_context(__action_done={})
-            return mail_automation.with_env(obj.env)
-
-        def _patched_send_mail(*args, **kwargs):
-            nonlocal send_mail_count
-            send_mail_count += 1
-
-        patchers = [
-            patch('odoo.addons.base_automation.models.base_automation.BaseAutomation._get_actions', _patched_get_actions),
-            patch('odoo.addons.mail.models.mail_template.MailTemplate.send_mail', _patched_send_mail),
-        ]
-
-        patchers[0].start()
-
-        lead = self.create_lead()
-        self.assertFalse(lead.priority)
-        self.assertFalse(lead.deadline)
-
-        patchers[1].start()
-
-        lead.write({'priority': True})
-
-        self.assertTrue(lead.priority)
-        self.assertTrue(lead.deadline)
-
-        for patcher in patchers:
-            patcher.stop()
-
-        self.assertEqual(send_mail_count, 1)
-
-    def test_12_recursive(self):
-        """ Check that a rule is executed recursively by a secondary change. """
-        lead = self.create_lead(state='open')
-        self.assertEqual(lead.state, 'open')
-        self.assertEqual(lead.user_id, self.user_root)
-        # change partner; this should trigger the rule that modifies the state
-        partner = self.res_partner_1
-        lead.write({'partner_id': partner.id})
-        self.assertEqual(lead.state, 'draft')
-
-    def test_20_direct_line(self):
-        """
-        Check that a rule is executed after creating a line record.
-        """
-        line = self.env['base.automation.line.test'].create({'name': "Line"})
-        self.assertEqual(line.user_id, self.user_demo)
-
-    def test_20_indirect_line(self):
-        """
-        Check that creating a lead with a line executes rules on both records.
-        """
-        lead = self.create_lead(line_ids=[(0, 0, {'name': "Line"})])
-        self.assertEqual(lead.state, 'draft', "Lead state should be 'draft'")
-        self.assertEqual(lead.user_id, self.user_demo, "Responsible should change on creation of Lead test line.")
-        self.assertEqual(len(lead.line_ids), 1, "New test line is not created")
-        self.assertEqual(lead.line_ids.user_id, self.user_demo, "Responsible should be change on creation of Lead test line.")
-
-    def test_21_trigger_fields(self):
-        """
-        Check that the rule with trigger is executed only once per pertinent update.
-        """
-        lead = self.create_lead(name="X")
-        lead.priority = True
-        partner1 = self.res_partner_1
-        lead.partner_id = partner1.id
-        self.assertEqual(lead.name, 'X', "No update until now.")
-
-        lead.state = 'open'
-        self.assertEqual(lead.name, 'XX', "One update should have happened.")
-        lead.state = 'done'
-        self.assertEqual(lead.name, 'XXX', "One update should have happened.")
-        lead.state = 'done'
-        self.assertEqual(lead.name, 'XXX', "No update should have happened.")
-        lead.state = 'cancel'
-        self.assertEqual(lead.name, 'XXXX', "One update should have happened.")
-
-        # change the rule to trigger on partner_id
-        rule = self.env['base.automation'].search([('name', '=', 'Base Automation: test rule with trigger')])
-        rule.write({'trigger_field_ids':  [(6, 0, [self.env.ref('base_automation.field_base_automation_lead_test__partner_id').id])]})
-
-        partner2 = self.env['res.partner'].create({'name': 'A new partner'})
-        lead.name = 'X'
-        lead.state = 'open'
-        self.assertEqual(lead.name, 'X', "No update should have happened.")
-        lead.partner_id = partner2
-        self.assertEqual(lead.name, 'XX', "One update should have happened.")
-        lead.partner_id = partner2
-        self.assertEqual(lead.name, 'XX', "No update should have happened.")
-        lead.partner_id = partner1
-        self.assertEqual(lead.name, 'XXX', "One update should have happened.")
diff --git a/addons/base_automation/tests/test_models.py b/addons/base_automation/tests/test_models.py
deleted file mode 100644
index c3b4ca4000f..00000000000
--- a/addons/base_automation/tests/test_models.py
+++ /dev/null
@@ -1,52 +0,0 @@
-# -*- coding: utf-8 -*-
-# Part of Odoo. See LICENSE file for full copyright and licensing details.
-
-from dateutil import relativedelta
-from odoo import fields, models, api
-
-# YTI TODO: Move this brol to a new test_base_automation module 
-class LeadTest(models.Model):
-    _name = "base.automation.lead.test"
-    _description = "Automated Rule Test"
-
-    name = fields.Char(string='Subject', required=True, index=True)
-    user_id = fields.Many2one('res.users', string='Responsible')
-    state = fields.Selection([('draft', 'New'), ('cancel', 'Cancelled'), ('open', 'In Progress'),
-                              ('pending', 'Pending'), ('done', 'Closed')],
-                             string="Status", readonly=True, default='draft')
-    active = fields.Boolean(default=True)
-    partner_id = fields.Many2one('res.partner', string='Partner')
-    date_action_last = fields.Datetime(string='Last Action', readonly=True)
-    employee = fields.Boolean(compute='_compute_employee_deadline', store=True)
-    line_ids = fields.One2many('base.automation.line.test', 'lead_id')
-
-    priority = fields.Boolean()
-    deadline = fields.Boolean(compute='_compute_employee_deadline', store=True)
-    is_assigned_to_admin = fields.Boolean(string='Assigned to admin user')
-
-    @api.depends('partner_id.employee', 'priority')
-    def _compute_employee_deadline(self):
-        # this method computes two fields on purpose; don't split it
-        for record in self:
-            record.employee = record.partner_id.employee
-            if not record.priority:
-                record.deadline = False
-            else:
-                record.deadline = record.create_date + relativedelta.relativedelta(days=3)
-
-    def write(self, vals):
-        result = super().write(vals)
-        # force recomputation of field 'deadline' via 'employee': the action
-        # based on 'deadline' must be triggered
-        self.mapped('employee')
-        return result
-
-
-
-class LineTest(models.Model):
-    _name = "base.automation.line.test"
-    _description = "Automated Rule Line Test"
-
-    name = fields.Char()
-    lead_id = fields.Many2one('base.automation.lead.test', ondelete='cascade')
-    user_id = fields.Many2one('res.users')