Skip to content

Conversation

@giles17
Copy link
Contributor

@giles17 giles17 commented Jan 27, 2026

Motivation and Context

Increases unit test coverage for the azure-ai package from 76% to 91%, exceeding the 85% target.
Resolves #3353

Description

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? If yes, add "[BREAKING]" prefix to the title of the PR.

Copilot AI review requested due to automatic review settings January 27, 2026 06:45
@github-actions github-actions bot changed the title Improved AzureAI Package Test Coverage Python: Improved AzureAI Package Test Coverage Jan 27, 2026
@markwallace-microsoft
Copy link
Member

markwallace-microsoft commented Jan 27, 2026

Python Test Coverage

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

Python Unit Test Overview

Tests Skipped Failures Errors Time
3429 213 💤 0 ❌ 0 🔥 1m 7s ⏱️

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

Increases unit test coverage for the python/packages/azure-ai package by adding new tests around shared conversion utilities and client/provider behaviors (notably MCP tooling and Azure Monitor configuration).

Changes:

  • Added a new test_shared.py suite covering _shared.py conversion helpers (tools, response_format, MCP config).
  • Expanded tests for AzureAIClient.configure_azure_monitor and MCP tool preparation logic.
  • Added coverage for provider/tool merging and additional Azure AI agent client edge cases.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
python/packages/azure-ai/tests/test_shared.py New tests for _shared conversion functions (tools, MCP, response format).
python/packages/azure-ai/tests/test_provider.py Adds a test ensuring tool-merge behavior skips dict function tools while preserving hosted tools.
python/packages/azure-ai/tests/test_azure_ai_client.py Adds tests for Azure Monitor configuration paths and MCP tool preparation.
python/packages/azure-ai/tests/test_azure_ai_agent_client.py Adds tests for response_format handling, tool resources, MCP resources, message preparation, and tool preparation.
python/packages/azure-ai/tests/test_agent_provider.py Adds tests validating dict-format function tools require implementations and related integration-test import refactors.

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.

Python: azure-ai package unit test coverage tracking

2 participants