elastic / elastic/apm-agent-python

[CI] BDD Tests that are not implemented: should the build fail or warn?

Offen
#1,201 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
agent-python question
Vorherrschende Sprache
Python
Sterne
431
Forks
239
Ø Merge
5 T. 10 Std.
Gemergte PRs (30 T.)
7

Beschreibung

We noticed that there are a few BDD files under https://github.com/elastic/apm-agent-python/tree/master/tests/bdd/features
- api_key
- azure_app_service_metadata
- outcome

But only the API Key ones are implemented.

>[2021-08-02T10:01:23.365Z] tests/bdd/__init__.py 0 0 0 0 100%
[2021-08-02T10:01:23.365Z] tests/bdd/api_key_tests.py 31 15 0 0 52%
[2021-08-02T10:01:23.365Z] tests/client/__init__.py 0 0 0 0 100%

You can check that they do not appear under the test list for that package: https://apm-ci.elastic.co/job/apm-agent-python/job/apm-agent-python-mbp/job/master/500/testReport/tests.bdd/

I'd like to ask about your expectations on it: should the build fail because they are not implemented, or show them as skipped?

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginnen Sie mit der Überprüfung der BDD-Dateien unter tests/bdd/features für api_key, azure_app_service_metadata und outcome, und vergleichen Sie sie anschließend mit tests/bdd/api_key_tests.py und dem verknüpften CI-Testbericht. Für das Issue muss vor der Festlegung des Implementierungsumfangs eine abgestimmte Richtlinie für nicht implementierte Szenarien festgelegt werden; als abgeschlossen gilt die Aufgabe, wenn das gewählte Fail-or-Skip-Verhalten explizit ist und im CI konsistent widergespiegelt wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
ci-cd, testing
Issue-Typ
Feature
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

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