Gauthier Wala (gawa)
ffcf2ee1a3
If you create an applicability and remove the company field, they are never used. An applicability like this should be valid for all companies. We put a 0.5 value for the company field so an applicability so it has a lesser priority than other fields. Same idea as the distribution models. opw-3847415 closes odoo/odoo#162152 Signed-off-by: Olivier Colson (oco) <oco@odoo.com>
101 lines
4.1 KiB
Python
101 lines
4.1 KiB
Python
# -*- coding: utf-8 -*-
|
|
import psycopg2
|
|
|
|
from odoo.exceptions import UserError
|
|
from odoo.tests import tagged
|
|
from odoo.tests.common import TransactionCase
|
|
from odoo.tools import mute_logger
|
|
|
|
|
|
@tagged('post_install', '-at_install')
|
|
class TestAnalyticPlanOperations(TransactionCase):
|
|
def test_delete_plan(self):
|
|
plan = self.env['account.analytic.plan'].create({'name': 'Test Plan'})
|
|
column = plan._column_name()
|
|
|
|
# columns exists
|
|
self.env.cr.execute(f"SELECT {column} FROM account_analytic_line LIMIT 1")
|
|
|
|
plan.unlink()
|
|
with self.assertRaises(psycopg2.errors.UndefinedColumn), mute_logger('odoo.sql_db'):
|
|
# column has been deleted
|
|
self.env.cr.execute(f"SELECT {column} FROM account_analytic_line LIMIT 1")
|
|
|
|
def test_delete_plan_with_view(self):
|
|
plan = self.env['account.analytic.plan'].create({'name': 'Test Plan'})
|
|
column = plan._column_name()
|
|
|
|
self.env['ir.ui.view'].create({
|
|
'name': 'Manual view',
|
|
'model': 'account.analytic.line',
|
|
'type': 'search',
|
|
'arch': f'<search><field name="{column}"/></search>',
|
|
})
|
|
|
|
# can't delete a plan still used in a view
|
|
with self.assertRaisesRegex(UserError, 'still present in views'):
|
|
plan.unlink()
|
|
|
|
def test_validate_deleted_account(self):
|
|
plan, mandatory_plan = self.env['account.analytic.plan'].create([{
|
|
'name': 'Test Plan',
|
|
}, {
|
|
'name': 'Mandatory Plan',
|
|
'default_applicability': 'mandatory',
|
|
}])
|
|
test_account, mandatory_account = self.env['account.analytic.account'].create([{
|
|
'name': 'Test Account',
|
|
'code': 'TAC',
|
|
'plan_id': plan.id,
|
|
}, {
|
|
'name': 'Mandatory Account',
|
|
'code': 'manda',
|
|
'plan_id': mandatory_plan.id,
|
|
}])
|
|
distribution_model = self.env['account.analytic.distribution.model'].create({}).with_context(validate_analytic=True)
|
|
|
|
# the configuration makes it raise an error
|
|
distribution_model.analytic_distribution = {f"{test_account.id}": 100}
|
|
with self.assertRaisesRegex(UserError, r'require a 100% analytic distribution'):
|
|
distribution_model._validate_distribution()
|
|
|
|
# once it is fixed, the error is not raised anymore
|
|
distribution_model.analytic_distribution = {f"{test_account.id},{mandatory_account.id}": 100}
|
|
distribution_model._validate_distribution()
|
|
|
|
# even by keeping a deleted account, the validation still works
|
|
test_account.unlink()
|
|
plan.unlink()
|
|
distribution_model._validate_distribution()
|
|
|
|
def test_validate_company_plans(self):
|
|
company_2 = self.env['res.company'].create({
|
|
'name': 'company_2',
|
|
})
|
|
plan = self.env['account.analytic.plan'].create([{
|
|
'name': 'Plan',
|
|
'default_applicability': 'optional',
|
|
}])
|
|
applicability = self.env['account.analytic.applicability'].create({
|
|
'business_domain': 'general',
|
|
'analytic_plan_id': plan.id,
|
|
'applicability': 'mandatory',
|
|
'company_id': company_2.id,
|
|
})
|
|
self.env['account.analytic.account'].create([{
|
|
'name': 'Mandatory Account',
|
|
'code': 'manda',
|
|
'plan_id': plan.id,
|
|
}])
|
|
distribution_model = self.env['account.analytic.distribution.model'].create({}).with_context(validate_analytic=True)
|
|
|
|
# mandatory applicability is only in company_2, should not raise for company_1
|
|
distribution_model._validate_distribution(business_domain='general', company_id=self.env.company.id)
|
|
|
|
applicability.company_id = False
|
|
# It should apply for all companies now
|
|
with self.assertRaisesRegex(UserError, r'require a 100% analytic distribution'):
|
|
distribution_model._validate_distribution(business_domain='general', company_id=self.env.company.id)
|
|
with self.assertRaisesRegex(UserError, r'require a 100% analytic distribution'):
|
|
distribution_model._validate_distribution(business_domain='general')
|