faster webserver with_dbs tests (ITISFoundation#2467)

Co-authored-by: Andrei Neagu <[email protected]>
Co-authored-by: Sylvain <[email protected]>

Author: 3 people

services/web/server/src/simcore_service_webserver/rest.py

@@ -6,6 +6,7 @@
 
 """
 import logging
+from functools import lru_cache
 from pathlib import Path
 from typing import Optional
 
@@ -14,7 +15,6 @@
 from aiohttp import web
 from aiohttp_swagger import setup_swagger
 from openapi_core.schema.specs.models import Spec as OpenApiSpecs
-
 from servicelib import openapi
 from servicelib.application_setup import ModuleCategory, app_module_setup
 from servicelib.rest_middlewares import (
@@ -25,8 +25,8 @@
 
 from . import rest_routes
 from ._meta import api_version_prefix
-from .rest_config import APP_OPENAPI_SPECS_KEY, assert_valid_config
 from .diagnostics_config import get_diagnostics_config
+from .rest_config import APP_OPENAPI_SPECS_KEY, assert_valid_config
 
 log = logging.getLogger(__name__)
 
@@ -38,6 +38,7 @@ def get_openapi_specs_path(api_version_dir: Optional[str] = None) -> Path:
     return resources.get_path(f"api/{api_version_dir}/openapi.yaml")
 
 
+@lru_cache  # required to boost tests speed, gains 3.5s per test
 def load_openapi_specs(spec_path: Optional[Path] = None) -> OpenApiSpecs:
     if spec_path is None:
         spec_path = get_openapi_specs_path()
@@ -116,4 +117,4 @@ def setup(app: web.Application, *, swagger_doc_enabled: bool = True):
 # alias
 setup_rest = setup
 
-__all__ = "setup_rest"
+__all__ = ["setup_rest"]

services/web/server/tests/unit/with_dbs/conftest.py

@@ -17,6 +17,7 @@
 from importlib import reload
 from pathlib import Path
 from typing import Callable, Dict, Iterator, List
+from unittest.mock import patch
 from uuid import uuid4
 
 import aioredis
@@ -33,9 +34,10 @@
 from pytest_simcore.helpers.utils_login import NewUser
 from servicelib.aiopg_utils import DSN
 from servicelib.application_keys import APP_CONFIG_KEY, APP_DB_ENGINE_KEY
+from simcore_service_webserver import rest
 from simcore_service_webserver.application import create_application
 from simcore_service_webserver.application_config import app_schema as app_schema
-from simcore_service_webserver.constants import APP_DB_ENGINE_KEY, INDEX_RESOURCE_NAME
+from simcore_service_webserver.constants import INDEX_RESOURCE_NAME
 from simcore_service_webserver.groups_api import (
     add_user_in_group,
     create_user_group,
@@ -51,6 +53,17 @@
 # DEPLOYED SERVICES FOR TESTSUITE SESSION -----------------------------------
 
 
+@pytest.fixture(autouse=True)
+def disable_swagger_doc_genertion() -> None:
+    """
+    by not enabling the swagger documentation, 1.8s per test is gained
+    """
+    with patch.dict(
+        rest.setup.__wrapped__.__kwdefaults__, {"swagger_doc_enabled": False}
+    ):
+        yield
+
+
 @pytest.fixture(scope="session")
 def default_app_cfg(osparc_simcore_root_dir):
     # NOTE: ONLY used at the session scopes