odoo-addon-shopfloor 18.0.0.1.2__py3-none-any.whl → 18.0.0.3.0__py3-none-any.whl

odoo/addons/shopfloor/tests/__init__.py

@@ -23,37 +23,38 @@ from . import test_actions_stock
 # from . import test_cluster_picking_stock_issue
 # from . import test_cluster_picking_change_pack_lot
 # from . import test_cluster_picking_unload
-# from . import test_checkout_base
-# from . import test_checkout_scan
-# from . import test_checkout_select
-# from . import test_checkout_scan_line
-# from . import test_checkout_scan_line_no_prefill_qty
-# from . import test_checkout_scan_line_base
-# from . import test_checkout_scan_dest_location
-# from . import test_checkout_select_line
-# from . import test_checkout_select_package_base
-# from . import test_checkout_set_qty
-# from . import test_checkout_scan_package_action
-# from . import test_checkout_scan_package_action_no_prefill_qty
-# from . import test_checkout_new_package
-# from . import test_checkout_no_package
-# from . import test_checkout_auto_post
-# from . import test_checkout_list_delivery_packaging
-# from . import test_checkout_list_package
-# from . import test_checkout_summary
-# from . import test_checkout_change_packaging
-# from . import test_checkout_cancel_line
-# from . import test_checkout_done
-# from . import test_delivery_base
-# from . import test_delivery_done
-# from . import test_delivery_scan_deliver
-# from . import test_delivery_reset_qty_done_line
-# from . import test_delivery_reset_qty_done_pack
-# from . import test_delivery_set_qty_done_pack
-# from . import test_delivery_set_qty_done_line
-# from . import test_delivery_sublocation
-# from . import test_delivery_list_stock_picking
-# from . import test_delivery_select
+from . import test_checkout_base
+from . import test_checkout_scan
+from . import test_checkout_select
+from . import test_checkout_scan_line
+from . import test_checkout_scan_line_no_prefill_qty
+from . import test_checkout_scan_line_base
+from . import test_checkout_scan_dest_location
+from . import test_checkout_select_line
+from . import test_checkout_select_package_base
+from . import test_checkout_set_qty
+from . import test_checkout_scan_package_action
+from . import test_checkout_scan_package_action_no_prefill_qty
+from . import test_checkout_new_package
+from . import test_checkout_no_package
+from . import test_checkout_auto_post
+from . import test_checkout_list_delivery_packaging
+from . import test_checkout_list_package
+from . import test_checkout_summary
+from . import test_checkout_change_packaging
+from . import test_checkout_cancel_line
+from . import test_checkout_done
+from . import test_delivery_base
+from . import test_delivery_done
+from . import test_delivery_scan_deliver
+from . import test_delivery_reset_qty_done_line
+from . import test_delivery_reset_qty_done_pack
+from . import test_delivery_set_qty_done_pack
+from . import test_delivery_set_qty_done_line
+from . import test_delivery_sublocation
+from . import test_delivery_list_stock_picking
+from . import test_delivery_select
+
 # from . import test_location_content_transfer_base
 # from . import test_location_content_transfer_start
 # from . import test_location_content_transfer_get_work
@@ -63,24 +64,25 @@ from . import test_actions_stock
 # from . import test_location_content_transfer_set_destination_package_or_line
 # from . import test_location_content_transfer_putaway
 # from . import test_location_content_transfer_mix
