Skip to content

Add direct unit tests for app/services/datadog/client.py #878

@VaibhavUpreti

Description

@VaibhavUpreti

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

  1. Add a dedicated DatadogClient service test module.
  2. Cover search_logs(), list_monitors(), and query_events() or equivalent event method names.
  3. 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

Metadata

Metadata

Assignees

Labels

reliabilityImproves stability and failure handlingtestsTest coverage and test organization

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions