odoo-addon-l10n-br-fiscal 18.0.2.0.0.10__py3-none-any.whl → 18.0.7.1.0__py3-none-any.whl

odoo/addons/l10n_br_fiscal/models/tax_classification.py

@@ -0,0 +1,81 @@
+# Copyright 2025 Marcel Savegnago <https://escodoo.com.br>
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+
+from odoo import api, fields, models
+
+from ..constants.fiscal import (
+    TAX_DOMAIN_CBS,
+    TAX_DOMAIN_IBS,
+    TAX_RATE_TYPE,
+    TAX_RATE_TYPE_DEFAULT,
+)
+
+
+class TaxClassification(models.Model):
+    _name = "l10n_br_fiscal.tax.classification"
+    _inherit = "l10n_br_fiscal.data.abstract"
+    _order = "code"
+    _description = "Tax Classification"
+
+    code = fields.Char(size=8)
+
+    description = fields.Text()
+
+    cst_code_prefix_like = fields.Char(
+        compute="_compute_cst_code_prefix_like",
+        help="Helper field to filter taxes by CST code prefix (3 chars) using LIKE.",
+    )
+
+    @api.depends("code")
+    def _compute_cst_code_prefix_like(self):
+        for rec in self:
+            prefix = (rec.code or "")[:3]
+            # Avoid matching all records when the prefix is not available yet.
+            rec.cst_code_prefix_like = (
+                f"{prefix}%" if len(prefix) == 3 else "__no_match__%"
+            )
+
+    tax_ibs_id = fields.Many2one(
+        comodel_name="l10n_br_fiscal.tax",
+        string="Tax IBS",
+        domain=(
+            f"[('tax_domain', '=', '{TAX_DOMAIN_IBS}'), '|', "
+            "('cst_in_id.code', 'like', cst_code_prefix_like), "
+            "('cst_out_id.code', 'like', cst_code_prefix_like)]"
+        ),
+    )
+
+    tax_cbs_id = fields.Many2one(
+        comodel_name="l10n_br_fiscal.tax",
+        string="Tax CBS",
+        domain=(
+            f"[('tax_domain', '=', '{TAX_DOMAIN_CBS}'), '|', "
+            "('cst_in_id.code', 'like', cst_code_prefix_like), "
+            "('cst_out_id.code', 'like', cst_code_prefix_like)]"
+        ),
+    )
+
+    regular_taxation = fields.Boolean(
+        default=False,
+    )
+
+    presumed_credit = fields.Boolean(
+        default=False,
+    )
+
+    credit_reversal = fields.Boolean(
+        default=False,
+    )
+
+    rate_type = fields.Selection(
+        selection=TAX_RATE_TYPE,
+        default=TAX_RATE_TYPE_DEFAULT,
+        required=True,
+    )
+
+    document_type_ids = fields.Many2many(
+        comodel_name="l10n_br_fiscal.document.type",
+        relation="tax_classification_document_type_rel",
+        string="Related DFes",
+        help="Related Digital Fiscal Documents",
+    )

odoo/addons/l10n_br_fiscal/security/fiscal_security.xml

@@ -38,44 +38,34 @@
         <field name="name">Fiscal Tax Estimate multi-company</field>
         <field name="model_id" ref="model_l10n_br_fiscal_tax_estimate" />
         <field eval="True" name="global" />
-        <field
-            name="domain_force"
-        >['|',('company_id','=',False),('company_id','in',company_ids)]</field>
+        <field name="domain_force">[('company_id', 'in', company_ids + [False])]</field>
     </record>
 
-    <record id="l10n_br_fiscal_operation_line_rule" model="ir.rule">
+    <record id="l10n_br_fiscal_operation_rule" model="ir.rule">
         <field name="name">Fiscal Operation multi-company</field>
         <field name="model_id" ref="model_l10n_br_fiscal_operation" />
         <field eval="True" name="global" />
