Description
The recent Kestra integration (PR #37) has zero test coverage. Critical functionality is untested.
Untested Components
-
Kestra Client (kestra_client.py)
- Flow triggering
- Status checking
- Error handling
-
Webhook Endpoints (webhooks.py)
- Task update handling
- Execution update handling
- WebSocket broadcasting
-
WebSocket Manager (websocket_manager.py)
- Connection management
- Broadcast functionality
- State tracking
Required Tests
Unit Tests
# backend/tests/test_kestra_client.py
async def test_trigger_flow_success()
async def test_trigger_flow_offline()
async def test_get_execution_status()
# backend/tests/test_webhooks.py
async def test_task_update_webhook()
async def test_execution_update_webhook()
async def test_webhook_broadcasts_to_websocket()
# backend/tests/test_websocket_manager.py
async def test_connection_lifecycle()
async def test_broadcast_to_multiple_clients()
async def test_execution_status_tracking()
Integration Tests
Acceptance Criteria
Location
Related Issues
Description
The recent Kestra integration (PR #37) has zero test coverage. Critical functionality is untested.
Untested Components
Kestra Client (
kestra_client.py)Webhook Endpoints (
webhooks.py)WebSocket Manager (
websocket_manager.py)Required Tests
Unit Tests
Integration Tests
Acceptance Criteria
KestraClient(>80% coverage)ConnectionManagerLocation
backend/tests/test_kestra_client.pybackend/tests/test_webhooks.pybackend/tests/test_websocket_manager.pyRelated Issues