fixes tests

Author: pcrespov

services/web/server/src/simcore_service_webserver/exporter/plugin.py

@@ -16,10 +16,9 @@
     logger=_logger,
 )
 def setup_exporter(app: web.Application) -> bool:
+    setup_scicrunch(app)
 
     # Rest-API routes: maps handlers with routes tags with "viewer" based on OAS operation_id
     app.router.add_routes(_handlers.routes)
 
-    setup_scicrunch(app)
-
     return True

services/web/server/src/simcore_service_webserver/scicrunch/settings.py

@@ -1,3 +1,5 @@
+from typing import Annotated
+
 from aiohttp import web
 from pydantic import Field, HttpUrl, SecretStr, TypeAdapter
 from settings_library.base import BaseCustomSettings
@@ -10,21 +12,17 @@
 
 class SciCrunchSettings(BaseCustomSettings):
 
-    SCICRUNCH_API_BASE_URL: HttpUrl = Field(
-        default=TypeAdapter(HttpUrl).validate_python(f"{SCICRUNCH_DEFAULT_URL}/api/1"),
-        description="Base url to scicrunch API's entrypoint",
-    )
+    SCICRUNCH_API_BASE_URL: Annotated[
+        HttpUrl, Field(description="Base url to scicrunch API's entrypoint")
+    ] = TypeAdapter(HttpUrl).validate_python(f"{SCICRUNCH_DEFAULT_URL}/api/1")
 
     # NOTE: Login in https://scicrunch.org and get API Key under My Account -> API Keys
     # WARNING: this needs to be setup in osparc-ops before deploying
     SCICRUNCH_API_KEY: SecretStr
 
-    SCICRUNCH_RESOLVER_BASE_URL: HttpUrl = Field(
-        default=TypeAdapter(HttpUrl).validate_python(
-            f"{SCICRUNCH_DEFAULT_URL}/resolver"
-        ),
-        description="Base url to scicrunch resolver entrypoint",
-    )
+    SCICRUNCH_RESOLVER_BASE_URL: Annotated[
+        HttpUrl, Field(description="Base url to scicrunch resolver entrypoint")
+    ] = TypeAdapter(HttpUrl).validate_python(f"{SCICRUNCH_DEFAULT_URL}/resolver")
 
 
 def get_plugin_settings(app: web.Application) -> SciCrunchSettings:

services/web/server/tests/integration/01/test_exporter_requests_handlers.py

@@ -14,6 +14,7 @@
 import pytest
 import redis.asyncio as aioredis
 from aiohttp.test_utils import TestClient
+from pytest_simcore.helpers.monkeypatch_envs import setenvs_from_dict
 from pytest_simcore.helpers.webserver_login import LoggedUser, UserInfoDict
 from pytest_simcore.helpers.webserver_projects import (
     create_project,
@@ -152,6 +153,7 @@ async def client(
     redis_client: aioredis.Redis,
     rabbit_service: RabbitSettings,
     simcore_services_ready: None,
+    monkeypatch: pytest.MonkeyPatch,
 ) -> Iterable[TestClient]:
     # test config & env vars ----------------------
     cfg = deepcopy(app_config)
@@ -163,11 +165,20 @@ async def client(
 
     monkeypatch_setenv_from_app_config(cfg)
 
+    monkeypatch.delenv("WEBSERVER_SCICRUNCH", raising=False)
+    setenvs_from_dict(
+        monkeypatch, {"SCICRUNCH_API_KEY": "REPLACE_ME_with_valid_api_key"}
+    )
+
     # app setup ----------------------------------
     app = create_safe_application(cfg)
 
     # activates only security+restAPI sub-modules
-    assert setup_settings(app)
+    app_settings = setup_settings(app)
+
+    assert app_settings.WEBSERVER_SCICRUNCH is not None
+    assert app_settings.WEBSERVER_RABBITMQ is not None
+    assert app_settings.WEBSERVER_EXPORTER is not None
     assert (
         exporter_settings.get_plugin_settings(app) is not None
     ), "Should capture defaults"