-# from . import test_zone_picking_base
-# from . import test_zone_picking_complete_mix_pack_flux
-# from . import test_zone_picking_start
-# from . import test_zone_picking_select_picking_type
-# from . import test_zone_picking_select_line
-# from . import test_zone_picking_select_line_no_prefill_qty
-# from . import test_zone_picking_select_line_first_scan_location
-# from . import test_zone_picking_set_line_destination
-# from . import test_zone_picking_set_line_destination_no_prefill_qty
-# from . import test_zone_picking_set_line_destination_pick_pack
-# from . import test_zone_picking_zero_check
-# from . import test_zone_picking_stock_issue
+from . import test_zone_picking_base
+from . import test_zone_picking_complete_mix_pack_flux
+from . import test_zone_picking_start
+from . import test_zone_picking_select_picking_type
+from . import test_zone_picking_select_line
+from . import test_zone_picking_select_line_no_prefill_qty
+from . import test_zone_picking_select_line_first_scan_location
+from . import test_zone_picking_set_line_destination
+from . import test_zone_picking_set_line_destination_no_prefill_qty
+from . import test_zone_picking_set_line_destination_pick_pack
+from . import test_zone_picking_zero_check
+from . import test_zone_picking_stock_issue
+
 # from . import test_zone_picking_change_pack_lot
-# from . import test_zone_picking_unload_buffer_lines
-# from . import test_zone_picking_unload_single
-# from . import test_zone_picking_unload_all
-# from . import test_zone_picking_unload_set_destination
-# from . import test_zone_picking_require_destination_package
+from . import test_zone_picking_unload_buffer_lines
+from . import test_zone_picking_unload_single
+from . import test_zone_picking_unload_all
+from . import test_zone_picking_unload_set_destination
+from . import test_zone_picking_require_destination_package
 from . import test_misc
 from . import test_move_action_assign
 from . import test_scan_anything

odoo/addons/shopfloor/tests/common.py

@@ -182,8 +182,10 @@ class CommonCase(BaseCommonCase):
         )
         # this method adds the quantity to the current quantity, so remove it
         quantity -= sum(quants.mapped("quantity"))
+        if not quantity:
+            return
         cls.env["stock.quant"]._update_available_quantity(
-            product, location, quantity, package_id=package, lot_id=lot
+            product, location, quantity=quantity, package_id=package, lot_id=lot
         )
 
     @classmethod

odoo/addons/shopfloor/tests/test_checkout_auto_post.py

@@ -14,11 +14,11 @@ class CheckoutAutoPostCase(CheckoutCommonCase):
         selected_move_line_a = picking.move_line_ids.filtered(
             lambda x: x.product_id == self.product_a
         )
-        selected_move_line_a.qty_done = 7
+        selected_move_line_a.qty_picked = 7
         selected_move_line_b = picking.move_line_ids.filtered(
             lambda x: x.product_id == self.product_b
         )
-        selected_move_line_b.qty_done = 9
+        selected_move_line_b.qty_picked = 9
         selected_move_line_c = picking.move_line_ids.filtered(
             lambda x: x.product_id == self.product_c
         )
@@ -44,10 +44,10 @@ class CheckoutAutoPostCase(CheckoutCommonCase):
         )
         self.assertEqual(picking, selected_move_line_c.picking_id)
 
-        # The lines in the new picking must have the expected qty_done,
+        # The lines in the new picking must have the expected picked qty,
         # and the split picking must be marked as "done".
-        self.assertEqual(selected_move_line_a.qty_done, 7)
-        self.assertEqual(selected_move_line_b.qty_done, 9)
+        self.assertEqual(selected_move_line_a.qty_picked, 7)
+        self.assertEqual(selected_move_line_b.qty_picked, 9)
         self.assertEqual(selected_move_line_a.picking_id.state, "done")
 
         # In the original picking, we should have three lines:
@@ -63,6 +63,7 @@ class CheckoutAutoPostCase(CheckoutCommonCase):
         self.assertEqual(line_b_in_original_picking.quantity, 11)
         self.assertEqual(selected_move_line_c.quantity, 30)
 
-        self.assertEqual(line_a_in_original_picking.qty_done, 0)
-        self.assertEqual(line_b_in_original_picking.qty_done, 0)
-        self.assertEqual(selected_move_line_c.qty_done, 0)
+        self.assertEqual(line_a_in_original_picking.qty_picked, 0)
+        self.assertEqual(line_b_in_original_picking.qty_picked, 0)
+        self.assertEqual(selected_move_line_c.qty_picked, 0)
+        self.assertFalse(selected_move_line_c.picked)

