From c54917fc50f4fd175e08349fba25f8a40a546c1b Mon Sep 17 00:00:00 2001 From: ZdenekSrotyr Date: Sun, 3 May 2026 15:40:41 +0200 Subject: [PATCH] fix(tests): drop stale 'healthy' from /api/health status assert Per Devin review on #166: /api/health returns 'ok' or 'unhealthy'; 'healthy' is the detailed endpoint's vocabulary (app/api/health.py:180). The pre-existing OR-tuple was dead code and inconsistent with the rest of this PR's alignment work. --- tests/test_e2e_docker.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/test_e2e_docker.py b/tests/test_e2e_docker.py index 5fb4b9a..51959c7 100644 --- a/tests/test_e2e_docker.py +++ b/tests/test_e2e_docker.py @@ -68,11 +68,13 @@ def docker_env(): class TestDockerHealth: def test_health_endpoint(self, docker_env): + # /api/health returns 'ok' or 'unhealthy' (never 'healthy' — that's + # the detailed endpoint's vocabulary). See app/api/health.py:111-118. import httpx resp = httpx.get(f"{docker_env}/api/health") assert resp.status_code == 200 data = resp.json() - assert data.get("status") in ("ok", "healthy") + assert data.get("status") == "ok" def test_health_has_duckdb(self, docker_env): # /api/health touches system.duckdb to read schema_version, so