odoo-addon-shopfloor 16.0.2.7.0.2__py3-none-any.whl → 16.0.2.8.0__py3-none-any.whl

odoo/addons/shopfloor/tests/test_actions_change_package_lot.py

@@ -716,9 +716,7 @@ class TestActionsChangePackageLot(CommonCase):
         # still we have to handle it). Forbid to pick.
         expected_message = self.msg_store.package_change_error(
             new_package,
-            "Package {} has been partially picked in another location".format(
-                new_package.display_name
-            ),
+            f"Package {new_package.display_name} has been partially picked in another location",  # noqa
         )
         self.change_package_lot.change_package(
             line,
@@ -921,6 +919,10 @@ class TestActionsChangePackageLot(CommonCase):
         line2 = picking2.move_line_ids
         line2.qty_done = 10
 
+        expected_msg = (
+            f"Package {package2.display_name} does not contain available product "
+            f"{line1.product_id.display_name}, cannot replace package."
+        )
         self.change_package_lot.change_package(
             line1,
             package2,
@@ -929,13 +931,7 @@ class TestActionsChangePackageLot(CommonCase):
             # failure callback
             lambda move_line, message=None: self.assertEqual(
                 message,
-                self.msg_store.package_change_error(
-                    package2,
-                    "Package {} does not contain available product {},"
-                    " cannot replace package.".format(
-                        package2.display_name, line1.product_id.display_name
-                    ),
-                ),
+                self.msg_store.package_change_error(package2, expected_msg),
             ),
         )
 

odoo/addons/shopfloor/tests/test_actions_data_base.py

@@ -219,7 +219,7 @@ class ActionsDataDetailCaseBase(ActionsDataCaseBase):
                     kw.get("move_lines", [])
                 ),
                 "products": self.data_detail._location_content(record),
-            }
+            },
         )
 
     def _expected_location_lot(self, record, **kw):
@@ -228,7 +228,7 @@ class ActionsDataDetailCaseBase(ActionsDataCaseBase):
             **{
                 "removal_date": record.removal_date or None,
                 "quantity": sum(record.quant_ids.mapped("quantity")),
-            }
+            },
         )
 
     def _expected_product_detail(self, record, **kw):
@@ -262,7 +262,7 @@ class ActionsDataDetailCaseBase(ActionsDataCaseBase):
         if kw.get("full"):
             detail.update(
                 {
-                    "image": "/web/image/product.product/{}/image_128".format(record.id)
+                    "image": f"/web/image/product.product/{record.id}/image_128"
                     if record.image_128
                     else None,
                     "locations": locations_info,
@@ -296,5 +296,5 @@ class ActionsDataDetailCaseBase(ActionsDataCaseBase):
                 "length_uom": record.length_uom_name,
                 "weight_uom": record.weight_uom_name,
                 "barcode": record.barcode,
-            }
+            },
         )

odoo/addons/shopfloor/tests/test_checkout_auto_post.py

@@ -12,20 +12,21 @@ class CheckoutAutoPostCase(CheckoutCommonCase):
         self._fill_stock_for_moves(picking.move_ids)
         picking.action_assign()
         selected_move_line_a = picking.move_line_ids.filtered(
-            lambda l: l.product_id == self.product_a
+            lambda x: x.product_id == self.product_a
         )
         selected_move_line_a.qty_done = 7
         selected_move_line_b = picking.move_line_ids.filtered(
-            lambda l: l.product_id == self.product_b
+            lambda x: x.product_id == self.product_b
         )
         selected_move_line_b.qty_done = 9
         selected_move_line_c = picking.move_line_ids.filtered(
-            lambda l: l.product_id == self.product_c
+            lambda x: x.product_id == self.product_c
         )
 
         # User has selected 7 units out of 10 for product_a,
         # and 9 units out of 20 for product_b.