-        <field
-            name="domain_force"
-        >['|',('company_id','=',False),('company_id','in',company_ids)]</field>
+        <field name="domain_force">[('company_id', 'in', company_ids + [False])]</field>
     </record>
 
     <record id="l10n_br_fiscal_document_serie_rule" model="ir.rule">
         <field name="name">Fiscal Document Serie multi-company</field>
         <field name="model_id" ref="model_l10n_br_fiscal_document_serie" />
         <field eval="True" name="global" />
-        <field
-            name="domain_force"
-        >['|',('company_id','=',False),('company_id','in',company_ids)]</field>
+        <field name="domain_force">[('company_id', 'in', company_ids + [False])]</field>
     </record>
 
     <record id="l10n_br_fiscal_document_rule" model="ir.rule">
         <field name="name">Fiscal Document multi-company</field>
         <field name="model_id" ref="model_l10n_br_fiscal_document" />
         <field eval="True" name="global" />
-        <field
-            name="domain_force"
-        >['|',('company_id','=',False),('company_id','in',company_ids)]</field>
+        <field name="domain_force">[('company_id', 'in', company_ids + [False])]</field>
     </record>
 
     <record id="l10n_br_fiscal_document_line_rule" model="ir.rule">
         <field name="name">Fiscal Document line multi-company</field>
         <field name="model_id" ref="model_l10n_br_fiscal_document_line" />
         <field eval="True" name="global" />
-        <field
-            name="domain_force"
-        >['|',('company_id','=',False),('company_id','in',company_ids)]</field>
+        <field name="domain_force">[('company_id', 'in', company_ids + [False])]</field>
     </record>
 </odoo>

odoo/addons/l10n_br_fiscal/security/ir.model.access.csv

@@ -38,6 +38,9 @@
 "l10n_br_fiscal_cest_user","Fiscal CEST for User","model_l10n_br_fiscal_cest","l10n_br_fiscal.group_user",1,0,0,0
 "l10n_br_fiscal_cest_manager","Fiscal CEST for Manager","model_l10n_br_fiscal_cest","l10n_br_fiscal.group_manager",1,0,0,0
 "l10n_br_fiscal_cest_maintenance","Fiscal CEST for Maintenance","model_l10n_br_fiscal_cest","l10n_br_fiscal.group_data_maintenance",1,1,1,1
+"l10n_br_fiscal_tax_classification_user","Fiscal Tax Classification for User","model_l10n_br_fiscal_tax_classification","l10n_br_fiscal.group_user",1,0,0,0
+"l10n_br_fiscal_tax_classification_manager","Fiscal Tax Classification for Manager","model_l10n_br_fiscal_tax_classification","l10n_br_fiscal.group_manager",1,0,0,0
+"l10n_br_fiscal_tax_classification_maintenance","Fiscal Tax Classification for Maintenance","model_l10n_br_fiscal_tax_classification","l10n_br_fiscal.group_data_maintenance",1,1,1,1
 "l10n_br_fiscal_product_genre_user","Fiscal Fiscal Product Genre for User","model_l10n_br_fiscal_product_genre","l10n_br_fiscal.group_user",1,0,0,0
 "l10n_br_fiscal_product_genre_manager","Fiscal Fiscal Product Genre for Manager","model_l10n_br_fiscal_product_genre","l10n_br_fiscal.group_manager",1,0,0,0
 "l10n_br_fiscal_product_genre_maintenance","Fiscal Fiscal Product Genre for Maintenance","model_l10n_br_fiscal_product_genre","l10n_br_fiscal.group_data_maintenance",1,1,1,1
@@ -67,6 +70,9 @@
 "l10n_br_fiscal_operation_line_manager","Fiscal Operation Line for Manager","model_l10n_br_fiscal_operation_line","l10n_br_fiscal.group_manager",1,1,1,1
 "l10n_br_fiscal_operation_document_type_user","Fiscal Operation Document Type for User","model_l10n_br_fiscal_operation_document_type","l10n_br_fiscal.group_user",1,0,0,0
 "l10n_br_fiscal_operation_document_type_manager","Fiscal Operation Document Type for Manager","model_l10n_br_fiscal_operation_document_type","l10n_br_fiscal.group_manager",1,1,1,1
