[MIG] base_rest: Migration to 19.0

Author: NachoAlesLopez

base_rest/__manifest__.py

@@ -20,8 +20,6 @@
     "assets": {
         "web.assets_frontend": [
             "base_rest/static/src/scss/base_rest.scss",
-            "base_rest/static/src/js/swagger_ui.js",
-            "base_rest/static/src/js/swagger.js",
         ],
     },
     "external_dependencies": {
@@ -33,5 +31,4 @@
             "apispec",
         ]
     },
-    "installable": True,
 }

base_rest/http.py

@@ -29,8 +29,8 @@
     ValidationError,
 )
 from odoo.http import (
-    CSRF_FREE_METHODS,
     MISSING_CSRF_WARNING,
+    SAFE_HTTP_METHODS,
     Dispatcher,
     SessionExpiredException,
     request,
@@ -162,7 +162,7 @@ def dispatch(self, endpoint, args):
 
         # Check for CSRF token for relevant requests
         if (
-            self.request.httprequest.method not in CSRF_FREE_METHODS
+            self.request.httprequest.method not in SAFE_HTTP_METHODS
             and endpoint.routing.get("csrf", True)
         ):
             token = params.pop("csrf_token", None)

base_rest/models/rest_service_registration.py

@@ -189,8 +189,8 @@ def build_registry(self, services_registry, states=None, exclude_addons=None):
         # dependencies to ensure that controllers defined in a more
         # specialized addon and overriding more generic one takes precedences
         # on the generic one into the registry
-        graph = odoo.modules.graph.Graph()
-        graph.add_module(self.env.cr, "base")
+        graph = odoo.modules.module_graph.ModuleGraph(self.env.cr)
+        graph.extend(["base"])
 
         query = "SELECT name FROM ir_module_module WHERE state IN %s "
         params = [tuple(states)]
@@ -200,7 +200,7 @@ def build_registry(self, services_registry, states=None, exclude_addons=None):
         self.env.cr.execute(query, params)
 
         module_list = [name for (name,) in self.env.cr.fetchall() if name not in graph]
-        graph.add_modules(self.env.cr, module_list)
+        graph.extend(module_list)
 
         for module in graph:
             self.load_services(module.name, services_registry)

base_rest/tests/common.py

@@ -95,6 +95,7 @@ def _setup_registry(class_or_instance):
             )
 
         # register our components
+        class_or_instance.comp_registry.load_components("component")
         class_or_instance.comp_registry.load_components("base_rest")
 
         # Define a base test controller here to avoid to have this controller
@@ -239,15 +240,15 @@ def setUpClass(cls):
 
 
 class BaseRestCase(TransactionComponentCase, RegistryMixin):
-    @classmethod
-    def setUpClass(cls):
-        super().setUpClass()
-        cls.setUpRegistry()
-        cls.base_url = cls.env["ir.config_parameter"].get_param("web.base.url")
-        cls.registry.enter_test_mode(cls.env.cr)
+    # pylint: disable=W8106
+    def setUp(self):
+        super().setUp()
+        self.setUpRegistry()
+        self.base_url = self.env["ir.config_parameter"].get_param("web.base.url")
+        self.registry_enter_test_mode(register_cleanup=False)
 
     # pylint: disable=W8110
     @classmethod
     def tearDownClass(cls):
-        cls.registry.leave_test_mode()
+        cls.registry_leave_test_mode()
         super().tearDownClass()

base_rest/tests/test_openapi_generator.py

@@ -2,7 +2,7 @@
 # License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl).
 
 from odoo.addons.component.core import Component
-from odoo.addons.website.tools import MockRequest
+from odoo.addons.http_routing.tests.common import MockRequest
 
 from .. import restapi
 from .common import TransactionRestServiceRegistryCase

base_rest/tests/test_service_context_provider.py

@@ -1,7 +1,7 @@
 # Copyright 2021 ACSONE SA/NV
 # License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl).
 from odoo.addons.component.core import Component
-from odoo.addons.website.tools import MockRequest
+from odoo.addons.http_routing.tests.common import MockRequest
 
 from .. import restapi
 from .common import BaseRestCase, TransactionRestServiceRegistryCase
@@ -147,4 +147,4 @@ def get(self, _id):
 class CommonCase(BaseRestCase):
     # dummy test method to pass codecov
     def test_04(self):
-        self.assertEqual(self.registry.test_cr, self.cr)
+        self.assertEqual(self.registry.db_name, self.cr.dbname)