-        # We would expect a split picking to be created with those two lines and qtys done.
+        # We would expect a split picking to be created
+        # with those two lines and qtys done.
         self.service.dispatch(
             "scan_package_action",
             params={
@@ -53,10 +54,10 @@ class CheckoutAutoPostCase(CheckoutCommonCase):
         # - the original line for product c, unchanged;
         # - two lines (products a and b) with the non-split qtys.
         line_a_in_original_picking = picking.move_line_ids.filtered(
-            lambda l: l.product_id == selected_move_line_a.product_id
+            lambda x: x.product_id == selected_move_line_a.product_id
         )
         line_b_in_original_picking = picking.move_line_ids.filtered(
-            lambda l: l.product_id == selected_move_line_b.product_id
+            lambda x: x.product_id == selected_move_line_b.product_id
         )
         self.assertEqual(line_a_in_original_picking.reserved_uom_qty, 3)
         self.assertEqual(line_b_in_original_picking.reserved_uom_qty, 11)

odoo/addons/shopfloor/tests/test_checkout_base.py

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

odoo/addons/shopfloor/tests/test_checkout_change_packaging.py

@@ -135,7 +135,7 @@ class CheckoutListSetPackagingCase(CheckoutCommonCase):
             },
             message={
                 "message_type": "success",
-                "body": "Packaging changed on package {}".format(self.package.name),
+                "body": f"Packaging changed on package {self.package.name}",
             },
         )
 

odoo/addons/shopfloor/tests/test_checkout_done.py

@@ -20,7 +20,7 @@ class CheckoutDoneCase(CheckoutCommonCase):
             next_state="select_document",
             message={
                 "message_type": "success",
-                "body": "Transfer {} done".format(picking.name),
+                "body": f"Transfer {picking.name} done",
             },
             data={"restrict_scan_first": False},
         )

odoo/addons/shopfloor/tests/test_checkout_list_delivery_packaging.py

@@ -31,7 +31,7 @@ class CheckoutListDeliveryPackagingCase(CheckoutCommonCase, CheckoutSelectPackag
     @classmethod
     def tearDownClass(cls):
         cls.loader.restore_registry()
-        super(CheckoutListDeliveryPackagingCase, cls).tearDownClass()
+        super().tearDownClass()
 
     @classmethod
     def setUpClassBaseData(cls, *args, **kwargs):

odoo/addons/shopfloor/tests/test_checkout_scan.py

@@ -84,7 +84,7 @@ class CheckoutScanCase(CheckoutCommonCase):
             next_state="select_document",
             message={
                 "message_type": "error",
-                "body": "Transfer {} is not available.".format(picking.name),
+                "body": f"Transfer {picking.name} is not available.",
             },
             data={"restrict_scan_first": False},
         )

odoo/addons/shopfloor/tests/test_checkout_scan_line.py

@@ -67,7 +67,7 @@ class CheckoutScanLineCase(CheckoutScanLineCaseBase):
         picking.action_assign()
         # The product a is scanned, so selected and quantity updated
         line_a = picking.move_line_ids.filtered(
-            lambda l: l.product_id == self.product_a
+            lambda x: x.product_id == self.product_a
         )
         # Because not part of a package other lines are selected also
         related_lines = picking.move_line_ids - line_a
@@ -84,7 +84,7 @@ class CheckoutScanLineCase(CheckoutScanLineCaseBase):
         picking.action_assign()
         # The product a is scanned, so selected and quantity updated
         lines_a = picking.move_line_ids.filtered(
-            lambda l: l.product_id == self.product_a
+            lambda x: x.product_id == self.product_a
         )
         # Because not part of a package other lines are selected also
         related_lines = picking.move_line_ids - lines_a
@@ -100,7 +100,7 @@ class CheckoutScanLineCase(CheckoutScanLineCaseBase):
         self._fill_stock_for_moves(picking.move_ids)
         picking.action_assign()
         lines_a = picking.move_line_ids.filtered(
-            lambda l: l.product_id == self.product_a
+            lambda x: x.product_id == self.product_a
         )
         # when we scan the packaging of the product, we should select the
         # lines as if the product was scanned
@@ -177,7 +177,7 @@ class CheckoutScanLineCase(CheckoutScanLineCaseBase):
             package.name,
             {
                 "message_type": "error",
-                "body": f"Package {package.name} not found in transfer {picking.name}",
+                "body": f"Package {package.name} not found in transfer {picking.name}",  # noqa
             },
         )
 

odoo/addons/shopfloor/tests/test_checkout_scan_line_no_prefill_qty.py

