4082 lines
164 KiB
Python
4082 lines
164 KiB
Python
|
# -*- coding: utf-8 -*-
|
||
|
|
# pylint: disable=bad-whitespace
|
||
|
|
from odoo.addons.account.tests.common import AccountTestInvoicingCommon
|
||
|
|
from odoo.tests.common import Form
|
||
|
|
from odoo.tests import tagged
|
||
|
|
from odoo import fields, Command
|
||
|
|
from odoo.exceptions import UserError
|
||
|
|
|
||
|
|
from collections import defaultdict
|
||
|
|
from unittest.mock import patch
|
||
|
|
from datetime import timedelta
|
||
|
|
from freezegun import freeze_time
|
||
|
|
|
||
|
|
|
||
|
|
|
||
|
|
@tagged('post_install', '-at_install')
|
||
|
|
class TestAccountMoveOutInvoiceOnchanges(AccountTestInvoicingCommon):
|
||
|
|
|
||
|
|
@classmethod
|
||
|
|
def setUpClass(cls, chart_template_ref=None):
|
||
|
|
super().setUpClass(chart_template_ref=chart_template_ref)
|
||
|
|
|
||
|
|
cls.invoice = cls.init_invoice('out_invoice', products=cls.product_a+cls.product_b)
|
||
|
|
|
||
|
|
cls.product_line_vals_1 = {
|
||
|
|
'name': cls.product_a.name,
|
||
|
|
'product_id': cls.product_a.id,
|
||
|
|
'account_id': cls.product_a.property_account_income_id.id,
|
||
|
|
'partner_id': cls.partner_a.id,
|
||
|
|
'product_uom_id': cls.product_a.uom_id.id,
|
||
|
|
'quantity': 1.0,
|
||
|
|
'discount': 0.0,
|
||
|
|
'price_unit': 1000.0,
|
||
|
|
'price_subtotal': 1000.0,
|
||
|
|
'price_total': 1150.0,
|
||
|
|
'tax_ids': cls.product_a.taxes_id.ids,
|
||
|
|
'tax_line_id': False,
|
||
|
|
'currency_id': cls.company_data['currency'].id,
|
||
|
|
'amount_currency': -1000.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 1000.0,
|
||
|
|
'date_maturity': False,
|
||
|
|
}
|
||
|
|
cls.product_line_vals_2 = {
|
||
|
|
'name': cls.product_b.name,
|
||
|
|
'product_id': cls.product_b.id,
|
||
|
|
'account_id': cls.product_b.property_account_income_id.id,
|
||
|
|
'partner_id': cls.partner_a.id,
|
||
|
|
'product_uom_id': cls.product_b.uom_id.id,
|
||
|
|
'quantity': 1.0,
|
||
|
|
'discount': 0.0,
|
||
|
|
'price_unit': 200.0,
|
||
|
|
'price_subtotal': 200.0,
|
||
|
|
'price_total': 260.0,
|
||
|
|
'tax_ids': cls.product_b.taxes_id.ids,
|
||
|
|
'tax_line_id': False,
|
||
|
|
'currency_id': cls.company_data['currency'].id,
|
||
|
|
'amount_currency': -200.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 200.0,
|
||
|
|
'date_maturity': False,
|
||
|
|
}
|
||
|
|
cls.tax_line_vals_1 = {
|
||
|
|
'name': cls.tax_sale_a.name,
|
||
|
|
'product_id': False,
|
||
|
|
'account_id': cls.company_data['default_account_tax_sale'].id,
|
||
|
|
'partner_id': cls.partner_a.id,
|
||
|
|
'product_uom_id': False,
|
||
|
|
'quantity': False,
|
||
|
|
'discount': 0.0,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_line_id': cls.tax_sale_a.id,
|
||
|
|
'currency_id': cls.company_data['currency'].id,
|
||
|
|
'amount_currency': -180.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 180.0,
|
||
|
|
'date_maturity': False,
|
||
|
|
}
|
||
|
|
cls.tax_line_vals_2 = {
|
||
|
|
'name': cls.tax_sale_b.name,
|
||
|
|
'product_id': False,
|
||
|
|
'account_id': cls.company_data['default_account_tax_sale'].id,
|
||
|
|
'partner_id': cls.partner_a.id,
|
||
|
|
'product_uom_id': False,
|
||
|
|
'quantity': False,
|
||
|
|
'discount': 0.0,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_line_id': cls.tax_sale_b.id,
|
||
|
|
'currency_id': cls.company_data['currency'].id,
|
||
|
|
'amount_currency': -30.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 30.0,
|
||
|
|
'date_maturity': False,
|
||
|
|
}
|
||
|
|
cls.term_line_vals_1 = {
|
||
|
|
'name': '',
|
||
|
|
'product_id': False,
|
||
|
|
'account_id': cls.company_data['default_account_receivable'].id,
|
||
|
|
'partner_id': cls.partner_a.id,
|
||
|
|
'product_uom_id': False,
|
||
|
|
'quantity': False,
|
||
|
|
'discount': 0.0,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_line_id': False,
|
||
|
|
'currency_id': cls.company_data['currency'].id,
|
||
|
|
'amount_currency': 1410.0,
|
||
|
|
'debit': 1410.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
'date_maturity': fields.Date.from_string('2019-01-01'),
|
||
|
|
}
|
||
|
|
cls.move_vals = {
|
||
|
|
'partner_id': cls.partner_a.id,
|
||
|
|
'currency_id': cls.company_data['currency'].id,
|
||
|
|
'journal_id': cls.company_data['default_journal_sale'].id,
|
||
|
|
'date': fields.Date.from_string('2019-01-01'),
|
||
|
|
'fiscal_position_id': False,
|
||
|
|
'payment_reference': '',
|
||
|
|
'invoice_payment_term_id': cls.pay_terms_a.id,
|
||
|
|
'amount_untaxed': 1200.0,
|
||
|
|
'amount_tax': 210.0,
|
||
|
|
'amount_total': 1410.0,
|
||
|
|
}
|
||
|
|
cls.env.user.groups_id += cls.env.ref('uom.group_uom')
|
||
|
|
|
||
|
|
def setUp(self):
|
||
|
|
super(TestAccountMoveOutInvoiceOnchanges, self).setUp()
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
self.product_line_vals_1,
|
||
|
|
self.product_line_vals_2,
|
||
|
|
self.tax_line_vals_1,
|
||
|
|
self.tax_line_vals_2,
|
||
|
|
self.term_line_vals_1,
|
||
|
|
], self.move_vals)
|
||
|
|
|
||
|
|
@freeze_time('2020-01-15')
|
||
|
|
def test_out_invoice_onchange_invoice_date(self):
|
||
|
|
for tax_date, invoice_date, accounting_date in [
|
||
|
|
('2019-03-31', '2019-05-12', '2019-05-12'),
|
||
|
|
('2019-03-31', '2019-02-10', '2019-12-31'),
|
||
|
|
('2019-05-31', '2019-06-15', '2019-06-15'),
|
||
|
|
]:
|
||
|
|
self.invoice.company_id.tax_lock_date = tax_date
|
||
|
|
invoice = self.invoice.copy()
|
||
|
|
with Form(invoice) as move_form:
|
||
|
|
move_form.invoice_date = invoice_date
|
||
|
|
invoice.action_post()
|
||
|
|
self.assertEqual(invoice.date, fields.Date.to_date(accounting_date))
|
||
|
|
|
||
|
|
def test_out_invoice_line_onchange_product_1(self):
|
||
|
|
move_form = Form(self.invoice)
|
||
|
|
with move_form.invoice_line_ids.edit(0) as line_form:
|
||
|
|
line_form.product_id = self.product_b
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'name': self.product_b.name,
|
||
|
|
'product_id': self.product_b.id,
|
||
|
|
'product_uom_id': self.product_b.uom_id.id,
|
||
|
|
'account_id': self.product_b.property_account_income_id.id,
|
||
|
|
'price_unit': 200.0,
|
||
|
|
'price_subtotal': 200.0,
|
||
|
|
'price_total': 260.0,
|
||
|
|
'tax_ids': self.product_b.taxes_id.ids,
|
||
|
|
'amount_currency': -200.0,
|
||
|
|
'credit': 200.0,
|
||
|
|
},
|
||
|
|
self.product_line_vals_2,
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'amount_currency': -60.0,
|
||
|
|
'credit': 60.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'amount_currency': -60.0,
|
||
|
|
'credit': 60.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'amount_currency': 520.0,
|
||
|
|
'debit': 520.0,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'amount_untaxed': 400.0,
|
||
|
|
'amount_tax': 120.0,
|
||
|
|
'amount_total': 520.0,
|
||
|
|
})
|
||
|
|
|
||
|
|
def test_out_invoice_line_onchange_product_2_with_fiscal_pos_1(self):
|
||
|
|
''' Test mapping a price-included tax (10%) with a price-excluded tax (20%) on a price_unit of 110.0.
|
||
|
|
The price_unit should be 100.0 after applying the fiscal position.
|
||
|
|
'''
|
||
|
|
tax_price_include = self.env['account.tax'].create({
|
||
|
|
'name': '10% incl',
|
||
|
|
'type_tax_use': 'sale',
|
||
|
|
'amount_type': 'percent',
|
||
|
|
'amount': 10,
|
||
|
|
'price_include': True,
|
||
|
|
'include_base_amount': True,
|
||
|
|
})
|
||
|
|
tax_price_exclude = self.env['account.tax'].create({
|
||
|
|
'name': '15% excl',
|
||
|
|
'type_tax_use': 'sale',
|
||
|
|
'amount_type': 'percent',
|
||
|
|
'amount': 15,
|
||
|
|
})
|
||
|
|
|
||
|
|
fiscal_position = self.env['account.fiscal.position'].create({
|
||
|
|
'name': 'fiscal_pos_a',
|
||
|
|
'tax_ids': [
|
||
|
|
(0, None, {
|
||
|
|
'tax_src_id': tax_price_include.id,
|
||
|
|
'tax_dest_id': tax_price_exclude.id,
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
|
||
|
|
product = self.env['product.product'].create({
|
||
|
|
'name': 'product',
|
||
|
|
'uom_id': self.env.ref('uom.product_uom_unit').id,
|
||
|
|
'lst_price': 110.0,
|
||
|
|
'taxes_id': [(6, 0, tax_price_include.ids)],
|
||
|
|
})
|
||
|
|
|
||
|
|
move_form = Form(self.env['account.move'].with_context(default_move_type='out_invoice'))
|
||
|
|
move_form.partner_id = self.partner_a
|
||
|
|
move_form.invoice_date = fields.Date.from_string('2019-01-01')
|
||
|
|
move_form.currency_id = self.currency_data['currency']
|
||
|
|
move_form.fiscal_position_id = fiscal_position
|
||
|
|
with move_form.invoice_line_ids.new() as line_form:
|
||
|
|
line_form.product_id = product
|
||
|
|
invoice = move_form.save()
|
||
|
|
|
||
|
|
self.assertInvoiceValues(invoice, [
|
||
|
|
{
|
||
|
|
'product_id': product.id,
|
||
|
|
'price_unit': 200.0,
|
||
|
|
'price_subtotal': 200.0,
|
||
|
|
'price_total': 230.0,
|
||
|
|
'tax_ids': tax_price_exclude.ids,
|
||
|
|
'tax_line_id': False,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -200.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 100.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'product_id': False,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_line_id': tax_price_exclude.id,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -30.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 15.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'product_id': False,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_line_id': False,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 230.0,
|
||
|
|
'debit': 115.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'fiscal_position_id': fiscal_position.id,
|
||
|
|
'amount_untaxed': 200.0,
|
||
|
|
'amount_tax': 30.0,
|
||
|
|
'amount_total': 230.0,
|
||
|
|
})
|
||
|
|
|
||
|
|
uom_dozen = self.env.ref('uom.product_uom_dozen')
|
||
|
|
with Form(invoice) as move_form:
|
||
|
|
with move_form.invoice_line_ids.edit(0) as line_form:
|
||
|
|
line_form.product_uom_id = uom_dozen
|
||
|
|
|
||
|
|
self.assertInvoiceValues(invoice, [
|
||
|
|
{
|
||
|
|
'product_id': product.id,
|
||
|
|
'product_uom_id': uom_dozen.id,
|
||
|
|
'price_unit': 2400.0,
|
||
|
|
'price_subtotal': 2400.0,
|
||
|
|
'price_total': 2760.0,
|
||
|
|
'tax_ids': tax_price_exclude.ids,
|
||
|
|
'tax_line_id': False,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -2400.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 1200.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'product_id': False,
|
||
|
|
'product_uom_id': False,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_line_id': tax_price_exclude.id,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -360.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 180.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'product_id': False,
|
||
|
|
'product_uom_id': False,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_line_id': False,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 2760.0,
|
||
|
|
'debit': 1380.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'fiscal_position_id': fiscal_position.id,
|
||
|
|
'amount_untaxed': 2400.0,
|
||
|
|
'amount_tax': 360.0,
|
||
|
|
'amount_total': 2760.0,
|
||
|
|
})
|
||
|
|
|
||
|
|
# Check rounding.
|
||
|
|
decimal_precision_name = self.env['account.move.line']._fields['price_unit']._digits
|
||
|
|
decimal_precision = self.env['decimal.precision'].search([('name', '=', decimal_precision_name)])
|
||
|
|
decimal_precision.digits = 4
|
||
|
|
|
||
|
|
product.lst_price = 90.0034
|
||
|
|
invoice = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'invoice_date': '2017-01-01',
|
||
|
|
'date': '2017-01-01',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'fiscal_position_id': fiscal_position.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'name': 'test line',
|
||
|
|
'product_id': product.id,
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
self.assertRecordValues(invoice.invoice_line_ids, [{
|
||
|
|
'price_unit': 163.6425, # 90.0034 / 1.10 * 2
|
||
|
|
'tax_ids': tax_price_exclude.ids,
|
||
|
|
'price_subtotal': 163.643,
|
||
|
|
'price_total': 188.189,
|
||
|
|
}])
|
||
|
|
|
||
|
|
def test_out_invoice_line_onchange_product_2_with_fiscal_pos_2(self):
|
||
|
|
''' Test mapping a price-included tax (10%) with another price-included tax (20%) on a price_unit of 110.0.
|
||
|
|
The price_unit should be 120.0 after applying the fiscal position.
|
||
|
|
'''
|
||
|
|
tax_price_include_1 = self.env['account.tax'].create({
|
||
|
|
'name': '10% incl',
|
||
|
|
'type_tax_use': 'sale',
|
||
|
|
'amount_type': 'percent',
|
||
|
|
'amount': 10,
|
||
|
|
'price_include': True,
|
||
|
|
'include_base_amount': True,
|
||
|
|
})
|
||
|
|
tax_price_include_2 = self.env['account.tax'].create({
|
||
|
|
'name': '20% incl',
|
||
|
|
'type_tax_use': 'sale',
|
||
|
|
'amount_type': 'percent',
|
||
|
|
'amount': 20,
|
||
|
|
'price_include': True,
|
||
|
|
'include_base_amount': True,
|
||
|
|
})
|
||
|
|
|
||
|
|
fiscal_position = self.env['account.fiscal.position'].create({
|
||
|
|
'name': 'fiscal_pos_a',
|
||
|
|
'tax_ids': [
|
||
|
|
(0, None, {
|
||
|
|
'tax_src_id': tax_price_include_1.id,
|
||
|
|
'tax_dest_id': tax_price_include_2.id,
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
|
||
|
|
product = self.env['product.product'].create({
|
||
|
|
'name': 'product',
|
||
|
|
'uom_id': self.env.ref('uom.product_uom_unit').id,
|
||
|
|
'lst_price': 110.0,
|
||
|
|
'taxes_id': [(6, 0, tax_price_include_1.ids)],
|
||
|
|
})
|
||
|
|
|
||
|
|
move_form = Form(self.env['account.move'].with_context(default_move_type='out_invoice'))
|
||
|
|
move_form.partner_id = self.partner_a
|
||
|
|
move_form.invoice_date = fields.Date.from_string('2019-01-01')
|
||
|
|
move_form.currency_id = self.currency_data['currency']
|
||
|
|
move_form.fiscal_position_id = fiscal_position
|
||
|
|
with move_form.invoice_line_ids.new() as line_form:
|
||
|
|
line_form.product_id = product
|
||
|
|
invoice = move_form.save()
|
||
|
|
|
||
|
|
self.assertInvoiceValues(invoice, [
|
||
|
|
{
|
||
|
|
'product_id': product.id,
|
||
|
|
'price_unit': 240.0,
|
||
|
|
'price_subtotal': 200.0,
|
||
|
|
'price_total': 240.0,
|
||
|
|
'tax_ids': tax_price_include_2.ids,
|
||
|
|
'tax_line_id': False,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -200.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 100.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'product_id': False,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_line_id': tax_price_include_2.id,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -40.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 20.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'product_id': False,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_line_id': False,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 240.0,
|
||
|
|
'debit': 120.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'fiscal_position_id': fiscal_position.id,
|
||
|
|
'amount_untaxed': 200.0,
|
||
|
|
'amount_tax': 40.0,
|
||
|
|
'amount_total': 240.0,
|
||
|
|
})
|
||
|
|
|
||
|
|
uom_dozen = self.env.ref('uom.product_uom_dozen')
|
||
|
|
with Form(invoice) as move_form:
|
||
|
|
with move_form.invoice_line_ids.edit(0) as line_form:
|
||
|
|
line_form.product_uom_id = uom_dozen
|
||
|
|
|
||
|
|
self.assertInvoiceValues(invoice, [
|
||
|
|
{
|
||
|
|
'product_id': product.id,
|
||
|
|
'product_uom_id': uom_dozen.id,
|
||
|
|
'price_unit': 2880.0,
|
||
|
|
'price_subtotal': 2400.0,
|
||
|
|
'price_total': 2880.0,
|
||
|
|
'tax_ids': tax_price_include_2.ids,
|
||
|
|
'tax_line_id': False,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -2400.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 1200.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'product_id': False,
|
||
|
|
'product_uom_id': False,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_line_id': tax_price_include_2.id,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -480.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 240.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'product_id': False,
|
||
|
|
'product_uom_id': False,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_line_id': False,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 2880.0,
|
||
|
|
'debit': 1440.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'fiscal_position_id': fiscal_position.id,
|
||
|
|
'amount_untaxed': 2400.0,
|
||
|
|
'amount_tax': 480.0,
|
||
|
|
'amount_total': 2880.0,
|
||
|
|
})
|
||
|
|
|
||
|
|
def test_out_invoice_line_onchange_business_fields_1(self):
|
||
|
|
move_form = Form(self.invoice)
|
||
|
|
with move_form.invoice_line_ids.edit(0) as line_form:
|
||
|
|
# Current price_unit is 1000.
|
||
|
|
# We set quantity = 4, discount = 50%, price_unit = 400. The debit/credit fields don't change because (4 * 500) * 0.5 = 1000.
|
||
|
|
line_form.quantity = 4
|
||
|
|
line_form.discount = 50
|
||
|
|
line_form.price_unit = 500
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'quantity': 4,
|
||
|
|
'discount': 50.0,
|
||
|
|
'price_unit': 500.0,
|
||
|
|
},
|
||
|
|
self.product_line_vals_2,
|
||
|
|
self.tax_line_vals_1,
|
||
|
|
self.tax_line_vals_2,
|
||
|
|
self.term_line_vals_1,
|
||
|
|
], self.move_vals)
|
||
|
|
|
||
|
|
move_form = Form(self.invoice)
|
||
|
|
with move_form.invoice_line_ids.edit(0) as line_form:
|
||
|
|
# Reset field except the discount that becomes 100%.
|
||
|
|
line_form.quantity = 1
|
||
|
|
line_form.discount = 100
|
||
|
|
line_form.price_unit = 1000
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'discount': 100.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'amount_currency': 0.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
self.product_line_vals_2,
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'amount_currency': -30.0,
|
||
|
|
'credit': 30.0,
|
||
|
|
},
|
||
|
|
self.tax_line_vals_2,
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'amount_currency': 260.0,
|
||
|
|
'debit': 260.0,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'amount_untaxed': 200.0,
|
||
|
|
'amount_tax': 60.0,
|
||
|
|
'amount_total': 260.0,
|
||
|
|
})
|
||
|
|
|
||
|
|
def test_out_invoice_line_onchange_partner_1(self):
|
||
|
|
move_form = Form(self.invoice)
|
||
|
|
move_form.partner_id = self.partner_b
|
||
|
|
move_form.payment_reference = 'turlututu'
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'partner_id': self.partner_b.id,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'partner_id': self.partner_b.id,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'partner_id': self.partner_b.id,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'partner_id': self.partner_b.id,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'name': 'turlututu installment #1',
|
||
|
|
'account_id': self.partner_b.property_account_receivable_id.id,
|
||
|
|
'partner_id': self.partner_b.id,
|
||
|
|
'amount_currency': 423.0,
|
||
|
|
'debit': 423.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'name': 'turlututu installment #2',
|
||
|
|
'account_id': self.partner_b.property_account_receivable_id.id,
|
||
|
|
'partner_id': self.partner_b.id,
|
||
|
|
'amount_currency': 987.0,
|
||
|
|
'debit': 987.0,
|
||
|
|
'date_maturity': fields.Date.from_string('2019-02-28'),
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'partner_id': self.partner_b.id,
|
||
|
|
'payment_reference': 'turlututu',
|
||
|
|
'fiscal_position_id': self.fiscal_pos_a.id,
|
||
|
|
'invoice_payment_term_id': self.pay_terms_b.id,
|
||
|
|
'amount_untaxed': 1200.0,
|
||
|
|
'amount_tax': 210.0,
|
||
|
|
'amount_total': 1410.0,
|
||
|
|
})
|
||
|
|
|
||
|
|
# Remove lines and recreate them to apply the fiscal position.
|
||
|
|
move_form = Form(self.invoice)
|
||
|
|
move_form.invoice_line_ids.remove(0)
|
||
|
|
move_form.invoice_line_ids.remove(0)
|
||
|
|
with move_form.invoice_line_ids.new() as line_form:
|
||
|
|
line_form.product_id = self.product_a
|
||
|
|
with move_form.invoice_line_ids.new() as line_form:
|
||
|
|
line_form.product_id = self.product_b
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'account_id': self.product_b.property_account_income_id.id,
|
||
|
|
'partner_id': self.partner_b.id,
|
||
|
|
'tax_ids': self.tax_sale_b.ids,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'partner_id': self.partner_b.id,
|
||
|
|
'price_total': 230.0,
|
||
|
|
'tax_ids': self.tax_sale_b.ids,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'name': self.tax_sale_b.name,
|
||
|
|
'partner_id': self.partner_b.id,
|
||
|
|
'tax_line_id': self.tax_sale_b.id,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'name': 'turlututu installment #1',
|
||
|
|
'account_id': self.partner_b.property_account_receivable_id.id,
|
||
|
|
'partner_id': self.partner_b.id,
|
||
|
|
'amount_currency': 414.0,
|
||
|
|
'debit': 414.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'name': 'turlututu installment #2',
|
||
|
|
'account_id': self.partner_b.property_account_receivable_id.id,
|
||
|
|
'partner_id': self.partner_b.id,
|
||
|
|
'amount_currency': 966.0,
|
||
|
|
'debit': 966.0,
|
||
|
|
'date_maturity': fields.Date.from_string('2019-02-28'),
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'partner_id': self.partner_b.id,
|
||
|
|
'payment_reference': 'turlututu',
|
||
|
|
'fiscal_position_id': self.fiscal_pos_a.id,
|
||
|
|
'invoice_payment_term_id': self.pay_terms_b.id,
|
||
|
|
'amount_untaxed': 1200.0,
|
||
|
|
'amount_tax': 180.0,
|
||
|
|
'amount_total': 1380.0,
|
||
|
|
})
|
||
|
|
|
||
|
|
def test_out_invoice_line_onchange_taxes_1(self):
|
||
|
|
move_form = Form(self.invoice)
|
||
|
|
with move_form.invoice_line_ids.edit(0) as line_form:
|
||
|
|
line_form.price_unit = 1200
|
||
|
|
line_form.tax_ids.add(self.tax_armageddon)
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
child_tax_1 = self.tax_armageddon.children_tax_ids[0]
|
||
|
|
child_tax_2 = self.tax_armageddon.children_tax_ids[1]
|
||
|
|
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'price_unit': 1200.0,
|
||
|
|
'price_subtotal': 1000.0,
|
||
|
|
'price_total': 1470.0,
|
||
|
|
'tax_ids': (self.tax_sale_a + self.tax_armageddon).ids,
|
||
|
|
},
|
||
|
|
self.product_line_vals_2,
|
||
|
|
self.tax_line_vals_1,
|
||
|
|
self.tax_line_vals_2,
|
||
|
|
{
|
||
|
|
'name': child_tax_1.name,
|
||
|
|
'product_id': False,
|
||
|
|
'account_id': self.company_data['default_account_revenue'].id,
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'product_uom_id': False,
|
||
|
|
'quantity': False,
|
||
|
|
'discount': 0.0,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': child_tax_2.ids,
|
||
|
|
'tax_line_id': child_tax_1.id,
|
||
|
|
'currency_id': self.company_data['currency'].id,
|
||
|
|
'amount_currency': -120.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 120.0,
|
||
|
|
'date_maturity': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'name': child_tax_1.name,
|
||
|
|
'product_id': False,
|
||
|
|
'account_id': self.company_data['default_account_tax_sale'].id,
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'product_uom_id': False,
|
||
|
|
'quantity': False,
|
||
|
|
'discount': 0.0,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': child_tax_2.ids,
|
||
|
|
'tax_line_id': child_tax_1.id,
|
||
|
|
'currency_id': self.company_data['currency'].id,
|
||
|
|
'amount_currency': -80.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 80.0,
|
||
|
|
'date_maturity': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'name': child_tax_2.name,
|
||
|
|
'product_id': False,
|
||
|
|
'account_id': child_tax_2.cash_basis_transition_account_id.id,
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'product_uom_id': False,
|
||
|
|
'quantity': False,
|
||
|
|
'discount': 0.0,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_line_id': child_tax_2.id,
|
||
|
|
'currency_id': self.company_data['currency'].id,
|
||
|
|
'amount_currency': -120.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 120.0,
|
||
|
|
'date_maturity': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'amount_currency': 1730.0,
|
||
|
|
'debit': 1730.0,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'amount_untaxed': 1200.0,
|
||
|
|
'amount_tax': 530.0,
|
||
|
|
'amount_total': 1730.0,
|
||
|
|
})
|
||
|
|
|
||
|
|
def test_out_invoice_line_onchange_rounding_price_subtotal_1(self):
|
||
|
|
''' Seek for rounding issue on the price_subtotal when dealing with a price_unit having more digits than the
|
||
|
|
foreign currency one.
|
||
|
|
'''
|
||
|
|
decimal_precision_name = self.env['account.move.line']._fields['price_unit']._digits
|
||
|
|
decimal_precision = self.env['decimal.precision'].search([('name', '=', decimal_precision_name)])
|
||
|
|
|
||
|
|
self.assertTrue(decimal_precision, "Decimal precision '%s' not found" % decimal_precision_name)
|
||
|
|
|
||
|
|
self.currency_data['currency'].rounding = 0.01
|
||
|
|
decimal_precision.digits = 4
|
||
|
|
|
||
|
|
def check_invoice_values(invoice):
|
||
|
|
self.assertInvoiceValues(invoice, [
|
||
|
|
{
|
||
|
|
'quantity': 1.0,
|
||
|
|
'price_unit': 0.025,
|
||
|
|
'price_subtotal': 0.03,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 0.02,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'quantity': False,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'debit': 0.02,
|
||
|
|
'credit': 0.0,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
'amount_untaxed': 0.03,
|
||
|
|
'amount_tax': 0.0,
|
||
|
|
'amount_total': 0.03,
|
||
|
|
})
|
||
|
|
|
||
|
|
# == Test at the creation of the invoice ==
|
||
|
|
|
||
|
|
invoice_1 = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'invoice_date': '2017-01-01',
|
||
|
|
'date': '2017-01-01',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'invoice_line_ids': [(0, 0, {
|
||
|
|
'name': 'test line',
|
||
|
|
'price_unit': 0.025,
|
||
|
|
'quantity': 1,
|
||
|
|
'account_id': self.company_data['default_account_revenue'].id,
|
||
|
|
})],
|
||
|
|
})
|
||
|
|
|
||
|
|
check_invoice_values(invoice_1)
|
||
|
|
|
||
|
|
# == Test when writing on the invoice ==
|
||
|
|
|
||
|
|
invoice_2 = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'invoice_date': '2017-01-01',
|
||
|
|
'date': '2017-01-01',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
})
|
||
|
|
invoice_2.write({
|
||
|
|
'invoice_line_ids': [(0, 0, {
|
||
|
|
'name': 'test line',
|
||
|
|
'price_unit': 0.025,
|
||
|
|
'quantity': 1,
|
||
|
|
'account_id': self.company_data['default_account_revenue'].id,
|
||
|
|
})],
|
||
|
|
})
|
||
|
|
|
||
|
|
check_invoice_values(invoice_2)
|
||
|
|
|
||
|
|
def test_out_invoice_line_onchange_rounding_price_subtotal_2(self):
|
||
|
|
""" Ensure the cyclic computations implemented using onchanges are not leading to rounding issues when using
|
||
|
|
price-included taxes.
|
||
|
|
For example:
|
||
|
|
100 / 1.21 ~= 82.64 but 82.64 * 1.21 ~= 99.99 != 100.0.
|
||
|
|
"""
|
||
|
|
|
||
|
|
def check_invoice_values(invoice):
|
||
|
|
self.assertInvoiceValues(invoice, [
|
||
|
|
{
|
||
|
|
'price_unit': 100.0,
|
||
|
|
'price_subtotal': 82.64,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 82.64,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 17.36,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'debit': 100.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
'amount_untaxed': 82.64,
|
||
|
|
'amount_tax': 17.36,
|
||
|
|
'amount_total': 100.0,
|
||
|
|
})
|
||
|
|
|
||
|
|
tax = self.env['account.tax'].create({
|
||
|
|
'name': '21%',
|
||
|
|
'amount': 21.0,
|
||
|
|
'price_include': True,
|
||
|
|
'include_base_amount': True,
|
||
|
|
})
|
||
|
|
|
||
|
|
# == Test assigning tax directly ==
|
||
|
|
|
||
|
|
invoice_create = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'invoice_date': '2017-01-01',
|
||
|
|
'date': '2017-01-01',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_line_ids': [(0, 0, {
|
||
|
|
'name': 'test line',
|
||
|
|
'price_unit': 100.0,
|
||
|
|
'account_id': self.company_data['default_account_revenue'].id,
|
||
|
|
'tax_ids': [(6, 0, tax.ids)],
|
||
|
|
})],
|
||
|
|
})
|
||
|
|
|
||
|
|
check_invoice_values(invoice_create)
|
||
|
|
|
||
|
|
move_form = Form(self.env['account.move'].with_context(default_move_type='out_invoice'))
|
||
|
|
move_form.invoice_date = fields.Date.from_string('2017-01-01')
|
||
|
|
move_form.partner_id = self.partner_a
|
||
|
|
with move_form.invoice_line_ids.new() as line_form:
|
||
|
|
line_form.name = 'test line'
|
||
|
|
line_form.price_unit = 100.0
|
||
|
|
line_form.account_id = self.company_data['default_account_revenue']
|
||
|
|
line_form.tax_ids.clear()
|
||
|
|
line_form.tax_ids.add(tax)
|
||
|
|
invoice_onchange = move_form.save()
|
||
|
|
|
||
|
|
check_invoice_values(invoice_onchange)
|
||
|
|
|
||
|
|
# == Test when the tax is set on a product ==
|
||
|
|
|
||
|
|
product = self.env['product.product'].create({
|
||
|
|
'name': 'product',
|
||
|
|
'lst_price': 100.0,
|
||
|
|
'property_account_income_id': self.company_data['default_account_revenue'].id,
|
||
|
|
'taxes_id': [(6, 0, tax.ids)],
|
||
|
|
})
|
||
|
|
|
||
|
|
move_form = Form(self.env['account.move'].with_context(default_move_type='out_invoice'))
|
||
|
|
move_form.invoice_date = fields.Date.from_string('2017-01-01')
|
||
|
|
move_form.partner_id = self.partner_a
|
||
|
|
with move_form.invoice_line_ids.new() as line_form:
|
||
|
|
line_form.product_id = product
|
||
|
|
invoice_onchange = move_form.save()
|
||
|
|
|
||
|
|
check_invoice_values(invoice_onchange)
|
||
|
|
|
||
|
|
# == Test with a fiscal position ==
|
||
|
|
|
||
|
|
fiscal_position = self.env['account.fiscal.position'].create({'name': 'fiscal_position'})
|
||
|
|
|
||
|
|
move_form = Form(self.env['account.move'].with_context(default_move_type='out_invoice'))
|
||
|
|
move_form.invoice_date = fields.Date.from_string('2017-01-01')
|
||
|
|
move_form.partner_id = self.partner_a
|
||
|
|
move_form.fiscal_position_id = fiscal_position
|
||
|
|
with move_form.invoice_line_ids.new() as line_form:
|
||
|
|
line_form.product_id = product
|
||
|
|
invoice_onchange = move_form.save()
|
||
|
|
|
||
|
|
check_invoice_values(invoice_onchange)
|
||
|
|
|
||
|
|
def test_out_invoice_line_onchange_taxes_2_price_unit_tax_included(self):
|
||
|
|
''' Seek for rounding issue in the price unit. Suppose a price_unit of 2300 with a 5.5% price-included tax
|
||
|
|
applied on it.
|
||
|
|
|
||
|
|
The computed balance will be computed as follow: 2300.0 / 1.055 = 2180.0948 ~ 2180.09.
|
||
|
|
Since accounting / business fields are synchronized, the inverse computation will try to recompute the
|
||
|
|
price_unit based on the balance: 2180.09 * 1.055 = 2299.99495 ~ 2299.99.
|
||
|
|
|
||
|
|
This test ensures the price_unit is not overridden in such case.
|
||
|
|
'''
|
||
|
|
tax_price_include = self.env['account.tax'].create({
|
||
|
|
'name': 'Tax 5.5% price included',
|
||
|
|
'amount': 5.5,
|
||
|
|
'amount_type': 'percent',
|
||
|
|
'price_include': True,
|
||
|
|
})
|
||
|
|
|
||
|
|
# == Single-currency ==
|
||
|
|
|
||
|
|
# price_unit=2300 with 15% tax (excluded) + 5.5% tax (included).
|
||
|
|
move_form = Form(self.invoice)
|
||
|
|
move_form.invoice_line_ids.remove(1)
|
||
|
|
with move_form.invoice_line_ids.edit(0) as line_form:
|
||
|
|
line_form.price_unit = 2300
|
||
|
|
line_form.tax_ids.add(tax_price_include)
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'price_unit': 2300.0,
|
||
|
|
'price_subtotal': 2180.09,
|
||
|
|
'price_total': 2627.01,
|
||
|
|
'tax_ids': (self.product_a.taxes_id + tax_price_include).ids,
|
||
|
|
'amount_currency': -2180.09,
|
||
|
|
'credit': 2180.09,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'amount_currency': -327.01,
|
||
|
|
'credit': 327.01,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'name': tax_price_include.name,
|
||
|
|
'product_id': False,
|
||
|
|
'account_id': self.product_line_vals_1['account_id'],
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'product_uom_id': False,
|
||
|
|
'quantity': False,
|
||
|
|
'discount': 0.0,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_line_id': tax_price_include.id,
|
||
|
|
'currency_id': self.company_data['currency'].id,
|
||
|
|
'amount_currency': -119.91,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 119.91,
|
||
|
|
'date_maturity': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'amount_currency': 2627.01,
|
||
|
|
'debit': 2627.01,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'amount_untaxed': 2180.09,
|
||
|
|
'amount_tax': 446.92,
|
||
|
|
'amount_total': 2627.01,
|
||
|
|
})
|
||
|
|
|
||
|
|
move_form = Form(self.invoice)
|
||
|
|
with move_form.invoice_line_ids.edit(0) as line_form:
|
||
|
|
line_form.price_unit = -2300
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'price_unit': -2300.0,
|
||
|
|
'price_subtotal': -2180.09,
|
||
|
|
'price_total': -2627.01,
|
||
|
|
'tax_ids': (self.product_a.taxes_id + tax_price_include).ids,
|
||
|
|
'amount_currency': 2180.09,
|
||
|
|
'debit': 2180.09,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'amount_currency': 327.01,
|
||
|
|
'debit': 327.01,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'name': tax_price_include.name,
|
||
|
|
'product_id': False,
|
||
|
|
'account_id': self.product_line_vals_1['account_id'],
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'product_uom_id': False,
|
||
|
|
'quantity': False,
|
||
|
|
'discount': 0.0,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_line_id': tax_price_include.id,
|
||
|
|
'currency_id': self.company_data['currency'].id,
|
||
|
|
'amount_currency': 119.91,
|
||
|
|
'debit': 119.91,
|
||
|
|
'credit': 0.0,
|
||
|
|
'date_maturity': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'amount_currency': -2627.01,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 2627.01,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'amount_untaxed': -2180.09,
|
||
|
|
'amount_tax': -446.92,
|
||
|
|
'amount_total': -2627.01,
|
||
|
|
})
|
||
|
|
|
||
|
|
# == Multi-currencies ==
|
||
|
|
|
||
|
|
move_form = Form(self.invoice)
|
||
|
|
move_form.currency_id = self.currency_data['currency']
|
||
|
|
with move_form.invoice_line_ids.edit(0) as line_form:
|
||
|
|
line_form.price_unit = 2300
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'price_unit': 2300.0,
|
||
|
|
'price_subtotal': 2180.095,
|
||
|
|
'price_total': 2627.014,
|
||
|
|
'tax_ids': (self.product_a.taxes_id + tax_price_include).ids,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -2180.095,
|
||
|
|
'credit': 1090.05,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -327.014,
|
||
|
|
'credit': 163.51,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'name': tax_price_include.name,
|
||
|
|
'product_id': False,
|
||
|
|
'account_id': self.product_line_vals_1['account_id'],
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'product_uom_id': False,
|
||
|
|
'quantity': False,
|
||
|
|
'discount': 0.0,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_line_id': tax_price_include.id,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -119.905,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 59.95,
|
||
|
|
'date_maturity': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 2627.014,
|
||
|
|
'debit': 1313.51,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_untaxed': 2180.095,
|
||
|
|
'amount_tax': 446.919,
|
||
|
|
'amount_total': 2627.014,
|
||
|
|
})
|
||
|
|
|
||
|
|
move_form = Form(self.invoice)
|
||
|
|
with move_form.invoice_line_ids.edit(0) as line_form:
|
||
|
|
line_form.price_unit = -2300
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'price_unit': -2300.0,
|
||
|
|
'price_subtotal': -2180.095,
|
||
|
|
'price_total': -2627.014,
|
||
|
|
'tax_ids': (self.product_a.taxes_id + tax_price_include).ids,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 2180.095,
|
||
|
|
'debit': 1090.05,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 327.014,
|
||
|
|
'debit': 163.51,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'name': tax_price_include.name,
|
||
|
|
'product_id': False,
|
||
|
|
'account_id': self.product_line_vals_1['account_id'],
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'product_uom_id': False,
|
||
|
|
'quantity': False,
|
||
|
|
'discount': 0.0,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_line_id': tax_price_include.id,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 119.905,
|
||
|
|
'debit': 59.95,
|
||
|
|
'credit': 0.0,
|
||
|
|
'date_maturity': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -2627.014,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 1313.51,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_untaxed': -2180.095,
|
||
|
|
'amount_tax': -446.919,
|
||
|
|
'amount_total': -2627.014,
|
||
|
|
})
|
||
|
|
|
||
|
|
def test_payment_term_line_fiscal_position(self):
|
||
|
|
"""Test the mapping of payment term line accounts with fiscal position."""
|
||
|
|
account_revenue_copy = self.company_data['default_account_revenue'].copy()
|
||
|
|
account_receivable_copy = self.company_data['default_account_receivable'].copy()
|
||
|
|
fp = self.env['account.fiscal.position'].create({
|
||
|
|
'name': 'Test FP',
|
||
|
|
'account_ids': [
|
||
|
|
Command.create({
|
||
|
|
'account_src_id': self.company_data['default_account_revenue'].id,
|
||
|
|
'account_dest_id': account_revenue_copy.id,
|
||
|
|
}),
|
||
|
|
Command.create({
|
||
|
|
'account_src_id': self.company_data['default_account_receivable'].id,
|
||
|
|
'account_dest_id': account_receivable_copy.id,
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
invoice = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'invoice_date': '2019-01-01',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'fiscal_position_id': fp.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'product_id': self.product_a.id,
|
||
|
|
'price_unit': 295.0,
|
||
|
|
'tax_ids': [(6, 0, self.product_a.taxes_id.ids)],
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
invoice.action_post()
|
||
|
|
|
||
|
|
self.assertIn(account_receivable_copy, invoice.line_ids.account_id)
|
||
|
|
self.assertIn(account_revenue_copy, invoice.line_ids.account_id)
|
||
|
|
|
||
|
|
def test_out_invoice_line_onchange_analytic(self):
|
||
|
|
self.env.user.groups_id += self.env.ref('analytic.group_analytic_accounting')
|
||
|
|
|
||
|
|
analytic_plan = self.env['account.analytic.plan'].create({'name': 'Plan Test'})
|
||
|
|
analytic_account = self.env['account.analytic.account'].create({
|
||
|
|
'name': 'test_analytic_account',
|
||
|
|
'partner_id': self.invoice.partner_id.id,
|
||
|
|
'plan_id': analytic_plan.id,
|
||
|
|
'code': 'TEST'
|
||
|
|
})
|
||
|
|
|
||
|
|
analytic_distribution = {str(analytic_account.id): 100.00}
|
||
|
|
|
||
|
|
move_form = Form(self.invoice)
|
||
|
|
with move_form.invoice_line_ids.edit(0) as line_form:
|
||
|
|
line_form.analytic_distribution = analytic_distribution
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
# The tax is not flagged as an analytic one. It should change nothing on the taxes.
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'analytic_distribution': analytic_distribution,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'analytic_distribution': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'analytic_distribution': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'analytic_distribution': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'analytic_distribution': False,
|
||
|
|
},
|
||
|
|
], self.move_vals)
|
||
|
|
|
||
|
|
move_form = Form(self.invoice)
|
||
|
|
with move_form.invoice_line_ids.edit(0) as line_form:
|
||
|
|
line_form.analytic_distribution = {}
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
# Enable the analytic
|
||
|
|
self.tax_sale_a.analytic = True
|
||
|
|
|
||
|
|
move_form = Form(self.invoice)
|
||
|
|
with move_form.invoice_line_ids.edit(0) as line_form:
|
||
|
|
line_form.analytic_distribution = analytic_distribution
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
# The tax is flagged as an analytic one.
|
||
|
|
# A new tax line must be generated.
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'analytic_distribution': analytic_distribution,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'analytic_distribution': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'amount_currency': -150.0,
|
||
|
|
'credit': 150.0,
|
||
|
|
'analytic_distribution': analytic_distribution,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'amount_currency': -30.0,
|
||
|
|
'credit': 30.0,
|
||
|
|
'analytic_distribution': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'analytic_distribution': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'analytic_distribution': False,
|
||
|
|
},
|
||
|
|
], self.move_vals)
|
||
|
|
|
||
|
|
move_form = Form(self.invoice)
|
||
|
|
with move_form.invoice_line_ids.edit(0) as line_form:
|
||
|
|
line_form.analytic_distribution = {}
|
||
|
|
with move_form.invoice_line_ids.edit(1) as line_form:
|
||
|
|
line_form.analytic_distribution = {}
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
# The tax line has been removed.
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'analytic_distribution': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'analytic_distribution': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'analytic_distribution': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'analytic_distribution': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'analytic_distribution': False,
|
||
|
|
},
|
||
|
|
], self.move_vals)
|
||
|
|
|
||
|
|
def test_out_invoice_line_onchange_analytic_2(self):
|
||
|
|
self.env.user.groups_id += self.env.ref('analytic.group_analytic_accounting')
|
||
|
|
|
||
|
|
analytic_plan = self.env['account.analytic.plan'].create({'name': 'Plan Test'})
|
||
|
|
analytic_account = self.env['account.analytic.account'].create({
|
||
|
|
'name': 'test_analytic_account1',
|
||
|
|
'plan_id': analytic_plan.id,
|
||
|
|
'code': 'TEST1'
|
||
|
|
})
|
||
|
|
|
||
|
|
analytic_distribution = {str(analytic_account.id): 100.00}
|
||
|
|
|
||
|
|
self.invoice.write({'invoice_line_ids': [(1, self.invoice.invoice_line_ids.ids[0], {
|
||
|
|
'analytic_distribution': analytic_distribution,
|
||
|
|
})]})
|
||
|
|
|
||
|
|
self.assertRecordValues(self.invoice.invoice_line_ids, [
|
||
|
|
{'analytic_distribution': analytic_distribution},
|
||
|
|
{'analytic_distribution': False},
|
||
|
|
])
|
||
|
|
|
||
|
|
# We can remove the analytic account, it is not recomputed by an invalidation
|
||
|
|
self.invoice.write({'invoice_line_ids': [(1, self.invoice.invoice_line_ids.ids[0], {
|
||
|
|
'analytic_distribution': False,
|
||
|
|
})]})
|
||
|
|
|
||
|
|
self.assertRecordValues(self.invoice.invoice_line_ids, [
|
||
|
|
{'analytic_distribution': False},
|
||
|
|
{'analytic_distribution': False},
|
||
|
|
])
|
||
|
|
|
||
|
|
def test_out_invoice_line_onchange_cash_rounding_1(self):
|
||
|
|
# Required for `invoice_cash_rounding_id` to be visible in the view
|
||
|
|
self.env.user.groups_id += self.env.ref('account.group_cash_rounding')
|
||
|
|
# Test 'add_invoice_line' rounding
|
||
|
|
move_form = Form(self.invoice)
|
||
|
|
# Add a cash rounding having 'add_invoice_line'.
|
||
|
|
move_form.invoice_cash_rounding_id = self.cash_rounding_a
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
# The cash rounding does nothing as the total is already rounded.
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
self.product_line_vals_1,
|
||
|
|
self.product_line_vals_2,
|
||
|
|
self.tax_line_vals_1,
|
||
|
|
self.tax_line_vals_2,
|
||
|
|
self.term_line_vals_1,
|
||
|
|
], self.move_vals)
|
||
|
|
|
||
|
|
move_form = Form(self.invoice)
|
||
|
|
with move_form.invoice_line_ids.edit(0) as line_form:
|
||
|
|
line_form.price_unit = 999.99
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'price_unit': 999.99,
|
||
|
|
'price_subtotal': 999.99,
|
||
|
|
'price_total': 1149.99,
|
||
|
|
'amount_currency': -999.99,
|
||
|
|
'credit': 999.99,
|
||
|
|
},
|
||
|
|
self.product_line_vals_2,
|
||
|
|
self.tax_line_vals_1,
|
||
|
|
self.tax_line_vals_2,
|
||
|
|
{
|
||
|
|
'name': 'add_invoice_line',
|
||
|
|
'product_id': False,
|
||
|
|
'account_id': self.cash_rounding_a.profit_account_id.id,
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'product_uom_id': False,
|
||
|
|
'quantity': False,
|
||
|
|
'discount': 0.0,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_line_id': False,
|
||
|
|
'currency_id': self.company_data['currency'].id,
|
||
|
|
'amount_currency': -0.01,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 0.01,
|
||
|
|
'date_maturity': False,
|
||
|
|
},
|
||
|
|
self.term_line_vals_1,
|
||
|
|
], self.move_vals)
|
||
|
|
|
||
|
|
# Test 'biggest_tax' rounding
|
||
|
|
|
||
|
|
self.company_data['company'].country_id = self.env.ref('base.us')
|
||
|
|
|
||
|
|
# Add a tag to product_a's default tax
|
||
|
|
tax_line_tag = self.env['account.account.tag'].create({
|
||
|
|
'name': "Tax tag",
|
||
|
|
'applicability': 'taxes',
|
||
|
|
'country_id': self.company_data['company'].country_id.id,
|
||
|
|
})
|
||
|
|
|
||
|
|
repartition_line = self.tax_sale_a.invoice_repartition_line_ids.filtered(lambda x: x.repartition_type == 'tax')
|
||
|
|
repartition_line.write({'tag_ids': [(4, tax_line_tag.id, 0)]})
|
||
|
|
|
||
|
|
# Create the invoice
|
||
|
|
biggest_tax_invoice = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'invoice_date': '2019-01-01',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_cash_rounding_id': self.cash_rounding_b.id,
|
||
|
|
'invoice_payment_term_id': self.pay_terms_a.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
(0, 0, {
|
||
|
|
'product_id': self.product_a.id,
|
||
|
|
'price_unit': 999.99,
|
||
|
|
'tax_ids': [(6, 0, self.product_a.taxes_id.ids)],
|
||
|
|
'product_uom_id': self.product_a.uom_id.id,
|
||
|
|
}),
|
||
|
|
|
||
|
|
(0, 0, {
|
||
|
|
'product_id': self.product_b.id,
|
||
|
|
'price_unit': self.product_b.lst_price,
|
||
|
|
'tax_ids': [(6, 0, self.product_b.taxes_id.ids)],
|
||
|
|
'product_uom_id': self.product_b.uom_id.id,
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
|
||
|
|
self.assertInvoiceValues(biggest_tax_invoice, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'price_unit': 999.99,
|
||
|
|
'price_subtotal': 999.99,
|
||
|
|
'price_total': 1149.99,
|
||
|
|
'amount_currency': -999.99,
|
||
|
|
'credit': 999.99,
|
||
|
|
'tax_repartition_line_id': None,
|
||
|
|
'tax_tag_ids': [],
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'tax_repartition_line_id': None,
|
||
|
|
'tax_tag_ids': [],
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'tax_repartition_line_id': repartition_line.id,
|
||
|
|
'tax_tag_ids': tax_line_tag.ids,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'tax_repartition_line_id': self.tax_sale_b.invoice_repartition_line_ids.filtered(lambda x: x.repartition_type == 'tax').id,
|
||
|
|
'tax_tag_ids': [],
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'name': '%s (rounding)' % self.tax_sale_a.name,
|
||
|
|
'product_id': False,
|
||
|
|
'account_id': self.company_data['default_account_tax_sale'].id,
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'product_uom_id': False,
|
||
|
|
'quantity': False,
|
||
|
|
'discount': 0.0,
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_line_id': self.tax_sale_a.id,
|
||
|
|
'tax_repartition_line_id': repartition_line.id,
|
||
|
|
'tax_tag_ids': tax_line_tag.ids,
|
||
|
|
'currency_id': self.company_data['currency'].id,
|
||
|
|
'amount_currency': 0.04,
|
||
|
|
'debit': 0.04,
|
||
|
|
'credit': 0.0,
|
||
|
|
'date_maturity': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'amount_currency': 1409.95,
|
||
|
|
'debit': 1409.95,
|
||
|
|
'tax_repartition_line_id': None,
|
||
|
|
'tax_tag_ids': [],
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'amount_untaxed': 1199.99,
|
||
|
|
'amount_tax': 209.96,
|
||
|
|
'amount_total': 1409.95,
|
||
|
|
})
|
||
|
|
|
||
|
|
def test_out_invoice_line_onchange_currency_1(self):
|
||
|
|
move_form = Form(self.invoice.with_context(dudu=True))
|
||
|
|
move_form.currency_id = self.currency_data['currency']
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -1000.0,
|
||
|
|
'credit': 500.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -200.0,
|
||
|
|
'credit': 100.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -180.0,
|
||
|
|
'credit': 90.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -30.0,
|
||
|
|
'credit': 15.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 1410.0,
|
||
|
|
'debit': 705.0,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
})
|
||
|
|
|
||
|
|
# Change the date to get another rate: 1/3 instead of 1/2.
|
||
|
|
with Form(self.invoice) as move_form:
|
||
|
|
move_form.invoice_date = fields.Date.from_string('2016-01-01')
|
||
|
|
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -1000.0,
|
||
|
|
'credit': 333.33,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -200.0,
|
||
|
|
'credit': 66.67,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -180.0,
|
||
|
|
'credit': 60.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -30.0,
|
||
|
|
'credit': 10.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 1410.0,
|
||
|
|
'debit': 470.0,
|
||
|
|
'date_maturity': fields.Date.from_string('2016-01-01'),
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'date': fields.Date.from_string('2016-01-01'),
|
||
|
|
})
|
||
|
|
|
||
|
|
move_form = Form(self.invoice)
|
||
|
|
with move_form.invoice_line_ids.edit(0) as line_form:
|
||
|
|
# 0.045 * 0.1 = 0.0045. As the foreign currency has a 0.001 rounding,
|
||
|
|
# the result should be 0.005 after rounding.
|
||
|
|
line_form.quantity = 0.1
|
||
|
|
line_form.price_unit = 0.045
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'quantity': 0.1,
|
||
|
|
'price_unit': 0.05,
|
||
|
|
'price_subtotal': 0.005,
|
||
|
|
'price_total': 0.006,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -0.005,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -200.0,
|
||
|
|
'credit': 66.67,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -30.001,
|
||
|
|
'credit': 10.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -30.0,
|
||
|
|
'credit': 10.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 260.006,
|
||
|
|
'debit': 86.67,
|
||
|
|
'date_maturity': fields.Date.from_string('2016-01-01'),
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'date': fields.Date.from_string('2016-01-01'),
|
||
|
|
'amount_untaxed': 200.005,
|
||
|
|
'amount_tax': 60.001,
|
||
|
|
'amount_total': 260.006,
|
||
|
|
})
|
||
|
|
|
||
|
|
# Exit the multi-currencies.
|
||
|
|
move_form = Form(self.invoice)
|
||
|
|
move_form.currency_id = self.company_data['currency']
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
self.assertInvoiceValues(self.invoice, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'quantity': 0.1,
|
||
|
|
'price_unit': 0.05,
|
||
|
|
'price_subtotal': 0.01,
|
||
|
|
'price_total': 0.01,
|
||
|
|
'amount_currency': -0.01,
|
||
|
|
'credit': 0.01,
|
||
|
|
},
|
||
|
|
self.product_line_vals_2,
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'amount_currency': -30.0,
|
||
|
|
'credit': 30.0,
|
||
|
|
},
|
||
|
|
self.tax_line_vals_2,
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'amount_currency': 260.01,
|
||
|
|
'debit': 260.01,
|
||
|
|
'date_maturity': fields.Date.from_string('2016-01-01'),
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'currency_id': self.company_data['currency'].id,
|
||
|
|
'date': fields.Date.from_string('2016-01-01'),
|
||
|
|
'amount_untaxed': 200.01,
|
||
|
|
'amount_tax': 60.0,
|
||
|
|
'amount_total': 260.01,
|
||
|
|
})
|
||
|
|
|
||
|
|
def test_out_invoice_line_tax_fixed_price_include_free_product(self):
|
||
|
|
''' Check that fixed tax include are correctly computed while the price_unit is 0
|
||
|
|
'''
|
||
|
|
fixed_tax_price_include = self.env['account.tax'].create({
|
||
|
|
'name': 'BEBAT 0.05',
|
||
|
|
'type_tax_use': 'sale',
|
||
|
|
'amount_type': 'fixed',
|
||
|
|
'amount': 0.05,
|
||
|
|
'price_include': True,
|
||
|
|
'include_base_amount': True,
|
||
|
|
})
|
||
|
|
invoice = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'invoice_date': '2022-03-03',
|
||
|
|
'date': '2022-03-03',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_line_ids': [(0, 0, {
|
||
|
|
'name': 'Free product',
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'account_id': self.company_data['default_account_revenue'].id,
|
||
|
|
'tax_ids': [(6, 0, fixed_tax_price_include.ids)],
|
||
|
|
})],
|
||
|
|
})
|
||
|
|
self.assertInvoiceValues(invoice, [
|
||
|
|
{ 'display_type': 'product',
|
||
|
|
'balance': 0.05,
|
||
|
|
'price_subtotal': -0.05,
|
||
|
|
'price_total': 0.0,
|
||
|
|
}, {
|
||
|
|
'display_type': 'tax',
|
||
|
|
'balance': -0.05,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
}, {
|
||
|
|
'display_type': 'payment_term',
|
||
|
|
'balance': 0,
|
||
|
|
'price_subtotal': 0.0,
|
||
|
|
'price_total': 0.0,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
'amount_untaxed': -0.05,
|
||
|
|
'amount_tax': 0.05,
|
||
|
|
'amount_total': 0.0,
|
||
|
|
})
|
||
|
|
|
||
|
|
def test_out_invoice_line_taxes_fixed_price_include_free_product(self):
|
||
|
|
''' Check that fixed tax include are correctly computed while the price_unit is 0
|
||
|
|
'''
|
||
|
|
# please ensure this test remains consistent with
|
||
|
|
# test_free_product_and_price_include_fixed_tax in the sale module
|
||
|
|
fixed_tax_price_include_1 = self.env['account.tax'].create({
|
||
|
|
'name': 'BEBAT 0.05',
|
||
|
|
'type_tax_use': 'sale',
|
||
|
|
'amount_type': 'fixed',
|
||
|
|
'amount': 0.05,
|
||
|
|
'price_include': True,
|
||
|
|
'include_base_amount': True,
|
||
|
|
})
|
||
|
|
fixed_tax_price_include_2 = self.env['account.tax'].create({
|
||
|
|
'name': 'Recupel 0.25',
|
||
|
|
'type_tax_use': 'sale',
|
||
|
|
'amount_type': 'fixed',
|
||
|
|
'amount': 0.25,
|
||
|
|
'price_include': True,
|
||
|
|
'include_base_amount': True,
|
||
|
|
})
|
||
|
|
invoice = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'invoice_date': '2022-03-03',
|
||
|
|
'date': '2022-03-03',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_line_ids': [(0, 0, {
|
||
|
|
'name': 'Free product',
|
||
|
|
'price_unit': 0.0,
|
||
|
|
'account_id': self.company_data['default_account_revenue'].id,
|
||
|
|
'tax_ids': [(6, 0, (fixed_tax_price_include_1 + fixed_tax_price_include_2).ids)],
|
||
|
|
})],
|
||
|
|
})
|
||
|
|
|
||
|
|
self.assertRecordValues(invoice, [{
|
||
|
|
'amount_untaxed': -0.30,
|
||
|
|
'amount_tax': 0.30,
|
||
|
|
'amount_total': 0.0,
|
||
|
|
}])
|
||
|
|
|
||
|
|
def test_out_invoice_create_refund(self):
|
||
|
|
self.invoice.action_post()
|
||
|
|
|
||
|
|
move_reversal = self.env['account.move.reversal'].with_context(active_model="account.move", active_ids=self.invoice.ids).create({
|
||
|
|
'date': fields.Date.from_string('2019-02-01'),
|
||
|
|
'reason': 'no reason',
|
||
|
|
'journal_id': self.invoice.journal_id.id,
|
||
|
|
})
|
||
|
|
reversal = move_reversal.refund_moves()
|
||
|
|
reverse_move = self.env['account.move'].browse(reversal['res_id'])
|
||
|
|
|
||
|
|
self.assertEqual(self.invoice.payment_state, 'not_paid', "Refunding with a draft credit note should keep the invoice 'not_paid'.")
|
||
|
|
self.assertInvoiceValues(reverse_move, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'amount_currency': 1000.0,
|
||
|
|
'debit': 1000.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
'tax_tag_invert': False,
|
||
|
|
'tax_base_amount': 0.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'amount_currency': 200.0,
|
||
|
|
'debit': 200.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
'tax_tag_invert': False,
|
||
|
|
'tax_base_amount': 0.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'amount_currency': 180.0,
|
||
|
|
'debit': 180.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
'tax_tag_invert': False,
|
||
|
|
'tax_base_amount': 1200.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'amount_currency': 30.0,
|
||
|
|
'debit': 30.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
'tax_tag_invert': False,
|
||
|
|
'tax_base_amount': 200.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'name': '',
|
||
|
|
'amount_currency': -1410.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 1410.0,
|
||
|
|
'date_maturity': move_reversal.date,
|
||
|
|
'tax_tag_invert': False,
|
||
|
|
'tax_base_amount': 0.0,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'invoice_payment_term_id': None,
|
||
|
|
'name': 'RINV/2019/00001',
|
||
|
|
'date': move_reversal.date,
|
||
|
|
'state': 'draft',
|
||
|
|
'ref': 'Reversal of: %s, %s' % (self.invoice.name, move_reversal.reason),
|
||
|
|
'payment_state': 'not_paid',
|
||
|
|
})
|
||
|
|
|
||
|
|
move_reversal = self.env['account.move.reversal'].with_context(active_model="account.move", active_ids=self.invoice.ids).create({
|
||
|
|
'date': fields.Date.from_string('2019-02-01'),
|
||
|
|
'reason': 'no reason',
|
||
|
|
'journal_id': self.invoice.journal_id.id,
|
||
|
|
})
|
||
|
|
reversal = move_reversal.modify_moves()
|
||
|
|
new_move = self.env['account.move'].browse(reversal['res_id'])
|
||
|
|
|
||
|
|
self.assertEqual(self.invoice.payment_state, 'reversed', "After cancelling it with a reverse invoice, an invoice should be in 'reversed' state.")
|
||
|
|
self.assertInvoiceValues(new_move, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'amount_currency': -1000.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 1000.0,
|
||
|
|
'tax_tag_invert': True,
|
||
|
|
'tax_base_amount': 0.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'amount_currency': -200.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 200.0,
|
||
|
|
'tax_tag_invert': True,
|
||
|
|
'tax_base_amount': 0.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'amount_currency': -180.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 180.0,
|
||
|
|
'tax_tag_invert': True,
|
||
|
|
'tax_base_amount': 1200.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'amount_currency': -30.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 30.0,
|
||
|
|
'tax_tag_invert': True,
|
||
|
|
'tax_base_amount': 200.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'name': '',
|
||
|
|
'amount_currency': 1410.0,
|
||
|
|
'debit': 1410.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
'date_maturity': move_reversal.date,
|
||
|
|
'tax_tag_invert': False,
|
||
|
|
'tax_base_amount': 0.0,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'invoice_payment_term_id': self.pay_terms_a.id,
|
||
|
|
'date': move_reversal.date,
|
||
|
|
'state': 'draft',
|
||
|
|
'ref': False,
|
||
|
|
'payment_state': 'not_paid',
|
||
|
|
})
|
||
|
|
|
||
|
|
def test_out_invoice_create_refund_multi_currency(self):
|
||
|
|
''' Test the account.move.reversal takes care about the currency rates when setting
|
||
|
|
a custom reversal date.
|
||
|
|
'''
|
||
|
|
with Form(self.invoice) as move_form:
|
||
|
|
move_form.invoice_date = '2016-01-01'
|
||
|
|
move_form.currency_id = self.currency_data['currency']
|
||
|
|
|
||
|
|
self.invoice.action_post()
|
||
|
|
|
||
|
|
# The currency rate changed from 1/3 to 1/2.
|
||
|
|
move_reversal = self.env['account.move.reversal'].with_context(active_model="account.move", active_ids=self.invoice.ids).create({
|
||
|
|
'date': fields.Date.from_string('2017-01-01'),
|
||
|
|
'reason': 'no reason',
|
||
|
|
'journal_id': self.invoice.journal_id.id,
|
||
|
|
})
|
||
|
|
reversal = move_reversal.refund_moves()
|
||
|
|
reverse_move = self.env['account.move'].browse(reversal['res_id'])
|
||
|
|
|
||
|
|
self.assertEqual(self.invoice.payment_state, 'not_paid', "Refunding with a draft credit note should keep the invoice 'not_paid'.")
|
||
|
|
self.assertInvoiceValues(reverse_move, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'amount_currency': 1000.0,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'debit': 500.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'amount_currency': 200.0,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'debit': 100.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'amount_currency': 180.0,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'debit': 90.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'amount_currency': 30.0,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'debit': 15.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'name': '',
|
||
|
|
'amount_currency': -1410.0,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 705.0,
|
||
|
|
'date_maturity': move_reversal.date,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'invoice_payment_term_id': None,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'date': move_reversal.date,
|
||
|
|
'state': 'draft',
|
||
|
|
'ref': 'Reversal of: %s, %s' % (self.invoice.name, move_reversal.reason),
|
||
|
|
'payment_state': 'not_paid',
|
||
|
|
})
|
||
|
|
|
||
|
|
move_reversal = self.env['account.move.reversal'].with_context(active_model="account.move", active_ids=self.invoice.ids).create({
|
||
|
|
'date': fields.Date.from_string('2017-01-01'),
|
||
|
|
'reason': 'no reason',
|
||
|
|
'journal_id': self.invoice.journal_id.id,
|
||
|
|
})
|
||
|
|
reversal = move_reversal.modify_moves()
|
||
|
|
new_move = self.env['account.move'].browse(reversal['res_id'])
|
||
|
|
|
||
|
|
self.assertEqual(self.invoice.payment_state, 'reversed', "After cancelling it with a reverse invoice, an invoice should be in 'reversed' state.")
|
||
|
|
self.assertInvoiceValues(new_move, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'amount_currency': -1000.0,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 500.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'amount_currency': -200.0,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 100.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'amount_currency': -180.0,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 90.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'amount_currency': -30.0,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 15.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'name': '',
|
||
|
|
'amount_currency': 1410.0,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'debit': 705.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
'date_maturity': move_reversal.date,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'invoice_payment_term_id': self.pay_terms_a.id,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'date': move_reversal.date,
|
||
|
|
'state': 'draft',
|
||
|
|
'ref': False,
|
||
|
|
'payment_state': 'not_paid',
|
||
|
|
})
|
||
|
|
|
||
|
|
def test_out_invoice_create_refund_auto_post(self):
|
||
|
|
self.invoice.action_post()
|
||
|
|
|
||
|
|
move_reversal = self.env['account.move.reversal'].with_context(active_model="account.move", active_ids=self.invoice.ids).create({
|
||
|
|
'date': fields.Date.today() + timedelta(days=7),
|
||
|
|
'reason': 'no reason',
|
||
|
|
'journal_id': self.invoice.journal_id.id,
|
||
|
|
})
|
||
|
|
move_reversal.modify_moves()
|
||
|
|
refund = self.env['account.move'].search([('move_type', '=', 'out_refund'), ('company_id', '=', self.invoice.company_id.id)])
|
||
|
|
|
||
|
|
self.assertRecordValues(refund, [{
|
||
|
|
'state': 'draft',
|
||
|
|
'auto_post': 'at_date',
|
||
|
|
}])
|
||
|
|
|
||
|
|
def test_out_invoice_create_1(self):
|
||
|
|
# Test creating an account_move with the least information.
|
||
|
|
move = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_date': fields.Date.from_string('2019-01-01'),
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'invoice_payment_term_id': self.pay_terms_a.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
(0, None, {
|
||
|
|
'product_id': self.product_a.id,
|
||
|
|
'product_uom_id': self.product_a.uom_id.id,
|
||
|
|
'quantity': 1.0,
|
||
|
|
'price_unit': 1000.0,
|
||
|
|
'tax_ids': [(6, 0, self.product_a.taxes_id.ids)],
|
||
|
|
}),
|
||
|
|
(0, None, {
|
||
|
|
'product_id': self.product_b.id,
|
||
|
|
'product_uom_id': self.product_b.uom_id.id,
|
||
|
|
'quantity': 1.0,
|
||
|
|
'price_unit': 200.0,
|
||
|
|
'tax_ids': [(6, 0, self.product_b.taxes_id.ids)],
|
||
|
|
}),
|
||
|
|
]
|
||
|
|
})
|
||
|
|
|
||
|
|
self.assertInvoiceValues(move, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -1000.0,
|
||
|
|
'credit': 500.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -200.0,
|
||
|
|
'credit': 100.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -180.0,
|
||
|
|
'credit': 90.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -30.0,
|
||
|
|
'credit': 15.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 1410.0,
|
||
|
|
'debit': 705.0,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
})
|
||
|
|
|
||
|
|
def test_out_invoice_create_child_partner(self):
|
||
|
|
# Test creating an account_move on a child partner.
|
||
|
|
# This needs to attach the lines to the parent partner id.
|
||
|
|
partner_a_child = self.env['res.partner'].create({
|
||
|
|
'name': 'partner_a_child',
|
||
|
|
'parent_id': self.partner_a.id
|
||
|
|
})
|
||
|
|
move = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'partner_id': partner_a_child.id,
|
||
|
|
'invoice_date': fields.Date.from_string('2019-01-01'),
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'invoice_payment_term_id': self.pay_terms_a.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'product_id': self.product_line_vals_1['product_id'],
|
||
|
|
'product_uom_id': self.product_line_vals_1['product_uom_id'],
|
||
|
|
'price_unit': self.product_line_vals_1['price_unit'],
|
||
|
|
'tax_ids': [Command.set(self.product_line_vals_1['tax_ids'])],
|
||
|
|
}),
|
||
|
|
Command.create({
|
||
|
|
'product_id': self.product_line_vals_2['product_id'],
|
||
|
|
'product_uom_id': self.product_line_vals_2['product_uom_id'],
|
||
|
|
'price_unit': self.product_line_vals_2['price_unit'],
|
||
|
|
'tax_ids': [Command.set(self.product_line_vals_2['tax_ids'])],
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
|
||
|
|
self.assertEqual(partner_a_child.id, move.partner_id.id, 'Keep child partner on the account move record')
|
||
|
|
self.assertEqual(self.partner_a.id, move.line_ids[0].partner_id.id, 'Set parent partner on the account move line records')
|
||
|
|
|
||
|
|
def test_out_invoice_write_1(self):
|
||
|
|
# Test creating an account_move with the least information.
|
||
|
|
move = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_date': fields.Date.from_string('2019-01-01'),
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'invoice_payment_term_id': self.pay_terms_a.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'product_id': self.product_line_vals_1['product_id'],
|
||
|
|
'product_uom_id': self.product_line_vals_1['product_uom_id'],
|
||
|
|
'price_unit': self.product_line_vals_1['price_unit'],
|
||
|
|
'tax_ids': [Command.set(self.product_line_vals_1['tax_ids'])],
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
move.write({
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'product_id': self.product_line_vals_2['product_id'],
|
||
|
|
'product_uom_id': self.product_line_vals_2['product_uom_id'],
|
||
|
|
'price_unit': self.product_line_vals_2['price_unit'],
|
||
|
|
'tax_ids': [Command.set(self.product_line_vals_2['tax_ids'])],
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
|
||
|
|
self.assertInvoiceValues(move, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -1000.0,
|
||
|
|
'credit': 500.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -200.0,
|
||
|
|
'credit': 100.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -180.0,
|
||
|
|
'credit': 90.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -30.0,
|
||
|
|
'credit': 15.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 1410.0,
|
||
|
|
'debit': 705.0,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
})
|
||
|
|
|
||
|
|
def test_out_invoice_write_2(self):
|
||
|
|
''' Ensure to not messing the invoice when writing a bad account type. '''
|
||
|
|
move = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
(0, None, {
|
||
|
|
'name': 'test_out_invoice_write_2',
|
||
|
|
'quantity': 1.0,
|
||
|
|
'price_unit': 2000,
|
||
|
|
}),
|
||
|
|
]
|
||
|
|
})
|
||
|
|
|
||
|
|
receivable_lines = move.line_ids.filtered(lambda line: line.account_id.account_type == 'asset_receivable')
|
||
|
|
not_receivable_lines = move.line_ids - receivable_lines
|
||
|
|
|
||
|
|
# Write a receivable account on a not-receivable line.
|
||
|
|
with self.assertRaises(UserError), self.cr.savepoint():
|
||
|
|
not_receivable_lines.write({'account_id': receivable_lines[0].account_id.copy().id})
|
||
|
|
|
||
|
|
# Write a not-receivable account on a receivable line.
|
||
|
|
with self.assertRaises(UserError), self.cr.savepoint():
|
||
|
|
receivable_lines.write({'account_id': not_receivable_lines[0].account_id.copy().id})
|
||
|
|
|
||
|
|
# Write another receivable account on a receivable line.
|
||
|
|
receivable_lines.write({'account_id': receivable_lines[0].account_id.copy().id})
|
||
|
|
|
||
|
|
def test_out_invoice_post_1(self):
|
||
|
|
''' Check the invoice_date will be set automatically at the post date. '''
|
||
|
|
frozen_today = fields.Date.today()
|
||
|
|
with patch.object(fields.Date, 'today', lambda *args, **kwargs: frozen_today), patch.object(fields.Date, 'context_today', lambda *args, **kwargs: frozen_today):
|
||
|
|
# Create an invoice with rate 1/3.
|
||
|
|
move = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_date': fields.Date.from_string('2016-01-01'),
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'invoice_payment_term_id': self.pay_terms_a.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'product_id': self.product_line_vals_1['product_id'],
|
||
|
|
'product_uom_id': self.product_line_vals_1['product_uom_id'],
|
||
|
|
'price_unit': self.product_line_vals_1['price_unit'],
|
||
|
|
'tax_ids': [Command.set(self.product_line_vals_1['tax_ids'])],
|
||
|
|
}),
|
||
|
|
Command.create({
|
||
|
|
'product_id': self.product_line_vals_2['product_id'],
|
||
|
|
'product_uom_id': self.product_line_vals_2['product_uom_id'],
|
||
|
|
'price_unit': self.product_line_vals_2['price_unit'],
|
||
|
|
'tax_ids': [Command.set(self.product_line_vals_2['tax_ids'])],
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
|
||
|
|
# Remove the invoice_date to check:
|
||
|
|
# - The invoice_date must be set automatically at today during the post.
|
||
|
|
# - As the invoice_date changed, date did too so the currency rate has changed (1/3 => 1/2).
|
||
|
|
# - A different invoice_date implies also a new date_maturity.
|
||
|
|
# Add a manual edition of a tax line:
|
||
|
|
# - The modification must be preserved in the business fields.
|
||
|
|
# - The journal entry must be balanced before / after the post.
|
||
|
|
move.write({
|
||
|
|
'invoice_date': False,
|
||
|
|
'line_ids': [
|
||
|
|
(1, move.line_ids.filtered(lambda line: line.tax_line_id.id == self.tax_line_vals_1['tax_line_id']).id, {
|
||
|
|
'amount_currency': -200.0,
|
||
|
|
}),
|
||
|
|
(1, move.line_ids.filtered(lambda line: line.date_maturity).id, {
|
||
|
|
'amount_currency': 1430.0,
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
|
||
|
|
move.action_post()
|
||
|
|
|
||
|
|
self.assertInvoiceValues(move, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -1000.0,
|
||
|
|
'credit': 500.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -200.0,
|
||
|
|
'credit': 100.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -180.0,
|
||
|
|
'credit': 90.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -30.0,
|
||
|
|
'credit': 15.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'name': move.name,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 1410.0,
|
||
|
|
'debit': 705.0,
|
||
|
|
'date_maturity': frozen_today,
|
||
|
|
'account_id': self.company_data['default_account_receivable'].id,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'payment_reference': move.name,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'date': frozen_today,
|
||
|
|
'invoice_date': frozen_today,
|
||
|
|
'invoice_date_due': frozen_today,
|
||
|
|
'amount_tax': 210.0,
|
||
|
|
'amount_total': 1410.0,
|
||
|
|
})
|
||
|
|
|
||
|
|
@freeze_time('2017-01-15')
|
||
|
|
def test_out_invoice_post_2(self):
|
||
|
|
''' Check the date will be set automatically at today due to the tax lock date. '''
|
||
|
|
# Create an invoice with rate 1/3.
|
||
|
|
move = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_date': fields.Date.from_string('2017-01-15'),
|
||
|
|
'date': fields.Date.from_string('2015-01-01'),
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'invoice_payment_term_id': self.pay_terms_a.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
(0, None, {
|
||
|
|
'name': self.product_line_vals_1['name'],
|
||
|
|
'product_id': self.product_line_vals_1['product_id'],
|
||
|
|
'product_uom_id': self.product_line_vals_1['product_uom_id'],
|
||
|
|
'quantity': self.product_line_vals_1['quantity'],
|
||
|
|
'price_unit': self.product_line_vals_1['price_unit'],
|
||
|
|
'tax_ids': self.product_line_vals_1['tax_ids'],
|
||
|
|
}),
|
||
|
|
(0, None, {
|
||
|
|
'name': self.product_line_vals_2['name'],
|
||
|
|
'product_id': self.product_line_vals_2['product_id'],
|
||
|
|
'product_uom_id': self.product_line_vals_2['product_uom_id'],
|
||
|
|
'quantity': self.product_line_vals_2['quantity'],
|
||
|
|
'price_unit': self.product_line_vals_2['price_unit'],
|
||
|
|
'tax_ids': self.product_line_vals_2['tax_ids'],
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
|
||
|
|
# Add a manual edition of a tax line:
|
||
|
|
# - The modification must be preserved in the business fields.
|
||
|
|
# - The journal entry must be balanced before / after the post.
|
||
|
|
move.write({
|
||
|
|
'line_ids': [
|
||
|
|
(1, move.line_ids.filtered(lambda line: line.tax_line_id.id == self.tax_line_vals_1['tax_line_id']).id, {
|
||
|
|
'amount_currency': -200.0,
|
||
|
|
}),
|
||
|
|
(1, move.line_ids.filtered(lambda line: line.date_maturity).id, {
|
||
|
|
'amount_currency': 1430.0,
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
|
||
|
|
# Set the tax lock date:
|
||
|
|
# - The date must be set automatically at the date of today (2017-01-15).
|
||
|
|
# - As the date changed, the currency rate has changed (1/3 => 1/2).
|
||
|
|
move.company_id.tax_lock_date = fields.Date.from_string('2016-12-31')
|
||
|
|
|
||
|
|
move.action_post()
|
||
|
|
|
||
|
|
self.assertInvoiceValues(move, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -1000.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 500.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -200.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 100.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -200.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 100.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -30.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 15.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'name': move.name,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 1430.0,
|
||
|
|
'debit': 715.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
'date_maturity': fields.Date.from_string('2017-01-15'),
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'payment_reference': move.name,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'date': fields.Date.from_string('2017-01-15'),
|
||
|
|
'amount_untaxed': 1200.0,
|
||
|
|
'amount_tax': 230.0,
|
||
|
|
'amount_total': 1430.0,
|
||
|
|
})
|
||
|
|
|
||
|
|
def test_out_invoice_switch_out_refund_1(self):
|
||
|
|
# Test creating an account_move with an out_invoice_type and switch it in an out_refund.
|
||
|
|
move = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_date': fields.Date.from_string('2019-01-01'),
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'invoice_payment_term_id': self.pay_terms_a.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'product_id': self.product_line_vals_1['product_id'],
|
||
|
|
'product_uom_id': self.product_line_vals_1['product_uom_id'],
|
||
|
|
'price_unit': self.product_line_vals_1['price_unit'],
|
||
|
|
'tax_ids': [Command.set(self.product_line_vals_1['tax_ids'])],
|
||
|
|
}),
|
||
|
|
Command.create({
|
||
|
|
'product_id': self.product_line_vals_2['product_id'],
|
||
|
|
'product_uom_id': self.product_line_vals_2['product_uom_id'],
|
||
|
|
'price_unit': self.product_line_vals_2['price_unit'],
|
||
|
|
'tax_ids': [Command.set(self.product_line_vals_2['tax_ids'])],
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
move.action_switch_move_type()
|
||
|
|
|
||
|
|
self.assertRecordValues(move, [{'move_type': 'out_refund'}])
|
||
|
|
self.assertInvoiceValues(move, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 1000.0,
|
||
|
|
'debit': 500.0,
|
||
|
|
'credit': 0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 200.0,
|
||
|
|
'debit': 100.0,
|
||
|
|
'credit': 0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 180.0,
|
||
|
|
'debit': 90.0,
|
||
|
|
'credit': 0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 30.0,
|
||
|
|
'debit': 15.0,
|
||
|
|
'credit': 0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -1410.0,
|
||
|
|
'credit': 705.0,
|
||
|
|
'debit': 0,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
})
|
||
|
|
|
||
|
|
def test_out_invoice_switch_out_refund_2(self):
|
||
|
|
# Test creating an account_move with an out_invoice_type and switch it in an out_refund and a negative quantity.
|
||
|
|
move = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_date': fields.Date.from_string('2019-01-01'),
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'invoice_payment_term_id': self.pay_terms_a.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'product_id': self.product_line_vals_1['product_id'],
|
||
|
|
'product_uom_id': self.product_line_vals_1['product_uom_id'],
|
||
|
|
'price_unit': self.product_line_vals_1['price_unit'],
|
||
|
|
'quantity': -self.product_line_vals_1['quantity'],
|
||
|
|
'tax_ids': [Command.set(self.product_line_vals_1['tax_ids'])],
|
||
|
|
}),
|
||
|
|
Command.create({
|
||
|
|
'product_id': self.product_line_vals_2['product_id'],
|
||
|
|
'product_uom_id': self.product_line_vals_2['product_uom_id'],
|
||
|
|
'price_unit': self.product_line_vals_2['price_unit'],
|
||
|
|
'quantity': -self.product_line_vals_2['quantity'],
|
||
|
|
'tax_ids': [Command.set(self.product_line_vals_2['tax_ids'])],
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
|
||
|
|
self.assertInvoiceValues(move, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 1000.0,
|
||
|
|
'price_subtotal': -1000.0,
|
||
|
|
'price_total': -1150.0,
|
||
|
|
'debit': 500.0,
|
||
|
|
'credit': 0,
|
||
|
|
'quantity': -1.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 200.0,
|
||
|
|
'price_subtotal': -200.0,
|
||
|
|
'price_total': -260.0,
|
||
|
|
'debit': 100.0,
|
||
|
|
'credit': 0,
|
||
|
|
'quantity': -1.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 180.0,
|
||
|
|
'debit': 90.0,
|
||
|
|
'credit': 0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 30.0,
|
||
|
|
'debit': 15.0,
|
||
|
|
'credit': 0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -1410.0,
|
||
|
|
'credit': 705.0,
|
||
|
|
'debit': 0,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_tax' : -self.move_vals['amount_tax'],
|
||
|
|
'amount_total' : -self.move_vals['amount_total'],
|
||
|
|
'amount_untaxed' : -self.move_vals['amount_untaxed'],
|
||
|
|
})
|
||
|
|
|
||
|
|
move.action_switch_move_type()
|
||
|
|
|
||
|
|
self.assertRecordValues(move, [{'move_type': 'out_refund'}])
|
||
|
|
self.assertInvoiceValues(move, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 1000.0,
|
||
|
|
'debit': 500.0,
|
||
|
|
'credit': 0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 200.0,
|
||
|
|
'debit': 100.0,
|
||
|
|
'credit': 0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 180.0,
|
||
|
|
'debit': 90.0,
|
||
|
|
'credit': 0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': 30.0,
|
||
|
|
'debit': 15.0,
|
||
|
|
'credit': 0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -1410.0,
|
||
|
|
'credit': 705.0,
|
||
|
|
'debit': 0,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_tax' : self.move_vals['amount_tax'],
|
||
|
|
'amount_total' : self.move_vals['amount_total'],
|
||
|
|
'amount_untaxed' : self.move_vals['amount_untaxed'],
|
||
|
|
})
|
||
|
|
|
||
|
|
def test_out_invoice_reverse_move_tags(self):
|
||
|
|
country = self.env.ref('base.us')
|
||
|
|
tags = self.env['account.account.tag'].create([{
|
||
|
|
'name': "Test tag %s" % i,
|
||
|
|
'applicability': 'taxes',
|
||
|
|
'country_id': country.id,
|
||
|
|
} for i in range(8)])
|
||
|
|
|
||
|
|
taxes = self.env['account.tax'].create([{
|
||
|
|
'name': "Test tax include_base_amount = %s" % include_base_amount,
|
||
|
|
'amount': 10.0,
|
||
|
|
'include_base_amount': include_base_amount,
|
||
|
|
'invoice_repartition_line_ids': [
|
||
|
|
(0, 0, {
|
||
|
|
'repartition_type': 'base',
|
||
|
|
'tag_ids': [(6, 0, tags[(i * 4)].ids)],
|
||
|
|
}),
|
||
|
|
(0, 0, {
|
||
|
|
'repartition_type': 'tax',
|
||
|
|
'tag_ids': [(6, 0, tags[(i * 4) + 1].ids)],
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
'refund_repartition_line_ids': [
|
||
|
|
(0, 0, {
|
||
|
|
'repartition_type': 'base',
|
||
|
|
'tag_ids': [(6, 0, tags[(i * 4) + 2].ids)],
|
||
|
|
}),
|
||
|
|
(0, 0, {
|
||
|
|
'repartition_type': 'tax',
|
||
|
|
'tag_ids': [(6, 0, tags[(i * 4) + 3].ids)],
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
} for i, include_base_amount in enumerate((True, False))])
|
||
|
|
|
||
|
|
invoice = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_date': fields.Date.from_string('2019-01-01'),
|
||
|
|
'invoice_payment_term_id': self.pay_terms_a.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
(0, 0, {
|
||
|
|
'product_id': self.product_a.id,
|
||
|
|
'price_unit': 1000.0,
|
||
|
|
'tax_ids': [(6, 0, taxes.ids)],
|
||
|
|
}),
|
||
|
|
]
|
||
|
|
})
|
||
|
|
invoice.action_post()
|
||
|
|
|
||
|
|
self.assertRecordValues(invoice.line_ids.sorted('tax_line_id'), [
|
||
|
|
# Product line
|
||
|
|
{'tax_line_id': False, 'tax_ids': taxes.ids, 'tax_tag_ids': (tags[0] + tags[4]).ids},
|
||
|
|
# Receivable line
|
||
|
|
{'tax_line_id': False, 'tax_ids': [], 'tax_tag_ids': []},
|
||
|
|
# Tax lines
|
||
|
|
{'tax_line_id': taxes[0].id, 'tax_ids': taxes[1].ids, 'tax_tag_ids': (tags[1] + tags[4]).ids},
|
||
|
|
{'tax_line_id': taxes[1].id, 'tax_ids': [], 'tax_tag_ids': tags[5].ids},
|
||
|
|
])
|
||
|
|
|
||
|
|
refund = invoice._reverse_moves(cancel=True)
|
||
|
|
|
||
|
|
self.assertRecordValues(refund.line_ids.sorted('tax_line_id'), [
|
||
|
|
# Product line
|
||
|
|
{'tax_line_id': False, 'tax_ids': taxes.ids, 'tax_tag_ids': (tags[2] + tags[6]).ids},
|
||
|
|
# Receivable line
|
||
|
|
{'tax_line_id': False, 'tax_ids': [], 'tax_tag_ids': []},
|
||
|
|
# Tax lines
|
||
|
|
{'tax_line_id': taxes[0].id, 'tax_ids': taxes[1].ids, 'tax_tag_ids': (tags[3] + tags[6]).ids},
|
||
|
|
{'tax_line_id': taxes[1].id, 'tax_ids': [], 'tax_tag_ids': tags[7].ids},
|
||
|
|
])
|
||
|
|
|
||
|
|
def test_out_invoice_change_period_accrual_1(self):
|
||
|
|
move = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'date': '2017-01-01',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_date': fields.Date.from_string('2017-01-01'),
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'invoice_payment_term_id': self.pay_terms_a.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
(0, None, {
|
||
|
|
'name': self.product_line_vals_1['name'],
|
||
|
|
'product_id': self.product_line_vals_1['product_id'],
|
||
|
|
'product_uom_id': self.product_line_vals_1['product_uom_id'],
|
||
|
|
'quantity': self.product_line_vals_1['quantity'],
|
||
|
|
'price_unit': self.product_line_vals_1['price_unit'],
|
||
|
|
'tax_ids': self.product_line_vals_1['tax_ids'],
|
||
|
|
}),
|
||
|
|
(0, None, {
|
||
|
|
'name': self.product_line_vals_2['name'],
|
||
|
|
'product_id': self.product_line_vals_2['product_id'],
|
||
|
|
'product_uom_id': self.product_line_vals_2['product_uom_id'],
|
||
|
|
'quantity': self.product_line_vals_2['quantity'],
|
||
|
|
'price_unit': self.product_line_vals_2['price_unit'],
|
||
|
|
'tax_ids': self.product_line_vals_2['tax_ids'],
|
||
|
|
}),
|
||
|
|
]
|
||
|
|
})
|
||
|
|
move.action_post()
|
||
|
|
|
||
|
|
wizard = self.env['account.automatic.entry.wizard']\
|
||
|
|
.with_context(active_model='account.move.line', active_ids=move.invoice_line_ids.ids).create({
|
||
|
|
'action': 'change_period',
|
||
|
|
'date': '2018-01-01',
|
||
|
|
'percentage': 60,
|
||
|
|
'journal_id': self.company_data['default_journal_misc'].id,
|
||
|
|
'expense_accrual_account': self.env['account.account'].create({
|
||
|
|
'name': 'Accrual Expense Account',
|
||
|
|
'code': '234567',
|
||
|
|
'account_type': 'expense',
|
||
|
|
'reconcile': True,
|
||
|
|
}).id,
|
||
|
|
'revenue_accrual_account': self.env['account.account'].create({
|
||
|
|
'name': 'Accrual Revenue Account',
|
||
|
|
'code': '765432',
|
||
|
|
'account_type': 'expense',
|
||
|
|
'reconcile': True,
|
||
|
|
}).id,
|
||
|
|
})
|
||
|
|
wizard_res = wizard.do_action()
|
||
|
|
|
||
|
|
self.assertInvoiceValues(move, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -1000.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 500.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -200.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 100.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -180.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 90.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -30.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 15.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'name': 'INV/2017/00001',
|
||
|
|
'amount_currency': 1410.0,
|
||
|
|
'debit': 705.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
'date_maturity': fields.Date.from_string('2017-01-01'),
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'date': fields.Date.from_string('2017-01-01'),
|
||
|
|
'payment_reference': 'INV/2017/00001',
|
||
|
|
})
|
||
|
|
|
||
|
|
accrual_lines = self.env['account.move'].browse(wizard_res['domain'][0][2]).line_ids.sorted('date')
|
||
|
|
self.assertRecordValues(accrual_lines, [
|
||
|
|
{'amount_currency': 600.0, 'debit': 300.0, 'credit': 0.0, 'account_id': self.product_line_vals_1['account_id'], 'reconciled': False},
|
||
|
|
{'amount_currency': -600.0, 'debit': 0.0, 'credit': 300.0, 'account_id': wizard.revenue_accrual_account.id, 'reconciled': True},
|
||
|
|
{'amount_currency': 120.0, 'debit': 60.0, 'credit': 0.0, 'account_id': self.product_line_vals_2['account_id'], 'reconciled': False},
|
||
|
|
{'amount_currency': -120.0, 'debit': 0.0, 'credit': 60.0, 'account_id': wizard.revenue_accrual_account.id, 'reconciled': True},
|
||
|
|
{'amount_currency': -600.0, 'debit': 0.0, 'credit': 300.0, 'account_id': self.product_line_vals_1['account_id'], 'reconciled': False},
|
||
|
|
{'amount_currency': 600.0, 'debit': 300.0, 'credit': 0.0, 'account_id': wizard.revenue_accrual_account.id, 'reconciled': True},
|
||
|
|
{'amount_currency': -120.0, 'debit': 0.0, 'credit': 60.0, 'account_id': self.product_line_vals_2['account_id'], 'reconciled': False},
|
||
|
|
{'amount_currency': 120.0, 'debit': 60.0, 'credit': 0.0, 'account_id': wizard.revenue_accrual_account.id, 'reconciled': True},
|
||
|
|
])
|
||
|
|
|
||
|
|
@freeze_time('2019-01-16')
|
||
|
|
def test_out_invoice_change_period_past_move_date(self):
|
||
|
|
move = self.init_invoice(
|
||
|
|
move_type='out_invoice',
|
||
|
|
partner=self.partner_a,
|
||
|
|
invoice_date=fields.Date.from_string('2019-01-01'),
|
||
|
|
amounts=[1000.0],
|
||
|
|
post=True,
|
||
|
|
)
|
||
|
|
|
||
|
|
context = {
|
||
|
|
'default_move_type': 'out_invoice',
|
||
|
|
'active_model': 'account.move.line',
|
||
|
|
'active_ids': move.mapped('invoice_line_ids').ids
|
||
|
|
}
|
||
|
|
wizard = self.env['account.automatic.entry.wizard'] \
|
||
|
|
.with_context(context) \
|
||
|
|
.create({
|
||
|
|
'action': 'change_period',
|
||
|
|
'journal_id': self.company_data['default_journal_misc'],
|
||
|
|
'revenue_accrual_account': self.company_data['default_account_assets'].id,
|
||
|
|
})
|
||
|
|
wizard_res = wizard.do_action()
|
||
|
|
|
||
|
|
accrual_moves = self.env['account.move'].browse(wizard_res['domain'][0][2])
|
||
|
|
self.assertRecordValues(accrual_moves, [
|
||
|
|
{'state': 'posted', 'date': fields.Date.from_string('2019-01-16')},
|
||
|
|
{'state': 'posted', 'date': fields.Date.from_string('2019-01-16')},
|
||
|
|
])
|
||
|
|
|
||
|
|
def test_out_invoice_multi_date_change_period_accrual(self):
|
||
|
|
dates = ['2017-01-01', '2017-01-01', '2017-02-01']
|
||
|
|
values = []
|
||
|
|
for date in dates:
|
||
|
|
values.append({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'date': date,
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_date': fields.Date.from_string(date),
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'invoice_payment_term_id': self.pay_terms_a.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
(0, None, {
|
||
|
|
'name': self.product_line_vals_1['name'],
|
||
|
|
'product_id': self.product_line_vals_1['product_id'],
|
||
|
|
'product_uom_id': self.product_line_vals_1['product_uom_id'],
|
||
|
|
'quantity': self.product_line_vals_1['quantity'],
|
||
|
|
'price_unit': self.product_line_vals_1['price_unit'],
|
||
|
|
'tax_ids': self.product_line_vals_1['tax_ids'],
|
||
|
|
}),
|
||
|
|
(0, None, {
|
||
|
|
'name': self.product_line_vals_2['name'],
|
||
|
|
'product_id': self.product_line_vals_2['product_id'],
|
||
|
|
'product_uom_id': self.product_line_vals_2['product_uom_id'],
|
||
|
|
'quantity': self.product_line_vals_2['quantity'],
|
||
|
|
'price_unit': self.product_line_vals_2['price_unit'],
|
||
|
|
'tax_ids': self.product_line_vals_2['tax_ids'],
|
||
|
|
}),
|
||
|
|
]
|
||
|
|
})
|
||
|
|
|
||
|
|
moves = self.env['account.move'].create(values)
|
||
|
|
moves.action_post()
|
||
|
|
|
||
|
|
wizard = self.env['account.automatic.entry.wizard'].with_context(
|
||
|
|
active_model='account.move.line',
|
||
|
|
active_ids=moves.invoice_line_ids.ids,
|
||
|
|
).create({
|
||
|
|
'action': 'change_period',
|
||
|
|
'date': '2018-01-01',
|
||
|
|
'percentage': 60,
|
||
|
|
'journal_id': self.company_data['default_journal_misc'].id,
|
||
|
|
'expense_accrual_account': self.env['account.account'].create({
|
||
|
|
'name': 'Accrual Expense Account',
|
||
|
|
'code': '234567',
|
||
|
|
'account_type': 'expense',
|
||
|
|
'reconcile': True,
|
||
|
|
}).id,
|
||
|
|
'revenue_accrual_account': self.env['account.account'].create({
|
||
|
|
'name': 'Accrual Revenue Account',
|
||
|
|
'code': '765432',
|
||
|
|
'account_type': 'expense',
|
||
|
|
'reconcile': True,
|
||
|
|
}).id,
|
||
|
|
})
|
||
|
|
wizard_res = wizard.do_action()
|
||
|
|
|
||
|
|
for date, move, ref in zip(dates, moves, ['INV/2017/00001', 'INV/2017/00002', 'INV/2017/00003']):
|
||
|
|
self.assertInvoiceValues(move, [
|
||
|
|
{
|
||
|
|
**self.product_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -1000.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 500.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.product_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -200.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 100.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -180.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 90.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.tax_line_vals_2,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'amount_currency': -30.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 15.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
**self.term_line_vals_1,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'name': ref,
|
||
|
|
'amount_currency': 1410.0,
|
||
|
|
'debit': 705.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
'date_maturity': fields.Date.from_string(date),
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
**self.move_vals,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'date': fields.Date.from_string(date),
|
||
|
|
'payment_reference': ref,
|
||
|
|
})
|
||
|
|
|
||
|
|
moves = self.env['account.move'].browse(wizard_res['domain'][0][2])
|
||
|
|
|
||
|
|
accrual_lines = moves.line_ids.sorted('date')
|
||
|
|
self.assertRecordValues(accrual_lines, [
|
||
|
|
{'amount_currency': 600.0, 'debit': 300.0, 'credit': 0.0, 'account_id': self.product_line_vals_1['account_id'], 'reconciled': False},
|
||
|
|
{'amount_currency': -600.0, 'debit': 0.0, 'credit': 300.0, 'account_id': wizard.revenue_accrual_account.id, 'reconciled': True},
|
||
|
|
{'amount_currency': 120.0, 'debit': 60.0, 'credit': 0.0, 'account_id': self.product_line_vals_2['account_id'], 'reconciled': False},
|
||
|
|
{'amount_currency': -120.0, 'debit': 0.0, 'credit': 60.0, 'account_id': wizard.revenue_accrual_account.id, 'reconciled': True},
|
||
|
|
{'amount_currency': 600.0, 'debit': 300.0, 'credit': 0.0, 'account_id': self.product_line_vals_1['account_id'], 'reconciled': False},
|
||
|
|
{'amount_currency': -600.0, 'debit': 0.0, 'credit': 300.0, 'account_id': wizard.revenue_accrual_account.id, 'reconciled': True},
|
||
|
|
{'amount_currency': 120.0, 'debit': 60.0, 'credit': 0.0, 'account_id': self.product_line_vals_2['account_id'], 'reconciled': False},
|
||
|
|
{'amount_currency': -120.0, 'debit': 0.0, 'credit': 60.0, 'account_id': wizard.revenue_accrual_account.id, 'reconciled': True},
|
||
|
|
{'amount_currency': 600.0, 'debit': 300.0, 'credit': 0.0, 'account_id': self.product_line_vals_1['account_id'], 'reconciled': False},
|
||
|
|
{'amount_currency': -600.0, 'debit': 0.0, 'credit': 300.0, 'account_id': wizard.revenue_accrual_account.id, 'reconciled': True},
|
||
|
|
{'amount_currency': 120.0, 'debit': 60.0, 'credit': 0.0, 'account_id': self.product_line_vals_2['account_id'], 'reconciled': False},
|
||
|
|
{'amount_currency': -120.0, 'debit': 0.0, 'credit': 60.0, 'account_id': wizard.revenue_accrual_account.id, 'reconciled': True},
|
||
|
|
|
||
|
|
{'amount_currency': -600.0, 'debit': 0.0, 'credit': 300.0, 'account_id': self.product_line_vals_1['account_id'], 'reconciled': False},
|
||
|
|
{'amount_currency': 600.0, 'debit': 300.0, 'credit': 0.0, 'account_id': wizard.revenue_accrual_account.id, 'reconciled': True},
|
||
|
|
{'amount_currency': -120.0, 'debit': 0.0, 'credit': 60.0, 'account_id': self.product_line_vals_2['account_id'], 'reconciled': False},
|
||
|
|
{'amount_currency': 120.0, 'debit': 60.0, 'credit': 0.0, 'account_id': wizard.revenue_accrual_account.id, 'reconciled': True},
|
||
|
|
{'amount_currency': -600.0, 'debit': 0.0, 'credit': 300.0, 'account_id': self.product_line_vals_1['account_id'], 'reconciled': False},
|
||
|
|
{'amount_currency': 600.0, 'debit': 300.0, 'credit': 0.0, 'account_id': wizard.revenue_accrual_account.id, 'reconciled': True},
|
||
|
|
{'amount_currency': -120.0, 'debit': 0.0, 'credit': 60.0, 'account_id': self.product_line_vals_2['account_id'], 'reconciled': False},
|
||
|
|
{'amount_currency': 120.0, 'debit': 60.0, 'credit': 0.0, 'account_id': wizard.revenue_accrual_account.id, 'reconciled': True},
|
||
|
|
{'amount_currency': -600.0, 'debit': 0.0, 'credit': 300.0, 'account_id': self.product_line_vals_1['account_id'], 'reconciled': False},
|
||
|
|
{'amount_currency': 600.0, 'debit': 300.0, 'credit': 0.0, 'account_id': wizard.revenue_accrual_account.id, 'reconciled': True},
|
||
|
|
{'amount_currency': -120.0, 'debit': 0.0, 'credit': 60.0, 'account_id': self.product_line_vals_2['account_id'], 'reconciled': False},
|
||
|
|
{'amount_currency': 120.0, 'debit': 60.0, 'credit': 0.0, 'account_id': wizard.revenue_accrual_account.id, 'reconciled': True},
|
||
|
|
])
|
||
|
|
|
||
|
|
def test_out_invoice_filter_zero_balance_lines(self):
|
||
|
|
zero_balance_payment_term = self.env['account.payment.term'].create({
|
||
|
|
'name': 'zero_balance_payment_term',
|
||
|
|
'line_ids': [
|
||
|
|
(0, 0, {
|
||
|
|
'value': 'percent',
|
||
|
|
'value_amount': 100.0,
|
||
|
|
'nb_days': 0,
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
|
||
|
|
zero_balance_tax = self.env['account.tax'].create({
|
||
|
|
'name': 'zero_balance_tax',
|
||
|
|
'amount_type': 'percent',
|
||
|
|
'amount': 0.0,
|
||
|
|
})
|
||
|
|
|
||
|
|
invoice = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_date': fields.Date.from_string('2019-01-01'),
|
||
|
|
'invoice_payment_term_id': zero_balance_payment_term.id,
|
||
|
|
'invoice_line_ids': [(0, None, {
|
||
|
|
'name': 'whatever',
|
||
|
|
'quantity': 1.0,
|
||
|
|
'price_unit': 1000.0,
|
||
|
|
'tax_ids': [(6, 0, zero_balance_tax.ids)],
|
||
|
|
})]
|
||
|
|
})
|
||
|
|
|
||
|
|
self.assertEqual(len(invoice.invoice_line_ids), 1)
|
||
|
|
self.assertEqual(len(invoice.line_ids), 2)
|
||
|
|
|
||
|
|
def test_out_invoice_recomputation_receivable_lines(self):
|
||
|
|
''' Test a tricky specific case caused by some framework limitations. Indeed, when
|
||
|
|
saving a record, some fields are written to the records even if the value is the same
|
||
|
|
as the previous one. It could lead to an unbalanced journal entry when the recomputed
|
||
|
|
line is the receivable/payable one.
|
||
|
|
|
||
|
|
For example, the computed price_subtotal are the following:
|
||
|
|
1471.95 / 0.14 = 10513.93
|
||
|
|
906468.18 / 0.14 = 6474772.71
|
||
|
|
1730.84 / 0.14 = 12363.14
|
||
|
|
17.99 / 0.14 = 128.50
|
||
|
|
SUM = 6497778.28
|
||
|
|
|
||
|
|
But when recomputing the receivable line:
|
||
|
|
909688.96 / 0.14 = 6497778.285714286 => 6497778.29
|
||
|
|
|
||
|
|
This recomputation was made because the framework was writing the same 'price_unit'
|
||
|
|
as the previous value leading to a recomputation of the debit/credit.
|
||
|
|
'''
|
||
|
|
self.env['decimal.precision'].search([
|
||
|
|
('name', '=', self.env['account.move.line']._fields['price_unit']._digits),
|
||
|
|
]).digits = 5
|
||
|
|
|
||
|
|
self.env['res.currency.rate'].create({
|
||
|
|
'name': '2019-01-01',
|
||
|
|
'rate': 0.14,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'company_id': self.company_data['company'].id,
|
||
|
|
})
|
||
|
|
|
||
|
|
invoice = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'invoice_date': '2019-01-01',
|
||
|
|
'date': '2019-01-01',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'invoice_payment_term_id': self.env.ref('account.account_payment_term_immediate').id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({'name': 'line1', 'price_unit': 38.73553, 'quantity': 38.0, 'tax_ids': []}),
|
||
|
|
Command.create({'name': 'line2', 'price_unit': 4083.19000, 'quantity': 222.0, 'tax_ids': []}),
|
||
|
|
Command.create({'name': 'line3', 'price_unit': 49.45257, 'quantity': 35.0, 'tax_ids': []}),
|
||
|
|
Command.create({'name': 'line4', 'price_unit': 17.99000, 'quantity': 1.0, 'tax_ids': []}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
|
||
|
|
# assertNotUnbalancedEntryWhenSaving
|
||
|
|
with Form(invoice) as move_form:
|
||
|
|
move_form.invoice_payment_term_id = self.env.ref('account.account_payment_term_30days')
|
||
|
|
|
||
|
|
def test_out_invoice_rounding_recomputation_receivable_lines(self):
|
||
|
|
''' Test rounding error due to the fact that subtracting then rounding is different from
|
||
|
|
rounding then subtracting.
|
||
|
|
'''
|
||
|
|
self.env['decimal.precision'].search([
|
||
|
|
('name', '=', self.env['account.move.line']._fields['price_unit']._digits),
|
||
|
|
]).digits = 5
|
||
|
|
|
||
|
|
self.env['res.currency.rate'].search([]).unlink()
|
||
|
|
|
||
|
|
invoice = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'invoice_date': '2019-01-01',
|
||
|
|
'date': '2019-01-01',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_payment_term_id': self.env.ref('account.account_payment_term_immediate').id,
|
||
|
|
})
|
||
|
|
|
||
|
|
# assertNotUnbalancedEntryWhenSaving
|
||
|
|
with Form(invoice) as move_form:
|
||
|
|
with move_form.invoice_line_ids.new() as line_form:
|
||
|
|
line_form.name = 'line1'
|
||
|
|
line_form.account_id = self.company_data['default_account_revenue']
|
||
|
|
line_form.tax_ids.clear()
|
||
|
|
line_form.price_unit = 0.89500
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
def test_out_invoice_multi_company(self):
|
||
|
|
''' Ensure the properties are found on the right company.
|
||
|
|
'''
|
||
|
|
|
||
|
|
product = self.env['product.product'].create({
|
||
|
|
'name': 'product',
|
||
|
|
'uom_id': self.env.ref('uom.product_uom_unit').id,
|
||
|
|
'lst_price': 1000.0,
|
||
|
|
'standard_price': 800.0,
|
||
|
|
'company_id': False,
|
||
|
|
})
|
||
|
|
|
||
|
|
partner = self.env['res.partner'].create({
|
||
|
|
'name': 'partner',
|
||
|
|
'company_id': False,
|
||
|
|
})
|
||
|
|
|
||
|
|
journal = self.env['account.journal'].create({
|
||
|
|
'name': 'test_out_invoice_multi_company',
|
||
|
|
'code': 'XXXXX',
|
||
|
|
'type': 'sale',
|
||
|
|
'company_id': self.company_data_2['company'].id,
|
||
|
|
})
|
||
|
|
|
||
|
|
product.with_company(self.company_data['company']).write({
|
||
|
|
'property_account_income_id': self.company_data['default_account_revenue'].id,
|
||
|
|
})
|
||
|
|
|
||
|
|
partner.with_company(self.company_data['company']).write({
|
||
|
|
'property_account_receivable_id': self.company_data['default_account_receivable'].id,
|
||
|
|
})
|
||
|
|
|
||
|
|
product.with_company(self.company_data_2['company']).write({
|
||
|
|
'property_account_income_id': self.company_data_2['default_account_revenue'].id,
|
||
|
|
})
|
||
|
|
|
||
|
|
partner.with_company(self.company_data_2['company']).write({
|
||
|
|
'property_account_receivable_id': self.company_data_2['default_account_receivable'].id,
|
||
|
|
})
|
||
|
|
|
||
|
|
def _check_invoice_values(invoice):
|
||
|
|
self.assertInvoiceValues(invoice, [
|
||
|
|
{
|
||
|
|
'product_id': product.id,
|
||
|
|
'account_id': self.company_data_2['default_account_revenue'].id,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 1000.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'product_id': False,
|
||
|
|
'account_id': self.company_data_2['default_account_receivable'].id,
|
||
|
|
'debit': 1000.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
], {
|
||
|
|
'amount_untaxed': 1000.0,
|
||
|
|
'amount_total': 1000.0,
|
||
|
|
})
|
||
|
|
|
||
|
|
invoice_create = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'invoice_date': '2017-01-01',
|
||
|
|
'date': '2017-01-01',
|
||
|
|
'partner_id': partner.id,
|
||
|
|
'journal_id': journal.id,
|
||
|
|
'invoice_line_ids': [(0, 0, {
|
||
|
|
'product_id': product.id,
|
||
|
|
'price_unit': 1000.0,
|
||
|
|
'tax_ids': [],
|
||
|
|
})],
|
||
|
|
})
|
||
|
|
|
||
|
|
_check_invoice_values(invoice_create)
|
||
|
|
|
||
|
|
move_form = Form(self.env['account.move'].with_context(default_move_type='out_invoice'))
|
||
|
|
move_form.journal_id = journal
|
||
|
|
move_form.partner_id = partner
|
||
|
|
move_form.invoice_date = fields.Date.from_string('2017-01-01')
|
||
|
|
with move_form.invoice_line_ids.new() as line_form:
|
||
|
|
line_form.product_id = product
|
||
|
|
line_form.tax_ids.clear()
|
||
|
|
invoice_onchange = move_form.save()
|
||
|
|
|
||
|
|
_check_invoice_values(invoice_onchange)
|
||
|
|
|
||
|
|
def test_out_invoice_multiple_switch_payment_terms(self):
|
||
|
|
''' When switching immediate payment term to 30% advance then back to immediate payment term, ensure the
|
||
|
|
receivable line is back to its previous value. If some business fields are not well updated, it could lead to a
|
||
|
|
recomputation of debit/credit when writing and then, an unbalanced journal entry.
|
||
|
|
'''
|
||
|
|
# assertNotUnbalancedEntryWhenSaving
|
||
|
|
with Form(self.invoice) as move_form:
|
||
|
|
move_form.invoice_payment_term_id = self.pay_terms_b # Switch to 30% in advance payment terms
|
||
|
|
move_form.invoice_payment_term_id = self.pay_terms_a # Back to immediate payment term
|
||
|
|
|
||
|
|
def test_out_invoice_copy_custom_date(self):
|
||
|
|
""" When creating a refund for a given invoice, the invoice is copied first. This test ensures the payment
|
||
|
|
terms are well recomputed in order to take the new date into account.
|
||
|
|
"""
|
||
|
|
invoice = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'invoice_date': '2017-01-01',
|
||
|
|
'invoice_date_due': '2017-01-01',
|
||
|
|
'date': '2017-01-01',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
(0, None, {
|
||
|
|
'product_id': self.product_a.id,
|
||
|
|
'product_uom_id': self.product_a.uom_id.id,
|
||
|
|
'quantity': 1.0,
|
||
|
|
'price_unit': 1000.0,
|
||
|
|
'tax_ids': [(6, 0, self.product_a.taxes_id.ids)],
|
||
|
|
}),
|
||
|
|
]
|
||
|
|
})
|
||
|
|
|
||
|
|
copy_invoice = invoice.copy(default={
|
||
|
|
'invoice_date_due': '2018-01-01',
|
||
|
|
'invoice_payment_term_id': False,
|
||
|
|
})
|
||
|
|
self.assertRecordValues(copy_invoice, [
|
||
|
|
{'invoice_date_due': fields.Date.from_string('2018-01-01')},
|
||
|
|
])
|
||
|
|
self.assertRecordValues(copy_invoice.line_ids.filtered('date_maturity'), [
|
||
|
|
{'date_maturity': fields.Date.from_string('2018-01-01')},
|
||
|
|
])
|
||
|
|
|
||
|
|
def test_out_invoice_note_and_tax_partner_is_set(self):
|
||
|
|
# Make sure that, when creating an invoice by giving a list of lines with the last one being a note,
|
||
|
|
# the partner_id of the tax line is properly set.
|
||
|
|
invoice_vals_list = [{
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'journal_id': self.company_data['default_journal_sale'].id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
(0, 0, {
|
||
|
|
'name': 'My super product.',
|
||
|
|
'quantity': 1.0,
|
||
|
|
'price_unit': 750.0,
|
||
|
|
'tax_ids': [(6, 0, self.product_a.taxes_id.ids)],
|
||
|
|
}),
|
||
|
|
(0, 0, {
|
||
|
|
'display_type': 'line_note',
|
||
|
|
'name': 'This is a note',
|
||
|
|
'account_id': False,
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
}]
|
||
|
|
moves = self.env['account.move'].create(invoice_vals_list)
|
||
|
|
tax_line = moves.line_ids.filtered('tax_line_id')
|
||
|
|
self.assertEqual(tax_line.partner_id.id, self.partner_a.id)
|
||
|
|
|
||
|
|
def test_out_invoice_reverse_caba(self):
|
||
|
|
tax_waiting_account = self.env['account.account'].create({
|
||
|
|
'name': 'TAX_WAIT',
|
||
|
|
'code': 'TWAIT',
|
||
|
|
'account_type': 'liability_current',
|
||
|
|
'reconcile': True,
|
||
|
|
'company_id': self.company_data['company'].id,
|
||
|
|
})
|
||
|
|
tax_final_account = self.env['account.account'].create({
|
||
|
|
'name': 'TAX_TO_DEDUCT',
|
||
|
|
'code': 'TDEDUCT',
|
||
|
|
'account_type': 'asset_current',
|
||
|
|
'company_id': self.company_data['company'].id,
|
||
|
|
})
|
||
|
|
tax_base_amount_account = self.env['account.account'].create({
|
||
|
|
'name': 'TAX_BASE',
|
||
|
|
'code': 'TBASE',
|
||
|
|
'account_type': 'asset_current',
|
||
|
|
'company_id': self.company_data['company'].id,
|
||
|
|
})
|
||
|
|
self.env.company.account_cash_basis_base_account_id = tax_base_amount_account
|
||
|
|
self.env.company.tax_exigibility = True
|
||
|
|
tax_tags = defaultdict(dict)
|
||
|
|
for line_type, repartition_type in [(l, r) for l in ('invoice', 'refund') for r in ('base', 'tax')]:
|
||
|
|
tax_tags[line_type][repartition_type] = self.env['account.account.tag'].create({
|
||
|
|
'name': '%s %s tag' % (line_type, repartition_type),
|
||
|
|
'applicability': 'taxes',
|
||
|
|
'country_id': self.env.ref('base.us').id,
|
||
|
|
})
|
||
|
|
tax = self.env['account.tax'].create({
|
||
|
|
'name': 'cash basis 10%',
|
||
|
|
'type_tax_use': 'sale',
|
||
|
|
'amount': 10,
|
||
|
|
'tax_exigibility': 'on_payment',
|
||
|
|
'cash_basis_transition_account_id': tax_waiting_account.id,
|
||
|
|
'invoice_repartition_line_ids': [
|
||
|
|
(0, 0, {
|
||
|
|
'repartition_type': 'base',
|
||
|
|
'tag_ids': [(6, 0, tax_tags['invoice']['base'].ids)],
|
||
|
|
}),
|
||
|
|
(0, 0, {
|
||
|
|
'repartition_type': 'tax',
|
||
|
|
'account_id': tax_final_account.id,
|
||
|
|
'tag_ids': [(6, 0, tax_tags['invoice']['tax'].ids)],
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
'refund_repartition_line_ids': [
|
||
|
|
(0, 0, {
|
||
|
|
'repartition_type': 'base',
|
||
|
|
'tag_ids': [(6, 0, tax_tags['refund']['base'].ids)],
|
||
|
|
}),
|
||
|
|
(0, 0, {
|
||
|
|
'repartition_type': 'tax',
|
||
|
|
'account_id': tax_final_account.id,
|
||
|
|
'tag_ids': [(6, 0, tax_tags['refund']['tax'].ids)],
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
# create invoice
|
||
|
|
move_form = Form(self.env['account.move'].with_context(default_move_type='out_invoice'))
|
||
|
|
move_form.partner_id = self.partner_a
|
||
|
|
move_form.invoice_date = fields.Date.from_string('2017-01-01')
|
||
|
|
with move_form.invoice_line_ids.new() as line_form:
|
||
|
|
line_form.product_id = self.product_a
|
||
|
|
line_form.tax_ids.clear()
|
||
|
|
line_form.tax_ids.add(tax)
|
||
|
|
invoice = move_form.save()
|
||
|
|
invoice.action_post()
|
||
|
|
# make payment
|
||
|
|
self.env['account.payment.register'].with_context(active_model='account.move', active_ids=invoice.ids).create({
|
||
|
|
'payment_date': invoice.date,
|
||
|
|
})._create_payments()
|
||
|
|
# check caba move
|
||
|
|
partial_rec = invoice.mapped('line_ids.matched_credit_ids')
|
||
|
|
caba_move = self.env['account.move'].search([('tax_cash_basis_rec_id', '=', partial_rec.id)])
|
||
|
|
expected_values = [
|
||
|
|
{
|
||
|
|
'tax_line_id': False,
|
||
|
|
'tax_repartition_line_id': False,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_tag_ids': [],
|
||
|
|
'account_id': tax_base_amount_account.id,
|
||
|
|
'debit': 1000.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'tax_line_id': False,
|
||
|
|
'tax_repartition_line_id': False,
|
||
|
|
'tax_ids': tax.ids,
|
||
|
|
'tax_tag_ids': tax_tags['invoice']['base'].ids,
|
||
|
|
'account_id': tax_base_amount_account.id,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 1000.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'tax_line_id': False,
|
||
|
|
'tax_repartition_line_id': False,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_tag_ids': [],
|
||
|
|
'account_id': tax_waiting_account.id,
|
||
|
|
'debit': 100.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'tax_line_id': tax.id,
|
||
|
|
'tax_repartition_line_id': tax.invoice_repartition_line_ids.filtered(lambda x: x.repartition_type == 'tax').id,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_tag_ids': tax_tags['invoice']['tax'].ids,
|
||
|
|
'account_id': tax_final_account.id,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 100.0,
|
||
|
|
},
|
||
|
|
]
|
||
|
|
self.assertRecordValues(caba_move.line_ids, expected_values)
|
||
|
|
# unreconcile
|
||
|
|
debit_aml = invoice.line_ids.filtered('debit')
|
||
|
|
debit_aml.remove_move_reconcile()
|
||
|
|
# check caba move reverse is same as caba move with only debit/credit inverted
|
||
|
|
reversed_caba_move = self.env['account.move'].search([('reversed_entry_id', '=', caba_move.id)])
|
||
|
|
for value in expected_values:
|
||
|
|
value.update({
|
||
|
|
'debit': value['credit'],
|
||
|
|
'credit': value['debit'],
|
||
|
|
})
|
||
|
|
self.assertRecordValues(reversed_caba_move.line_ids, expected_values)
|
||
|
|
|
||
|
|
def test_out_invoice_with_down_payment_caba(self):
|
||
|
|
tax_waiting_account = self.env['account.account'].create({
|
||
|
|
'name': 'TAX_WAIT',
|
||
|
|
'code': 'TWAIT',
|
||
|
|
'account_type': 'liability_current',
|
||
|
|
'reconcile': True,
|
||
|
|
'company_id': self.company_data['company'].id,
|
||
|
|
})
|
||
|
|
tax_final_account = self.env['account.account'].create({
|
||
|
|
'name': 'TAX_TO_DEDUCT',
|
||
|
|
'code': 'TDEDUCT',
|
||
|
|
'account_type': 'asset_current',
|
||
|
|
'company_id': self.company_data['company'].id,
|
||
|
|
})
|
||
|
|
default_income_account = self.company_data['default_account_revenue']
|
||
|
|
not_default_income_account = self.env['account.account'].create({
|
||
|
|
'name': 'NOT_DEFAULT_INCOME',
|
||
|
|
'code': 'NDI',
|
||
|
|
'account_type': 'income',
|
||
|
|
'company_id': self.company_data['company'].id,
|
||
|
|
})
|
||
|
|
self.env.company.tax_exigibility = True
|
||
|
|
tax_tags = defaultdict(dict)
|
||
|
|
for line_type, repartition_type in [(l, r) for l in ('invoice', 'refund') for r in ('base', 'tax')]:
|
||
|
|
tax_tags[line_type][repartition_type] = self.env['account.account.tag'].create({
|
||
|
|
'name': '%s %s tag' % (line_type, repartition_type),
|
||
|
|
'applicability': 'taxes',
|
||
|
|
'country_id': self.env.ref('base.us').id,
|
||
|
|
})
|
||
|
|
tax = self.env['account.tax'].create({
|
||
|
|
'name': 'cash basis 10%',
|
||
|
|
'type_tax_use': 'sale',
|
||
|
|
'amount': 10,
|
||
|
|
'tax_exigibility': 'on_payment',
|
||
|
|
'cash_basis_transition_account_id': tax_waiting_account.id,
|
||
|
|
'invoice_repartition_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'repartition_type': 'base',
|
||
|
|
'tag_ids': [Command.set(tax_tags['invoice']['base'].ids)],
|
||
|
|
}),
|
||
|
|
Command.create({
|
||
|
|
'repartition_type': 'tax',
|
||
|
|
'account_id': tax_final_account.id,
|
||
|
|
'tag_ids': [Command.set(tax_tags['invoice']['tax'].ids)],
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
'refund_repartition_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'repartition_type': 'base',
|
||
|
|
'tag_ids': [Command.set(tax_tags['refund']['base'].ids)],
|
||
|
|
}),
|
||
|
|
Command.create({
|
||
|
|
'repartition_type': 'tax',
|
||
|
|
'account_id': tax_final_account.id,
|
||
|
|
'tag_ids': [Command.set(tax_tags['refund']['tax'].ids)],
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
# create invoice
|
||
|
|
# one downpayment on default account and one product line on not default account, both with the caba tax
|
||
|
|
invoice = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_date': fields.Date.from_string('2017-01-01'),
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'account_id': not_default_income_account.id,
|
||
|
|
'product_id': self.product_a.id,
|
||
|
|
'tax_ids': [Command.set(tax.ids)],
|
||
|
|
}),
|
||
|
|
Command.create({
|
||
|
|
'name': 'Down payment',
|
||
|
|
'price_unit': 300,
|
||
|
|
'quantity': -1,
|
||
|
|
'tax_ids': [Command.set(tax.ids)],
|
||
|
|
}),
|
||
|
|
]
|
||
|
|
})
|
||
|
|
invoice.action_post()
|
||
|
|
# make payment
|
||
|
|
self.env['account.payment.register'].with_context(active_model='account.move', active_ids=invoice.ids).create({
|
||
|
|
'payment_date': invoice.date,
|
||
|
|
})._create_payments()
|
||
|
|
# check caba move
|
||
|
|
partial_rec = invoice.mapped('line_ids.matched_credit_ids')
|
||
|
|
caba_move = self.env['account.move'].search([('tax_cash_basis_rec_id', '=', partial_rec.id)])
|
||
|
|
# all amls with tax_tag should all have tax_tag_invert at True since the caba move comes from an invoice
|
||
|
|
expected_values = [
|
||
|
|
{
|
||
|
|
'tax_line_id': False,
|
||
|
|
'tax_repartition_line_id': False,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_tag_ids': [],
|
||
|
|
'account_id': not_default_income_account.id,
|
||
|
|
'debit': 1000.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
'tax_tag_invert': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'tax_line_id': False,
|
||
|
|
'tax_repartition_line_id': False,
|
||
|
|
'tax_ids': tax.ids,
|
||
|
|
'tax_tag_ids': tax_tags['invoice']['base'].ids,
|
||
|
|
'account_id': not_default_income_account.id,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 1000.0,
|
||
|
|
'tax_tag_invert': True,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'tax_line_id': False,
|
||
|
|
'tax_repartition_line_id': False,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_tag_ids': [],
|
||
|
|
'account_id': default_income_account.id,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 300.0,
|
||
|
|
'tax_tag_invert': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'tax_line_id': False,
|
||
|
|
'tax_repartition_line_id': False,
|
||
|
|
'tax_ids': tax.ids,
|
||
|
|
'tax_tag_ids': tax_tags['invoice']['base'].ids,
|
||
|
|
'account_id': default_income_account.id,
|
||
|
|
'debit': 300.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
'tax_tag_invert': True,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'tax_line_id': False,
|
||
|
|
'tax_repartition_line_id': False,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_tag_ids': [],
|
||
|
|
'account_id': tax_waiting_account.id,
|
||
|
|
'debit': 70.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
'tax_tag_invert': False,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'tax_line_id': tax.id,
|
||
|
|
'tax_repartition_line_id': tax.invoice_repartition_line_ids.filtered(lambda x: x.repartition_type == 'tax').id,
|
||
|
|
'tax_ids': [],
|
||
|
|
'tax_tag_ids': tax_tags['invoice']['tax'].ids,
|
||
|
|
'account_id': tax_final_account.id,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 70.0,
|
||
|
|
'tax_tag_invert': True,
|
||
|
|
},
|
||
|
|
]
|
||
|
|
self.assertRecordValues(caba_move.line_ids, expected_values)
|
||
|
|
|
||
|
|
def test_tax_grid_remove_tax(self):
|
||
|
|
# Add a tag to tax_sale_a
|
||
|
|
tax_line_tag = self.env['account.account.tag'].create({
|
||
|
|
'name': "Tax tag",
|
||
|
|
'applicability': 'taxes',
|
||
|
|
'country_id': self.company_data['company'].country_id.id,
|
||
|
|
})
|
||
|
|
|
||
|
|
repartition_line = self.tax_sale_a.invoice_repartition_line_ids\
|
||
|
|
.filtered(lambda x: x.repartition_type == 'tax')
|
||
|
|
repartition_line.tag_ids |= tax_line_tag
|
||
|
|
|
||
|
|
# Create the invoice
|
||
|
|
invoice = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'invoice_date': fields.Date.from_string('2022-02-20'),
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'product_id': self.product_a.id,
|
||
|
|
'price_unit': 999.99,
|
||
|
|
'tax_ids': [Command.set(self.product_a.taxes_id.ids)],
|
||
|
|
}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
|
||
|
|
with Form(invoice) as form:
|
||
|
|
with form.invoice_line_ids.edit(0) as line_form:
|
||
|
|
line_form.tax_ids.clear()
|
||
|
|
|
||
|
|
# Tags should be empty since the tax has been removed from the invoice line
|
||
|
|
self.assertRecordValues(invoice.line_ids, [{'tax_tag_ids': []}, {'tax_tag_ids': []}])
|
||
|
|
|
||
|
|
def test_quick_edit_total_amount(self):
|
||
|
|
move_form = Form(self.env['account.move'].with_context(default_move_type='out_invoice'))
|
||
|
|
move_form.invoice_date = fields.Date.from_string('2022-01-01')
|
||
|
|
move_form.partner_id = self.partner_a
|
||
|
|
|
||
|
|
# Quick edit total amount not activated yet
|
||
|
|
# As quick edit total is not yet activated, it's invisible by default in the view
|
||
|
|
move_form._view['modifiers']['quick_edit_total_amount']['invisible'] = 'False'
|
||
|
|
move_form.quick_edit_total_amount = 100.0
|
||
|
|
invoice = move_form.save()
|
||
|
|
self.assertEqual(invoice.amount_total, 0.0)
|
||
|
|
self.assertEqual(len(invoice.invoice_line_ids), 0)
|
||
|
|
|
||
|
|
# Quick edit total amount activated
|
||
|
|
self.env.company.quick_edit_mode = "out_and_in_invoices"
|
||
|
|
self.env.company.account_sale_tax_id = self.env['account.tax'].create({
|
||
|
|
'name': '21%',
|
||
|
|
'amount': 21,
|
||
|
|
'type_tax_use': 'sale',
|
||
|
|
}) # 21% tax of a total amount may create a rounding error (99.99 or 100.01)
|
||
|
|
|
||
|
|
# Let's make sure it does not (the rounding cent should be on the tax)
|
||
|
|
with Form(invoice) as move_form:
|
||
|
|
move_form.quick_edit_total_amount = 100.0
|
||
|
|
self.assertEqual(invoice.amount_total, 100)
|
||
|
|
self.assertEqual(invoice.amount_untaxed, 82.64)
|
||
|
|
self.assertEqual(invoice.amount_tax, 17.36)
|
||
|
|
self.assertEqual(len(invoice.invoice_line_ids), 1)
|
||
|
|
|
||
|
|
# Modify one invoice line
|
||
|
|
with Form(invoice) as move_form:
|
||
|
|
with move_form.invoice_line_ids.edit(0) as line_form:
|
||
|
|
line_form.price_unit = 50
|
||
|
|
self.assertEqual(invoice.amount_total, 60.5)
|
||
|
|
self.assertEqual(invoice.amount_untaxed, 50)
|
||
|
|
self.assertEqual(invoice.amount_tax, 10.5)
|
||
|
|
self.assertEqual(len(invoice.invoice_line_ids), 1)
|
||
|
|
|
||
|
|
# Suggest the new amount such that the total is equal to the quick amount
|
||
|
|
with Form(invoice) as move_form:
|
||
|
|
with move_form.invoice_line_ids.new() as line_form:
|
||
|
|
self.assertEqual(line_form.price_unit, 32.64)
|
||
|
|
self.assertEqual(invoice.amount_total, 100)
|
||
|
|
self.assertEqual(invoice.amount_untaxed, 82.64)
|
||
|
|
self.assertEqual(invoice.amount_tax, 17.36)
|
||
|
|
self.assertEqual(len(invoice.invoice_line_ids), 2)
|
||
|
|
|
||
|
|
def test_quick_edit_total_amount_with_mixed_epd(self):
|
||
|
|
move_form = Form(self.env['account.move'].with_context(default_move_type='out_invoice'))
|
||
|
|
move_form.invoice_date = fields.Date.from_string('2022-01-01')
|
||
|
|
|
||
|
|
# Quick edit total amount activated
|
||
|
|
self.env.company.quick_edit_mode = "out_and_in_invoices"
|
||
|
|
# 21% sale tax
|
||
|
|
self.env.company.account_sale_tax_id = self.env['account.tax'].create({
|
||
|
|
'name': '21%',
|
||
|
|
'amount': 21,
|
||
|
|
'type_tax_use': 'sale',
|
||
|
|
})
|
||
|
|
# Create a payment term with early payment discount of 2% and computation set to mixed (Always (upon invoice))
|
||
|
|
epd_payment_term = self.env['account.payment.term'].create({
|
||
|
|
'name': "2/7 Term",
|
||
|
|
'discount_days': 7,
|
||
|
|
'discount_percentage': 2,
|
||
|
|
'early_discount': True,
|
||
|
|
'early_pay_discount_computation': 'mixed',
|
||
|
|
})
|
||
|
|
# Set the payment term to the one we just created
|
||
|
|
move_form.invoice_payment_term_id = epd_payment_term
|
||
|
|
|
||
|
|
invoice = move_form.save()
|
||
|
|
|
||
|
|
# Invoice of one item of price 100, discount 2% and tax 21%:
|
||
|
|
# 21% tax = 100 * (1 - 0.2) * 0.21 = 20.58
|
||
|
|
# total_amount = 100 + 20.58 = 120.58
|
||
|
|
|
||
|
|
# Make sure the quick edit added one line with the correct values
|
||
|
|
with Form(invoice) as move_form:
|
||
|
|
move_form.quick_edit_total_amount = 120.58
|
||
|
|
self.assertRecordValues(invoice, [{'amount_total': 120.58, 'amount_untaxed': 100, 'amount_tax': 20.58}])
|
||
|
|
self.assertEqual(len(invoice.invoice_line_ids), 1)
|
||
|
|
|
||
|
|
# Modify one invoice line
|
||
|
|
with Form(invoice) as move_form:
|
||
|
|
with move_form.invoice_line_ids.edit(0) as line_form:
|
||
|
|
line_form.price_unit = 70
|
||
|
|
self.assertRecordValues(invoice, [{'amount_total': 84.41, 'amount_untaxed': 70, 'amount_tax': 14.41}])
|
||
|
|
self.assertEqual(len(invoice.invoice_line_ids), 1)
|
||
|
|
|
||
|
|
# Suggest the new amount such that the total is equal to the quick amount
|
||
|
|
with Form(invoice) as move_form:
|
||
|
|
with move_form.invoice_line_ids.new() as line_form:
|
||
|
|
self.assertEqual(line_form.price_unit, 30)
|
||
|
|
self.assertRecordValues(invoice, [{'amount_total': 120.58, 'amount_untaxed': 100, 'amount_tax': 20.58}])
|
||
|
|
self.assertEqual(len(invoice.invoice_line_ids), 2)
|
||
|
|
|
||
|
|
def test_out_invoice_depreciated_account(self):
|
||
|
|
move = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'journal_id': self.company_data['default_journal_sale'].id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
(0, 0, {
|
||
|
|
'name': 'My super product.',
|
||
|
|
'quantity': 1.0,
|
||
|
|
'price_unit': 750.0,
|
||
|
|
'account_id': self.product_a.property_account_income_id.id,
|
||
|
|
})
|
||
|
|
],
|
||
|
|
})
|
||
|
|
self.product_a.property_account_income_id.deprecated = True
|
||
|
|
with self.assertRaises(UserError), self.cr.savepoint():
|
||
|
|
move.action_post()
|
||
|
|
|
||
|
|
def test_change_currency_id(self):
|
||
|
|
"""
|
||
|
|
Test that we are able to change currency on invoice,
|
||
|
|
even when a default currency is set on journal
|
||
|
|
"""
|
||
|
|
self.company_data['default_journal_sale'].currency_id = self.company_data['currency']
|
||
|
|
move = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'journal_id': self.company_data['default_journal_sale'].id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'name': 'My super product.',
|
||
|
|
'quantity': 1.0,
|
||
|
|
'price_unit': 750.0,
|
||
|
|
'account_id': self.product_a.property_account_income_id.id,
|
||
|
|
'tax_ids': False,
|
||
|
|
})
|
||
|
|
],
|
||
|
|
})
|
||
|
|
|
||
|
|
self.assertEqual(move.currency_id, self.company_data['currency'])
|
||
|
|
move.currency_id = self.currency_data['currency']
|
||
|
|
self.assertEqual(move.currency_id, self.currency_data['currency'])
|
||
|
|
self.assertRecordValues(move.line_ids, [
|
||
|
|
{
|
||
|
|
'display_type': 'product',
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 375.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'display_type': 'payment_term',
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
'debit': 375.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
])
|
||
|
|
|
||
|
|
move.currency_id = self.company_data['currency']
|
||
|
|
with Form(move) as move_form:
|
||
|
|
move_form.currency_id = self.currency_data['currency']
|
||
|
|
self.assertEqual(move.currency_id, self.currency_data['currency'])
|
||
|
|
self.assertEqual(move.line_ids.currency_id, self.currency_data['currency'])
|
||
|
|
|
||
|
|
with Form(self.env['account.move'].with_context(default_move_type='out_invoice')) as move_form:
|
||
|
|
move_form.journal_id = self.company_data['default_journal_sale']
|
||
|
|
with move_form.invoice_line_ids.new() as line_form:
|
||
|
|
line_form.product_id = self.product_a
|
||
|
|
line_form.tax_ids.clear()
|
||
|
|
move_form.currency_id = self.currency_data['currency']
|
||
|
|
self.assertEqual(move_form.currency_id, self.currency_data['currency'])
|
||
|
|
move = move_form.save()
|
||
|
|
self.assertEqual(move.currency_id, self.currency_data['currency'])
|
||
|
|
self.assertEqual(move.line_ids.currency_id, self.currency_data['currency'])
|
||
|
|
|
||
|
|
def test_change_journal_currency(self):
|
||
|
|
second_journal = self.company_data['default_journal_sale'].copy({
|
||
|
|
'currency_id': self.currency_data['currency'].id,
|
||
|
|
})
|
||
|
|
move = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'journal_id': self.company_data['default_journal_sale'].id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'name': 'My super product.',
|
||
|
|
'quantity': 1.0,
|
||
|
|
'price_unit': 750.0,
|
||
|
|
'account_id': self.product_a.property_account_income_id.id,
|
||
|
|
'tax_ids': False,
|
||
|
|
})
|
||
|
|
],
|
||
|
|
})
|
||
|
|
|
||
|
|
self.assertEqual(move.currency_id, self.company_data['currency'])
|
||
|
|
move.journal_id = second_journal
|
||
|
|
self.assertEqual(move.currency_id, self.currency_data['currency'])
|
||
|
|
|
||
|
|
@freeze_time('2019-01-01')
|
||
|
|
def test_date_reversal_exchange_move(self):
|
||
|
|
"""
|
||
|
|
Test the date of the reversal of an exchange move created when unreconciling a payment made in the past, when no lock date is set.
|
||
|
|
It should be the last day of the month of the exchange move date if sequence is incremented by month,
|
||
|
|
and the last day of the year of the exchange move date if sequence is incremented by year.
|
||
|
|
"""
|
||
|
|
for format_incrementor, expected_date in (('month', '2017-01-31'), ('year', '2017-12-31')):
|
||
|
|
with self.subTest(format_incrementor=format_incrementor, expected_date=expected_date):
|
||
|
|
invoice = self.init_invoice(move_type='out_invoice', partner=self.partner_a, invoice_date='2016-01-20', post=True, amounts=[750.0], currency=self.currency_data['currency'])
|
||
|
|
|
||
|
|
new_exchange_journal = self.env['account.journal'].create({
|
||
|
|
'name': f'Exchange Journal for {invoice.name}',
|
||
|
|
'code': f'EXCH{invoice.sequence_number}',
|
||
|
|
'type': 'general',
|
||
|
|
'company_id': self.env.company.id,
|
||
|
|
})
|
||
|
|
|
||
|
|
# Need a first move in the new journal to initiate the sequence with a right incrementor, depending on the wanted format
|
||
|
|
self.env['account.move'].create({
|
||
|
|
'journal_id': new_exchange_journal.id,
|
||
|
|
'name': 'EXCH/2019/00001' if format_incrementor == 'year' else 'EXCH/2019/01/0001',
|
||
|
|
'line_ids': [
|
||
|
|
(0, 0, {
|
||
|
|
'account_id': self.company_data['default_account_receivable'].id,
|
||
|
|
'debit': 125.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
}),
|
||
|
|
(0, 0, {
|
||
|
|
'account_id': self.company_data['default_account_revenue'].id,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 125.0,
|
||
|
|
})
|
||
|
|
]
|
||
|
|
})
|
||
|
|
|
||
|
|
self.env.company.currency_exchange_journal_id = new_exchange_journal
|
||
|
|
|
||
|
|
self.env['account.payment.register'].with_context(active_model='account.move', active_ids=invoice.ids).create({
|
||
|
|
'payment_date': '2017-01-20',
|
||
|
|
})._create_payments()
|
||
|
|
|
||
|
|
line_receivable = invoice.line_ids.filtered(lambda l: l.account_id.account_type == 'asset_receivable')
|
||
|
|
|
||
|
|
exchange_move = line_receivable.full_reconcile_id.partial_reconcile_ids.exchange_move_id
|
||
|
|
|
||
|
|
# Date of the exchange move should be the end of the month/year of the payment
|
||
|
|
self.assertEqual(exchange_move.date, fields.Date.to_date(expected_date))
|
||
|
|
|
||
|
|
line_receivable.remove_move_reconcile()
|
||
|
|
|
||
|
|
exchange_move_reversal = exchange_move.reversal_move_id
|
||
|
|
|
||
|
|
# Date of the reversal of the exchange move should be the last day of the month/year of the payment depending on the sequence format
|
||
|
|
self.assertEqual(exchange_move_reversal.date, fields.Date.to_date(expected_date))
|
||
|
|
|
||
|
|
@freeze_time('2023-01-01')
|
||
|
|
def test_change_first_journal_move_sequence(self):
|
||
|
|
"""Invoice name should not be reset when posting the invoice"""
|
||
|
|
new_sale_journal = self.company_data['default_journal_sale'].copy()
|
||
|
|
invoice = self.env['account.move'].with_context(default_move_type='out_invoice').create({
|
||
|
|
'journal_id': new_sale_journal.id,
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'name': 'INV1/2023/00010',
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'name': 'My super product.',
|
||
|
|
'quantity': 1.0,
|
||
|
|
'price_unit': 750.0,
|
||
|
|
'account_id': self.company_data['default_account_revenue'].id,
|
||
|
|
})
|
||
|
|
]
|
||
|
|
})
|
||
|
|
invoice.action_post()
|
||
|
|
self.assertEqual(invoice.name, 'INV1/2023/00010')
|
||
|
|
|
||
|
|
def test_invoice_mass_posting(self):
|
||
|
|
"""
|
||
|
|
With some particular setup (in this case, rounding issue), partner get mixed up in the
|
||
|
|
invoice lines after mass posting.
|
||
|
|
"""
|
||
|
|
currency = self.company_data['currency']
|
||
|
|
currency.rounding = 0.0001
|
||
|
|
invoice1 = self.init_invoice(move_type='out_invoice', partner=self.partner_a, invoice_date='2016-01-20', products=self.product_a)
|
||
|
|
invoice1.invoice_line_ids.price_unit = 12.36
|
||
|
|
invoice2 = self.init_invoice(move_type='out_invoice', partner=self.partner_b, invoice_date='2016-01-20', products=self.product_a)
|
||
|
|
|
||
|
|
vam = self.env["validate.account.move"].create({"force_post": True})
|
||
|
|
vam.with_context(active_model='account.move', active_ids=[invoice2.id, invoice1.id]).validate_move()
|
||
|
|
|
||
|
|
for aml in invoice1.line_ids:
|
||
|
|
self.assertEqual(aml.partner_id, self.partner_a)
|
||
|
|
for aml in invoice2.line_ids:
|
||
|
|
self.assertEqual(aml.partner_id, self.partner_b)
|
||
|
|
|
||
|
|
@freeze_time('2023-01-01')
|
||
|
|
def test_post_valid_invoices_when_auto_post(self):
|
||
|
|
valid_invoice = self.init_invoice(move_type='out_invoice', products=self.product_a, invoice_date='2023-01-01')
|
||
|
|
|
||
|
|
# missing partner
|
||
|
|
invalid_invoice_1 = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'invoice_date': '2023-01-01',
|
||
|
|
'date': '2023-01-01',
|
||
|
|
'invoice_line_ids': [(0, 0, {
|
||
|
|
'name': 'test line',
|
||
|
|
'price_unit': 10,
|
||
|
|
'quantity': 1,
|
||
|
|
'account_id': self.company_data['default_account_revenue'].id,
|
||
|
|
})],
|
||
|
|
})
|
||
|
|
|
||
|
|
# missing invoice lines
|
||
|
|
invalid_invoice_2 = self.init_invoice(move_type='out_invoice', invoice_date='2023-01-01')
|
||
|
|
|
||
|
|
(valid_invoice + invalid_invoice_1 + invalid_invoice_2).auto_post = 'at_date'
|
||
|
|
|
||
|
|
self.env['account.move']._autopost_draft_entries()
|
||
|
|
self.assertEqual(valid_invoice.state, 'posted')
|
||
|
|
self.assertEqual(invalid_invoice_1.state, 'draft')
|
||
|
|
|
||
|
|
self.assertTrue(any(
|
||
|
|
message.body == "<p>The move could not be posted for the following reason: The field 'Customer' is required, please complete it to validate the Customer Invoice.</p>"
|
||
|
|
for message in invalid_invoice_1.message_ids))
|
||
|
|
|
||
|
|
self.assertEqual(invalid_invoice_2.state, 'draft')
|
||
|
|
self.assertTrue(any(
|
||
|
|
message.body == "<p>The move could not be posted for the following reason: You need to add a line before posting.</p>"
|
||
|
|
for message in invalid_invoice_2.message_ids))
|
||
|
|
|
||
|
|
def test_no_taxes_on_payment_term_line(self):
|
||
|
|
''' No tax should be set on payment_term lines'''
|
||
|
|
|
||
|
|
receivable_account = self.partner_a.property_account_receivable_id
|
||
|
|
receivable_account.tax_ids = [Command.set(self.company_data['default_tax_sale'].ids)]
|
||
|
|
|
||
|
|
invoice = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'name': 'test line',
|
||
|
|
'quantity': 1,
|
||
|
|
'price_unit': 100,
|
||
|
|
})
|
||
|
|
],
|
||
|
|
})
|
||
|
|
|
||
|
|
self.assertRecordValues(invoice.line_ids.filtered(lambda l: l.display_type == 'payment_term'), [
|
||
|
|
{'account_id': receivable_account.id, 'tax_ids': []},
|
||
|
|
])
|
||
|
|
|
||
|
|
def test_invoice_journal_account_check_constraints(self):
|
||
|
|
"""
|
||
|
|
Test account-journal constraint check is working as expected in a complex write operation
|
||
|
|
Setup:
|
||
|
|
- journal_a accepts account_a but not account_b
|
||
|
|
- journal_b accepts account_b but not account_a
|
||
|
|
We expect that constraints are checked as usual when creating/writing records, and in particular
|
||
|
|
changing account and journal at the same time should work
|
||
|
|
"""
|
||
|
|
|
||
|
|
account_a = self.company_data['default_account_revenue'].copy()
|
||
|
|
journal_a = self.company_data['default_journal_sale'].copy({'default_account_id': account_a.id})
|
||
|
|
account_b = account_a.copy()
|
||
|
|
journal_b = journal_a.copy({'default_account_id': account_b.id})
|
||
|
|
journal_a.account_control_ids = account_a | self.company_data['default_account_tax_sale'] | self.company_data['default_account_receivable']
|
||
|
|
journal_b.account_control_ids = account_b | self.company_data['default_account_tax_sale'] | self.company_data['default_account_receivable']
|
||
|
|
|
||
|
|
# Should not raise
|
||
|
|
invoice = self.env['account.move'].with_context(default_move_type='out_invoice').create({
|
||
|
|
'journal_id': journal_a.id,
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'name': 'My super product.',
|
||
|
|
'quantity': 1.0,
|
||
|
|
'price_unit': 750.0,
|
||
|
|
'account_id': account_a.id,
|
||
|
|
})
|
||
|
|
]
|
||
|
|
})
|
||
|
|
|
||
|
|
# Should not raise
|
||
|
|
invoice.write({'journal_id': journal_b.id, 'invoice_line_ids': [Command.update(invoice.invoice_line_ids.id, {'account_id': account_b.id})]})
|
||
|
|
|
||
|
|
with self.assertRaises(UserError), self.cr.savepoint():
|
||
|
|
invoice.write({'journal_id': journal_a.id})
|
||
|
|
with self.assertRaises(UserError), self.cr.savepoint():
|
||
|
|
# we want to test the update of both records in the same write operation
|
||
|
|
invoice.write({'invoice_line_ids': [Command.update(invoice.invoice_line_ids.id, {'account_id': account_a.id})]})
|
||
|
|
|
||
|
|
def test_discount_allocation_account_on_invoice(self):
|
||
|
|
# Ensure two aml of display_type 'discount' are correctly created when setting an account for discounts in the settings
|
||
|
|
discount_account = self.company_data['default_account_expense'].copy()
|
||
|
|
self.company_data['company'].account_discount_expense_allocation_id = discount_account
|
||
|
|
|
||
|
|
invoice = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'product_id': self.product_a.id,
|
||
|
|
'quantity': 1,
|
||
|
|
'discount': 5,
|
||
|
|
})
|
||
|
|
],
|
||
|
|
})
|
||
|
|
product_line_account = invoice.line_ids.filtered(lambda x: x.product_id).account_id
|
||
|
|
self.assertRecordValues(invoice.line_ids.filtered(lambda l: l.display_type == 'discount'), [
|
||
|
|
{
|
||
|
|
'account_id': product_line_account.id,
|
||
|
|
'tax_ids': [],
|
||
|
|
'amount_currency': -50.0,
|
||
|
|
'debit': 0.0,
|
||
|
|
'credit': 50.0,
|
||
|
|
},
|
||
|
|
{
|
||
|
|
'account_id': discount_account.id,
|
||
|
|
'tax_ids': [],
|
||
|
|
'amount_currency': 50.0,
|
||
|
|
'debit': 50.0,
|
||
|
|
'credit': 0.0,
|
||
|
|
},
|
||
|
|
])
|
||
|
|
|
||
|
|
def test_keep_receivable(self):
|
||
|
|
"""Duplicating an invoice with a different receivable account should keep the account."""
|
||
|
|
receivable_account = self.partner_a.property_account_receivable_id
|
||
|
|
other_receivable_account = receivable_account.copy()
|
||
|
|
|
||
|
|
invoice = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'name': 'test line',
|
||
|
|
'quantity': 1,
|
||
|
|
'price_unit': 100,
|
||
|
|
})
|
||
|
|
],
|
||
|
|
})
|
||
|
|
|
||
|
|
invoice.line_ids.filtered(lambda l: l.display_type == 'payment_term').account_id = other_receivable_account
|
||
|
|
duplicate_invoice = invoice.copy()
|
||
|
|
|
||
|
|
self.assertEqual(
|
||
|
|
duplicate_invoice.line_ids.filtered(lambda l: l.display_type == 'payment_term').account_id,
|
||
|
|
other_receivable_account
|
||
|
|
)
|
||
|
|
|
||
|
|
def test_account_on_invoice_line_product_removal(self):
|
||
|
|
"""Removing a product from an invoice line should preserve that line's account."""
|
||
|
|
other_income_account = self.product_a.property_account_income_id.copy()
|
||
|
|
|
||
|
|
invoice = self.env['account.move'].create({
|
||
|
|
'move_type': 'out_invoice',
|
||
|
|
'partner_id': self.partner_a.id,
|
||
|
|
'invoice_date': fields.Date.from_string('2019-01-01'),
|
||
|
|
'invoice_line_ids': [
|
||
|
|
Command.create({
|
||
|
|
'product_id': self.product_a.id,
|
||
|
|
}),
|
||
|
|
]
|
||
|
|
})
|
||
|
|
invoice.invoice_line_ids.account_id = other_income_account
|
||
|
|
invoice.invoice_line_ids.product_id = False
|
||
|
|
|
||
|
|
self.assertEqual(
|
||
|
|
invoice.invoice_line_ids.account_id,
|
||
|
|
other_income_account,
|
||
|
|
"Removing a product from an invoice line should no change the account."
|
||
|
|
)
|
||
|
|
|
||
|
|
def test_compute_name_payment_reference(self):
|
||
|
|
"""
|
||
|
|
Test that the label of the payment_term line is consistent with the payment reference
|
||
|
|
Also tests that it won't affect the hash inalterability report
|
||
|
|
"""
|
||
|
|
self.company_data['default_journal_sale'].restrict_mode_hash_table = True
|
||
|
|
|
||
|
|
move_form = Form(self.env['account.move'].with_context(default_move_type='out_invoice'))
|
||
|
|
move_form.partner_id = self.partner_b
|
||
|
|
with move_form.invoice_line_ids.new() as line_form:
|
||
|
|
line_form.product_id = self.product_a
|
||
|
|
invoice = move_form.save()
|
||
|
|
payment_term_lines = invoice.line_ids.filtered(lambda line: line.display_type == 'payment_term')
|
||
|
|
|
||
|
|
self.assertRecordValues(payment_term_lines, [
|
||
|
|
{'name': 'installment #1'},
|
||
|
|
{'name': 'installment #2'},
|
||
|
|
])
|
||
|
|
|
||
|
|
move_form.payment_reference = "Super Reference"
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
self.assertRecordValues(payment_term_lines, [
|
||
|
|
{'name': 'Super Reference installment #1'},
|
||
|
|
{'name': 'Super Reference installment #2'},
|
||
|
|
])
|
||
|
|
|
||
|
|
move_form.payment_reference = "Great Reference"
|
||
|
|
invoice = move_form.save()
|
||
|
|
|
||
|
|
self.assertRecordValues(payment_term_lines, [
|
||
|
|
{'name': 'Great Reference installment #1'},
|
||
|
|
{'name': 'Great Reference installment #2'},
|
||
|
|
])
|
||
|
|
|
||
|
|
invoice.action_post()
|
||
|
|
move_form.payment_reference = "Bad Reference"
|
||
|
|
move_form.save()
|
||
|
|
|
||
|
|
# The integrity check should work
|
||
|
|
integrity_check = invoice.company_id._check_hash_integrity()['results'][0]
|
||
|
|
self.assertEqual(integrity_check['msg_cover'], 'All entries are hashed.')
|
||
|
|
|
||
|
|
def test_out_invoice_create_cross_branch_refund(self):
|
||
|
|
"""You should not be able to reverse moves from different branches."""
|
||
|
|
|
||
|
|
# create a new branch
|
||
|
|
self.env.company.write({
|
||
|
|
'child_ids': [
|
||
|
|
Command.create({'name': 'Branch A'}),
|
||
|
|
],
|
||
|
|
})
|
||
|
|
self.cr.precommit.run() # load the CoA
|
||
|
|
|
||
|
|
# create an invoice on the new branch
|
||
|
|
branch_a = self.env.company.child_ids
|
||
|
|
branch_invoice = self.init_invoice('out_invoice', products=self.product_a, company=branch_a)
|
||
|
|
|
||
|
|
branch_invoice.action_post()
|
||
|
|
self.invoice.action_post()
|
||
|
|
|
||
|
|
with self.assertRaises(UserError) as error_catcher:
|
||
|
|
# attempt to reverse both the parent's and the branch's move at once
|
||
|
|
move_reversal = self.env['account.move.reversal'].with_context(
|
||
|
|
active_model="account.move",
|
||
|
|
active_ids=(branch_invoice + self.invoice).ids,
|
||
|
|
).create({})
|
||
|
|
|
||
|
|
move_reversal.refund_moves()
|
||
|
|
|
||
|
|
self.assertEqual(error_catcher.exception.args[0], "All selected moves for reversal must belong to the same company.")
|