|
| 1 | +from unittest.mock import MagicMock, patch |
| 2 | + |
| 3 | +import pytest |
| 4 | +from datacommons_mcp.middleware import APIKeyMiddleware |
| 5 | +from starlette.applications import Starlette |
| 6 | +from starlette.middleware import Middleware |
| 7 | +from starlette.responses import JSONResponse |
| 8 | +from starlette.routing import Route |
| 9 | +from starlette.testclient import TestClient |
| 10 | + |
| 11 | + |
| 12 | +async def homepage(request): # noqa: ARG001 |
| 13 | + return JSONResponse({"status": "ok"}) |
| 14 | + |
| 15 | + |
| 16 | +@pytest.fixture |
| 17 | +def client(): |
| 18 | + # Define a simple Starlette app with the middleware |
| 19 | + middleware = [Middleware(APIKeyMiddleware)] |
| 20 | + routes = [Route("/", homepage)] |
| 21 | + app = Starlette(routes=routes, middleware=middleware) |
| 22 | + return TestClient(app) |
| 23 | + |
| 24 | + |
| 25 | +def test_api_key_header_present(client): |
| 26 | + """Verify use_api_key is called when X-API-Key header is present.""" |
| 27 | + with patch("datacommons_mcp.middleware.use_api_key") as mock_use_api_key: |
| 28 | + # Configuration for the mock context manager |
| 29 | + mock_context_manager = MagicMock() |
| 30 | + mock_use_api_key.return_value.__enter__.return_value = mock_context_manager |
| 31 | + |
| 32 | + headers = {"X-API-Key": "test-key-123"} |
| 33 | + response = client.get("/", headers=headers) |
| 34 | + |
| 35 | + assert response.status_code == 200 |
| 36 | + mock_use_api_key.assert_called_once_with("test-key-123") |
| 37 | + # Ensure the context manager was actually entered |
| 38 | + mock_use_api_key.return_value.__enter__.assert_called_once() |
| 39 | + mock_use_api_key.return_value.__exit__.assert_called_once() |
| 40 | + |
| 41 | + |
| 42 | +def test_api_key_header_missing(client): |
| 43 | + """Verify use_api_key is NOT called when X-API-Key header is missing.""" |
| 44 | + with patch("datacommons_mcp.middleware.use_api_key") as mock_use_api_key: |
| 45 | + response = client.get("/") |
| 46 | + |
| 47 | + assert response.status_code == 200 |
| 48 | + mock_use_api_key.assert_not_called() |
| 49 | + |
| 50 | + |
| 51 | +def test_api_key_header_empty(client): |
| 52 | + """Verify use_api_key is NOT called when X-API-Key header is empty.""" |
| 53 | + with patch("datacommons_mcp.middleware.use_api_key") as mock_use_api_key: |
| 54 | + headers = {"X-API-Key": ""} |
| 55 | + response = client.get("/", headers=headers) |
| 56 | + |
| 57 | + assert response.status_code == 200 |
| 58 | + mock_use_api_key.assert_not_called() |
0 commit comments