Skip to content

Add direct unit tests for app/services/tracer_client/tracer_logs.py #890

@VaibhavUpreti

Description

@VaibhavUpreti

Why This Matters

this mixin is simple but still easy to break because it returns raw dicts.

Read First

  • app/services/tracer_client/tracer_logs.py

Scope

  1. Add direct tests for parameter construction.
  2. Test both the trace_id and run_id call paths.
  3. Keep the tests small and focused.

Tests / Coverage

  • tests/services/tracer_client/test_tracer_logs.py
  • cover default size
  • cover trace_id precedence
  • cover run_id fallback

Acceptance Criteria

  • parameter-building behavior is covered directly
  • the mixin can be refactored safely later
  • the tests stay lightweight

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