odoo/addons/shopfloor/tests/test_checkout_base.py

@@ -91,6 +91,6 @@ class CheckoutCommonCase(CommonCase):
                 "message_type": "warning",
                 "body": "The quantity scanned for one or more lines cannot be "
                 "higher than the maximum allowed. "
-                f"({line.product_id.name} : {str(line.qty_done)} > {str(line.quantity)})",  # noqa
+                f"({line.product_id.name} : {str(line.qty_picked)} > {str(line.quantity)})",  # noqa
             },
         )

odoo/addons/shopfloor/tests/test_checkout_cancel_line.py

@@ -35,7 +35,7 @@ class CheckoutRemovePackageCase(CheckoutCommonCase):
         new_package = self.env["stock.quant.package"].create({})
         (pack1_lines | raw_line).write(
             {
-                "qty_done": 10,
+                "qty_picked": 10,
                 "result_package_id": new_package.id,
                 "shopfloor_checkout_done": True,
             }
@@ -43,7 +43,7 @@ class CheckoutRemovePackageCase(CheckoutCommonCase):
         new_package2 = self.env["stock.quant.package"].create({})
         pack2_lines.write(
             {
-                "qty_done": 10,
+                "qty_picked": 10,
                 "result_package_id": new_package2.id,
                 "shopfloor_checkout_done": True,
             }
@@ -59,26 +59,30 @@ class CheckoutRemovePackageCase(CheckoutCommonCase):
             pack1_lines + raw_line + pack2_lines,
             [
                 {
-                    "qty_done": 0,
+                    "qty_picked": 0,
+                    "picked": False,
                     # reset to origin package
                     "result_package_id": pack1_lines.mapped("package_id").id,
                     "shopfloor_checkout_done": False,
                 },
                 {
-                    "qty_done": 0,
+                    "qty_picked": 0,
+                    "picked": False,
                     # reset to origin package
                     "result_package_id": pack1_lines.mapped("package_id").id,
                     "shopfloor_checkout_done": False,
                 },
                 {
-                    "qty_done": 0,
+                    "qty_picked": 0,
+                    "picked": False,
                     # result to an empty package (raw product)
                     "result_package_id": False,
                     "shopfloor_checkout_done": False,
                 },
                 # different package, leave untouched
                 {
-                    "qty_done": 10,
+                    "qty_picked": 10,
+                    "picked": True,
                     "result_package_id": new_package2.id,
                     "shopfloor_checkout_done": True,
                 },
@@ -97,7 +101,7 @@ class CheckoutRemovePackageCase(CheckoutCommonCase):
 
         raw_line = self.raw_move.move_line_ids
 
-        raw_line.write({"qty_done": 10, "shopfloor_checkout_done": True})
+        raw_line.write({"qty_picked": 10, "shopfloor_checkout_done": True})
 
         # and now, we want to drop the new_package
         response = self.service.dispatch(
@@ -107,7 +111,7 @@ class CheckoutRemovePackageCase(CheckoutCommonCase):
 
         self.assertRecordValues(
             raw_line,
-            [{"qty_done": 0, "shopfloor_checkout_done": False}],
+            [{"qty_picked": 0, "picked": False, "shopfloor_checkout_done": False}],
         )
 
         self.assert_response(

odoo/addons/shopfloor/tests/test_checkout_done.py

@@ -10,7 +10,7 @@ class CheckoutDoneCase(CheckoutCommonCase):
         self._fill_stock_for_moves(picking.move_ids, in_package=True)
         picking.action_assign()
         # line is done
-        picking.move_line_ids.write({"qty_done": 10, "shopfloor_checkout_done": True})
+        picking.move_line_ids.write({"qty_picked": 10, "shopfloor_checkout_done": True})
         response = self.service.dispatch("done", params={"picking_id": picking.id})
 
         self.assertRecordValues(picking, [{"state": "done"}])
@@ -37,8 +37,8 @@ class CheckoutDonePartialCase(CheckoutCommonCase):
         picking.action_assign()
         cls.line1 = picking.move_line_ids[0]
         cls.line2 = picking.move_line_ids[1]
-        cls.line1.write({"qty_done": 10, "shopfloor_checkout_done": True})
-        cls.line2.write({"qty_done": 2, "shopfloor_checkout_done": True})
+        cls.line1.write({"qty_picked": 10, "shopfloor_checkout_done": True})
+        cls.line2.write({"qty_picked": 2, "shopfloor_checkout_done": True})
 
         cls.dest_location = picking.location_dest_id
         cls.child_location = (
@@ -119,12 +119,12 @@ class CheckoutDoneRawUnpackedCase(CheckoutCommonCase):
         cls.package = cls.env["stock.quant.package"].create({})
         cls.line1.write(
             {
-                "qty_done": 10,
+                "qty_picked": 10,
                 "shopfloor_checkout_done": True,
                 "result_package_id": cls.package.id,
             }
         )
-        cls.line2.write({"qty_done": 10, "shopfloor_checkout_done": False})
+        cls.line2.write({"qty_picked": 10, "shopfloor_checkout_done": False})
 
     def test_done_partial(self):
         # line is done

odoo/addons/shopfloor/tests/test_checkout_list_delivery_packaging.py

@@ -105,8 +105,6 @@ class CheckoutListDeliveryPackagingCase(CheckoutCommonCase, CheckoutSelectPackag
         self._fill_stock_for_moves(self.picking.move_ids, in_package=True)
         self.picking.action_assign()
         selected_lines = self.picking.move_line_ids
-        # for line in selected_lines:
-        #     line.qty_done = line.quantity
         response = self.service.dispatch(
             "list_delivery_packaging",
             params={

odoo/addons/shopfloor/tests/test_checkout_list_package.py

@@ -137,9 +137,9 @@ class CheckoutScanSetDestPackageCase(CheckoutCommonCase, SelectDestPackageMixin)
         # them
         cls.move_line1.result_package_id = cls.delivery_package
         # We'll put only product A and B in the destination package
-        cls.move_line1.qty_done = cls.move_line1.quantity
-        cls.move_line2.qty_done = cls.move_line2.quantity
-        cls.move_line3.qty_done = 0
+        cls.move_line1.qty_picked = cls.move_line1.quantity
+        cls.move_line2.qty_picked = cls.move_line2.quantity
+        cls.move_line3.qty_picked = 0
 
         cls.picking = picking
 
@@ -161,7 +161,7 @@ class CheckoutScanSetDestPackageCase(CheckoutCommonCase, SelectDestPackageMixin)
                     "result_package_id": self.delivery_package.id,
                     "shopfloor_checkout_done": True,
                 },
-                # qty_done was zero so we don't set it as packed
+                # picked qty was zero so we don't set it as packed
                 {"result_package_id": self.pack1.id, "shopfloor_checkout_done": False},
             ],
         )
@@ -232,9 +232,9 @@ class CheckoutScanSetDestPackageCase(CheckoutCommonCase, SelectDestPackageMixin)
         )
         self._assert_package_set(response)
 
-    def test_set_dest_package_ok_on_partial_qty_done(self):
+    def test_set_dest_package_ok_on_partial_qty_picked(self):
         # Partially process line three 3 quantiy out of 10
-        self.move_line3.qty_done = 3
+        self.move_line3.qty_picked = 3
         response = self.service.dispatch(
             "set_dest_package",
             params={
@@ -250,23 +250,30 @@ class CheckoutScanSetDestPackageCase(CheckoutCommonCase, SelectDestPackageMixin)
                 {
                     "result_package_id": self.delivery_package.id,
                     "shopfloor_checkout_done": True,
+                    "quantity": 10,
+                    "qty_picked": 10,
+                    "picked": True,
                 },
                 {
                     "result_package_id": self.delivery_package.id,
                     "shopfloor_checkout_done": True,
+                    "quantity": 10,
+                    "qty_picked": 10,
+                    "picked": True,
                 },
                 # Line 3 has been split
                 {
                     "result_package_id": self.delivery_package.id,
                     "shopfloor_checkout_done": True,
-                    "product_uom_qty": 3,
-                    "qty_done": 3,
+                    "quantity": 3,
+                    "qty_picked": 3,
+                    "picked": True,
                 },
             ],
         )
         # Left quantity to do from line 3
         new_move_line = self.picking.move_line_ids.filtered(
-            lambda line: line.qty_done == 0 and line.quantity == 7
+            lambda line: not line.picked and line.quantity == 7
         )
         self.assertTrue(new_move_line)
         self.assertFalse(new_move_line.shopfloor_checkout_done)
@@ -313,12 +320,12 @@ class CheckoutScanSetDestPackageCase(CheckoutCommonCase, SelectDestPackageMixin)
             message=self.service.msg_store.dest_package_not_valid(package),
         )
 
-    def test_set_dest_package_error_qty_done_above(self):
-        # If the qty_done of a selected line goes beyond
+    def test_set_dest_package_error_qty_picked_above(self):
+        # If the picked qty of a selected line goes beyond
         # the maximum allowed, a message should be displayed
         # and the user shouldn't be allowed to select a package.
         line = fields.first(self.picking.move_line_ids)
-        line.qty_done = line.quantity + 1
+        line.qty_picked = line.quantity + 1
         response = self.service.dispatch(
             "list_dest_package",
             params={

odoo/addons/shopfloor/tests/test_checkout_new_package.py

@@ -28,9 +28,9 @@ class CheckoutNewPackageCase(CheckoutCommonCase, CheckoutSelectPackageMixin):
 
         move_line1, move_line2, move_line3 = selected_lines
         # we'll put only the first 2 lines (product A and B) in the new package
-        move_line1.qty_done = move_line1.quantity
-        move_line2.qty_done = move_line2.quantity
-        move_line3.qty_done = 0
+        move_line1.qty_picked = move_line1.quantity
+        move_line2.qty_picked = move_line2.quantity
+        move_line3.qty_picked = 0
 
         response = self.service.dispatch(
             "new_package",
@@ -50,7 +50,7 @@ class CheckoutNewPackageCase(CheckoutCommonCase, CheckoutSelectPackageMixin):
         )
         self.assertRecordValues(
             move_line3,
-            # qty_done was zero so we don't set it as packed and it remains in
+            # picked qty was zero so we don't set it as packed and it remains in
             # the same package
             [{"result_package_id": pack1.id, "shopfloor_checkout_done": False}],
         )
@@ -62,7 +62,7 @@ class CheckoutNewPackageCase(CheckoutCommonCase, CheckoutSelectPackageMixin):
             message=self.msg_store.goods_packed_in(new_package),
         )
 
-    def test_set_dest_package_error_qty_done_above(self):
+    def test_set_dest_package_error_qty_picked_above(self):
         picking = self._create_picking(
             lines=[
                 (self.product_a, 10),
@@ -72,12 +72,12 @@ class CheckoutNewPackageCase(CheckoutCommonCase, CheckoutSelectPackageMixin):
         moves = picking.move_ids
         self._fill_stock_for_moves(moves, in_package=True)
         picking.action_assign()
-        # If the qty_done of a selected line goes beyond
+        # If the picked qty of a selected line goes beyond
         # the maximum allowed, a message should be displayed
         # and the user shouldn't be allowed to select a package.
         selected_lines = moves.move_line_ids
         line = fields.first(selected_lines)
-        line.qty_done = line.quantity + 1
+        line.qty_picked = line.quantity + 1
         response = self.service.dispatch(
             "list_dest_package",
             params={

odoo/addons/shopfloor/tests/test_checkout_no_package.py

@@ -33,9 +33,9 @@ class CheckoutNoPackageCase(CheckoutCommonCase, CheckoutSelectPackageMixin):
         selected_lines = move_line1 + move_line2
 
         # we'll put only the first 2 lines (product A and B) w/ no package
-        move_line1.qty_done = move_line1.quantity
-        move_line2.qty_done = move_line2.quantity
-        move_line3.qty_done = 0
+        move_line1.qty_picked = move_line1.quantity
+        move_line2.qty_picked = move_line2.quantity
+        move_line3.qty_picked = 0
         response = self.service.dispatch(
             "no_package",
             params={
@@ -79,12 +79,12 @@ class CheckoutNoPackageCase(CheckoutCommonCase, CheckoutSelectPackageMixin):
             )
             self.assertEqual(repr(err), "`checkout.no_package` endpoint is not enabled")
 
-    def test_set_dest_package_error_qty_done_above(self):
-        # If the qty_done of a selected line goes beyond
+    def test_set_dest_package_error_qty_picked_above(self):
+        # If the picked qty of a selected line goes beyond
         # the maximum allowed, a message should be displayed
         # and the user shouldn't be allowed to select a package.
         line = fields.first(self.picking.move_line_ids)
-        line.qty_done = line.quantity + 1
+        line.qty_picked = line.quantity + 1
         response = self.service.dispatch(
             "list_dest_package",
             params={

odoo/addons/shopfloor/tests/test_checkout_scan.py

@@ -189,7 +189,7 @@ class CheckoutScanCase(CheckoutCommonCase):
             "select_line",
             params={"picking_id": picking.id, "package_id": package.id},
         )
-        self.assertTrue(all(m.qty_done for m in picking.move_line_ids))
+        self.assertTrue(all(m.qty_picked for m in picking.move_line_ids))
         self.assertEqual(picking.move_line_ids.shopfloor_user_id, self.env.user)
         # He restarts the scenario and try to select again the previous line
         # to continue its job

odoo/addons/shopfloor/tests/test_checkout_scan_dest_location.py

@@ -17,8 +17,8 @@ class CheckoutSelectChildLocationCase(CheckoutCommonCase):
         picking.action_assign()
         cls.line1 = picking.move_line_ids[0]
         cls.line2 = picking.move_line_ids[1]
-        cls.line1.write({"qty_done": 10, "shopfloor_checkout_done": True})
-        cls.line2.write({"qty_done": 2, "shopfloor_checkout_done": True})
+        cls.line1.write({"qty_picked": 10, "shopfloor_checkout_done": True})
+        cls.line2.write({"qty_picked": 2, "shopfloor_checkout_done": True})
 
         cls.dest_location = picking.location_dest_id
         cls.child_location = (

odoo/addons/shopfloor/tests/test_checkout_scan_line.py

@@ -382,7 +382,9 @@ class CheckoutScanLineCase(CheckoutScanLineCaseBase):
         self._fill_stock_for_moves(picking.move_ids, in_package=True)
         picking.action_assign()
         # set all lines as done
-        picking.move_line_ids.write({"qty_done": 10.0, "shopfloor_checkout_done": True})
+        picking.move_line_ids.write(
+            {"qty_picked": 10.0, "shopfloor_checkout_done": True}
+        )
         response = self.service.dispatch(
             "scan_line",
             params={

odoo/addons/shopfloor/tests/test_checkout_scan_package_action.py

@@ -21,7 +21,7 @@ class CheckoutScanPackageActionCase(CheckoutCommonCase, CheckoutSelectPackageMix
 
         # we have selected the pack that contains product a
         line_a = picking.move_line_ids[0]
-        line_a.qty_done = origin_qty_func(line_a)
+        line_a.qty_picked = origin_qty_func(line_a)
 
         response = self.service.dispatch(
             "scan_package_action",
@@ -33,7 +33,7 @@ class CheckoutScanPackageActionCase(CheckoutCommonCase, CheckoutSelectPackageMix
         )
 
         # since we scanned the barcode of the product and we had a
-        # qty_done, the qty_done should flip to 0
+        # picked qty, the picked qty should flip to 0
         self._assert_selected_qties(
             response, line_a, {line_a: expected_qty_func(line_a)}
         )
@@ -86,7 +86,7 @@ class CheckoutScanPackageActionCase(CheckoutCommonCase, CheckoutSelectPackageMix
         self._fill_stock_for_moves(picking.move_ids, in_package=True)
         picking.action_assign()
         move_line = picking.move_line_ids
-        origin_qty_done = move_line.qty_done
+        origin_qty_picked = move_line.qty_picked
         response = self.service.dispatch(
             "scan_package_action",
             params={
@@ -99,7 +99,7 @@ class CheckoutScanPackageActionCase(CheckoutCommonCase, CheckoutSelectPackageMix
             response,
             move_line,
             # no change as the scan was not valid
-            {move_line: origin_qty_done},
+            {move_line: origin_qty_picked},
             message={
                 "message_type": "warning",
                 "body": "Product tracked by lot, please scan one.",
@@ -135,9 +135,9 @@ class CheckoutScanPackageActionCase(CheckoutCommonCase, CheckoutSelectPackageMix
 
         move_line1, move_line2, move_line3 = selected_lines
         # We'll put only product A and B in the package
-        move_line1.qty_done = move_line1.quantity
-        move_line2.qty_done = move_line2.quantity
-        move_line3.qty_done = 0
+        move_line1.qty_picked = move_line1.quantity
+        move_line2.qty_picked = move_line2.quantity
+        move_line3.qty_picked = 0
 
         response = self.service.dispatch(
             "scan_package_action",
@@ -161,7 +161,7 @@ class CheckoutScanPackageActionCase(CheckoutCommonCase, CheckoutSelectPackageMix
         )
         self.assertRecordValues(
             move_line3,
-            # qty_done was zero so it hasn't been done anyway
+            # picked qty was zero so it hasn't been done anyway
             [{"result_package_id": pack1.id, "shopfloor_checkout_done": False}],
         )
         self.assert_response(
@@ -243,7 +243,7 @@ class CheckoutScanPackageActionCase(CheckoutCommonCase, CheckoutSelectPackageMix
 
         selected_lines = pack1_moves.move_line_ids
         # they are all selected
-        selected_lines.write({"qty_done": 10.0})
+        selected_lines.write({"qty_picked": 10.0})
 
         response = self.service.dispatch(
             "scan_package_action",
@@ -296,9 +296,9 @@ class CheckoutScanPackageActionCase(CheckoutCommonCase, CheckoutSelectPackageMix
 
         move_line1, move_line2, move_line3 = selected_lines
         # we'll put only the first 2 lines (product A and B) in the new package
-        move_line1.qty_done = move_line1.quantity
-        move_line2.qty_done = move_line2.quantity
-        move_line3.qty_done = 0
+        move_line1.qty_picked = move_line1.quantity
+        move_line2.qty_picked = move_line2.quantity
+        move_line3.qty_picked = 0
 
         packaging = (
             self.env["stock.package.type"]
@@ -349,7 +349,7 @@ class CheckoutScanPackageActionCase(CheckoutCommonCase, CheckoutSelectPackageMix
         )
         self.assertRecordValues(
             move_line3,
-            # qty_done was zero so we don't set it as packed and it remains in
+            # picked qty was zero so we don't set it as packed and it remains in
             # the same package
             [{"result_package_id": pack1.id, "shopfloor_checkout_done": False}],
         )
@@ -368,7 +368,7 @@ class CheckoutScanPackageActionCase(CheckoutCommonCase, CheckoutSelectPackageMix
         self._fill_stock_for_moves(pack1_moves, in_package=True)
         picking.action_assign()
         selected_lines = pack1_moves.move_line_ids
-        selected_lines.qty_done = selected_lines.quantity
+        selected_lines.qty_picked = selected_lines.quantity
 
         packaging = (
             self.env["stock.package.type"]

odoo/addons/shopfloor/tests/test_checkout_scan_package_action_no_prefill_qty.py

@@ -21,7 +21,7 @@ class CheckoutScanPackageActionCaseNoPrefillQty(
         self._fill_stock_for_moves(picking.move_ids, in_package=False)
         picking.action_assign()
         move_line = picking.move_line_ids
-        origin_qty_done = move_line.qty_done = 2
+        origin_qty_picked = move_line.qty_picked = 2
         response = self.service.dispatch(
             "scan_package_action",
             params={
@@ -33,7 +33,7 @@ class CheckoutScanPackageActionCaseNoPrefillQty(
         self._assert_selected_qties(
             response,
             move_line,
-            {move_line: origin_qty_done + 1},
+            {move_line: origin_qty_picked + 1},
         )
 
     def test_scan_package_action_scan_product2_to_increment_qty(self):
@@ -68,7 +68,7 @@ class CheckoutScanPackageActionCaseNoPrefillQty(
 
         # First line is fully processed,
         # so we expect the second line to be incremented.
-        move_lines[0].qty_done = 3.0
+        move_lines[0].qty_picked = 3.0
         self.service.dispatch(
             "scan_package_action",
             params={
@@ -77,8 +77,8 @@ class CheckoutScanPackageActionCaseNoPrefillQty(
                 "barcode": self.product_a.barcode,
             },
         )
-        self.assertEqual(move_lines[0].qty_done, 3.0)
-        self.assertEqual(move_lines[1].qty_done, 1.0)
+        self.assertEqual(move_lines[0].qty_picked, 3.0)
+        self.assertEqual(move_lines[1].qty_picked, 1.0)
 
     def test_scan_package_action_scan_lot_to_increment_qty(self):
         """ """
@@ -86,7 +86,7 @@ class CheckoutScanPackageActionCaseNoPrefillQty(
         self._fill_stock_for_moves(picking.move_ids, in_lot=True)
         picking.action_assign()
         move_line = picking.move_line_ids
-        origin_qty_done = move_line.qty_done = 2
+        origin_qty_picked = move_line.qty_picked = 2
         response = self.service.dispatch(
             "scan_package_action",
             params={
@@ -98,7 +98,7 @@ class CheckoutScanPackageActionCaseNoPrefillQty(
         self._assert_selected_qties(
             response,
             move_line,
-            {move_line: origin_qty_done + 1},
+            {move_line: origin_qty_picked + 1},
         )
 
     def test_scan_package_action_scan_packaging_to_increment_qty(self):
@@ -107,7 +107,7 @@ class CheckoutScanPackageActionCaseNoPrefillQty(
         self._fill_stock_for_moves(picking.move_ids, in_package=True, in_lot=False)
         picking.action_assign()
         move_line = picking.move_line_ids
-        origin_qty_done = move_line.qty_done = 0
+        origin_qty_picked = move_line.qty_picked = 0
         response = self.service.dispatch(
             "scan_package_action",
             params={
@@ -119,5 +119,5 @@ class CheckoutScanPackageActionCaseNoPrefillQty(
         self._assert_selected_qties(
             response,
             move_line,
-            {move_line: origin_qty_done + self.product_a_packaging.qty},
+            {move_line: origin_qty_picked + self.product_a_packaging.qty},
         )