cmu-delphi / delphi-epidata

An open API for epidemiological data.
https://cmu-delphi.github.io/delphi-epidata/
MIT License
102 stars 68 forks source link

Add basic integration tests for all endpoints #1068

Closed rzats closed 1 year ago

rzats commented 1 year ago

This PR https://github.com/cmu-delphi/delphi-epidata/pull/1062 had some endpoints with missing imports, but passed integration testing. We should make sure all endpoints are covered by basic integration tests ("doesn't throw an error") to prevent this from happening.

Missing imports were fixed in https://github.com/cmu-delphi/delphi-epidata/pull/1067.

Starter list of endpoints:

For each we need to:

  1. see if it has coverage in integration tests
  2. if so, check it off
  3. if not, add a test, then check it off
melange396 commented 1 year ago

possibly helpful: Dmitry "dug through the old endpoints and got one non-trivial response query for almost all of them" (slack message)

melange396 commented 1 year ago

done in #1241