+"l10n_br_fiscal_operation_indicator_user","Operation Indicator for User","model_l10n_br_fiscal_operation_indicator","l10n_br_fiscal.group_user",1,0,0,0
+"l10n_br_fiscal_operation_indicator_manager","Operation Indicator for Manager","model_l10n_br_fiscal_operation_indicator","l10n_br_fiscal.group_manager",1,1,1,1
+"l10n_br_fiscal_operation_indicator_maintenance","Operation Indicator for Maintenance","model_l10n_br_fiscal_operation_indicator","l10n_br_fiscal.group_data_maintenance",1,1,1,1
 "l10n_br_fiscal_partner_profile_user","Fiscal Partner Profile for User","model_l10n_br_fiscal_partner_profile","base.group_user",1,0,0,0
 "l10n_br_fiscal_partner_profile_manager","Fiscal Partner Profile for Manager","model_l10n_br_fiscal_partner_profile","l10n_br_fiscal.group_manager",1,1,1,1
 "l10n_br_fiscal_document_user","Fiscal Document for User","model_l10n_br_fiscal_document","l10n_br_fiscal.group_user",1,1,1,0
@@ -94,6 +100,5 @@
 "l10n_br_fiscal_invalidate_number_manager","manager_l10n_br_fiscal_invalidate_number","model_l10n_br_fiscal_invalidate_number","l10n_br_fiscal.group_manager",1,1,1,1
 "l10n_br_fiscal_city_taxation_code_user","Fiscal City Taxation Code for User","model_l10n_br_fiscal_city_taxation_code","l10n_br_fiscal.group_user",1,1,1,0
 "l10n_br_fiscal_city_taxation_code_manager","Fiscal City Taxation Code for Manager","model_l10n_br_fiscal_city_taxation_code","l10n_br_fiscal.group_user",1,1,1,1
-"l10n_br_fiscal_base_wizard_mixin_user",l10n_br_fiscal_base_wizard_mixin,model_l10n_br_fiscal_base_wizard_mixin,base.group_user,1,1,1,1
 "l10n_br_fiscal_document_status_wizard_user",l10n_br_fiscal_document_status_wizard,model_l10n_br_fiscal_document_status_wizard,base.group_user,1,1,1,1
-"l10n_br_fiscal_document_import_wizard_mixin_user",l10n_br_fiscal_document_import_wizard_mixin_user,model_l10n_br_fiscal_document_import_wizard_mixin,base.group_user,1,1,1,1
+"l10n_br_fiscal_document_import_wizard_user",l10n_br_fiscal_document_import_wizard_user,model_l10n_br_fiscal_document_import_wizard,base.group_user,1,1,1,1

odoo/addons/l10n_br_fiscal/static/description/index.html

