pymc-devs/pymc

Fail tests with unexpected warnings

Open

#6.408 geöffnet am 21. Dez. 2022

Auf GitHub ansehen
 (6 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (1.902 Forks)batch import
beginner friendlyhelp wantedmaintenancetests

Repository-Metriken

Stars
 (7.926 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 11T 6h) (12 gemergte PRs in 30 T)

Beschreibung

Description

Some great maintenance work to cleanup warnings was done in #6067 and #6074 (probably more)

More recently, I started adding pytestmark = pytest.mark.filterwarnings("error"), as in #6407

Ideally, we would move to failing with warnings in the entire test suite, so that we don't regress and catch deprecation issues earlier on. The approach taken in #6407 might be a good start if there's too much going on.

https://docs.pytest.org/en/7.1.x/how-to/capture-warnings.html

Contributor Guide

Research-Richtung
Beginnen Sie mit der Untersuchung des Ansatzes in PR #6407, der 'pytestmark = pytest.mark.filterwarnings("error")' zu einigen Tests hinzugefügt hat. Erwägen Sie dann, denselben Marker global in der Testkonfiguration anzuwenden, optional unter Verwendung der pytest Konfigurationsdateien, um alle Warnungen als Fehler zu behandeln. Seien Sie bereit, alle Tests zu beheben, die derzeit Warnungen auslösen. Dies kann die Aktualisierung veralteter Verwendungen oder das Hinzufügen erwarteter Warnfilter umfassen.
Tech Stack
python
Domain
testing
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Pythonpytest
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60