Azure / Azure/azure-sdk-for-python
Define common `SampleExecutor` in devtools_testutils
- Dominant language
- Python
- Stars
- 5.6k
- Forks
- 3.4k
- Avg merge
- 1d 21h
- Merged PRs (30d)
- 193
Description
@howieleung did an excellent job writing up a TestClass that allows them to easily wrap their `samples` in recordings without adjustments to the code of the sample itself (beyond the [customizations](https://github.com/Azure/azure-sdk-for-python/blob/de53a2eda95601ff20df68575f7bae67efa8c9b2/sdk/ai/azure-ai-projects/tests/samples/test_samples.py#L301) that enable the playback for the recording).
There is _some_ code here that I'm uncertain will be valid -- namely some of the output processing is rightly tightly bound to AI specifically.
We should A) abstract this code into devtools_testutils B) update the AI SampleExecutor to use this new common class and C) ensure that the common class has customizability to re-enable the newly replaced AI samples.
Contributor guide
Assessment
This issue has not been assessed yet.