From 29ff7940f6e40997efe824760e560345f97016e4 Mon Sep 17 00:00:00 2001 From: Cursor Agent Date: Sat, 7 Mar 2026 04:01:43 +0000 Subject: [PATCH] fix: accept both 'healthy' and 'connected' in health check test The test_health_and_chat_completion test runs against both source builds (which return 'healthy') and pip-installed versions (which may return 'connected'). Accept both values. Co-authored-by: Ishaan Jaff --- tests/basic_proxy_startup_tests/test_basic_proxy_startup.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/basic_proxy_startup_tests/test_basic_proxy_startup.py b/tests/basic_proxy_startup_tests/test_basic_proxy_startup.py index c330895a578..218fbae71db 100644 --- a/tests/basic_proxy_startup_tests/test_basic_proxy_startup.py +++ b/tests/basic_proxy_startup_tests/test_basic_proxy_startup.py @@ -23,7 +23,9 @@ async def test_health_and_chat_completion(): async with session.get("http://0.0.0.0:4000/health/readiness") as response: assert response.status == 200 readiness_response = await response.json() - assert readiness_response["status"] == "healthy" + # Accept both "healthy" (new format) and "connected" (legacy format) + # since this test runs against both source builds and pip-installed versions + assert readiness_response["status"] in ("healthy", "connected") # Test liveness endpoint async with session.get("http://0.0.0.0:4000/health/liveness") as response: