diff options
| author | Christian Cleberg <[email protected]> | 2026-04-11 12:34:11 -0500 |
|---|---|---|
| committer | Christian Cleberg <[email protected]> | 2026-04-11 12:34:11 -0500 |
| commit | 1878dff520ee2e424b088a963c8f6417f5106ce9 (patch) | |
| tree | df094ec09d8b26bac56c08d1408a2fab35ac9e7d /tests/test_contributions_api.py | |
| parent | acbff854f2da96bddcaede1385e7fefeba0fb34b (diff) | |
| download | hutch-stats-1878dff520ee2e424b088a963c8f6417f5106ce9.tar.gz hutch-stats-1878dff520ee2e424b088a963c8f6417f5106ce9.tar.bz2 hutch-stats-1878dff520ee2e424b088a963c8f6417f5106ce9.zip | |
feat: harden contribution api for production use
Diffstat (limited to 'tests/test_contributions_api.py')
| -rw-r--r-- | tests/test_contributions_api.py | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/tests/test_contributions_api.py b/tests/test_contributions_api.py index d90aceb..e221602 100644 --- a/tests/test_contributions_api.py +++ b/tests/test_contributions_api.py @@ -6,14 +6,18 @@ from srht_contrib.main import create_app from srht_contrib.models import ContributionEvent -def test_api_routes_require_api_key(settings, db_engine, session_factory) -> None: +def test_read_only_contribution_routes_are_public_and_write_routes_require_api_key(settings, db_engine, session_factory) -> None: app = create_app(settings, engine=db_engine, session_factory=session_factory) with TestClient(app) as open_client: response = open_client.get("/health") + public_contributions = open_client.get("/api/contributions/~ccleberg?from=2026-03-28&to=2026-03-30") + public_stats = open_client.get("/api/contributions/~ccleberg/stats?from=2026-03-28&to=2026-03-30") assert response.status_code == 200 + assert public_contributions.status_code == 200 + assert public_stats.status_code == 200 with TestClient(app) as unauthorized: - unauthorized_response = unauthorized.get("/api/contributions/~ccleberg?from=2026-03-28&to=2026-03-30") + unauthorized_response = unauthorized.post("/api/contributions/poll?actor=~ccleberg") assert unauthorized_response.status_code == 401 with TestClient(app) as invalid: |