@@ -37,7 +37,7 @@ class CheckoutScanLineNoPrefillQtyCase(CheckoutScanLineCaseBase):
             "scan_line", params={"picking_id": picking.id, "barcode": barcode}
         )
         response_lines = response["data"]["select_package"]["selected_move_lines"]
-        for response_line, qty in zip(response_lines, qties):
+        for response_line, qty in zip(response_lines, qties, strict=False):
             self.assertEqual(response_line["qty_done"], qty)
 
     def test_scan_line_product_exist_in_two_lines(self):
@@ -65,7 +65,7 @@ class CheckoutScanLineNoPrefillQtyCase(CheckoutScanLineCaseBase):
         self._fill_stock_for_moves(picking.move_ids)
         picking.action_assign()
         line_a = picking.move_line_ids.filtered(
-            lambda l: l.product_id == self.product_a
+            lambda x: x.product_id == self.product_a
         )
         # When no_prefill_qty is enabled in the checkout menu, prefilled qty
         # should be 1.0 if a product is scanned
@@ -79,7 +79,7 @@ class CheckoutScanLineNoPrefillQtyCase(CheckoutScanLineCaseBase):
         self._fill_stock_for_moves(picking.move_ids)
         picking.action_assign()
         lines_a = picking.move_line_ids.filtered(
-            lambda l: l.product_id == self.product_a
+            lambda x: x.product_id == self.product_a
         )
         # When no_prefill_qty is enabled in the checkout menu, prefilled qty
         # should be the packaging qty, if a packaging is scanned

odoo/addons/shopfloor/tests/test_checkout_scan_package_action.py

@@ -40,40 +40,40 @@ class CheckoutScanPackageActionCase(CheckoutCommonCase, CheckoutSelectPackageMix
 
     def test_scan_package_action_select_product(self):
         self._test_select_product(
-            lambda l: l.product_id.barcode, lambda l: l.reserved_uom_qty, lambda __: 0
+            lambda x: x.product_id.barcode, lambda x: x.reserved_uom_qty, lambda __: 0
         )
 
     def test_scan_package_action_deselect_product(self):
         self._test_select_product(
-            lambda l: l.product_id.barcode, lambda __: 0, lambda l: l.reserved_uom_qty
+            lambda x: x.product_id.barcode, lambda __: 0, lambda x: x.reserved_uom_qty
         )
 
     def test_scan_package_action_select_product_packaging(self):
         self._test_select_product(
-            lambda l: l.product_id.packaging_ids.barcode,
-            lambda l: l.reserved_uom_qty,
+            lambda x: x.product_id.packaging_ids.barcode,
+            lambda x: x.reserved_uom_qty,
             lambda __: 0,
         )
 
     def test_scan_package_action_deselect_product_packaging(self):
         self._test_select_product(
-            lambda l: l.product_id.packaging_ids.barcode,
+            lambda x: x.product_id.packaging_ids.barcode,
             lambda __: 0,
-            lambda l: l.reserved_uom_qty,
+            lambda x: x.reserved_uom_qty,
         )
 
     def test_scan_package_action_select_product_lot(self):
         self._test_select_product(
-            lambda l: l.lot_id.name,
+            lambda x: x.lot_id.name,
             lambda __: 0,
-            lambda l: l.reserved_uom_qty,
+            lambda x: x.reserved_uom_qty,
             in_lot=True,
         )
 
     def test_scan_package_action_deselect_product_lot(self):
         self._test_select_product(
-            lambda l: l.lot_id.name,
-            lambda l: l.reserved_uom_qty,
+            lambda x: x.lot_id.name,
+            lambda x: x.reserved_uom_qty,
             lambda __: 0,
             in_lot=True,
         )

odoo/addons/shopfloor/tests/test_cluster_picking_is_zero.py

@@ -57,11 +57,7 @@ class ClusterPickingIsZeroCase(ClusterPickingCommonCase):
             data=self._line_data(self.next_line),
             message={
                 "message_type": "success",
-                "body": "{} {} put in {}".format(
-                    self.line.qty_done,
-                    self.line.product_id.display_name,
-                    self.bin1.name,
-                ),
+                "body": f"{self.line.qty_done} {self.line.product_id.display_name} put in {self.bin1.name}",  # noqa
             },
         )
 
