Why This Matters
Datadog is one of the core integrations, but there is no direct DatadogClient test module under tests/services/.
Read First
app/services/datadog/client.py
tests/tools/test_datadog_logs_tool.py
tests/tools/test_datadog_monitors_tool.py
tests/tools/test_datadog_events_tool.py
Scope
- Add a dedicated
DatadogClient service test module.
- Cover
search_logs(), list_monitors(), and query_events() or equivalent event method names.
- Keep the tests at the service layer, not the tool layer.
Tests / Coverage
tests/services/test_datadog_client.py
- cover success and HTTP failure for logs, monitors, and events
- cover empty
data arrays
- cover auth/configured checks if present
Acceptance Criteria
- Datadog client logic is directly protected by tests
- no tool test has to implicitly prove client behavior
- no real Datadog credentials are needed
Why This Matters
Datadog is one of the core integrations, but there is no direct
DatadogClienttest module undertests/services/.Read First
app/services/datadog/client.pytests/tools/test_datadog_logs_tool.pytests/tools/test_datadog_monitors_tool.pytests/tools/test_datadog_events_tool.pyScope
DatadogClientservice test module.search_logs(),list_monitors(), andquery_events()or equivalent event method names.Tests / Coverage
tests/services/test_datadog_client.pydataarraysAcceptance Criteria