@@ -372,7 +372,7 @@ ul.auto-toc {
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-!! source digest: sha256:b7471fd6bd126d628ba34ff9f474314e408b56c0cde25924e2312b4a1c047a5c
+!! source digest: sha256:5cd8cb0b01f3ea46d8329418b2478d777a70350607e16eee38900f6a08e7e3f2
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
 <p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Production/Stable" src="https://img.shields.io/badge/maturity-Production%2FStable-green.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/license-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/l10n-brazil/tree/18.0/l10n_br_fiscal"><img alt="OCA/l10n-brazil" src="https://img.shields.io/badge/github-OCA%2Fl10n--brazil-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/l10n-brazil-18-0/l10n-brazil-18-0-l10n_br_fiscal"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/l10n-brazil&amp;target_branch=18.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
 <p><img alt="image" src="https://raw.githubusercontent.com/OCA/l10n-brazil/18.0/l10n_br_fiscal/static/img/fiscal_dashboard.png" /></p>

odoo/addons/l10n_br_fiscal/tests/__init__.py

@@ -2,9 +2,11 @@
 
 from . import (
     test_cnae,
+    test_fiscal_document_serie,
     test_fiscal_document_generic,
     test_fiscal_document_nfse,
     test_fiscal_tax,
+    test_tax_classification,
     test_tax_benefit,
     test_document_edition,
     test_ibpt_product,

odoo/addons/l10n_br_fiscal/tests/test_document_edition.py

@@ -4,9 +4,10 @@
 from unittest import mock
 
 from odoo import Command
-from odoo.tests import Form, TransactionCase
+from odoo.tests import Form, TransactionCase, tagged
 
 
+@tagged("post_install", "-at_install")
 class TestDocumentEdition(TransactionCase):
     @classmethod
     def setUpClass(cls):
@@ -35,6 +36,10 @@ class TestDocumentEdition(TransactionCase):
         cls.env = cls.env(
             user=cls.user, 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(
@@ -98,20 +103,46 @@ class TestDocumentEdition(TransactionCase):
                 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 = False
-            # self.assertEqual(len(line_form.fiscal_tax_ids), 0)
+            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()
-        self.assertEqual(doc.fiscal_line_ids[0].price_unit, 100)
-        self.assertEqual(doc.fiscal_line_ids[0].fiscal_price, 100)
-        self.assertEqual(doc.fiscal_line_ids[0].quantity, 2)
-        self.assertEqual(doc.fiscal_line_ids[0].fiscal_quantity, 2)
-        self.assertEqual(len(doc.fiscal_line_ids[0].fiscal_tax_ids), 4)
+        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(
-            doc.fiscal_line_ids[0].icms_tax_id.id,
-            self.ref("l10n_br_fiscal.tax_icms_12"),
+            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(
@@ -162,3 +193,137 @@ class TestDocumentEdition(TransactionCase):
         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

@@ -40,15 +40,10 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_same_state(self):
         """Test NFe same state."""
         for line in self.nfe_same_state.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-
             # Restore the original price_unit value,
             # as the product change might have altered it.
             line.price_unit = 100
 
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -163,10 +158,6 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_other_state(self):
         """Test NFe other state."""
         for line in self.nfe_other_state.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -278,10 +269,6 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_not_taxpayer(self):
         """Test NFe not taxpayer."""
         for line in self.nfe_not_taxpayer.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -380,10 +367,6 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_not_taxpayer_not_company(self):
         """Test NFe not taxpayer not Company."""
         for line in self.nfe_not_taxpayer_pf.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -482,10 +465,6 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_export(self):
         """Test NFe export."""
         for line in self.nfe_export.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -576,8 +555,6 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_sn_same_state(self):
         """Test NFe Simples Nacional same state."""
         for line in self.nfe_sn_same_state.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-
             # set fake estimate tax
             line.ncm_id.tax_estimate_ids.create(
                 {
@@ -588,9 +565,9 @@ class TestFiscalDocumentGeneric(TransactionCase):
                     "federal_taxes_national": 33.00,
                 }
             )
-
-            line._onchange_fiscal_operation_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(
@@ -689,10 +666,6 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_sn_other_state(self):
         """Test NFe SN other state."""
         for line in self.nfe_sn_other_state.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -787,10 +760,6 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_sn_not_taxpayer(self):
         """Test NFe SN not taxpayer."""
         for line in self.nfe_sn_not_taxpayer.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -872,10 +841,6 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_sn_export(self):
         """Test NFe SN export."""
         for line in self.nfe_sn_export.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_taxes()
-
             if "Revenda" in line.fiscal_operation_line_id.name:
                 self.assertEqual(
                     line.cfop_id.code,
@@ -983,10 +948,16 @@ class TestFiscalDocumentGeneric(TransactionCase):
     def test_nfe_comments(self):
         self.nfe_not_taxpayer._document_comment()
         additional_data = self.nfe_not_taxpayer.fiscal_line_ids[0].additional_data
-        self.assertEqual(
-            additional_data,
-            "manual comment test",
-        )
+        if self.env.ref("base.lang_pt_BR").active:
+            self.assertEqual(
+                additional_data,
+                "manual comment test - Valor Aprox. dos Tributos: R$\xa00,00",
+            )
+        else:  # no pt_BR amount formatting available
+            self.assertEqual(
+                additional_data,
+                "manual comment test - Valor Aprox. dos Tributos: R$\xa00.00",
+            )
 
     def test_fields_freight_insurance_other_costs(self):
         """Test fields Freight, Insurance and Other Costs when

odoo/addons/l10n_br_fiscal/tests/test_fiscal_document_nfse.py

@@ -13,12 +13,7 @@ class TestFiscalDocumentNFSe(TransactionCase):
 
     def test_nfse_same_state(self):
         """Test NFSe same state."""
-
         for line in self.nfse_same_state.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_taxes()
-
             self.assertEqual(
                 line.fiscal_operation_line_id.name,
                 "Prestação de Serviço",

odoo/addons/l10n_br_fiscal/tests/test_fiscal_document_serie.py

@@ -0,0 +1,60 @@
+# Copyright (C) 2025  Renato Lima - Akretion <renato.lima@akretion.com.br>
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
+
+from psycopg2 import IntegrityError
+
+from odoo.exceptions import ValidationError
+from odoo.tests import TransactionCase
+from odoo.tools import mute_logger
+
+
+class TestFiscalDocumentSerie(TransactionCase):
+    @classmethod
+    def setUpClass(cls):
+        super().setUpClass()
+        cls.env = cls.env(context=dict(cls.env.context, tracking_disable=True))
+
+        # Company
+        cls.company_sn = cls.env.ref("l10n_br_base.empresa_simples_nacional")
+
+        # Fiscal Document Type
+        cls.document_type_nfe = cls.env.ref("l10n_br_fiscal.document_55")
+
+        cls.document_serie_nfe_5 = cls.env["l10n_br_fiscal.document.serie"].create(
+            {
+                "code": "5",
+                "name": "Serie 5",
+                "document_type_id": cls.document_type_nfe.id,
+                "company_id": cls.company_sn.id,
+            }
+        )
+
+        # Fiscal Document
+        cls.document = cls.env["l10n_br_fiscal.document"].create(
+            {
+                "company_id": cls.company_sn.id,
+                "document_type_id": cls.document_type_nfe.id,
+                "document_serie_id": cls.document_serie_nfe_5.id,
+                "partner_id": cls.env.ref("l10n_br_base.res_partner_cliente1_sp").id,
+                "state_edoc": "cancelada",
+            }
+        )
+
+    def test_document_serie_duplicated(self):
+        """Test document serie duplicate constraint."""
+        document_serie = self.env["l10n_br_fiscal.document.serie"]
+        document_serie_values = {
+            "code": "10",
+            "name": "Serie 10",
+            "document_type_id": self.document_type_nfe.id,
+            "company_id": self.company_sn.id,
+        }
+
+        with self.assertRaises(IntegrityError), mute_logger("odoo.sql_db"):
+            for _ in range(2):
+                document_serie.create(document_serie_values)
+
+    def test_document_serie_code_in_use(self):
+        """Test document serie code in use constraint."""
+        with self.assertRaises(ValidationError):
+            self.document_serie_nfe_5.write({"code": "7"})

odoo/addons/l10n_br_fiscal/tests/test_tax_benefit.py

@@ -35,15 +35,12 @@ class TestTaxBenefit(TransactionCase):
                 "state": "approved",
             }
         )
+        # force update
+        cls.nfe_tax_benefit.fiscal_line_ids._compute_fiscal_tax_ids()
 
     def test_nfe_tax_benefit(self):
         """Test NFe with tax benefit."""
-
         for line in self.nfe_tax_benefit.fiscal_line_ids:
-            line._onchange_product_id_fiscal()
-            line._onchange_fiscal_operation_id()
-            line._onchange_fiscal_taxes()
-
             self.assertEqual(
                 line.icms_tax_benefit_id,
                 self.tax_benefit,