Skip to content

Commit f9ff8a7

Browse files
dittopsclaude
andcommitted
fix(tests): update cluster health test mock to use correct response format
The health.py action expects response in format {"data": {...}} but the tests were mocking {"health": {...}}. This caused the test to fail because the code would not find any health data and mark all checks as unavailable. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent a1a1606 commit f9ff8a7

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

services/budpipeline/tests/actions/cluster/test_cluster_actions.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ async def test_execute_success(self) -> None:
7373
"invoke_service",
7474
new_callable=AsyncMock,
7575
return_value={
76-
"health": {
76+
"data": {
7777
"nodes": {"healthy": True, "message": "All nodes ready"},
7878
"api": {"healthy": True, "message": "API server responding"},
7979
}
@@ -97,7 +97,7 @@ async def test_execute_degraded(self) -> None:
9797
"invoke_service",
9898
new_callable=AsyncMock,
9999
return_value={
100-
"health": {
100+
"data": {
101101
"nodes": {"healthy": False, "message": "2 nodes not ready"},
102102
"api": {"healthy": True, "message": "API server responding"},
103103
}

0 commit comments

Comments
 (0)