[IMP] repair_stock_consumption_step: use default location defined on picking type

Author: nicolas-delbovier-acsone

repair_stock_consumption_step/models/repair_order.py

@@ -1,7 +1,7 @@
 # Copyright 2025 ACSONE SA/NV
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 
-from odoo import fields, models
+from odoo import Command, fields, models
 
 
 class RepairOrder(models.Model):
@@ -44,8 +44,7 @@ def action_repair_done(self):
                 {
                     "picking_type_id": rec.warehouse_id.repair_consumption_picking_type_id.id,
                     "origin": rec.name,
-                    "location_id": moves[0].location_id.id,
-                    "location_dest_id": moves[0].location_dest_id.id,
+                    "move_ids": [Command.set(moves.ids)],
                 }
             )
 
@@ -54,7 +53,6 @@ def action_repair_done(self):
             for move in moves:
                 move_id_lots_ids_map[move.id] = move.move_line_ids.mapped("lot_id").ids
 
-            moves.picking_id = picking.id
             moves.move_line_ids.unlink()
             moves._action_confirm()
             moves._action_assign()

repair_stock_consumption_step/tests/test_repair_stock_consumption_step.py

@@ -87,6 +87,14 @@ def test_repair_done_with_consumption_step(self):
         self.assertEqual(self.repair.operations.lot_id, pick.move_line_ids.lot_id)
         self.assertTrue(moves)
         self.assertEqual(pick.move_ids, moves)
+        self.assertEqual(
+            pick.location_id,
+            self.warehouse.repair_consumption_picking_type_id.default_location_src_id,
+        )
+        self.assertEqual(
+            pick.location_dest_id,
+            self.warehouse.repair_consumption_picking_type_id.default_location_dest_id,
+        )
         self.assertSetEqual(set(moves.mapped("state")), {"assigned"})
         self.assertIn(pick.state, "assigned")
         self.assertEqual(pick.move_line_ids, moves.move_line_ids)