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

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},
         )

odoo/addons/shopfloor/tests/test_checkout_select_line.py

@@ -109,7 +109,7 @@ class CheckoutSelectLineCase(CheckoutCommonCase, CheckoutSelectPackageMixin):
     def test_select_line_all_lines_done(self):
         # set all lines as done
         self.picking.move_line_ids.write(
-            {"qty_done": 10.0, "shopfloor_checkout_done": True}
+            {"qty_picked": 10.0, "shopfloor_checkout_done": True}
         )
         response = self.service.dispatch(
             "select_line",

odoo/addons/shopfloor/tests/test_checkout_select_package_base.py

@@ -42,9 +42,11 @@ class CheckoutSelectPackageMixin:
             sorted(selected_lines.ids), sorted([line.id for line in lines_quantities])
         )
         for line, quantity in lines_quantities.items():
-            self.assertEqual(line.qty_done, quantity)
+            self.assertEqual(line.qty_picked, quantity)
         for line in deselected_lines:
-            self.assertEqual(line.qty_done, 0, "Lines deselected must have no qty done")
+            self.assertEqual(
+                line.qty_picked, 0, "Lines deselected must have no qty done"
+            )
         self._assert_selected_response(
             response, selected_lines, message=message, packing_info=packing_info
         )
@@ -57,11 +59,11 @@ class CheckoutSelectPackageMixin:
         unselected_lines = picking.move_line_ids - selected_lines
         for line in selected_lines - related_lines:
             self.assertEqual(
-                line.qty_done,
+                line.qty_picked,
                 line.quantity,
                 "Scanned lines must have their qty done set to the reserved quantity",
             )
         for line in unselected_lines + related_lines:
-            self.assertEqual(line.qty_done, 0)
+            self.assertEqual(line.qty_picked, 0)
         package_lines = selected_lines + related_lines
         self._assert_selected_response(response, package_lines, message=message, **kw)

odoo/addons/shopfloor/tests/test_checkout_set_qty.py

@@ -24,15 +24,15 @@ class CheckoutSetQtyCommonCase(CheckoutCommonCase, CheckoutSelectPackageMixin):
     def setUp(self):
         super().setUp()
         # we assume we have called /select_line on pack one, so by default, we
-        # expect the lines for product a and b to have their qty_done set to
+        # expect the lines for product a and b to have their picked qty set to
         # their reserved quantity at the start of the tests
         self.selected_lines = self.moves_pack1.move_line_ids
         self.deselected_lines = self.moves_pack2.move_line_ids
         self.service._select_lines(self.selected_lines)
         self.assertTrue(
-            all(line.qty_done == line.quantity for line in self.selected_lines)
+            all(line.qty_picked == line.quantity for line in self.selected_lines)
         )
-        self.assertTrue(all(line.qty_done == 0 for line in self.deselected_lines))
+        self.assertTrue(all(not line.picked for line in self.deselected_lines))
 
 
 class CheckoutResetLineQtyCase(CheckoutSetQtyCommonCase):
@@ -83,7 +83,7 @@ class CheckoutSetLineQtyCase(CheckoutSetQtyCommonCase):
     def test_set_line_qty_ok(self):
         selected_lines = self.moves_pack1.move_line_ids
         # do as if the user removed the qties of the 2 selected lines
-        selected_lines.qty_done = 0
+        selected_lines.qty_picked = 0
         line_to_set = selected_lines[0]
         line_no_qty = selected_lines[1]
         # we want to check that when we give the package id, we get
@@ -96,8 +96,9 @@ class CheckoutSetLineQtyCase(CheckoutSetQtyCommonCase):
                 "move_line_id": line_to_set.id,
             },
         )
-        self.assertEqual(line_to_set.qty_done, line_to_set.quantity)
-        self.assertEqual(line_no_qty.qty_done, 0)
+        self.assertEqual(line_to_set.qty_picked, line_to_set.quantity)
+        self.assertEqual(line_no_qty.qty_picked, 0)
+        self.assertFalse(line_no_qty.picked)
         self._assert_selected_qties(
             response,
             selected_lines,
@@ -145,8 +146,8 @@ class CheckoutSetCustomQtyCase(CheckoutSetQtyCommonCase):
                 "qty_done": new_qty,
             },
         )
-        self.assertEqual(line_to_change.qty_done, new_qty)
-        self.assertEqual(line_keep_qty.qty_done, line_keep_qty.quantity)
+        self.assertEqual(line_to_change.qty_picked, new_qty)
+        self.assertEqual(line_keep_qty.qty_picked, line_keep_qty.quantity)
         self._assert_selected_qties(
             response,
             selected_lines,
@@ -180,7 +181,7 @@ class CheckoutSetCustomQtyCase(CheckoutSetQtyCommonCase):
         selected_lines = self.moves_pack1.move_line_ids
         line1 = selected_lines[0]
         # modify so we can check that a too high quantity set the max
-        line1.qty_done = 1
+        line1.qty_picked = 1
         line2 = selected_lines[1]
         response = self.service.dispatch(
             "set_custom_qty",
@@ -240,8 +241,8 @@ class CheckoutSetCustomQtyCase(CheckoutSetQtyCommonCase):
                 "qty_done": new_qty,
             },
         )
-        self.assertEqual(line_to_change.qty_done, new_qty)
-        self.assertEqual(line_keep_qty.qty_done, line_keep_qty.quantity)
+        self.assertEqual(line_to_change.qty_picked, new_qty)
+        self.assertEqual(line_keep_qty.qty_picked, line_keep_qty.quantity)
         new_lines = [
             x for x in self.moves_pack1.move_line_ids if x not in selected_lines
         ]

odoo/addons/shopfloor/tests/test_checkout_summary.py

@@ -35,7 +35,7 @@ class CheckoutSummaryCase(CheckoutCommonCase):
         self.picking.action_assign()
         # satisfy only few lines
         for ml in self.picking.move_line_ids[:2]:
-            ml.qty_done = ml.quantity
+            ml.picked = True
             ml.shopfloor_checkout_done = True
         response = self.service.dispatch(
             "summary", params={"picking_id": self.picking.id}
@@ -54,7 +54,7 @@ class CheckoutSummaryCase(CheckoutCommonCase):
         self.picking.action_assign()
         # satisfy only all lines
         for ml in self.picking.move_line_ids:
-            ml.qty_done = ml.quantity
+            ml.picked = True
             ml.shopfloor_checkout_done = True
         response = self.service.dispatch(
             "summary", params={"picking_id": self.picking.id}