ci: lint fix

Author: alukach

src/stac_auth_proxy/app.py

@@ -13,6 +13,7 @@
 
 from .config import Settings
 from .handlers import HealthzHandler, ReverseProxyHandler, SwaggerUI
+from .lifespan import lifespan
 from .middleware import (
     AddProcessTimeHeaderMiddleware,
     AuthenticationExtensionMiddleware,
@@ -25,7 +26,6 @@
     ProcessLinksMiddleware,
     RemoveRootPathMiddleware,
 )
-from .lifespan import lifespan
 
 logger = logging.getLogger(__name__)
 
@@ -135,6 +135,8 @@ def configure_app(app: FastAPI, settings: Optional[Settings] = None) -> FastAPI:
         )
 
     return app
+
+
 def create_app(settings: Optional[Settings] = None) -> FastAPI:
     """FastAPI Application Factory."""
     settings = settings or Settings()

src/stac_auth_proxy/lifespan.py

@@ -1,7 +1,7 @@
 """Reusable lifespan handler for FastAPI applications."""
 
-from contextlib import asynccontextmanager
 import logging
+from contextlib import asynccontextmanager
 from typing import Any
 
 from fastapi import FastAPI
@@ -13,7 +13,8 @@
 
 
 def lifespan(settings: Settings | None = None, **settings_kwargs: Any):
-    """Create a lifespan handler that runs startup checks.
+    """
+    Create a lifespan handler that runs startup checks.
 
     Parameters
     ----------
@@ -29,13 +30,13 @@ def lifespan(settings: Settings | None = None, **settings_kwargs: Any):
     Callable[[FastAPI], AsyncContextManager[Any]]
         A callable suitable for the ``lifespan`` parameter of ``FastAPI``.
     """
-
     if settings is None:
         settings = Settings(**settings_kwargs)
-    assert settings is not None
 
     @asynccontextmanager
     async def _lifespan(app: FastAPI):
+        assert settings is not None  # Required for type checking
+
         # Wait for upstream servers to become available
         if settings.wait_for_upstream:
             logger.info("Running upstream server health checks...")
@@ -62,4 +63,3 @@ async def _lifespan(app: FastAPI):
 
 
 __all__ = ["lifespan", "check_conformance", "check_server_health"]
-

tests/test_lifespan.py

@@ -9,11 +9,8 @@
 from starlette.middleware import Middleware
 from starlette.types import ASGIApp
 
-from stac_auth_proxy import (
-    check_conformance,
-    check_server_health,
-    lifespan as lifespan_handler,
-)
+from stac_auth_proxy import check_conformance, check_server_health
+from stac_auth_proxy import lifespan as lifespan_handler
 from stac_auth_proxy.utils.middleware import required_conformance
 
 
@@ -109,6 +106,4 @@ def test_lifespan_reusable():
             pass
         assert mock_health.await_count == 2
         expected_upstream = upstream_url.rstrip("/") + "/"
-        mock_conf.assert_awaited_once_with(
-            app.user_middleware, expected_upstream
-        )
+        mock_conf.assert_awaited_once_with(app.user_middleware, expected_upstream)