fix pylint

Author: bisgaard-itis

services/api-server/src/simcore_service_api_server/exceptions/service_errors_utils.py

@@ -1,3 +1,4 @@
+# pylint: disable=dangerous-default-value
 import asyncio
 import logging
 from collections.abc import Callable, Coroutine, Mapping
@@ -164,6 +165,8 @@ def service_exception_mapper(
     [Callable[Concatenate[Self, P], Coroutine[Any, Any, R]]],
     Callable[Concatenate[Self, P], Coroutine[Any, Any, R]],
 ]:
+    assert len(http_status_map.keys()) > 0 or len(rpc_exception_map.keys()) > 0
+
     def _decorator(member_func: Callable[Concatenate[Self, P], Coroutine[Any, Any, R]]):
         _assert_correct_kwargs(
             func=member_func,

services/api-server/tests/unit/test_exceptions.py

@@ -1,7 +1,7 @@
 # pylint: disable=unused-variable
 # pylint: disable=unused-argument
 # pylint: disable=redefined-outer-name
-
+# pylint: disable=too-many-arguments
 
 from http import HTTPStatus
 from uuid import UUID
@@ -25,8 +25,8 @@
 
 async def test_backend_service_exception_mapper():
     @service_exception_mapper(
-        "DummyService",
-        {status.HTTP_400_BAD_REQUEST: ProfileNotFoundError},
+        service_name="DummyService",
+        http_status_map={status.HTTP_400_BAD_REQUEST: ProfileNotFoundError},
     )
     async def my_endpoint(status_code: int):
         raise HTTPStatusError(
@@ -103,26 +103,26 @@ async def coro1(project_id):
         pass
 
     with pytest.raises(AssertionError):
-        _assert_correct_kwargs(func=coro1, status_map=status_map)
+        _assert_correct_kwargs(func=coro1, exception_types=set(status_map.values()))
 
     async def coro2(project_id=UUID("9c201eb7-ba04-4d9b-abe6-f16b406ca86d")):
         pass
 
     with pytest.raises(AssertionError) as exc:
-        _assert_correct_kwargs(func=coro2, status_map=status_map)
+        _assert_correct_kwargs(func=coro2, exception_types=set(status_map.values()))
 
     async def coro3(*, project_id):
         pass
 
-    _assert_correct_kwargs(func=coro3, status_map=status_map)
+    _assert_correct_kwargs(func=coro3, exception_types=set(status_map.values()))
 
     async def coro4(*, project_id=UUID("ce56af2e-e9e5-46a4-8067-662077de5528")):
         pass
 
-    _assert_correct_kwargs(func=coro4, status_map=status_map)
+    _assert_correct_kwargs(func=coro4, exception_types=set(status_map.values()))
 
     async def coro5(*, project_uuid):
         pass
 
     with pytest.raises(AssertionError):
-        _assert_correct_kwargs(func=coro5, status_map=status_map)
+        _assert_correct_kwargs(func=coro5, exception_types=set(status_map.values()))

services/api-server/tests/unit/test_licensed_items.py

@@ -1,3 +1,7 @@
+# pylint: disable=redefined-outer-name
+# pylint: disable=unused-argument
+# pylint: disable=unused-variable
+# pylint: disable=too-many-arguments
 import asyncio
 
 import pytest