odoo-addon-l10n-br-fiscal 16.0.2.17.0__py3-none-any.whl → 16.0.19.4.0__py3-none-any.whl

odoo/addons/l10n_br_fiscal/tests/test_document_edition.py

@@ -0,0 +1,308 @@
+# Copyright 2025-TODAY  Akretion - Raphaël Valyi
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
+
+from unittest import mock
+
+from odoo.tests import TransactionCase
+from odoo.tests.common import Form, tagged
+
+
+@tagged("post_install", "-at_install")
+class TestDocumentEdition(TransactionCase):
+    @classmethod
+    def setUpClass(cls):
+        super().setUpClass()
+        cls.env = cls.env(context=dict(cls.env.context, tracking_disable=True))
+        cls.user = cls.env.user
+        cls.company = cls.env.ref("l10n_br_base.empresa_lucro_presumido")
+        cls.user.company_ids |= cls.company
+        cls.user.company_id = cls.company.id
+
+    def test_basic_doc_edition(self):
+        doc_form = Form(
+            self.env["l10n_br_fiscal.document"].with_context(
+                default_fiscal_operation_type="out",
+            )
+        )
+        doc_form.company_id = self.env.ref("l10n_br_base.empresa_lucro_presumido")
+        doc_form.partner_id = self.env.ref("l10n_br_base.res_partner_cliente1_sp")
+        doc_form.fiscal_operation_id = self.env.ref("l10n_br_fiscal.fo_venda")
+        doc_form.ind_final = "1"
+        product_id = self.env.ref("product.product_product_6")
+        product_id.list_price = 150  # we will later check we can set price_unit to 100
+        with doc_form.fiscal_line_ids.new() as line_form:
+            original_method = type(
+                self.env["l10n_br_fiscal.operation.line"]
+            ).map_fiscal_taxes
+
+            def wrapped_method(self, *args, **kwargs):
+                return original_method(self, *args, **kwargs)
+
+            with mock.patch.object(
+                type(self.env["l10n_br_fiscal.operation.line"]),
+                "map_fiscal_taxes",
+                side_effect=wrapped_method,
+                autospec=True,
+            ) as mocked:
+                line_form.product_id = product_id
+
+            # ensure the tax engine is called with the proper
+            # parameters, especially ind_final
+            # as it is related=document_id.ind_final
+            # which is converted to move_id.ind_final to work live
+            mocked.assert_called_with(
+                self.env.ref("l10n_br_fiscal.fo_venda_revenda"),
+                company=doc_form.company_id,
+                partner=doc_form.partner_id,
+                product=product_id,
+                ncm=product_id.ncm_id,
+                nbm=self.env["l10n_br_fiscal.nbm"],
+                nbs=self.env["l10n_br_fiscal.nbs"],
+                cest=self.env["l10n_br_fiscal.cest"],
+                city_taxation_code=self.env["l10n_br_fiscal.city.taxation.code"],
+                service_type=self.env["l10n_br_fiscal.service.type"],
+                ind_final="1",
+            )
+
+            line_form.price_unit = 50
+            line_form.quantity = 2
+            self.assertEqual(len(line_form.fiscal_tax_ids), 4)
+            self.assertEqual(
+                line_form.icms_tax_id, self.env.ref("l10n_br_fiscal.tax_icms_12")
+            )
+            self.assertEqual(line_form.icms_value, 12.0)
+            line_form.price_unit = 100
+            self.assertEqual(
+                line_form.icms_tax_id, self.env.ref("l10n_br_fiscal.tax_icms_12")
+            )
+            self.assertEqual(line_form.icms_value, 24.0)
+            self.assertEqual(
+                line_form.fiscal_operation_line_id,
+                self.env.ref("l10n_br_fiscal.fo_venda_revenda"),
+            )
+            self.assertEqual(
+                line_form.ipi_tax_id, self.env.ref("l10n_br_fiscal.tax_ipi_nt")
+            )
+
+            line_form.fiscal_operation_line_id = self.env.ref(
+                "l10n_br_fiscal.fo_venda_venda"
+            )
+            self.assertEqual(
+                line_form.ipi_tax_id, self.env.ref("l10n_br_fiscal.tax_ipi_3_25")
+            )
+
+            # ensure manually setting a xx_tax_id is properly saved (not recomputed):
+            line_form.icms_tax_id = self.env.ref("l10n_br_fiscal.tax_icms_18")
+            self.assertEqual(line_form.icms_value, 37.17)
+            self.assertEqual(
+                line_form.ipi_tax_id, self.env.ref("l10n_br_fiscal.tax_ipi_3_25")
+            )
+            line_form.icmsfcp_base = line_form.price_unit
+            line_form.icmsfcp_value = 3  # ensure manually setting FCP value works
+
+        doc = doc_form.save()
+        line = doc.fiscal_line_ids[0]
+        self.assertEqual(line.price_unit, 100)
+        self.assertEqual(line.fiscal_price, 100)
+        self.assertEqual(line.quantity, 2)
+        self.assertEqual(line.fiscal_quantity, 2)
+        self.assertEqual(len(line.fiscal_tax_ids), 4)
+
+        self.assertEqual(
+            line.fiscal_operation_line_id,
+            self.env.ref("l10n_br_fiscal.fo_venda_venda"),
+        )
+        self.assertEqual(
+            line.icms_tax_id.id,
+            self.ref("l10n_br_fiscal.tax_icms_18"),
+        )
+        self.assertEqual(line.ipi_tax_id, self.env.ref("l10n_br_fiscal.tax_ipi_3_25"))
+        self.assertEqual(line.icms_value, 37.17)
+        self.assertEqual(line.icmsfcp_base, line.price_unit)
+        self.assertEqual(line.icmsfcp_value, 3)
+
+    def test_product_fiscal_factor(self):
+        doc_form = Form(
+            self.env["l10n_br_fiscal.document"].with_context(
+                default_fiscal_operation_type="out",
+            )
+        )
+        doc_form.company_id = self.env.ref("l10n_br_base.empresa_lucro_presumido")
+        doc_form.partner_id = self.env.ref("l10n_br_base.res_partner_cliente1_sp")
+        doc_form.fiscal_operation_id = self.env.ref("l10n_br_fiscal.fo_venda")
+        doc_form.ind_final = "1"
+        product_id = self.env.ref("product.product_product_6")
+        product_id.uot_factor = 2
+        with doc_form.fiscal_line_ids.new() as line_form:
+            line_form.product_id = product_id
+            line_form.price_unit = 100
+            line_form.quantity = 10
+
+        doc = doc_form.save()
+        self.assertEqual(doc.fiscal_line_ids[0].price_unit, 100)
+        self.assertEqual(doc.fiscal_line_ids[0].fiscal_price, 50)
+        self.assertEqual(doc.fiscal_line_ids[0].quantity, 10)
+        self.assertEqual(doc.fiscal_line_ids[0].fiscal_quantity, 20)
+
+    def test_product_fiscal_price_and_qty_edition(self):
+        doc_form = Form(
+            self.env["l10n_br_fiscal.document"].with_context(
+                default_fiscal_operation_type="out",
+            )
+        )
+        doc_form.company_id = self.env.ref("l10n_br_base.empresa_lucro_presumido")
+        doc_form.partner_id = self.env.ref("l10n_br_base.res_partner_cliente1_sp")
+        doc_form.fiscal_operation_id = self.env.ref("l10n_br_fiscal.fo_venda")
+        doc_form.ind_final = "1"
+        product_id = self.env.ref("product.product_product_6")
+        product_id.list_price = 100
+        with doc_form.fiscal_line_ids.new() as line_form:
+            line_form.product_id = product_id
+            line_form.price_unit = 110
+            line_form.quantity = 10
+            line_form.fiscal_price = 112
+            line_form.fiscal_quantity = 5
+            self.assertEqual(line_form.price_unit, 110)
+            self.assertEqual(line_form.fiscal_price, 112)
+            self.assertEqual(line_form.quantity, 10)
+            self.assertEqual(line_form.fiscal_quantity, 5)
+
+        doc = doc_form.save()
+        self.assertEqual(doc.fiscal_line_ids[0].price_unit, 110)
+        self.assertEqual(doc.fiscal_line_ids[0].fiscal_price, 112)
+        self.assertEqual(doc.fiscal_line_ids[0].quantity, 10)
+        self.assertEqual(doc.fiscal_line_ids[0].fiscal_quantity, 5)
+
+    def test_landed_costs_by_line_and_by_total(self):
+        """
+        Tests both landed cost scenarios: 'by line' and 'by total'.
+        1. By Line: Enters costs on lines and verifies the header totals.
+        2. By Total: Enters costs on the header and verifies lines distribution.
+        """
+        self.env.user.groups_id |= self.env.ref("l10n_br_fiscal.group_user")
+        product1 = self.env.ref("product.product_product_6")
+        product2 = self.env.ref("product.product_product_7")
+
+        # Part 1: Test with delivery_costs = 'line'
+        # ----------------------------------------------------
+        self.company.delivery_costs = "line"
+        doc_form = Form(self.env["l10n_br_fiscal.document"])
+        doc_form.company_id = self.company
+        doc_form.partner_id = self.env.ref("l10n_br_base.res_partner_cliente1_sp")
+        doc_form.fiscal_operation_id = self.env.ref("l10n_br_fiscal.fo_venda")
+
+        with doc_form.fiscal_line_ids.new() as line1:
+            line1.product_id = product1
+            line1.fiscal_operation_line_id = self.env.ref(
+                "l10n_br_fiscal.fo_venda_venda"
+            )
+            line1.price_unit = 1000.0
+            line1.quantity = 2.0  # Gross: 2000
+            line1.freight_value = 10.0
+            line1.insurance_value = 20.0
+            line1.other_value = 5.0
+
+        with doc_form.fiscal_line_ids.new() as line2:
+            line2.product_id = product2
+            line2.fiscal_operation_line_id = self.env.ref(
+                "l10n_br_fiscal.fo_venda_venda"
+            )
+            line2.price_unit = 500.0
+            line2.quantity = 1.0  # Gross: 500
+            line2.freight_value = 4.0
+            line2.insurance_value = 6.0
+            line2.other_value = 2.0
+
+        doc = doc_form.save()
+
+        self.assertEqual(doc.company_id.delivery_costs, "line")
+        # Assert header totals are the SUM of line values
+        self.assertAlmostEqual(doc.amount_freight_value, 14.0)  # 10.0 + 4.0
+        self.assertAlmostEqual(doc.amount_insurance_value, 26.0)  # 20.0 + 6.0
+        self.assertAlmostEqual(doc.amount_other_value, 7.0)  # 5.0 + 2.0
+
+        # Assert final fiscal totals (bottom-up calculation)
+        # price_gross = (1000*2) + (500*1) = 2500
+        # landed_costs = 14 + 26 + 7 = 47
+        # fiscal_amount_untaxed (IPI Base) = 2500 + 47 = 2547
+        self.assertAlmostEqual(doc.fiscal_amount_untaxed, 2547.00)
+        # fiscal_amount_tax (IPI) = (2035 * 3.25%) + (512 * 5%) = 66.14 + 25.60 = 91.74
+        self.assertAlmostEqual(doc.fiscal_amount_tax, 91.74, places=2)
+        # fiscal_amount_total = 2547.00 + 91.74 = 2638.74
+        self.assertAlmostEqual(doc.fiscal_amount_total, 2638.74, places=2)
+
+        # Part 2: Test with delivery_costs = 'total'
+        # ----------------------------------------------------
+        self.company.delivery_costs = "total"
+        doc_form_edit = Form(doc)
+        # Set new header totals, which should trigger inverse methods to distribute
+        doc_form_edit.amount_freight_value = 30.0
+        doc_form_edit.amount_insurance_value = 60.0
+        doc_form_edit.amount_other_value = 90.0
+        doc_after_total_update = doc_form_edit.save()
+
+        line1 = doc_after_total_update.fiscal_line_ids[0]
+        line2 = doc_after_total_update.fiscal_line_ids[1]
+
+        # Assert values were distributed proportionally to price_gross
+        # (2000 vs 500 -> 80% vs 20%)
+        # Freight: 30.0 * 0.8 = 24.0 | 30.0 * 0.2 = 6.0
+        self.assertAlmostEqual(line1.freight_value, 24.0)
+        self.assertAlmostEqual(line2.freight_value, 6.0)
+        # Insurance: 60.0 * 0.8 = 48.0 | 60.0 * 0.2 = 12.0
+        self.assertAlmostEqual(line1.insurance_value, 48.0)
+        self.assertAlmostEqual(line2.insurance_value, 12.0)
+        # Other: 90.0 * 0.8 = 72.0 | 90.0 * 0.2 = 18.0
+        self.assertAlmostEqual(line1.other_value, 72.0)
+        self.assertAlmostEqual(line2.other_value, 18.0)
+
+        # Assert final fiscal totals are recomputed correctly (top-down calculation)
+        # price_gross = 2500
+        # landed_costs = 30 + 60 + 90 = 180
+        # fiscal_amount_untaxed (IPI Base) = 2500 + 180 = 2680
+        self.assertAlmostEqual(doc_after_total_update.fiscal_amount_untaxed, 2680.00)
+        # Line 1 IPI Base = 2000 (product) + 24 (freight) + 48 (insurance)
+        # + 72 (other) = 2144
+        # Line 1 IPI Value = 2144 * 3.25% = 69.68
+        self.assertAlmostEqual(line1.ipi_base, 2144.00)
+        self.assertAlmostEqual(line1.ipi_value, 69.68, places=2)
+
+        # Line 2 IPI Base = 500 (product) + 6 (freight) + 12 (insurance)
+        # + 18 (other) = 536
+        # Line 2 IPI Value = 536 * 5% = 26.80
+        self.assertAlmostEqual(line2.ipi_base, 536.00)
+        self.assertAlmostEqual(line2.ipi_value, 26.80, places=2)
+
+        # fiscal_amount_tax (IPI) = 69.68 + 26.80 = 96.48
+        self.assertAlmostEqual(
+            doc_after_total_update.fiscal_amount_tax, 96.48, places=2
+        )
+        # fiscal_amount_total = 2680.00 + 96.48 = 2776.48
+        self.assertAlmostEqual(
+            doc_after_total_update.fiscal_amount_total, 2776.48, places=2
+        )
+
+    def test_difal_calculation(self):
+        partner = self.env.ref("l10n_br_base.res_partner_cliente5_pe")
+        partner.ind_ie_dest = "9"
+        doc_form = Form(
+            self.env["l10n_br_fiscal.document"].with_context(
+                default_fiscal_operation_type="out",
+            )
+        )
+        doc_form.company_id = self.company
+        doc_form.partner_id = partner
+        doc_form.fiscal_operation_id = self.env.ref("l10n_br_fiscal.fo_venda")
+
+        product = self.env.ref("product.product_product_6")
+        with doc_form.fiscal_line_ids.new() as line_form:
+            line_form.product_id = product
+            line_form.price_unit = 100.0
+            line_form.quantity = 1.0
+
+        doc = doc_form.save()
+        line = doc.fiscal_line_ids[0]
+        self.assertEqual(line.icms_destination_base, 100.0)
+        self.assertEqual(line.icms_origin_percent, 7.0)
+        self.assertEqual(line.icms_destination_percent, 20.5)
+        self.assertEqual(line.icms_destination_value, 13.5)

