cmu-delphi/delphi-epidata

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

Offen

#896 geöffnet am 11.04.2022

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Python (67 Forks)auto 404
acquisitionhelp wantedtestsv4

Repository-Metriken

Stars
 (104 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 63T 20h) (11 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuche die vorhandenen Integrationstests in integrations/acquisition/covidcast/test *.py als Vorlagen. Verstehe das Datenbankschema und die Logik zum Setzen von is latest issue in signal load und signal latest. Schreibe Tests, die folgende Szenarien abdecken: Einfügen neuer Tagesdaten mit höherer Issue, Einfügen von Datenpatches mit niedrigerer Issue, und sicherstellen, dass signal latest nur überschrieben wird, wenn die neue Issue >= ist. Teste auch, dass Zeilen in * dim nur bei Bedarf hinzugefügt werden.
Tech Stack
python
Domain
backenddatabasetesting
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
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
PythonpytestSQLAlchemyPostgreSQL
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50