fix test

services/web/server/src/simcore_service_webserver/catalog/_handlers.py

@@ -5,6 +5,7 @@
 
 """
 import logging
+import urllib.parse
 from typing import Any, Iterator
 
 import orjson
@@ -17,7 +18,7 @@
 )
 from models_library.users import UserID
 from pint import UnitRegistry
-from pydantic import BaseModel, Extra, Field
+from pydantic import BaseModel, Extra, Field, validator
 from servicelib.aiohttp.requests_validation import (
     handle_validation_as_http_error,
     parse_request_path_parameters_as,
@@ -81,6 +82,14 @@ class Config:
         allow_population_by_field_name = True
         extra = Extra.forbid
 
+    @validator("service_key", pre=True)
+    @classmethod
+    def ensure_unquoted(cls, v):
+        # NOTE: this is needed as in pytest mode, the aiohttp server does not seem to unquote automatically
+        if v is not None:
+            return urllib.parse.unquote(v)
+        return v
+
 
 @routes.get(f"{VTAG}/catalog/services")
 @login_required
@@ -284,7 +293,6 @@ async def get_service_resources_handler(request: Request):
     """
     ctx = _RequestContext.create(request)
     path_params = parse_request_path_parameters_as(_ServicePathParams, request)
-
     service_resources: ServiceResourcesDict = await client.get_service_resources(
         request.app,
         user_id=ctx.user_id,

services/web/server/tests/unit/with_dbs/01/test_catalog_api.py

@@ -3,6 +3,7 @@
 # pylint:disable=redefined-outer-name
 
 import re
+import urllib.parse
 
 import pytest
 from aiohttp import web
@@ -118,7 +119,7 @@ async def test_get_service_resources(
     assert client.app
     assert client.app.router
     url = client.app.router["get_service_resources_handler"].url_for(
-        service_key="simcore%2Fservices%2Fdynamic%2Fsomeservice",
+        service_key=urllib.parse.quote("simcore/services/dynamic/someservice", safe=""),
         service_version="3.4.5",
     )
     response = await client.get(f"{url}")