cmu-delphi/delphi-epidata

Write integration tests for src/acquisition/covidcast/database.py

Aperta

#896 aperta il 11 apr 2022

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Python (67 fork)auto 404
acquisitionhelp wantedtestsv4

Metriche repository

Star
 (104 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Integration tests because we do want to exercise the actual database engine here, and not a mock.

  • make sure is_latest_issue is being set correctly -
    • when adding new daily data (data in signal_load has > issue than data in signal_latest)
    • when uploading data patches (data in signal load has < issue than data in signal_latest)
  • make sure data is being correctly transferred from signal_load into signal_* and *_dim
    • make sure signal_latest is only overwritten when signal_load issue is >=
    • make sure rows are added to *_dim only when needed

+Any other corner cases that worry you.

Use integrations/acquisition/covidcast/test_*.py as templates.

Guida contributor

Direzione di ricerca
Esamina i test di integrazione esistenti in integrations/acquisition/covidcast/test *.py come modelli. Comprendi lo schema del database e la logica per impostare is latest issue in signal load e signal latest. Scrivi test che coprano: inserimento di nuovi dati giornalieri con issue più alto, inserimento di patch di dati con issue più basso, e assicurati che signal latest venga sovrascritto solo quando il nuovo issue è >=. Testa anche che le righe in * dim vengano aggiunte solo quando necessario.
Tech stack
python
Dominio
backenddatabasetesting
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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
PythonpytestSQLAlchemyPostgreSQL
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50