[MIG] stock_picking_delivery_link: Migration to 18.0

Author: sebalix

stock_picking_delivery_link/__manifest__.py

@@ -4,11 +4,11 @@
 {
     "name": "Stock Picking Delivery Link",
     "summary": "Adds link to the delivery on all intermediate operations.",
-    "version": "16.0.1.1.4",
+    "version": "18.0.1.0.0",
     "author": "Camptocamp, Odoo Community Association (OCA)",
     "website": "https://github.com/OCA/delivery-carrier",
     "category": "Warehouse Management",
-    "depends": ["stock", "delivery"],
+    "depends": ["stock_delivery"],
     "data": ["views/stock_picking.xml", "views/stock_picking_type.xml"],
     "installable": True,
     "license": "AGPL-3",

stock_picking_delivery_link/tests/test_delivery_link.py

@@ -11,7 +11,7 @@ class TestStockPickingDeliveryLink(StockPickingDeliveryLinkCommonCase):
     def setUpClass(cls):
         super().setUpClass()
         cls.product = cls.env["product.product"].create(
-            {"name": "Test Product", "type": "product"}
+            {"name": "Test Product", "type": "consu", "is_storable": True}
         )
         cls.customer_location = cls.env.ref("stock.stock_location_customers")
         test_carrier_product = cls.env["product.product"].create(
@@ -159,9 +159,11 @@ def test_put_in_pack_from_pick_with_wizard(self):
         # force wizard on pick operation picking_type_id
         pick_picking.picking_type_id.set_delivery_package_type_on_put_in_pack = True
         pick_picking.action_assign()
-        pick_picking.move_line_ids.filtered(
+        pick_line = pick_picking.move_line_ids.filtered(
             lambda ml: ml.product_id == self.product
-        ).qty_done = 5.0
+        )
+        self.assertEqual(pick_line.quantity, 5)
+        pick_line.picked = True
         pip_action = pick_picking.action_put_in_pack()
         # check the action is a dict
         self.assertIsInstance(pip_action, dict)
@@ -215,9 +217,11 @@ def test_put_in_pack_from_pick_with_wizard_carrier(self):
         # force wizard on pick operation picking_type_id
         pick_picking.picking_type_id.set_delivery_package_type_on_put_in_pack = True
         pick_picking.action_assign()
-        pick_picking.move_line_ids.filtered(
+        pick_line = pick_picking.move_line_ids.filtered(
             lambda ml: ml.product_id == self.product
-        ).qty_done = 5.0
+        )
+        self.assertEqual(pick_line.quantity, 5)
+        pick_line.picked = True
         pip_action = pick_picking.action_put_in_pack()
         # check the action is a dict
         self.assertIsInstance(pip_action, dict)
@@ -268,9 +272,11 @@ def test_put_in_pack_from_pick_without_wizard(self):
         # set a carrier on shipment picking but do not force wizard on picking type
         ship_picking.carrier_id = self.test_carrier
         pick_picking.action_assign()
-        pick_picking.move_line_ids.filtered(
+        pick_line = pick_picking.move_line_ids.filtered(
             lambda ml: ml.product_id == self.product
-        ).qty_done = 5.0
+        )
+        self.assertEqual(pick_line.quantity, 5)
+        pick_line.picked = True
         pip_action = pick_picking.action_put_in_pack()
         # the action is not a dict so not a wizard
         self.assertNotIsInstance(pip_action, dict)

stock_picking_delivery_link/views/stock_picking.xml

@@ -11,7 +11,7 @@
                 <group
                     name="ship_data"
                     string="Delivery Links"
-                    attrs="{'invisible': [('ship_picking_id', '=', False)]}"
+                    invisible="not ship_picking_id"
                 >
                     <field name="ship_picking_id" />
                     <field name="ship_carrier_id" />

test-requirements.txt

@@ -0,0 +1 @@
+odoo-test-helper