mirror of
https://github.com/LukeHagar/connexion.git
synced 2025-12-06 12:27:45 +00:00
1beb0539f5
Fixes #1682 This PR inspects the parameters of a Middleware class before passing in the `lifespan` keyword argument. I was doubting to check the class instead (`isinstance(LifespanMiddleware)`), but that's less flexible if users want to write custom middleware to handle the lifespan.
52 lines
1.4 KiB
Python
52 lines
1.4 KiB
Python
import sys
|
|
from unittest import mock
|
|
|
|
import pytest
|
|
from connexion.middleware import ConnexionMiddleware
|
|
from starlette.datastructures import MutableHeaders
|
|
|
|
from conftest import build_app_from_fixture
|
|
|
|
|
|
class TestMiddleware:
|
|
"""Middleware to check if operation is accessible on scope."""
|
|
|
|
__test__ = False
|
|
|
|
def __init__(self, app):
|
|
self.app = app
|
|
|
|
async def __call__(self, scope, receive, send):
|
|
operation_id = scope["extensions"]["connexion_routing"]["operation_id"]
|
|
|
|
async def patched_send(message):
|
|
if message["type"] != "http.response.start":
|
|
await send(message)
|
|
return
|
|
|
|
message.setdefault("headers", [])
|
|
headers = MutableHeaders(scope=message)
|
|
headers["operation_id"] = operation_id
|
|
|
|
await send(message)
|
|
|
|
await self.app(scope, receive, patched_send)
|
|
|
|
|
|
@pytest.fixture(scope="session")
|
|
def middleware_app(spec, app_class):
|
|
middlewares = ConnexionMiddleware.default_middlewares + [TestMiddleware]
|
|
return build_app_from_fixture(
|
|
"simple", app_class=app_class, spec_file=spec, middlewares=middlewares
|
|
)
|
|
|
|
|
|
def test_routing_middleware(middleware_app):
|
|
app_client = middleware_app.test_client()
|
|
|
|
response = app_client.post("/v1.0/greeting/robbe")
|
|
|
|
assert (
|
|
response.headers.get("operation_id") == "fakeapi.hello.post_greeting"
|
|
), response.status_code
|