437 lines
19 KiB
Python
437 lines
19 KiB
Python
# -*- coding: utf-8 -*-
|
|
from odoo.addons.account.tests.common import AccountTestInvoicingCommon
|
|
from odoo.tests import tagged, Form
|
|
from odoo.exceptions import UserError, ValidationError
|
|
from odoo import Command
|
|
|
|
|
|
@tagged('post_install', '-at_install')
|
|
class TestAccountAnalyticAccount(AccountTestInvoicingCommon):
|
|
|
|
@classmethod
|
|
def setUpClass(cls, chart_template_ref=None):
|
|
super().setUpClass(chart_template_ref=chart_template_ref)
|
|
|
|
cls.env.user.groups_id += cls.env.ref('analytic.group_analytic_accounting')
|
|
|
|
# By default, tests are run with the current user set on the first company.
|
|
cls.env.user.company_id = cls.company_data['company']
|
|
|
|
cls.default_plan = cls.env['account.analytic.plan'].create({'name': 'Default'})
|
|
cls.analytic_account_a = cls.env['account.analytic.account'].create({
|
|
'name': 'analytic_account_a',
|
|
'plan_id': cls.default_plan.id,
|
|
'company_id': False,
|
|
})
|
|
cls.analytic_account_b = cls.env['account.analytic.account'].create({
|
|
'name': 'analytic_account_b',
|
|
'plan_id': cls.default_plan.id,
|
|
'company_id': False,
|
|
})
|
|
|
|
cls.cross_plan = cls.env['account.analytic.plan'].create({'name': 'Cross'})
|
|
cls.analytic_account_c = cls.env['account.analytic.account'].create({
|
|
'name': 'analytic_account_c',
|
|
'plan_id': cls.cross_plan.id,
|
|
'company_id': False,
|
|
})
|
|
|
|
def create_invoice(self, partner, product):
|
|
return self.env['account.move'].create([{
|
|
'move_type': 'out_invoice',
|
|
'partner_id': partner.id,
|
|
'date': '2017-01-01',
|
|
'invoice_date': '2017-01-01',
|
|
'invoice_line_ids': [Command.create({
|
|
'product_id': product.id,
|
|
})]
|
|
}])
|
|
|
|
def test_changing_analytic_company(self):
|
|
""" Ensure you can't change the company of an account.analytic.account if there are analytic lines linked to
|
|
the account
|
|
"""
|
|
self.env['account.analytic.line'].create({
|
|
'name': 'company specific account',
|
|
'account_id': self.analytic_account_a.id,
|
|
'amount': 100,
|
|
})
|
|
|
|
# Set a different company on the analytic account.
|
|
with self.assertRaises(UserError), self.cr.savepoint():
|
|
self.analytic_account_a.company_id = self.company_data_2['company']
|
|
|
|
# Making the analytic account not company dependent is allowed.
|
|
self.analytic_account_a.company_id = False
|
|
|
|
def test_analytic_lines(self):
|
|
''' Ensures analytic lines are created when posted and are recreated when editing the account.move'''
|
|
def get_analytic_lines():
|
|
return self.env['account.analytic.line'].search([
|
|
('move_line_id', 'in', out_invoice.line_ids.ids)
|
|
]).sorted('amount')
|
|
|
|
out_invoice = self.env['account.move'].create([{
|
|
'move_type': 'out_invoice',
|
|
'partner_id': self.partner_a.id,
|
|
'date': '2017-01-01',
|
|
'invoice_date': '2017-01-01',
|
|
'invoice_line_ids': [Command.create({
|
|
'product_id': self.product_a.id,
|
|
'price_unit': 200.0,
|
|
'analytic_distribution': {
|
|
self.analytic_account_a.id: 100,
|
|
self.analytic_account_b.id: 50,
|
|
},
|
|
})]
|
|
}])
|
|
|
|
out_invoice.action_post()
|
|
|
|
# Analytic lines are created when posting the invoice
|
|
self.assertRecordValues(get_analytic_lines(), [{
|
|
'amount': 100,
|
|
self.default_plan._column_name(): self.analytic_account_b.id,
|
|
'partner_id': self.partner_a.id,
|
|
'product_id': self.product_a.id,
|
|
}, {
|
|
'amount': 200,
|
|
self.default_plan._column_name(): self.analytic_account_a.id,
|
|
'partner_id': self.partner_a.id,
|
|
'product_id': self.product_a.id,
|
|
}])
|
|
|
|
# Analytic lines are updated when a posted invoice's distribution changes
|
|
out_invoice.invoice_line_ids.analytic_distribution = {
|
|
self.analytic_account_a.id: 100,
|
|
self.analytic_account_b.id: 25,
|
|
}
|
|
self.assertRecordValues(get_analytic_lines(), [{
|
|
'amount': 50,
|
|
self.default_plan._column_name(): self.analytic_account_b.id,
|
|
}, {
|
|
'amount': 200,
|
|
self.default_plan._column_name(): self.analytic_account_a.id,
|
|
}])
|
|
|
|
# Analytic lines are deleted when resetting to draft
|
|
out_invoice.button_draft()
|
|
self.assertFalse(get_analytic_lines())
|
|
|
|
def test_model_score(self):
|
|
"""Test that the models are applied correctly based on the score"""
|
|
|
|
self.env['account.analytic.distribution.model'].create([{
|
|
'product_id': self.product_a.id,
|
|
'analytic_distribution': {self.analytic_account_a.id: 100}
|
|
}, {
|
|
'partner_id': self.partner_a.id,
|
|
'product_id': self.product_a.id,
|
|
'analytic_distribution': {self.analytic_account_b.id: 100}
|
|
}])
|
|
|
|
# Partner and product match, score 2
|
|
invoice = self.create_invoice(self.partner_a, self.product_a)
|
|
self.assertEqual(invoice.invoice_line_ids.analytic_distribution, {str(self.analytic_account_b.id): 100})
|
|
|
|
# Match the partner but not the product, score 0
|
|
invoice = self.create_invoice(self.partner_a, self.product_b)
|
|
self.assertEqual(invoice.invoice_line_ids.analytic_distribution, False)
|
|
|
|
# Product match, score 1
|
|
invoice = self.create_invoice(self.partner_b, self.product_a)
|
|
self.assertEqual(invoice.invoice_line_ids.analytic_distribution, {str(self.analytic_account_a.id): 100})
|
|
|
|
# No rule match with the product, score 0
|
|
invoice = self.create_invoice(self.partner_b, self.product_b)
|
|
self.assertEqual(invoice.invoice_line_ids.analytic_distribution, False)
|
|
|
|
def test_model_application(self):
|
|
"""Test that the distribution is recomputed if and only if it is needed when changing the partner."""
|
|
self.env['account.analytic.distribution.model'].create([{
|
|
'partner_id': self.partner_a.id,
|
|
'analytic_distribution': {self.analytic_account_a.id: 100},
|
|
'company_id': False,
|
|
}, {
|
|
'partner_id': self.partner_b.id,
|
|
'analytic_distribution': {self.analytic_account_b.id: 100},
|
|
'company_id': False,
|
|
}])
|
|
|
|
invoice = self.create_invoice(self.env['res.partner'], self.product_a)
|
|
# No model is found, don't put anything
|
|
self.assertEqual(invoice.invoice_line_ids.analytic_distribution, False)
|
|
|
|
# A model is found, set the new values
|
|
invoice.partner_id = self.partner_a
|
|
self.assertEqual(invoice.invoice_line_ids.analytic_distribution, {str(self.analytic_account_a.id): 100})
|
|
|
|
# A model is found, set the new values
|
|
invoice.partner_id = self.partner_b
|
|
self.assertEqual(invoice.invoice_line_ids.analytic_distribution, {str(self.analytic_account_b.id): 100})
|
|
|
|
# No model is found, don't change previously set values
|
|
invoice.partner_id = invoice.company_id.partner_id
|
|
self.assertEqual(invoice.invoice_line_ids.analytic_distribution, {str(self.analytic_account_b.id): 100})
|
|
|
|
# No model is found, don't change previously set values
|
|
invoice.partner_id = False
|
|
self.assertEqual(invoice.invoice_line_ids.analytic_distribution, {str(self.analytic_account_b.id): 100})
|
|
|
|
# It manual value is not erased in form view when saving
|
|
with Form(invoice) as invoice_form:
|
|
invoice_form.partner_id = self.partner_a
|
|
with invoice_form.invoice_line_ids.edit(0) as line_form:
|
|
self.assertEqual(line_form.analytic_distribution, {str(self.analytic_account_a.id): 100})
|
|
line_form.analytic_distribution = {self.analytic_account_b.id: 100}
|
|
self.assertEqual(invoice.invoice_line_ids.analytic_distribution, {str(self.analytic_account_b.id): 100})
|
|
|
|
def test_mandatory_plan_validation(self):
|
|
invoice = self.create_invoice(self.partner_b, self.product_a)
|
|
self.default_plan.write({
|
|
'applicability_ids': [Command.create({
|
|
'business_domain': 'invoice',
|
|
'product_categ_id': self.product_a.categ_id.id,
|
|
'applicability': 'mandatory',
|
|
})]
|
|
})
|
|
|
|
# ValidationError is raised only when validate_analytic is in the context and the distribution is != 100
|
|
with self.assertRaisesRegex(ValidationError, '100% analytic distribution.'):
|
|
invoice.with_context({'validate_analytic': True}).action_post()
|
|
|
|
invoice.invoice_line_ids.analytic_distribution = {self.analytic_account_b.id: 100.01}
|
|
with self.assertRaisesRegex(ValidationError, '100% analytic distribution.'):
|
|
invoice.with_context({'validate_analytic': True}).action_post()
|
|
|
|
invoice.invoice_line_ids.analytic_distribution = {self.analytic_account_b.id: 99.9}
|
|
with self.assertRaisesRegex(ValidationError, '100% analytic distribution.'):
|
|
invoice.with_context({'validate_analytic': True}).action_post()
|
|
|
|
invoice.invoice_line_ids.analytic_distribution = {self.analytic_account_b.id: 100}
|
|
invoice.with_context({'validate_analytic': True}).action_post()
|
|
self.assertEqual(invoice.state, 'posted')
|
|
|
|
# reset and post without the validate_analytic context key
|
|
invoice.button_draft()
|
|
invoice.invoice_line_ids.analytic_distribution = {self.analytic_account_b.id: 0.9}
|
|
invoice.action_post()
|
|
self.assertEqual(invoice.state, 'posted')
|
|
|
|
def test_mandatory_plan_validation_mass_posting(self):
|
|
"""
|
|
In case of mass posting, we should still check for mandatory analytic plans. This may raise a RedirectWarning,
|
|
if more than one entry was selected for posting, or a ValidationError if only one entry was selected.
|
|
"""
|
|
invoice1 = self.create_invoice(self.partner_a, self.product_a)
|
|
invoice2 = self.create_invoice(self.partner_b, self.product_a)
|
|
self.default_plan.write({
|
|
'applicability_ids': [Command.create({
|
|
'business_domain': 'invoice',
|
|
'product_categ_id': self.product_a.categ_id.id,
|
|
'applicability': 'mandatory',
|
|
})]
|
|
})
|
|
|
|
vam = self.env['validate.account.move'].create({'force_post': True})
|
|
for invoices in [invoice1, invoice1 | invoice2]:
|
|
with self.subTest(invoices=invoices):
|
|
with self.assertRaises(Exception):
|
|
vam.with_context({
|
|
'active_model': 'account.move',
|
|
'active_ids': [invoice1.id, invoice2.id],
|
|
'validate_analytic': True,
|
|
}).validate_move()
|
|
self.assertTrue('posted' not in invoices.mapped('state'))
|
|
|
|
def test_cross_analytics_computing(self):
|
|
|
|
out_invoice = self.env['account.move'].create([{
|
|
'move_type': 'out_invoice',
|
|
'partner_id': self.partner_a.id,
|
|
'date': '2017-01-01',
|
|
'invoice_date': '2017-01-01',
|
|
'invoice_line_ids': [Command.create({
|
|
'product_id': self.product_b.id,
|
|
'price_unit': 200.0,
|
|
'analytic_distribution': {
|
|
f'{self.analytic_account_a.id},{self.analytic_account_c.id}': 20,
|
|
f'{self.analytic_account_a.id},{self.analytic_account_b.id}': 80,
|
|
},
|
|
})]
|
|
}])
|
|
out_invoice.action_post()
|
|
in_invoice = self.env['account.move'].create([{
|
|
'move_type': 'in_invoice',
|
|
'partner_id': self.partner_b.id,
|
|
'date': '2017-01-01',
|
|
'invoice_date': '2017-01-01',
|
|
'invoice_line_ids': [
|
|
Command.create({
|
|
'product_id': self.product_a.id,
|
|
'price_unit': 200.0,
|
|
'analytic_distribution': {
|
|
f'{self.analytic_account_a.id},{self.analytic_account_b.id}': 100,
|
|
},
|
|
}),
|
|
Command.create({
|
|
'product_id': self.product_a.id,
|
|
'price_unit': 200.0,
|
|
'analytic_distribution': {
|
|
f'{self.analytic_account_a.id},{self.analytic_account_c.id}': 50,
|
|
self.analytic_account_b.id: 50,
|
|
},
|
|
})
|
|
]
|
|
}])
|
|
in_invoice.action_post()
|
|
|
|
self.analytic_account_a._compute_invoice_count()
|
|
self.assertEqual(self.analytic_account_a.invoice_count, 1)
|
|
self.analytic_account_a._compute_vendor_bill_count()
|
|
self.assertEqual(self.analytic_account_a.vendor_bill_count, 1)
|
|
|
|
def test_applicability_score(self):
|
|
""" Tests which applicability is chosen if several ones are valid """
|
|
applicability_without_company, applicability_with_company = self.env['account.analytic.applicability'].create([
|
|
{
|
|
'business_domain': 'invoice',
|
|
'product_categ_id': self.product_a.categ_id.id,
|
|
'applicability': 'mandatory',
|
|
'analytic_plan_id': self.default_plan.id,
|
|
'company_id': False,
|
|
},
|
|
{
|
|
'business_domain': 'invoice',
|
|
'applicability': 'unavailable',
|
|
'analytic_plan_id': self.default_plan.id,
|
|
'company_id': self.env.company.id,
|
|
},
|
|
])
|
|
|
|
applicability = self.default_plan._get_applicability(business_domain='invoice', company_id=self.env.company.id, product=self.product_a.id)
|
|
self.assertEqual(applicability, 'mandatory', "product takes precedence over company")
|
|
|
|
# If the model that asks for a validation does not have a company_id,
|
|
# the score shouldn't take into account the company of the applicability
|
|
score = applicability_without_company._get_score(business_domain='invoice', product=self.product_a.id)
|
|
self.assertEqual(score, 2)
|
|
score = applicability_with_company._get_score(business_domain='invoice', product=self.product_a.id)
|
|
self.assertEqual(score, 1)
|
|
|
|
def test_analytic_lines_partner_compute(self):
|
|
''' Ensures analytic lines partner is changed when changing partner on move line'''
|
|
def get_analytic_lines():
|
|
return self.env['account.analytic.line'].search([
|
|
('move_line_id', 'in', entry.line_ids.ids)
|
|
]).sorted('amount')
|
|
|
|
entry = self.env['account.move'].create([{
|
|
'move_type': 'entry',
|
|
'partner_id': self.partner_a.id,
|
|
'line_ids': [
|
|
Command.create({
|
|
'account_id': self.company_data['default_account_receivable'].id,
|
|
'debit': 200.0,
|
|
'partner_id': self.partner_a.id,
|
|
}),
|
|
Command.create({
|
|
'account_id': self.company_data['default_account_revenue'].id,
|
|
'credit': 200.0,
|
|
'partner_id': self.partner_b.id,
|
|
'analytic_distribution': {
|
|
self.analytic_account_a.id: 100,
|
|
},
|
|
}),
|
|
]
|
|
}])
|
|
entry.action_post()
|
|
|
|
# Analytic lines are created when posting the invoice
|
|
analytic_line = get_analytic_lines()
|
|
self.assertRecordValues(analytic_line, [{
|
|
'amount': 200,
|
|
self.default_plan._column_name(): self.analytic_account_a.id,
|
|
'partner_id': self.partner_b.id,
|
|
}])
|
|
# Change the move line on the analytic line, partner changes on the analytic line
|
|
analytic_line.move_line_id = entry.line_ids[0]
|
|
self.assertRecordValues(analytic_line, [{
|
|
'amount': 200,
|
|
self.default_plan._column_name(): self.analytic_account_a.id,
|
|
'partner_id': self.partner_a.id,
|
|
}])
|
|
# Change the move line's partner, partner changes on the analytic line
|
|
entry.line_ids.write({'partner_id': self.partner_b.id})
|
|
self.assertRecordValues(analytic_line, [{
|
|
'amount': 200,
|
|
self.default_plan._column_name(): self.analytic_account_a.id,
|
|
'partner_id': self.partner_b.id,
|
|
}])
|
|
|
|
def test_analytic_distribution_with_discount(self):
|
|
"""Ensure that discount lines include analytic distribution when a discount expense account is set."""
|
|
|
|
# Create discount expense account
|
|
self.company_data['company'].account_discount_expense_allocation_id = self.env['account.account'].create({
|
|
'name': 'Discount Expense',
|
|
'code': 'DIS',
|
|
'account_type': 'expense',
|
|
'reconcile': False,
|
|
'company_id': self.company_data['company'].id,
|
|
})
|
|
|
|
# Create invoice with 2 lines: each has a discount and analytic distribution
|
|
out_invoice = self.env['account.move'].create([{
|
|
'move_type': 'out_invoice',
|
|
'partner_id': self.partner_a.id,
|
|
'date': '2017-01-01',
|
|
'invoice_date': '2017-01-01',
|
|
'invoice_line_ids': [Command.create({
|
|
'product_id': self.product_a.id,
|
|
'tax_ids': [Command.clear()],
|
|
'price_unit': 200.0,
|
|
'discount': 20, # 40.0 discount
|
|
'analytic_distribution': {
|
|
self.analytic_account_a.id: 100,
|
|
},
|
|
}), Command.create({
|
|
'product_id': self.product_b.id,
|
|
'tax_ids': [Command.clear()],
|
|
'price_unit': 200.0,
|
|
'discount': 10, # 20.0 discount
|
|
'analytic_distribution': {
|
|
self.analytic_account_b.id: 100,
|
|
},
|
|
})]
|
|
}])
|
|
out_invoice.action_post()
|
|
self.assertRecordValues(out_invoice.line_ids, [{
|
|
'display_type': 'product',
|
|
'balance': -160.0,
|
|
'analytic_distribution': {str(self.analytic_account_a.id): 100},
|
|
}, {
|
|
'display_type': 'product',
|
|
'balance': -180.0,
|
|
'analytic_distribution': {str(self.analytic_account_b.id): 100},
|
|
}, {
|
|
'display_type': 'discount',
|
|
'balance': -40.0,
|
|
'analytic_distribution': {str(self.analytic_account_a.id): 100}
|
|
}, {
|
|
'display_type': 'discount',
|
|
'balance': 60.0,
|
|
'analytic_distribution': {
|
|
str(self.analytic_account_a.id): 66.67,
|
|
str(self.analytic_account_b.id): 33.33,
|
|
}
|
|
}, {
|
|
'display_type': 'discount',
|
|
'balance': -20.0,
|
|
'analytic_distribution': {str(self.analytic_account_b.id): 100}
|
|
}, {
|
|
'display_type': 'payment_term',
|
|
'balance': 340.0,
|
|
'analytic_distribution': False,
|
|
}])
|