@sanderegg review: rm session scope

Author: pcrespov

api/specs/web-server/_common.py

@@ -58,7 +58,7 @@ def as_query(model_class: type[BaseModel]) -> type[BaseModel]:
             "title": field_info.title,
             "description": field_info.description,
             "metadata": field_info.metadata,
-            "json_schema_extra": field_info.json_schema_extra or {},
+            "json_schema_extra": field_info.json_schema_extra,
         }
 
         annotation = _replace_basemodel_in_annotation(

packages/pytest-simcore/src/pytest_simcore/openapi_specs.py

@@ -27,7 +27,7 @@ class Entrypoint(NamedTuple):
     path: str
 
 
-@pytest.fixture
+@pytest.fixture(scope="session")
 def openapi_specs_path() -> Path:
     # NOTE: cannot be defined as a session scope because it is designed to be overriden
     pytest.fail(reason="Must be overriden in caller test suite")
@@ -53,7 +53,7 @@ def _load(file: Path, base_uri: str = "") -> dict:
     return data
 
 
-@pytest.fixture
+@pytest.fixture(scope="session")
 def openapi_specs(openapi_specs_path: Path) -> dict[str, Any]:
     assert openapi_specs_path.is_file()
     openapi: dict[str, Any] = _load(
@@ -62,7 +62,7 @@ def openapi_specs(openapi_specs_path: Path) -> dict[str, Any]:
     return deepcopy(openapi)
 
 
-@pytest.fixture
+@pytest.fixture(scope="session")
 def openapi_specs_entrypoints(
     openapi_specs: dict,
 ) -> set[Entrypoint]:

services/web/server/tests/unit/with_dbs/03/test__openapi_specs.py

@@ -18,7 +18,7 @@
 from simcore_service_webserver.rest._utils import get_openapi_specs_path
 
 
-@pytest.fixture
+@pytest.fixture(scope="session")
 def openapi_specs_path(api_version_prefix: str) -> Path:
     # overrides pytest_simcore.openapi_specs.app_openapi_specs_path fixture
     return get_openapi_specs_path(api_version_prefix)