@@ -89,10 +85,6 @@ class ClusterPickingIsZeroCase(ClusterPickingCommonCase):
             data=self._line_data(self.next_line),
             message={
                 "message_type": "success",
-                "body": "{} {} put in {}".format(
-                    self.line.qty_done,
-                    self.line.product_id.display_name,
-                    self.bin1.name,
-                ),
+                "body": f"{self.line.qty_done} {self.line.product_id.display_name} put in {self.bin1.name}",  # noqa
             },
         )

odoo/addons/shopfloor/tests/test_cluster_picking_scan_destination.py

@@ -64,9 +64,7 @@ class ClusterPickingScanDestinationPackCase(ClusterPickingCommonCase):
             data=self._line_data(next_line),
             message={
                 "message_type": "success",
-                "body": "{} {} put in {}".format(
-                    line.qty_done, line.product_id.display_name, self.bin1.name
-                ),
+                "body": f"{line.qty_done} {line.product_id.display_name} put in {self.bin1.name}",  # noqa
             },
         )
 
@@ -149,8 +147,8 @@ class ClusterPickingScanDestinationPackCase(ClusterPickingCommonCase):
             data=self._line_data(line, qty_done=10.0),
             message={
                 "message_type": "error",
-                "body": "The destination bin {} is not empty,"
-                " please take another.".format(self.bin1.name),
+                "body": f"The destination bin {self.bin1.name} is not empty, "
+                "please take another.",
             },
         )
 
@@ -233,9 +231,7 @@ class ClusterPickingScanDestinationPackCase(ClusterPickingCommonCase):
             data=self._line_data(line, qty_done=11.0),
             message={
                 "message_type": "error",
-                "body": "You must not pick more than {} units.".format(
-                    line.reserved_uom_qty
-                ),
+                "body": f"You must not pick more than {line.reserved_uom_qty} units.",
             },
         )
 
@@ -271,9 +267,7 @@ class ClusterPickingScanDestinationPackCase(ClusterPickingCommonCase):
             data=self._line_data(new_line),
             message={
                 "message_type": "success",
-                "body": "{} {} put in {}".format(
-                    line.qty_done, line.product_id.display_name, self.bin1.name
-                ),
+                "body": f"{line.qty_done} {line.product_id.display_name} put in {self.bin1.name}",  # noqa
             },
         )
 
@@ -369,8 +363,6 @@ class ClusterPickingScanDestinationPackCase(ClusterPickingCommonCase):
             data=self._line_data(next_line),
             message={
                 "message_type": "success",
-                "body": "{} {} put in {}".format(
-                    line.qty_done, line.product_id.display_name, self.bin1.name
-                ),
+                "body": f"{line.qty_done} {line.product_id.display_name} put in {self.bin1.name}",  # noqa
             },
         )

odoo/addons/shopfloor/tests/test_cluster_picking_scan_line_location_or_pack_first.py

@@ -103,8 +103,7 @@ class ClusterPickingScanLineLocationOrPackFirstCase(ClusterPickingLineCommonCase
         """Check scanning a location then a product without package.
 
         When there is multiple product in the location and the location is scanned,
-        The user needs to scan the product but the system does not remember the location ?
-
+        User needs to scan the product but the system does not remember the location.
         """
         self._simulate_batch_selected(self.batch, in_package=False, in_lot=False)
         line = self.batch.picking_ids.move_line_ids

odoo/addons/shopfloor/tests/test_cluster_picking_stock_issue.py

@@ -125,7 +125,7 @@ class ClusterPickingStockIssue(ClusterPickingCommonCase):
             + self.move2.product_uom_qty
             + sum(
                 self.batch_other.picking_ids.move_line_ids.filtered(
-                    lambda l: l.location_id == self.shelf2
+                    lambda x: x.location_id == self.shelf2
                 ).mapped("reserved_uom_qty")
             )
         )
@@ -162,10 +162,10 @@ class ClusterPickingStockIssue(ClusterPickingCommonCase):
         self.assertEqual(self.move5.move_line_ids.location_id, self.shelf2)
 
         line_shelf1 = self.move3.move_line_ids.filtered(
-            lambda l: l.location_id == self.shelf1
+            lambda x: x.location_id == self.shelf1
         )
         line_shelf2 = self.move3.move_line_ids.filtered(
-            lambda l: l.location_id == self.shelf2
+            lambda x: x.location_id == self.shelf2
         )
 
         # pick the first 2 moves

