Update multipart file type variant tests#47750
Closed
dargilco wants to merge 78 commits into
Closed
Conversation
* sample update routines * Add test for routines samples and update sample skipping logic
* sample update routines * Add test for routines samples and update sample skipping logic
…agent_card, memory TTL timedelta) (#47658)
* Refactor skill zip handling and enhance RBAC setup options - Updated skill zip creation to use a new `zip` function for consistency. - Added environment variable support for storing agent responses. - Enhanced RBAC setup to allow skipping via an environment variable. - Introduced new test cases for hosted agents and skills with environment variable configurations. - Added binary assets for testing purposes. * test data * Update asset tag and remove unused environment variable in sample tests * fix test * fix test * fix test * test * use lazy load for replay
…ive model updates) (#47680)
…ath". Add `overwrite` option when downloading to path. (#47673)
#47679) * Update hosted agent samples and tests to include responses invocation and refactor code handling * Update hosted agent samples and tests to include environment variable setup and enhance OpenAI response handling * fix test * fix test * resolve conflict * fix test
dargilco
requested review from
bobogogo1990,
glharper,
howieleung,
kingernupur,
nick863,
trangevi and
trrwilson
as code owners
June 29, 2026 19:40
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The method
_normalize_multipart_file_entryinazure\ai\projects\_utils\utils.pynow uses "application/octet-stream" as the content type default, as seen here: https://github.com/Azure/azure-sdk-for-python/blob/feature/azure-ai-projects/2.3.0/sdk/ai/azure-ai-projects/azure/ai/projects/_utils/utils.py#L66 . This is a result of an emitter change. There was no default before (see this commit, which added this new default following using new emitter: 7f2a28c#diff-956744d63fbe9ee2ad5d51c6959f9f17f7d173656de8fabad16753ce7fe156cfR66 )Update unit-tests to reflect that change in default value.
This was the original CoPilot comment in the PR: #47500
This helper now injects an explicit content-type ("application/octet-stream") when normalizing bare file values, which changes the normalized multipart tuples produced by prepare_multipart_form_data. The offline unit tests in tests/models/test_multipart_filetype_variants.py currently assume that variants normalize to an equivalent representation when the caller did not explicitly provide a content-type; with this change they will need to be updated accordingly (e.g., update the expected/default content-type and/or the explicit content-type variant).