Azure / Azure/azure-sdk-for-python
Define common `SampleExecutor` in devtools_testutils
- Vorherrschende Sprache
- Python
- Sterne
- 5.6k
- Forks
- 3.4k
- Ø Merge
- 2 T. 2 Std.
- Gemergte PRs (30 T.)
- 213
Beschreibung
@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.
Beitragsleitfaden
Rechercherichtung
Beginne damit, den AI SampleExecutor und die gemeinsam genutzten devtools_testutils-Hilfsprogramme zu finden, und überprüfe anschließend die verlinkte Anpassung in test_samples.py um Zeile 301. Extrahiere das wiederverwendbare Verhalten in devtools_testutils, aktualisiere den AI SampleExecutor so, dass er es verwendet, und verifiziere, dass die gemeinsame Klasse weiterhin die Anpassungen und Aufzeichnungen der AI-Samples unterstützt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- devtools, testing-qa
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100