Skip to content

Conversation

@giles17
Copy link
Contributor

@giles17 giles17 commented Jan 27, 2026

Motivation and Context

Addresses #3356 - improving unit test coverage for the core observability module.

Description

Adds unit tests for observability module. Coverage: 72% to 86%.

Contribution Checklist

  • The code builds clean without any errors or warnings
  • The PR follows the Contribution Guidelines
  • All unit tests pass, and I have added new tests where possible
  • Is this a breaking change? No

Copilot AI review requested due to automatic review settings January 27, 2026 20:34
@github-actions github-actions bot changed the title Add observability unit tests to improve coverage from 72% to 86% Python: Add observability unit tests to improve coverage from 72% to 86% Jan 27, 2026
@markwallace-microsoft
Copy link
Member

markwallace-microsoft commented Jan 27, 2026

Python Test Coverage

Python Test Coverage Report •
FileStmtsMissCoverMissing
TOTAL16171232385% 
report-only-changed-files is enabled. No files were changed during this commit :)

Python Unit Test Overview

Tests Skipped Failures Errors Time
3517 234 💤 0 ❌ 0 🔥 1m 6s ⏱️

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request adds comprehensive unit tests for the observability module to improve test coverage from 72% to 86%, addressing issue #3356. The new tests cover a wide range of observability functionality including OTel attributes, metric views, instrumentation decorators, span creation, exception handling, and various utility functions.

Changes:

  • Added 1,169 lines of new test code covering observability features
  • Tests include coverage for agent instrumentation, chat client telemetry, workflow spans, and configuration
  • Tests for both enabled and disabled instrumentation scenarios

@giles17 giles17 changed the title Python: Add observability unit tests to improve coverage from 72% to 86% Python: Add observability unit tests to improve coverage Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants