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

odoo/addons/shopfloor/services/delivery.py

@@ -233,7 +233,7 @@ class Delivery(Component):
 
     def _deliver_package(self, picking, package, location):
         lines = package.move_line_ids.filtered(
-            lambda l: l.state in ("assigned", "partially_available")
+            lambda x: x.state in ("assigned", "partially_available")
         )
         if not lines:
             return self._response_for_deliver(
@@ -372,7 +372,7 @@ class Delivery(Component):
         if message:
             return self._response_for_deliver(location=location, message=message)
         lines = lines.filtered(
-            lambda l: l.move_id.picking_type_id in self.picking_types
+            lambda x: x.move_id.picking_type_id in self.picking_types
         )
         # State of the picking might change while we reach this point: check again!
         message = self._check_picking_status(lines.mapped("picking_id"))
@@ -421,7 +421,7 @@ class Delivery(Component):
                 )
         # We focus only on lines on which we can increase the 'qty_done'
         lines = lines.filtered(
-            lambda l: (l.qty_done + product_qty) <= l.reserved_uom_qty
+            lambda x: (x.qty_done + product_qty) <= x.reserved_uom_qty
         )
         # Filter lines to keep only ones from one delivery operation
         # (we do not want to process lines of another delivery operation)

odoo/addons/shopfloor/services/service.py

@@ -28,7 +28,6 @@ class BaseShopfloorService(AbstractComponent):
 
 
 class BaseShopfloorProcess(AbstractComponent):
-
     _inherit = "base.shopfloor.process"
 
     def _get_process_picking_types(self):

odoo/addons/shopfloor/services/single_pack_transfer.py

@@ -126,7 +126,6 @@ class SinglePackTransfer(Component):
                     message=self.msg_store.package_already_picked_by(package, picking)
                 )
             elif other_move_lines and self.work.menu.allow_unreserve_other_moves:
-
                 unreserved_moves = other_move_lines.move_id
                 other_package_levels = other_move_lines.package_level_id
                 other_package_levels.explode_package()

odoo/addons/shopfloor/services/zone_picking.py

@@ -238,9 +238,11 @@ class ZonePicking(Component):
         data = self._data_for_move_line(move_line)
         data["move_line"].update(kw)
         data["confirmation_required"] = confirmation_required
-        data[
-            "allow_alternative_destination_package"
-        ] = self.work.menu.allow_alternative_destination_package
+        # fmt: off
+        data["allow_alternative_destination_package"] = (
+            self.work.menu.allow_alternative_destination_package
+        )
+        # fmt: on
         data["handle_complete_mix_pack"] = self._handle_complete_mix_pack(
             move_line.package_id
         )
@@ -368,13 +370,15 @@ class ZonePicking(Component):
             data_move_line["handle_complete_mix_pack"] = handle_complete_mix_pack
             # `location_will_be_empty` flag states if, by processing this move line
             # and picking the product, the location will be emptied.
-            data_move_line[
-                "location_will_be_empty"
-            ] = move_line.location_id.planned_qty_in_location_is_empty(
-                move_line.package_id.move_line_ids
-                if handle_complete_mix_pack
-                else move_line
+            # fmt: off
+            data_move_line["location_will_be_empty"] = (
+                move_line.location_id.planned_qty_in_location_is_empty(
+                    move_line.package_id.move_line_ids
+                    if handle_complete_mix_pack
+                    else move_line
+                )
             )
+            # fmt: on
         return data
 
     def _data_for_location(self, location, zone_location=None, picking_type=None):
@@ -1140,8 +1144,8 @@ class ZonePicking(Component):
         When the barcode is the product (or its packaging) or the lot on the line:
             * The done quantity is incremented by one or the packaging quantity.
 
-        The `handle_complete_mix_pack` option, when it is set to true. Will move all they
-        lines contained in the package of the move line passed in parameter.
+        The `handle_complete_mix_pack` option, when it is set to true,
+        will move all lines contained in the package of the move line given.
 
         Transitions:
         * select_line: destination has been set, showing the next lines to pick
@@ -1214,7 +1218,6 @@ class ZonePicking(Component):
         # When the barcode is a package
         package = search.package_from_scan(barcode)
         if package:
-
             if not moving_full_quantity and move_line.package_id == package:
                 # Check we're not using the source package as transfer package.
                 message = self.msg_store.dest_package_not_valid(package)
@@ -1230,7 +1233,7 @@ class ZonePicking(Component):
                 and move_line.result_package_id
                 and move_line.result_package_id != package
             ):
-                # Check whether the user can move a whole package to a different package.
+                # Check whether the user can move a whole package to a different one.
                 message = self.msg_store.package_transfer_not_allowed_scan_location()
                 return self._response_for_set_line_destination(
                     move_line, message=message, qty_done=quantity

odoo/addons/shopfloor/static/description/index.html

@@ -367,7 +367,7 @@ ul.auto-toc {
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-!! source digest: sha256:986a6c228f6ee438330907a4be64b74f7291bb1acfc6498a2617ed5e69777d9e
+!! source digest: sha256:8f4a1f9cca9cb3a4830c65a6a9d8a65ec3334a34722525fe5da87e77b0e90de2
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
 <p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/wms/tree/16.0/shopfloor"><img alt="OCA/wms" src="https://img.shields.io/badge/github-OCA%2Fwms-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/wms-16-0/wms-16-0-shopfloor"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/wms&amp;target_branch=16.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
 <p>Shopfloor is a barcode scanner application for internal warehouse operations.</p>

odoo/addons/shopfloor/tests/common.py

@@ -268,7 +268,6 @@ class CommonCase(BaseCommonCase):
 
 
 class PickingBatchMixin:
-
     BatchProduct = namedtuple(
         "BatchProduct",
         # browse record of the product,

odoo/addons/shopfloor/tests/models.py

@@ -7,6 +7,7 @@ Add a new "test" value for 'delivery_type' field of carrier and
 Shopfloor do not depend on 'delivery_*' modules adding the different
 delivery types.
 """
+
 from odoo import fields, models
 
 

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