Skip to content

Python: Added Azure OpenAI Responses File Search sample + Integration test update#2645

Merged
eavanvalkenburg merged 2 commits intomicrosoft:mainfrom
giles17:aoai_file_search
Dec 5, 2025
Merged

Python: Added Azure OpenAI Responses File Search sample + Integration test update#2645
eavanvalkenburg merged 2 commits intomicrosoft:mainfrom
giles17:aoai_file_search

Conversation

@giles17
Copy link
Contributor

@giles17 giles17 commented Dec 4, 2025

Motivation and Context

Adds Azure OpenAI File Search sample and update integration tests because the file search feature now supports TokenCredential

Resolves #750

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 December 4, 2025 21:38
@markwallace-microsoft markwallace-microsoft added documentation Improvements or additions to documentation python labels Dec 4, 2025
@github-actions github-actions bot changed the title Added Azure OpenAI Responses File Search sample + Integration test update Python: Added Azure OpenAI Responses File Search sample + Integration test update Dec 4, 2025
@markwallace-microsoft
Copy link
Member

markwallace-microsoft commented Dec 4, 2025

Python Test Coverage

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

Python Unit Test Overview

Tests Skipped Failures Errors Time
2296 130 💤 0 ❌ 0 🔥 59.083s ⏱️

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 PR adds a new Azure OpenAI File Search sample that demonstrates using HostedFileSearchTool with the Azure OpenAI Responses Client. It also updates integration tests to remove skip markers now that file search supports TokenCredential authentication.

  • Adds a new sample demonstrating document-based question answering with file search
  • Updates integration tests to enable file search tests that were previously skipped
  • Corrects test comments to accurately describe file search functionality

Reviewed changes

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

File Description
python/samples/getting_started/agents/azure_openai/azure_responses_client_with_file_search.py New sample demonstrating Azure OpenAI Responses Client with HostedFileSearchTool for document-based question answering, including vector store creation and cleanup
python/samples/getting_started/agents/azure_openai/README.md Adds entry to the samples table documenting the new file search sample
python/packages/core/tests/azure/test_azure_responses_client.py Removes skip markers from file search tests now that TokenCredential is supported, and fixes comment from "web search" to "file search"

@eavanvalkenburg eavanvalkenburg added this pull request to the merge queue Dec 5, 2025
Merged via the queue into microsoft:main with commit f5f909f Dec 5, 2025
24 checks passed
arisng pushed a commit to arisng/agent-framework that referenced this pull request Feb 2, 2026
… test update (microsoft#2645)

* added azure responses file search sample + int test update

* sample update to use token credential
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Azure Responses Client File Search Tests Failing Due to API Key Authentication Requirement

5 participants