pymc-devs/pymc

Fail tests with unexpected warnings

Aperta

#6408 aperta il 21 dic 2022

Vedi su GitHubGuida contributor
 (6 commenti) (0 reazioni) (0 assegnatari)Python (1902 fork)batch import
beginner friendlyhelp wantedmaintenancetests

Metriche repository

Star
 (7926 stelle)
Metriche merge PR
 (Merge medio 11g) (7 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Inizia esaminando l'approccio adottato nella PR #6407, che ha aggiunto 'pytestmark = pytest.mark.filterwarnings("error")' ad alcuni test. Quindi, considera di applicare lo stesso marcatore globalmente nella configurazione dei test, eventualmente utilizzando i file di configurazione di pytest per trattare tutti i warning come errori. Sii pronto a correggere eventuali test che attualmente attivano warning, il che potrebbe comportare l'aggiornamento di utilizzi deprecati o l'aggiunta di filtri di warning previsti.
Tech stack
python
Dominio
testing
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Pythonpytest
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60