MetOffice/CSET

Make constraints tests more robust

Open

#838 aperta il 17 set 2024

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Python (16 fork)auto 404
cleanupgood first issue

Metriche repository

Star
 (28 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

I don't think checking the result of repr is sufficient for many of the tests in this file. In particular those where the repr just says a lambda or function. The relevant lambda or function could be changed to something that produces complete nonsense and these tests would never catch it. It's not really related to this PR though so I'd suggest opening a new issue.

Originally posted by @SGallagherMet in https://github.com/MetOffice/CSET/pull/765#discussion_r1740967262

We should make the tests in test_constraints.py more robust. This will likely involve making most of them into filter tests.

Guida contributor

Direzione di ricerca
Esaminare i test dei vincoli in test constraints.py e convertirli in test di filtro per garantire robustezza. Verificare che i test verifichino effettivamente la logica dei vincoli.
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.
Chiara
Prerequisiti
GitPython
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70