Howie/sample-recording-7

[howieleung]

Description

Please add an informative description that covers that changes made by the pull request and link all relevant issues.

If an SDK is being regenerated based on a new API spec, a link to the pull request containing these API spec changes should be included above.

All SDK Contribution checklist:

• The pull request does not introduce [breaking changes]
• CHANGELOG is updated for new features, bug fixes or other significant changes.
• I have read the contribution guidelines.

General Guidelines and Best Practices

• Title of the pull request is clear and informative.
• There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

• Pull request includes test coverage for the included changes.

[Copilot]

Pull request overview

This pull request refactors the sample testing infrastructure for the Azure AI Projects SDK to improve modularity, add support for optional environment variables in recorded tests, and provide comprehensive documentation.

Key Changes

• Splits the monolithic sample_executor_helpers.py into separate, focused modules: sample_executor.py (core executors and decorators), and test_samples_helpers.py (shared test configuration)
• Introduces the @additionalSampleTests decorator to enable testing samples with optional environment variables through additional parameterized test cases
• Adds comprehensive README.md documentation with usage examples for sync/async sample testing, including record/playback workflows

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
sdk/ai/azure-ai-projects/tests/samples/sample_executor_helpers.py	Deleted old monolithic helper file (206 lines removed)
sdk/ai/azure-ai-projects/tests/samples/sample_executor.py	New comprehensive module with sync/async executors, path discovery functions, and additionalSampleTests decorator for optional environment variables
sdk/ai/azure-ai-projects/tests/samples/test_samples_helpers.py	New shared configuration module containing LLM validation instructions and environment variable mappings
sdk/ai/azure-ai-projects/tests/samples/test_samples.py	Updated to use new imports and demonstrates usage of @additionalSampleTests for the computer_use sample
sdk/ai/azure-ai-projects/tests/samples/test_samples_async.py	Updated to use new imports and refactored async sample executor
sdk/ai/azure-ai-projects/tests/samples/README.md	New comprehensive documentation for sample testing patterns, including examples and best practices
sdk/ai/azure-ai-projects/assets.json	Updated asset tag reference for test recordings

[Copilot]

Inconsistent naming: the decorator is referenced as @samplePathPasser() but should be @SamplePathPasser() (capital S and P) to match the class name used throughout the codebase and in the earlier examples.

[Copilot]

The variable name env_var_sets_by_sample uses an unclear abbreviation 'env_var'. Consider renaming to environment_variable_sets_by_sample for better clarity, or at minimum env_variable_sets_by_sample to make it clear this refers to environment variables.