Azure / Azure/azure-sdk-for-python

Define common `SampleExecutor` in devtools_testutils

Offen
#44,440 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
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

Beitragsleitfaden öffnen

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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.