[IMP] delivery_roulier_geodis_fr : fix tests + add maintainer

Author: florian-dacosta

delivery_roulier_geodis_fr/__manifest__.py

@@ -5,7 +5,7 @@
     "version": "16.0.1.0.0",
     "author": "Akretion, Odoo Community Association (OCA)",
     "summary": "Generate Label for Geodis logistic",
-    "maintainer": "Akretion, Odoo Community Association (OCA)",
+    "maintainers": ["florian-dacosta"],
     "category": "Warehouse",
     "depends": [
         "delivery_roulier",

delivery_roulier_geodis_fr/tests/test_geodis_labels.py

@@ -28,6 +28,8 @@ def setUp(self, *args, **kwargs):
                 "geodis_fr_customer_id": "787000",
             }
         )
+        carrier = self.env.ref("delivery_roulier_geodis_fr.delivery_carrier_mes")
+        carrier.carrier_account_id = self.account.id
         self.agency = self.env["delivery.carrier.agency"].create(
             {
                 "name": "Lille Agency",
@@ -61,6 +63,13 @@ def _get_vcr_kwargs(self, **kwargs):
             "before_record_request": self._hide_sensitive_data,
         }
 
+    def _transfer_order_picking(self):
+        for move in self.picking.move_ids:
+            move.quantity_done = move.product_uom_qty
+        move_lines = self.picking.move_line_ids
+        self.picking._put_in_pack(move_lines)
+        return super()._transfer_order_picking()
+
     def _product_data(self):
         data = super()._product_data()
         data.update(
@@ -88,27 +97,23 @@ def _partner_data(self):
         )
         return data
 
-    def test_labels(self):
+    def test_labels_and_edi(self):
         res = super().test_labels()
         self.assertTrue(self.picking.geodis_shippingid)
-        return res
-
-    def test_edi(self):
         deposit = self.env["deposit.slip"].create(
             {
                 "name": "test",
                 "delivery_type": "geodis_fr",
                 "picking_ids": [(6, 0, self.picking.ids)],
             }
         )
-        # filled on label generation / simulate it
-        self.picking.write({"geodis_shippingid": "0123456"})
         deposit.validate_deposit()
         attachment = self.env["ir.attachment"].search(
             [("res_id", "=", deposit.id), ("res_model", "=", "deposit.slip")]
         )
         self.assertEqual(len(attachment), 1)
         self.assertTrue(attachment.datas)
+        return res
 
     def test_addresses(self):
         addresses = self.picking._geodis_fr_get_address_proposition()