Skip to content

Add direct unit tests for app/services/cloudwatch_client.py #885

@VaibhavUpreti

Description

@VaibhavUpreti

Why This Matters

CloudWatch helpers are reused by AWS tools, but the service functions themselves are untested directly.

Read First

  • app/services/cloudwatch_client.py
  • tests/tools/test_cloudwatch_logs_tool.py

Scope

  1. Add a direct test module for metric and log helper functions.
  2. Mock make_boto3_client() and require_aws_credentials().
  3. Keep tool-registration assertions out of this test module.

Tests / Coverage

  • tests/services/test_cloudwatch_client.py
  • cover get_metric_statistics()
  • cover filter_log_events()
  • cover get_log_events()
  • cover missing boto3 and missing credentials flows

Acceptance Criteria

  • CloudWatch service functions have direct tests
  • no tool tests are needed to prove service behavior
  • the test names make each helper's contract obvious

Metadata

Metadata

Assignees

No one assigned

    Labels

    good first issueGood for newcomerstestsTest coverage and test organization

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions