cmu-delphi/delphi-epidata

Update `test_covidcast_meta.py` with hard value comparisons

Aperta

#924 aperta il 2 giu 2022

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Python (67 fork)auto 404
help wanted

Metriche repository

Star
 (104 stelle)
Metriche merge PR
 (Merge medio 63g 20h) (11 PR mergiate in 30 g)

Descrizione

just want to point out that nothing in these tests is actually testing that the meta values were computed correctly, only that the length of the responses returned was correct and that fields like min_time are present. i don't believe that the changes to compute_covidcast_meta warrant and update to that, but it would be good to have at least a few tests that compare hard values in the future.

Originally posted by @dshemetov in https://github.com/cmu-delphi/delphi-epidata/pull/903#discussion_r887367780

Related: #897

Guida contributor

Direzione di ricerca
Studia il test covidcast meta.py esistente per comprenderne la struttura, quindi identifica i valori meta chiave che possono essere hardcodati e confrontali con l'output effettivo di compute covidcast meta.
Tech stack
python
Dominio
backendapidata
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
Pythonpytest
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70