DataBiosphere / data-explorer

BSD 3-Clause "New" or "Revised" License
10 stars 6 forks source link

Delete most API server unit tests #262

Closed melissachang closed 5 years ago

melissachang commented 5 years ago

API server unit tests are a pain to set up and haven't been useful -- they haven't caught bugs AFAIK. So rather than fix them in an upcoming PR, just delete them. (FYI -- the e2e tests are useful, they catch a ton of bugs.)

Also move number_to_range() from facets_controller.py to elasticsearch_util.py, so it can be next to range_to_number().