2021-04-05 16:27:30 +02:00
|
|
|
# Copyright 2015-2019 ACSONE SA/NV (<http://acsone.eu>)
|
|
|
|
# Copyright 2021 CorporateHub (https://corporatehub.eu)
|
|
|
|
# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
|
|
|
|
|
|
|
|
from datetime import timedelta
|
|
|
|
|
|
|
|
from odoo import fields
|
|
|
|
from odoo.exceptions import UserError
|
|
|
|
from odoo.tests import common
|
|
|
|
|
|
|
|
|
2022-03-22 16:31:02 +01:00
|
|
|
class TestAccountInvoiceConstraintChronology(common.TransactionCase):
|
2021-04-05 16:27:30 +02:00
|
|
|
@classmethod
|
|
|
|
def setUpClass(cls):
|
|
|
|
super().setUpClass()
|
|
|
|
|
|
|
|
cls.company = cls.env.ref("base.main_company")
|
|
|
|
cls.partner_2 = cls.env.ref("base.res_partner_2")
|
|
|
|
cls.today = fields.Date.today()
|
|
|
|
cls.yesterday = cls.today - timedelta(days=1)
|
|
|
|
cls.tomorrow = cls.today + timedelta(days=1)
|
|
|
|
|
|
|
|
cls.AccountJournal = cls.env["account.journal"]
|
|
|
|
cls.sale_journal = cls.AccountJournal.create(
|
|
|
|
{
|
|
|
|
"name": "Sale journal",
|
|
|
|
"code": "SALE",
|
|
|
|
"type": "sale",
|
|
|
|
"check_chronology": True,
|
|
|
|
}
|
|
|
|
)
|
|
|
|
|
|
|
|
cls.ProductProduct = cls.env["product.product"]
|
|
|
|
cls.product = cls.ProductProduct.create({"name": "Product"})
|
|
|
|
|
|
|
|
cls.AccountMove = cls.env["account.move"]
|
|
|
|
with common.Form(
|
2021-04-06 20:41:36 +02:00
|
|
|
cls.AccountMove.with_context(default_move_type="out_invoice")
|
2021-04-05 16:27:30 +02:00
|
|
|
) as invoice_form:
|
|
|
|
invoice_form.invoice_date = cls.today
|
|
|
|
invoice_form.partner_id = cls.partner_2
|
|
|
|
with invoice_form.invoice_line_ids.new() as line_form:
|
|
|
|
line_form.product_id = cls.product
|
|
|
|
cls.invoice_1 = invoice_form.save()
|
2022-10-06 17:05:07 +02:00
|
|
|
cls.invoice_1.update({"journal_id": cls.sale_journal})
|
2021-04-05 16:27:30 +02:00
|
|
|
cls.invoice_2 = cls.invoice_1.copy()
|
|
|
|
|
|
|
|
cls.AccountMoveReversal = cls.env["account.move.reversal"]
|
|
|
|
|
|
|
|
def test_journal_type_change(self):
|
|
|
|
self.assertTrue(self.sale_journal.check_chronology)
|
|
|
|
|
|
|
|
with common.Form(self.sale_journal) as form:
|
|
|
|
form.type = "general"
|
|
|
|
self.assertFalse(self.sale_journal.check_chronology)
|
|
|
|
|
|
|
|
with common.Form(self.sale_journal) as form:
|
|
|
|
form.type = "sale"
|
|
|
|
self.assertFalse(self.sale_journal.check_chronology)
|
|
|
|
|
|
|
|
with common.Form(self.sale_journal) as form:
|
|
|
|
form.check_chronology = True
|
|
|
|
self.assertTrue(self.sale_journal.check_chronology)
|
|
|
|
|
|
|
|
def test_invoice_draft(self):
|
|
|
|
self.invoice_1.invoice_date = self.yesterday
|
|
|
|
self.invoice_2.invoice_date = self.today
|
|
|
|
with self.assertRaises(UserError):
|
|
|
|
self.invoice_2.action_post()
|
|
|
|
|
|
|
|
def test_invoice_draft_nocheck(self):
|
|
|
|
self.invoice_1.invoice_date = self.yesterday
|
|
|
|
self.invoice_2.invoice_date = self.today
|
|
|
|
self.sale_journal.check_chronology = False
|
|
|
|
self.invoice_2.action_post()
|
|
|
|
|
|
|
|
def test_invoice_validate(self):
|
|
|
|
self.invoice_1.invoice_date = self.tomorrow
|
|
|
|
self.invoice_1.action_post()
|
|
|
|
self.invoice_2.invoice_date = self.today
|
|
|
|
with self.assertRaises(UserError):
|
|
|
|
self.invoice_2.action_post()
|
|
|
|
|
|
|
|
def test_invoice_without_date(self):
|
|
|
|
self.invoice_1.invoice_date = self.yesterday
|
|
|
|
self.invoice_2.invoice_date = False
|
|
|
|
with self.assertRaises(UserError):
|
|
|
|
self.invoice_2.action_post()
|
|
|
|
|
|
|
|
def test_invoice_refund_before(self):
|
|
|
|
self.invoice_1.invoice_date = self.tomorrow
|
|
|
|
self.invoice_1.action_post()
|
|
|
|
refund = (
|
|
|
|
self.AccountMoveReversal.with_context(
|
2021-04-06 20:40:53 +02:00
|
|
|
active_model="account.move",
|
|
|
|
active_ids=self.invoice_1.ids,
|
2021-04-05 16:27:30 +02:00
|
|
|
)
|
|
|
|
.create(
|
2022-03-31 16:39:29 +02:00
|
|
|
{
|
|
|
|
"date": self.today,
|
|
|
|
"reason": "no reason",
|
|
|
|
"refund_method": "refund",
|
|
|
|
"journal_id": self.invoice_1.journal_id.id,
|
|
|
|
}
|
2021-04-05 16:27:30 +02:00
|
|
|
)
|
|
|
|
.reverse_moves()
|
|
|
|
)
|
|
|
|
refund = self.AccountMove.browse(refund["res_id"])
|
|
|
|
refund.action_post()
|
|
|
|
|
|
|
|
def test_invoice_refund_before_same_sequence(self):
|
|
|
|
self.sale_journal.refund_sequence = False
|
|
|
|
self.invoice_1.invoice_date = self.tomorrow
|
|
|
|
self.invoice_1.action_post()
|
|
|
|
refund = (
|
|
|
|
self.AccountMoveReversal.with_context(
|
2021-04-06 20:40:53 +02:00
|
|
|
active_model="account.move",
|
|
|
|
active_ids=self.invoice_1.ids,
|
2021-04-05 16:27:30 +02:00
|
|
|
)
|
|
|
|
.create(
|
2022-03-31 16:39:29 +02:00
|
|
|
{
|
|
|
|
"date": self.today,
|
|
|
|
"reason": "no reason",
|
|
|
|
"refund_method": "refund",
|
|
|
|
"journal_id": self.invoice_1.journal_id.id,
|
|
|
|
}
|
2021-04-05 16:27:30 +02:00
|
|
|
)
|
|
|
|
.reverse_moves()
|
|
|
|
)
|
|
|
|
refund = self.AccountMove.browse(refund["res_id"])
|
|
|
|
with self.assertRaises(UserError):
|
|
|
|
refund.action_post()
|
2021-04-08 07:41:03 +02:00
|
|
|
|
|
|
|
def test_invoice_higher_number(self):
|
|
|
|
self.invoice_1.invoice_date = self.yesterday
|
|
|
|
self.invoice_1.action_post()
|
|
|
|
self.invoice_1.button_draft()
|
|
|
|
self.invoice_1.invoice_date = False
|
|
|
|
|
|
|
|
self.invoice_2.invoice_date = self.today
|
|
|
|
self.invoice_2.action_post()
|
|
|
|
|
|
|
|
self.invoice_1.invoice_date = self.tomorrow
|
|
|
|
with self.assertRaisesRegex(UserError, "higher number"):
|
|
|
|
self.invoice_1.action_post()
|
2022-02-17 10:26:39 +01:00
|
|
|
|
|
|
|
def test_modify_validated_past_invoice(self):
|
|
|
|
"""We've got an invoice from yesterday that we need to modify but new ones
|
|
|
|
have been posted since. As the invoice already has a name, we should be able
|
|
|
|
to validate it"""
|
|
|
|
self.invoice_1.invoice_date = self.yesterday
|
|
|
|
self.invoice_1.action_post()
|
|
|
|
self.invoice_2.invoice_date = self.today
|
|
|
|
self.invoice_2.action_post()
|
|
|
|
self.invoice_1.button_cancel()
|
|
|
|
self.invoice_1.button_draft()
|
|
|
|
self.invoice_1.action_post()
|
|
|
|
self.invoice_1_5 = self.invoice_1.copy()
|
|
|
|
self.invoice_1_5.invoice_date = self.yesterday
|
|
|
|
with self.assertRaises(UserError):
|
|
|
|
self.invoice_1_5.action_post()
|
2023-06-15 14:45:29 +02:00
|
|
|
|
|
|
|
def test_modify_invoice_date_validated_past_invoice(self):
|
|
|
|
# INV5 YYYYMM20 posted
|
|
|
|
# INV4 YYYYMM15 posted
|
|
|
|
# INV3 YYYYMM10 posted
|
|
|
|
# INV2 YYYYMM05 posted
|
|
|
|
# INV1 YYYYMM01 posted
|
|
|
|
# if we pass INV3 to draft and change the date to YYYYYMM15 or YYYYYMM05
|
|
|
|
# it should be able to validate, but if we change the date
|
|
|
|
# higher than YYYYYMM15 or lower than YYYYYMM05
|
|
|
|
# it should not be able to validate.
|
|
|
|
after_5_days = self.today + timedelta(days=5)
|
|
|
|
after_10_days = self.today + timedelta(days=10)
|
|
|
|
after_15_days = self.today + timedelta(days=15)
|
|
|
|
after_20_days = self.today + timedelta(days=20)
|
|
|
|
after_25_days = self.today + timedelta(days=25)
|
|
|
|
|
|
|
|
self.invoice_1.action_post()
|
|
|
|
|
|
|
|
self.invoice_1_a_5 = self.invoice_1.copy()
|
|
|
|
self.invoice_1_a_5.invoice_date = after_5_days
|
|
|
|
self.invoice_1_a_5.action_post()
|
|
|
|
self.invoice_1_a_10 = self.invoice_1.copy()
|
|
|
|
self.invoice_1_a_10.invoice_date = after_10_days
|
|
|
|
self.invoice_1_a_10.action_post()
|
|
|
|
self.invoice_1_a_15 = self.invoice_1.copy()
|
|
|
|
self.invoice_1_a_15.invoice_date = after_15_days
|
|
|
|
self.invoice_1_a_15.action_post()
|
|
|
|
self.invoice_1_a_20 = self.invoice_1.copy()
|
|
|
|
self.invoice_1_a_20.invoice_date = after_20_days
|
|
|
|
self.invoice_1_a_20.action_post()
|
|
|
|
self.invoice_1_a_25 = self.invoice_1.copy()
|
|
|
|
self.invoice_1_a_25.invoice_date = after_25_days
|
|
|
|
self.invoice_1_a_25.action_post()
|
|
|
|
|
|
|
|
self.invoice_1_a_15.button_cancel()
|
|
|
|
self.invoice_1_a_15.button_draft()
|
|
|
|
self.invoice_1_a_15.invoice_date = after_10_days
|
|
|
|
self.invoice_1_a_15.action_post()
|
|
|
|
|
|
|
|
self.invoice_1_a_15.button_cancel()
|
|
|
|
self.invoice_1_a_15.button_draft()
|
|
|
|
self.invoice_1_a_15.invoice_date = after_10_days - timedelta(days=1)
|
|
|
|
with self.assertRaisesRegex(
|
|
|
|
UserError,
|
|
|
|
"Chronology conflict: Invoice {} cannot be before invoice {}.".format(
|
|
|
|
self.invoice_1_a_15.name, self.invoice_1_a_10.name
|
|
|
|
),
|
|
|
|
):
|
|
|
|
self.invoice_1_a_15.action_post()
|
|
|
|
|
|
|
|
self.invoice_1_a_15.button_cancel()
|
|
|
|
self.invoice_1_a_15.button_draft()
|
|
|
|
self.invoice_1_a_15.invoice_date = after_20_days
|
|
|
|
self.invoice_1_a_15.action_post()
|
|
|
|
|
|
|
|
self.invoice_1_a_15.button_cancel()
|
|
|
|
self.invoice_1_a_15.button_draft()
|
|
|
|
self.invoice_1_a_15.invoice_date = after_20_days + timedelta(days=1)
|
|
|
|
with self.assertRaisesRegex(
|
|
|
|
UserError,
|
|
|
|
"Chronology conflict: Invoice {} cannot be after invoice {}.".format(
|
|
|
|
self.invoice_1_a_15.name, self.invoice_1_a_20.name
|
|
|
|
),
|
|
|
|
):
|
|
|
|
self.invoice_1_a_15.action_post()
|