[FIX] stock_move_line_devaluation: fix return wizard quantity in tests

Author: marcos-mendez

stock_move_line_devaluation/tests/test_stock_move_line_devaluation.py

@@ -8,7 +8,7 @@ def setUpClass(cls):
         super().setUpClass()
 
         cls.partner = cls.env["res.partner"].create(
-            {"name": "Test Devaluation Customer", "customer_rank": 1}
+            {"name": "Test Devaluation Customer"}
         )
         cls.pricelist = cls.env["product.pricelist"].create(
             {"name": "Test Devaluation Pricelist"}
@@ -26,10 +26,8 @@ def setUpClass(cls):
         cls.stock_location = cls.env.ref("stock.stock_location_stock")
         cls.customer_location = cls.env.ref("stock.stock_location_customers")
         cls.picking_type_out = cls.env.ref("stock.picking_type_out")
-        cls.picking_type_in = cls.env.ref("stock.picking_type_in")
         cls.uom_unit = cls.env.ref("uom.product_uom_unit")
 
-        # Ensure stock availability
         cls.env["stock.quant"].with_context(inventory_mode=True).create(
             {
                 "product_id": cls.product.id,
@@ -67,19 +65,36 @@ def _create_outgoing_picking(self, qty=5.0):
         return picking
 
     def _create_return(self, picking, qty):
-        """Create a return for a picking using the stock return wizard."""
-        return_wizard = (
-            self.env["stock.return.picking"]
-            .with_context(active_id=picking.id, active_model="stock.picking")
-            .create({})
+        """Create a return for a picking manually linking returned moves."""
+        return_type = picking.picking_type_id.return_picking_type_id or (
+            picking.picking_type_id
         )
-        for line in return_wizard.product_return_moves:
-            if line.product_id == self.product:
-                line.quantity = qty
-        result = return_wizard.create_returns()
-        return_picking = self.env["stock.picking"].browse(result["res_id"])
+        return_picking = self.env["stock.picking"].create(
+            {
+                "partner_id": picking.partner_id.id,
+                "picking_type_id": return_type.id,
+                "location_id": picking.location_dest_id.id,
+                "location_dest_id": picking.location_id.id,
+                "origin": "Return of %s" % picking.name,
+            }
+        )
+        original_move = picking.move_ids[0]
+        self.env["stock.move"].create(
+            {
+                "name": "Return of %s" % original_move.name,
+                "product_id": self.product.id,
+                "product_uom_qty": qty,
+                "product_uom": self.uom_unit.id,
+                "picking_id": return_picking.id,
+                "location_id": picking.location_dest_id.id,
+                "location_dest_id": picking.location_id.id,
+                "origin_returned_move_id": original_move.id,
+            }
+        )
+        return_picking.action_confirm()
+        return_picking.action_assign()
         for ml in return_picking.move_line_ids:
-            ml.qty_done = qty
+            ml.qty_done = ml.reserved_uom_qty or qty
         return_picking._action_done()
         return return_picking