odoo/addons/shopfloor/tests/test_cluster_picking_unload.py

@@ -430,7 +430,7 @@ class ClusterPickingSetDestinationAllCase(ClusterPickingUnloadingCommonCase):
         )
 
     def test_set_destination_all_check_confirmation(self):
-        """Endpoint called confirming with a different location, ask confirmation again"""
+        """Endpoint called confirming with a different location, ask confirmation"""
         move_lines = self.move_lines
         self._set_dest_package_and_done(move_lines, self.bin1)
         move_lines.write({"location_dest_id": self.packing_a_location.id})
@@ -933,8 +933,8 @@ class ClusterPickingUnloadScanDestinationCase(ClusterPickingUnloadingCommonCase)
             response,
             next_state="unload_single",
             popup={
-                "body": "Last operation of transfer {}. Next operation "
-                "({}) is ready to proceed.".format(picking.name, next_picking.name)
+                "body": f"Last operation of transfer {picking.name}. Next operation "
+                f"({next_picking.name}) is ready to proceed."
             },
             data=data,
         )

odoo/addons/shopfloor/tests/test_delivery_scan_deliver.py

@@ -680,7 +680,7 @@ class DeliveryScanDeliverSpecialCase(DeliveryCommonCase):
             response,
             message={
                 "message_type": "error",
-                "body": "Transfer {} is not available.".format(picking.name),
+                "body": f"Transfer {picking.name} is not available.",
             },
         )
 

odoo/addons/shopfloor/tests/test_location_content_transfer_get_work.py

@@ -123,11 +123,14 @@ class TestLocationContentTransferGetWork(LocationContentTransferCommonCase):
 
     def test_find_work_custom_sort_key(self):
         # fmt: off
+        custom_code = (
+            f"key = ("
+            f"-1 if line.location_id.id == {self.content_loc.id} else 10, )"
+        )
         self.menu.sudo().write(
             {
                 "move_line_search_sort_order": "custom_code",
-                "move_line_search_sort_order_custom_code":
-                    f"key = (-1 if line.location_id.id == {self.content_loc.id} else 10, )",
+                "move_line_search_sort_order_custom_code": custom_code,
             }
         )
         # fmt: on

odoo/addons/shopfloor/tests/test_location_content_transfer_mix.py

@@ -220,7 +220,7 @@ class LocationContentTransferMixCase(LocationContentTransferCommonCase):
         # done, the operator is currently moving the goods to the destination location)
         pack_move_line1 = pick_move_line1.move_id.move_dest_ids.filtered(
             lambda m: m.state not in ("cancel", "done")
-        ).move_line_ids.filtered(lambda l: not l.shopfloor_user_id)
+        ).move_line_ids.filtered(lambda x: not x.shopfloor_user_id)
         self._location_content_transfer_process_line(pack_move_line1)
         # Operator-1 process the second pallet with the "zone picking" scenario
         self._zone_picking_process_line(pick_move_line2)
@@ -228,11 +228,12 @@ class LocationContentTransferMixCase(LocationContentTransferCommonCase):
         # the location where this second pallet is
         pack_move_line2 = pick_move_line2.move_id.move_dest_ids.filtered(
             lambda m: m.state not in ("cancel", "done")
-        ).move_line_ids.filtered(lambda l: not l.shopfloor_user_id)
-        assert (
-            len(pack_move_line2) == 1
-        ), "Operator-3 should end up with one move line taken from {}".format(
-            pack_move_line2.picking_id.name
+        ).move_line_ids.filtered(lambda x: not x.shopfloor_user_id)
+        self.assertEqual(
+            len(pack_move_line2),
+            1,
+            f"Operator-3 should find only "
+            f"one move line in {pack_move_line2.location_id.name}",
         )
         self._location_content_transfer_process_line(pack_move_line2)
 
