AI-Cognitive-Leap/agnes-the-ai-analyst
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(tests): drop stale 'healthy' from /api/health status assert

Browse source 
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.
This commit is contained in:
ZdenekSrotyr 2026-05-03 15:40:41 +02:00
parent f348296685
commit c54917fc50
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
tests/test_e2e_docker.py
View file

@ -68,11 +68,13 @@ def docker_env():
class TestDockerHealth: class TestDockerHealth:
def test_health_endpoint(self, docker_env): 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 import httpx
resp = httpx.get(f"{docker_env}/api/health") resp = httpx.get(f"{docker_env}/api/health")
assert resp.status_code == 200 assert resp.status_code == 200
data = resp.json() data = resp.json()
assert data.get("status") in ("ok", "healthy") assert data.get("status") == "ok"
def test_health_has_duckdb(self, docker_env): def test_health_has_duckdb(self, docker_env):
# /api/health touches system.duckdb to read schema_version, so # /api/health touches system.duckdb to read schema_version, so