[IMP] mrp_bom_attribute_match: increase test strenght: add another component to bom

Author: ivantodorovich

mrp_bom_attribute_match/tests/common.py

@@ -56,6 +56,13 @@ def setUpClass(cls):
                 "route_ids": [Command.link(cls.route_manufacture.id)],
             }
         )
+        cls.p4 = cls.env["product.template"].create(
+            {
+                "name": "P4",
+                "type": "product",
+                "route_ids": [Command.link(cls.route_manufacture.id)],
+            }
+        )
         cls.product_9 = cls.env["product.product"].create(
             {
                 "name": "Paper",
@@ -97,6 +104,10 @@ def setUpClass(cls):
                     component_template_id=cls.product_plastic.id,
                     product_qty=1,
                 ),
+                dict(
+                    product_id=cls.product_9,
+                    product_qty=1,
+                ),
             ],
         )
         cls.fin_bom_id = cls._create_bom(
@@ -139,7 +150,7 @@ def setUpClass(cls):
             cls.p3,
             [
                 dict(
-                    product_id=cls.p1.product_variant_ids[0],
+                    product_id=cls.p4.product_variant_ids[0],
                     product_qty=1,
                 ),
             ],

mrp_bom_attribute_match/tests/test_mrp_bom_attribute_match.py

@@ -67,18 +67,18 @@ def test_bom_2(self):
         plastic_smells_like_orchid.unlink()
 
     def test_manufacturing_order_1(self):
+        sword_cyan = self.product_sword.product_variant_ids[0]
+        plastic_cyan = self.product_plastic.product_variant_ids[0]
         mo_form = Form(self.env["mrp.production"])
-        mo_form.product_id = self.product_sword.product_variant_ids.filtered(
-            lambda x: x.name == "Plastic Sword"
-        )[0]
+        mo_form.product_id = sword_cyan
         mo_form.bom_id = self.bom_id
         mo_form.product_qty = 1
         self.mo_sword = mo_form.save()
         self.mo_sword.action_confirm()
         # Assert correct component variant was selected automatically
         self.assertEqual(
-            self.mo_sword.move_raw_ids.product_id.display_name,
-            "Plastic Component (Cyan)",
+            self.mo_sword.move_raw_ids.product_id,
+            plastic_cyan + self.product_9,
         )
 
     def test_manufacturing_order_2(self):
@@ -181,7 +181,11 @@ def test_mrp_report_bom_structure(self):
         self.assertEqual(res["lines"]["product"], sword_cyan)
         self.assertEqual(
             res["lines"]["components"][0]["line_id"],
-            self.bom_id.bom_line_ids.id,
+            self.bom_id.bom_line_ids[0].id,
+        )
+        self.assertEqual(
+            res["lines"]["components"][1]["line_id"],
+            self.bom_id.bom_line_ids[1].id,
         )
         self.assertEqual(
             res["lines"]["components"][0]["parent_id"],