@@ -302,12 +303,12 @@ class LocationContentTransferMixCase(LocationContentTransferCommonCase):
         )
         self.assertEqual(pack_move_a, self.pack_move_a)
         pack_first_pallet = pack_move_a.move_line_ids.filtered(
-            lambda l: not l.shopfloor_user_id and l.location_id == dest_location1
+            lambda x: not x.shopfloor_user_id and x.location_id == dest_location1
         )
         self.assertEqual(pack_first_pallet.reserved_uom_qty, 6)
         self.assertEqual(pack_first_pallet.qty_done, 0)
         pack_second_pallet = pack_move_a.move_line_ids.filtered(
-            lambda l: not l.shopfloor_user_id and l.location_id == dest_location2
+            lambda x: not x.shopfloor_user_id and x.location_id == dest_location2
         )
         self.assertEqual(pack_second_pallet.reserved_uom_qty, 4)
         self.assertEqual(pack_second_pallet.qty_done, 0)
@@ -367,7 +368,7 @@ class LocationContentTransferMixCase(LocationContentTransferCommonCase):
         )
         self.assertEqual(pack_move_a, self.pack_move_a)
         pack_second_pallet = pack_move_a.move_line_ids.filtered(
-            lambda l: not l.shopfloor_user_id and l.location_id == dest_location2
+            lambda x: not x.shopfloor_user_id and x.location_id == dest_location2
         )
         picking_before = pack_second_pallet.picking_id
         move_lines = self.service.search_move_line.search_move_lines(
@@ -443,7 +444,7 @@ class LocationContentTransferMixCase(LocationContentTransferCommonCase):
         )
         self.assertEqual(pack_move_a1, self.pack_move_a)
         pack_first_pallet = pack_move_a1.move_line_ids.filtered(
-            lambda l: not l.shopfloor_user_id and l.location_id == dest_location1
+            lambda x: not x.shopfloor_user_id and x.location_id == dest_location1
         )
         self.assertEqual(pack_first_pallet.reserved_uom_qty, 6)
         self.assertEqual(pack_first_pallet.qty_done, 0)
@@ -469,7 +470,7 @@ class LocationContentTransferMixCase(LocationContentTransferCommonCase):
             lambda m: m.move_line_ids.package_id == self.package_2
         )
         pack_second_pallet = pack_move_a2.move_line_ids.filtered(
-            lambda l: not l.shopfloor_user_id and l.location_id == dest_location2
+            lambda x: not x.shopfloor_user_id and x.location_id == dest_location2
         )
         self.assertEqual(pack_second_pallet.reserved_uom_qty, 4)
         self.assertEqual(pack_second_pallet.qty_done, 0)

odoo/addons/shopfloor/tests/test_menu_contrains.py

@@ -7,7 +7,6 @@ from .test_menu_base import MenuCountersCommonCase
 
 class TestMenuContrains(MenuCountersCommonCase):
     def test_move_line_search_sort_order_custom_code_invalid(self):
-
         with self.assertRaises(exceptions.ValidationError):
             # wrong indentation in python code
             self.menu1.sudo().write(

odoo/addons/shopfloor/tests/test_single_pack_transfer.py

@@ -154,7 +154,7 @@ class TestSinglePackTransfer(SinglePackTransferCommonBase):
             next_state="start",
             message={
                 "message_type": "error",
-                "body": "No pending operation for package {}.".format(self.pack_a.name),
+                "body": f"No pending operation for package {self.pack_a.name}.",
             },
         )
 
@@ -302,8 +302,7 @@ class TestSinglePackTransfer(SinglePackTransferCommonBase):
             next_state="start",
             message={
                 "message_type": "error",
-                "body": "Location %s doesn't contain any package."
-                % (self.shelf2.name,),
+                "body": f"Location {self.shelf2.name} doesn't contain any package.",
             },
         )
 
@@ -340,8 +339,8 @@ class TestSinglePackTransfer(SinglePackTransferCommonBase):
             next_state="start",
             message={
                 "message_type": "warning",
-                "body": "Several packages found in %s, please scan a package."
-                % (self.shelf1.name,),
+                "body": f"Several packages found in {self.shelf1.name}, "
+                "please scan a package.",
             },
         )
 
