[MIG] fastapi_auth_api_key: Migration to 19.0

Author: agiuseppe28

fastapi_auth_api_key/README.rst

@@ -21,13 +21,13 @@ Fastapi Auth API Key
     :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
     :alt: License: AGPL-3
 .. |badge3| image:: https://img.shields.io/badge/github-OCA%2Frest--framework-lightgray.png?logo=github
-    :target: https://github.com/OCA/rest-framework/tree/18.0/fastapi_auth_api_key
+    :target: https://github.com/OCA/rest-framework/tree/19.0/fastapi_auth_api_key
     :alt: OCA/rest-framework
 .. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
-    :target: https://translation.odoo-community.org/projects/rest-framework-18-0/rest-framework-18-0-fastapi_auth_api_key
+    :target: https://translation.odoo-community.org/projects/rest-framework-19-0/rest-framework-19-0-fastapi_auth_api_key
     :alt: Translate me on Weblate
 .. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png
-    :target: https://runboat.odoo-community.org/builds?repo=OCA/rest-framework&target_branch=18.0
+    :target: https://runboat.odoo-community.org/builds?repo=OCA/rest-framework&target_branch=19.0
     :alt: Try me on Runboat
 
 |badge1| |badge2| |badge3| |badge4| |badge5|
@@ -84,7 +84,7 @@ Bug Tracker
 Bugs are tracked on `GitHub Issues <https://github.com/OCA/rest-framework/issues>`_.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us to smash it by providing a detailed and welcomed
-`feedback <https://github.com/OCA/rest-framework/issues/new?body=module:%20fastapi_auth_api_key%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+`feedback <https://github.com/OCA/rest-framework/issues/new?body=module:%20fastapi_auth_api_key%0Aversion:%2019.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
 
 Do not contact contributors directly about support or help with technical issues.
 
@@ -101,6 +101,7 @@ Contributors
 
 - Matthieu Méquignon <matthieu.mequignon@camptocamp.com>
 - Son Ho <sonhd@trobz.com>
+- Giuseppe Ardoselli <agiuseppe28@gmail.com>
 
 Other credits
 -------------
@@ -129,6 +130,6 @@ Current `maintainer <https://odoo-community.org/page/maintainer-role>`__:
 
 |maintainer-mmequignon| 
 
-This module is part of the `OCA/rest-framework <https://github.com/OCA/rest-framework/tree/18.0/fastapi_auth_api_key>`_ project on GitHub.
+This module is part of the `OCA/rest-framework <https://github.com/OCA/rest-framework/tree/19.0/fastapi_auth_api_key>`_ project on GitHub.
 
 You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.

fastapi_auth_api_key/__manifest__.py

@@ -3,7 +3,7 @@
 
 {
     "name": "Fastapi Auth API Key",
-    "version": "18.0.1.0.0",
+    "version": "19.0.1.0.0",
     "category": "Others",
     "website": "https://github.com/OCA/rest-framework",
     "author": "Camptocamp, Odoo Community Association (OCA)",

fastapi_auth_api_key/dependencies.py

@@ -3,17 +3,18 @@
 import os
 from typing import Annotated
 
+from fastapi.exceptions import HTTPException
+
 from odoo import SUPERUSER_ID
 from odoo.api import Environment
 from odoo.exceptions import ValidationError
 
 from odoo.addons.auth_api_key.models.auth_api_key import AuthApiKey
-from odoo.addons.base.models.res_partner import Partner
+from odoo.addons.base.models.res_partner import ResPartner
 from odoo.addons.fastapi.dependencies import fastapi_endpoint, odoo_env
 from odoo.addons.fastapi.models.fastapi_endpoint import FastapiEndpoint
 
 from fastapi import Depends, status
-from fastapi.exceptions import HTTPException
 from fastapi.security import APIKeyHeader
 
 HTTP_API_KEY_HEADER = os.environ.get("FASTAPI_AUTH_HTTP_API_KEY_HEADER", "HTTP-API-KEY")
@@ -27,8 +28,10 @@ def authenticated_auth_api_key(
     if not key:
         raise HTTPException(
             status_code=status.HTTP_401_UNAUTHORIZED,
-            detail=env._("Missing %(HTTP_API_KEY_HEADER)s header")
-            % {"HTTP_API_KEY_HEADER": HTTP_API_KEY_HEADER},
+            detail=env._(
+                "Missing %(HTTP_API_KEY_HEADER)s header",
+                HTTP_API_KEY_HEADER=HTTP_API_KEY_HEADER,
+            ),
             headers={"WWW-Authenticate": HTTP_API_KEY_HEADER},
         )
     admin_env = Environment(env.cr, SUPERUSER_ID, {})
@@ -55,7 +58,7 @@ def authenticated_auth_api_key(
 
 def authenticated_partner_by_api_key(
     auth_api_key: Annotated[AuthApiKey, Depends(authenticated_auth_api_key)],
-) -> Partner:
+) -> ResPartner:
     return auth_api_key.user_id.partner_id
 
 

fastapi_auth_api_key/readme/CONTRIBUTORS.md

@@ -1,2 +1,3 @@
 - Matthieu Méquignon \<<matthieu.mequignon@camptocamp.com>\>
 - Son Ho \<<sonhd@trobz.com>\>
+- Giuseppe Ardoselli \<<agiuseppe28@gmail.com>\>

fastapi_auth_api_key/tests/test_fastapi_api_key_dependencies.py

@@ -1,3 +1,5 @@
+from fastapi.exceptions import HTTPException
+
 from odoo.tests import tagged
 from odoo.tests.common import TransactionCase
 
@@ -7,16 +9,20 @@
     authenticated_partner_by_api_key,
 )
 
-from fastapi.exceptions import HTTPException
-
 
 @tagged("-at_install", "post_install")
 class TestFastapiAuthApiKey(TransactionCase):
     @classmethod
     def setUpClass(cls):
         super().setUpClass()
         cls.env = cls.env(context=dict(cls.env.context, tracking_disable=True))
-        demo_user = cls.env.ref("base.user_demo")
+        # Odoo 19 no longer loads demo data by default, so we cannot rely on
+        # ``base.user_demo``; create a dedicated user instead.
+        demo_user = (
+            cls.env["res.users"]
+            .with_context(no_reset_password=True)
+            .create({"name": "Endpoint Demo User", "login": "endpoint_demo_user"})
+        )
         cls.demo_env = demo_user.with_user(demo_user).env
         cls.demo_endpoint = cls.env["fastapi.endpoint"].create(
             {
@@ -79,7 +85,7 @@ def test_authenticated_auth_api_key_no_api_key_found(self):
         # An exception is raised when no api key record is found
         with self.assertRaises(HTTPException) as error:
             authenticated_auth_api_key("404", self.demo_env, self.demo_endpoint)
-        self.assertEqual(error.exception.detail, ("The key 404 is not allowed",))
+        self.assertEqual(error.exception.detail, ("The key '404' is not allowed",))
 
     def test_authenticated_auth_api_key_unauthorized_key(self):
         # An exception is raised when unauthorized api key record is found