[FIX] account_invoice_line_description: unbalanced invoice error in test

coopdevs · Jun 13, 2022 · 50a37f7 · 50a37f7
1 parent 3f3e016
commit 50a37f7
Showing 1 changed file with 28 additions and 38 deletions.
diff --git a/account_invoice_line_description/tests/test_account_invoice_line_description.py b/account_invoice_line_description/tests/test_account_invoice_line_description.py
@@ -1,6 +1,7 @@
 # Copyright 2017 - Tecnativa, S.L. - Luis M. Ontalba
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl)
 
+from odoo.tests import Form
 from odoo.tests.common import SavepointCase, tagged
 
 
@@ -41,26 +42,7 @@ def setUpClass(cls):
                 "user_type_id": cls.account_type_regular.id,
             }
         )
-        cls.invoice_sale_vals = [
-            (
-                0,
-                0,
-                {
-                    "product_id": cls.product_sale.id,
-                    "name": "Test Invoice Line",
-                    "account_id": cls.account.id,
-                    "price_unit": 500.00,
-                },
-            )
-        ]
-        cls.invoice_sale = cls.env["account.move"].create(
-            {
-                "partner_id": cls.partner.id,
-                "journal_id": cls.journal_sale.id,
-                "move_type": "out_invoice",
-                "invoice_line_ids": cls.invoice_sale_vals,
-            }
-        )
+
         cls.journal_purchase = cls.env["account.journal"].create(
             {"name": "Test Purchase Journal", "code": "TPJ", "type": "purchase"}
         )
@@ -74,26 +56,34 @@ def setUpClass(cls):
                 "lst_price": 0,
             }
         )
-        cls.invoice_purchase_vals = [
-            (
-                0,
-                0,
-                {
-                    "product_id": cls.product_purchase.id,
-                    "name": "Test Invoice Line",
-                    "account_id": cls.account.id,
-                    "price_unit": 500.00,
-                },
+        invoice_sale = Form(
+            cls.env["account.move"].with_context(
+                default_move_type="out_invoice",
+            )
+        )
+        invoice_sale.partner_id = cls.partner
+        invoice_sale.journal_id = cls.journal_sale
+        with invoice_sale.invoice_line_ids.new() as line_form:
+            line_form.name = "Test Invoice Line"
+            line_form.price_unit = 500.0
+            line_form.quantity = 1
+            line_form.product_id = cls.product_sale
+            line_form.account_id = cls.account
+        cls.invoice_sale = invoice_sale.save()
+
+        invoice_purchase = Form(
+            cls.env["account.move"].with_context(
+                default_move_type="in_invoice",
             )
-        ]
-        cls.invoice_purchase = cls.env["account.move"].create(
-            {
-                "partner_id": cls.partner.id,
-                "journal_id": cls.journal_purchase.id,
-                "move_type": "in_invoice",
-                "invoice_line_ids": cls.invoice_purchase_vals,
-            }
         )
+        invoice_purchase.partner_id = cls.partner
+        invoice_purchase.journal_id = cls.journal_purchase
+        with invoice_purchase.invoice_line_ids.new() as line_form:
+            line_form.name = "Test Invoice Line"
+            line_form.price_unit = 500.0
+            line_form.quantity = 1
+            line_form.product_id = cls.product_purchase
+        cls.invoice_purchase = invoice_purchase.save()
 
     def test_onchange_product_id_sale(self):
         self.invoice_sale.invoice_line_ids._onchange_product_id()