odoo/addons/l10n_br_fiscal/tests/test_fiscal_document_generic.py

@@ -2,7 +2,6 @@
 #   Magno Costa <magno.costa@akretion.com.br>
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
-
 from odoo.tests import TransactionCase
 
 from ..constants.icms import ICMS_ORIGIN_TAX_IMPORTED
@@ -12,6 +11,8 @@ class TestFiscalDocumentGeneric(TransactionCase):
     @classmethod
     def setUpClass(cls):
         super().setUpClass()
+        cls.env = cls.env(context=dict(cls.env.context, tracking_disable=True))
+
         # Contribuinte
         cls.nfe_same_state = cls.env.ref("l10n_br_fiscal.demo_nfe_same_state")
         cls.nfe_other_state = cls.env.ref("l10n_br_fiscal.demo_nfe_other_state")
@@ -38,26 +39,10 @@ class TestFiscalDocumentGeneric(TransactionCase):
 
     def test_nfe_same_state(self):
         """Test NFe same state."""
-
-        self.nfe_same_state._onchange_document_serie_id()
-        self.nfe_same_state._onchange_fiscal_operation_id()
-
         for line in self.nfe_same_state.fiscal_line_ids:
-            # Save the original price_unit value of the line as defined in
-            # the NFe demo data.
-            original_price_unit = line.price_unit
-
-            line._onchange_product_id_fiscal()
-
             # Restore the original price_unit value,
             # as the product change might have altered it.