@@ -366,8 +365,9 @@ class TestSinglePackTransfer(SinglePackTransferCommonBase):
             next_state="start",
             message={
                 "message_type": "error",
-                "body": "You cannot work on a package (%s) outside of locations: %s"
-                % (self.pack_a.name, self.picking_type.default_location_src_id.name),
+                "body": f"You cannot work on a package ({self.pack_a.name}) "
+                "outside of locations: "
+                f"{self.picking_type.default_location_src_id.name}",
             },
         )
 
@@ -533,8 +533,8 @@ class TestSinglePackTransfer(SinglePackTransferCommonBase):
             response,
             next_state="start",
             popup={
-                "body": "Last operation of transfer {}. Next operation "
-                "({}) is ready to proceed.".format(self.picking.name, next_picking.name)
+                "body": f"Last operation of transfer {self.picking.name}. "
+                f"Next operation ({next_picking.name}) is ready to proceed."
             },
             message={
                 "message_type": "success",

odoo/addons/shopfloor/tests/test_stock_split.py

@@ -10,7 +10,7 @@ from odoo.tests.common import TransactionCase
 class TestStockSplit(TransactionCase):
     @classmethod
     def setUpClass(cls):
-        super(TestStockSplit, cls).setUpClass()
+        super().setUpClass()
         cls.env = cls.env(context=dict(cls.env.context, tracking_disable=True))
         cls.warehouse = cls.env.ref("stock.warehouse0")
         cls.warehouse.delivery_steps = "pick_pack_ship"

odoo/addons/shopfloor/tests/test_zone_picking_base.py

@@ -343,12 +343,14 @@ class ZonePickingCommonCase(CommonCase):
             data["sublocation"] = self.data.location(sublocation)
         for data_move_line in data["move_lines"]:
             move_line = self.env["stock.move.line"].browse(data_move_line["id"])
-            data_move_line[
-                "location_will_be_empty"
-            ] = move_line.location_id.planned_qty_in_location_is_empty(move_line)
-            data_move_line[
-                "handle_complete_mix_pack"
-            ] = self.service._handle_complete_mix_pack(move_line.package_id)
+            # fmt: off
+            data_move_line["location_will_be_empty"] = (
+                move_line.location_id.planned_qty_in_location_is_empty(move_line)
+            )
+            data_move_line["handle_complete_mix_pack"] = (
+                self.service._handle_complete_mix_pack(move_line.package_id)
+            )
+            # fmt: on
         self.assert_response(
             response,
             next_state=state,
@@ -401,9 +403,6 @@ class ZonePickingCommonCase(CommonCase):
         expected_move_line = self.data.move_line(move_line, with_picking=True)
         if qty_done is not None:
             expected_move_line["qty_done"] = qty_done
-        allow_alternative_destination_package = (
-            self.menu.allow_alternative_destination_package
-        )
         self.assert_response(
             response,
             next_state=state,
@@ -412,7 +411,9 @@ class ZonePickingCommonCase(CommonCase):
                 "picking_type": self.data.picking_type(picking_type),
                 "move_line": expected_move_line,
                 "confirmation_required": confirmation_required,
-                "allow_alternative_destination_package": allow_alternative_destination_package,
+                "allow_alternative_destination_package": (
+                    self.menu.allow_alternative_destination_package
+                ),
                 "handle_complete_mix_pack": handle_complete_mix_pack,
             },
             message=message,

odoo/addons/shopfloor/tests/test_zone_picking_complete_mix_pack_flux.py

@@ -24,7 +24,7 @@ class ZonePickingCompleteMixPackageFluxCase(ZonePickingCommonCase):
         move_lines = self.service._find_location_move_lines(
             package=package,
         )
-        move_lines = move_lines.sorted(lambda l: l.move_id.priority, reverse=True)
+        move_lines = move_lines.sorted(lambda x: x.move_id.priority, reverse=True)
         move_line = move_lines[0]
         self.assert_response_set_line_destination(
             response,
@@ -49,7 +49,7 @@ class ZonePickingCompleteMixPackageFluxCase(ZonePickingCommonCase):
         )
         # Check response
         move_lines = self.service._find_location_move_lines()
-        move_lines = move_lines.sorted(lambda l: l.move_id.priority, reverse=True)
+        move_lines = move_lines.sorted(lambda x: x.move_id.priority, reverse=True)
         self.assert_response_select_line(
             response,
             zone_location=self.zone_location,