[17.0][MIG] delivery_roulier_chronopost_fr: Migration to 17.0

Author: yankinmax

delivery_roulier_chronopost_fr/__manifest__.py

@@ -2,13 +2,11 @@
 
 {
     "name": "Delivery Chronopost France",
-    "version": "14.0.1.0.1",
+    "version": "17.0.1.0.1",
     "author": "Akretion,Odoo Community Association (OCA)",
     "category": "Delivery",
     "complexity": "normal",
-    "depends": [
-        "delivery_roulier_option",
-    ],
+    "depends": ["delivery_roulier_option"],
     "website": "https://github.com/OCA/delivery-carrier",
     "data": [
         "views/carrier_account.xml",

delivery_roulier_chronopost_fr/tests/test_label_chronopost.py

@@ -1,21 +1,23 @@
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 
-from vcr_unittest import VCRMixin
+import pytest
 
 from odoo.addons.base_delivery_carrier_label.tests import carrier_label_case
 
 
-class ChronopostLabelCase(VCRMixin, carrier_label_case.CarrierLabelCase):
-    def setUp(self, *args, **kwargs):
+class ChronopostLabelCase(carrier_label_case.TestCarrierLabel):
+    @classmethod
+    def setUpClass(cls):
         # need it to be defined before super to avoid failure in _hide_sensitive_data
-        self.account = False
-        super().setUp(*args, **kwargs)
+        cls.account = False
+        super().setUpClass()
+        cls.env = cls.env(context=dict(cls.env.context, tracking_disable=True))
         # french carrier sender need to be from France
-        self.picking.company_id.partner_id.write(
-            {"country_id": self.env.ref("base.fr").id}
+        cls.picking.company_id.partner_id.write(
+            {"country_id": cls.env.ref("base.fr").id}
         )
         # change account and password with valid credentials to regenerate the cassette
-        self.account = self.env["carrier.account"].create(
+        cls.account = cls.env["carrier.account"].create(
             {
                 "name": "Chronopost Test Account",
                 "delivery_type": "chronopost_fr",
@@ -25,6 +27,7 @@ def setUp(self, *args, **kwargs):
             }
         )
 
+    @classmethod
     def _hide_sensitive_data(self, request):
         password = self.account and self.account.password or "dummy"
         account = self.account and self.account.account or "dummy"
@@ -34,8 +37,12 @@ def _hide_sensitive_data(self, request):
         request.body = body
         return request
 
-    def _get_vcr_kwargs(self, **kwargs):
+    @classmethod
+    @pytest.fixture(scope="module")
+    def vcr_config(self):
         return {
+            "filter_headers": ["authorization"],
+            "ignore_localhost": True,
             "record_mode": "once",
             "match_on": ["method", "path"],
             "decode_compressed_response": True,

delivery_roulier_chronopost_fr/views/carrier_account.xml

@@ -1,23 +1,24 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <odoo>
 
-    <record id="view_chronopost_carrier_account_form" model="ir.ui.view">
+     <record id="view_chronopost_carrier_account_form" model="ir.ui.view">
         <field name="model">carrier.account</field>
         <field
             name="inherit_id"
-            ref="base_delivery_carrier_label.carrier_account_view_form"
+            ref="delivery_carrier_account.carrier_account_view_form"
         />
         <field name="arch" type="xml">
             <field name="account" position="after">
                 <field
                     name="chronopost_fr_subaccount"
-                    attrs="{'invisible': [('delivery_type', '!=', 'chronopost_fr')]}"
+                    invisible="delivery_type != 'chronopost_fr'"
                 />
             </field>
             <field name="password" position="after">
                 <field
                     name="chronopost_fr_file_format"
-                    attrs="{'invisible': [('delivery_type', '!=', 'chronopost_fr')], 'required': [('name', '=', 'chronopost')]}"
+                    invisible="delivery_type != 'chronopost_fr'"
+                    required="name != 'chronopost'"
                 />
             </field>
         </field>

test-requirements.txt

@@ -1,2 +1,4 @@
 # delivery_roulier
 odoo_test_helper
+# delivery_roulier_chronopost_fr
+pytest-recording