-            line.price_unit = original_price_unit
-
-            line._onchange_commercial_quantity()
-            line._onchange_ncm_id()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_operation_line_id()
-            line._onchange_fiscal_taxes()
+            line.price_unit = 100
 
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
@@ -87,9 +72,10 @@ class TestFiscalDocumentGeneric(TransactionCase):
             # ICMS
             self.assertTrue(
                 is_icms_internal,
-                "Error to mapping ICMS Inernal for {}"
+                "Error to mapping ICMS Inernal for "
+                f"{self.nfe_same_state.partner_id.state_id.name}"
                 " for Venda de Contribuinte Dentro do "
-                "Estado.".format(self.nfe_same_state.partner_id.state_id.name),
+                "Estado.",
             )
             self.assertEqual(
                 line.icms_cst_id.code,
@@ -171,18 +157,7 @@ class TestFiscalDocumentGeneric(TransactionCase):
 
     def test_nfe_other_state(self):
         """Test NFe other state."""
-
-        self.nfe_other_state._onchange_document_serie_id()
-        self.nfe_other_state._onchange_fiscal_operation_id()
-
         for line in self.nfe_other_state.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_commercial_quantity()
-            line._onchange_ncm_id()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_operation_line_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -293,18 +268,7 @@ class TestFiscalDocumentGeneric(TransactionCase):
 
     def test_nfe_not_taxpayer(self):
         """Test NFe not taxpayer."""
-
-        self.nfe_not_taxpayer._onchange_document_serie_id()
-        self.nfe_not_taxpayer._onchange_fiscal_operation_id()
-
         for line in self.nfe_not_taxpayer.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_commercial_quantity()
-            line._onchange_ncm_id()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_operation_line_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -402,18 +366,7 @@ class TestFiscalDocumentGeneric(TransactionCase):
 
     def test_nfe_not_taxpayer_not_company(self):
         """Test NFe not taxpayer not Company."""
-
-        self.nfe_not_taxpayer_pf._onchange_document_serie_id()
-        self.nfe_not_taxpayer_pf._onchange_fiscal_operation_id()
-
         for line in self.nfe_not_taxpayer_pf.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_commercial_quantity()
-            line._onchange_ncm_id()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_operation_line_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -511,18 +464,7 @@ class TestFiscalDocumentGeneric(TransactionCase):
 
     def test_nfe_export(self):
         """Test NFe export."""
-
-        self.nfe_export._onchange_document_serie_id()
-        self.nfe_export._onchange_fiscal_operation_id()
-
         for line in self.nfe_export.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_commercial_quantity()
-            line._onchange_ncm_id()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_operation_line_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -586,8 +528,7 @@ class TestFiscalDocumentGeneric(TransactionCase):
             self.assertEqual(
                 line.pis_tax_id.name,
                 "PIS 0,65%",
-                "Error to mapping PIS 0,65%"
-                " for Venda de Contribuinte p/ o Exterior.",
+                "Error to mapping PIS 0,65% for Venda de Contribuinte p/ o Exterior.",
             )
             self.assertEqual(
                 line.pis_cst_id.code,
@@ -601,8 +542,7 @@ class TestFiscalDocumentGeneric(TransactionCase):
             self.assertEqual(
                 line.cofins_tax_id.name,
                 "COFINS 3%",
-                "Error to mapping COFINS 3%"
-                " for Venda de Contribuinte p/ o Exterior.",
+                "Error to mapping COFINS 3% for Venda de Contribuinte p/ o Exterior.",
             )
             self.assertEqual(
                 line.cofins_cst_id.code,
@@ -614,14 +554,7 @@ class TestFiscalDocumentGeneric(TransactionCase):
 
     def test_nfe_sn_same_state(self):
         """Test NFe Simples Nacional same state."""
-
-        self.nfe_sn_same_state._onchange_document_serie_id()
-        self.nfe_sn_same_state._onchange_fiscal_operation_id()
-
         for line in self.nfe_sn_same_state.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_commercial_quantity()
-
             # set fake estimate tax
             line.ncm_id.tax_estimate_ids.create(
                 {
@@ -632,11 +565,9 @@ class TestFiscalDocumentGeneric(TransactionCase):
                     "federal_taxes_national": 33.00,
                 }
             )
-
-            line._onchange_ncm_id()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_operation_line_id()
-            line._onchange_fiscal_taxes()
+            # força o compute, pois não é chamado automaticamente
+            # quando uma informação externa muda.
+            line._compute_tax_fields()
 
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
@@ -734,18 +665,7 @@ class TestFiscalDocumentGeneric(TransactionCase):
 
     def test_nfe_sn_other_state(self):
         """Test NFe SN other state."""
-
-        self.nfe_sn_other_state._onchange_document_serie_id()
-        self.nfe_sn_other_state._onchange_fiscal_operation_id()
-
         for line in self.nfe_sn_other_state.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_commercial_quantity()
-            line._onchange_ncm_id()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_operation_line_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -839,18 +759,7 @@ class TestFiscalDocumentGeneric(TransactionCase):
 
     def test_nfe_sn_not_taxpayer(self):
         """Test NFe SN not taxpayer."""
-
-        self.nfe_sn_not_taxpayer._onchange_document_serie_id()
-        self.nfe_sn_not_taxpayer._onchange_fiscal_operation_id()
-
         for line in self.nfe_sn_not_taxpayer.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_commercial_quantity()
-            line._onchange_ncm_id()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_operation_line_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -890,8 +799,7 @@ class TestFiscalDocumentGeneric(TransactionCase):
             self.assertEqual(
                 line.ipi_tax_id.name,
                 "IPI 5%",
-                "Erro ao mapear IPI 5%"
-                " para Venda de Simples Nacional Fora do Estado.",
+                "Erro ao mapear IPI 5% para Venda de Simples Nacional Fora do Estado.",
             )
             self.assertEqual(
                 line.ipi_cst_id.code,
@@ -932,18 +840,7 @@ class TestFiscalDocumentGeneric(TransactionCase):
 
     def test_nfe_sn_export(self):
         """Test NFe SN export."""
-
-        self.nfe_sn_export._onchange_document_serie_id()
-        self.nfe_sn_export._onchange_fiscal_operation_id()
-
         for line in self.nfe_sn_export.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_commercial_quantity()
-            line._onchange_ncm_id()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_operation_line_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -1053,7 +950,7 @@ class TestFiscalDocumentGeneric(TransactionCase):
         additional_data = self.nfe_not_taxpayer.fiscal_line_ids[0].additional_data
         self.assertEqual(
             additional_data,
-            "manual comment test",
+            "manual comment test - Valor Aprox. dos Tributos: R$\xa00,00",
         )
 
     def test_fields_freight_insurance_other_costs(self):
@@ -1063,6 +960,7 @@ class TestFiscalDocumentGeneric(TransactionCase):
 
         # Teste definindo os valores Por Linha
         for line in self.nfe_same_state.fiscal_line_ids:
+            line.price_unit = 100
             line.freight_value = 10.0
             line.insurance_value = 10.0
             line.other_value = 10.0
@@ -1070,19 +968,17 @@ class TestFiscalDocumentGeneric(TransactionCase):
         self.assertEqual(
             self.nfe_same_state.amount_freight_value,
             20.0,
-            "Unexpected value for the field" " Amount Freight in Fiscal Document line",
+            "Unexpected value for the field Amount Freight in Fiscal Document line",
         )
         self.assertEqual(
             self.nfe_same_state.amount_insurance_value,
             20.0,
-            "Unexpected value for the field"
-            " Amount Insurance in Fiscal Document line",
+            "Unexpected value for the field Amount Insurance in Fiscal Document line",
         )
         self.assertEqual(
             self.nfe_same_state.amount_other_value,
             20.0,
-            "Unexpected value for the field"
-            " Amount Other Value in Fiscal Document line",
+            "Unexpected value for the field Amount Other Value in Fiscal Document line",
         )
 
         # Teste definindo os valores Por Total
@@ -1098,18 +994,17 @@ class TestFiscalDocumentGeneric(TransactionCase):
             self.assertEqual(
                 line.freight_value,
                 5.0,
-                "Unexpected value for the field" " Freight in Fiscal Document line",
+                "Unexpected value for the field Freight in Fiscal Document line",
             )
             self.assertEqual(
                 line.insurance_value,
                 5.0,
-                "Unexpected value for the field" " Insurance in Fiscal Document line",
+                "Unexpected value for the field Insurance in Fiscal Document line",
             )
             self.assertEqual(
                 line.other_value,
                 5.0,
-                "Unexpected value for the field"
-                " Other Values in Fiscal Document line",
+                "Unexpected value for the field Other Values in Fiscal Document line",
             )
 
         # Caso que os Campos na Linha não tem valor
@@ -1126,16 +1021,15 @@ class TestFiscalDocumentGeneric(TransactionCase):
             self.assertEqual(
                 line.freight_value,
                 10.0,
-                "Unexpected value for the field" " Freight in Fiscal Document line",
+                "Unexpected value for the field Freight in Fiscal Document line",
             )
             self.assertEqual(
                 line.insurance_value,
                 10.0,
-                "Unexpected value for the field" " Insurance in Fiscal Document line",
+                "Unexpected value for the field Insurance in Fiscal Document line",
             )
             self.assertEqual(
                 line.other_value,
                 10.0,
-                "Unexpected value for the field"
-                " Other Values in Fiscal Document line",
+                "Unexpected value for the field Other Values in Fiscal Document line",
             )

odoo/addons/l10n_br_fiscal/tests/test_fiscal_document_nfse.py

@@ -5,25 +5,15 @@ from odoo.tests.common import TransactionCase
 
 
 class TestFiscalDocumentNFSe(TransactionCase):
-    def setUp(self):
-        super().setUp()
-
-        self.nfse_same_state = self.env.ref("l10n_br_fiscal.demo_nfse_same_state")
+    @classmethod
+    def setUpClass(cls):
+        super().setUpClass()
+        cls.env = cls.env(context=dict(cls.env.context, tracking_disable=True))
+        cls.nfse_same_state = cls.env.ref("l10n_br_fiscal.demo_nfse_same_state")
 
     def test_nfse_same_state(self):
         """Test NFSe same state."""
-
-        self.nfse_same_state._onchange_document_serie_id()
-        self.nfse_same_state._onchange_fiscal_operation_id()
-
         for line in self.nfse_same_state.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_commercial_quantity()
-            line._onchange_ncm_id()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_operation_line_id()
-            line._onchange_fiscal_taxes()
-
             self.assertEqual(
                 line.fiscal_operation_line_id.name,
                 "Prestação de Serviço",