arXiv / arxiv-search

arXiv Search UI & APIs
https://arxiv.github.io/arxiv-search/
MIT License
96 stars 15 forks source link

ARXIVNG-2224 ARXIVNG-2225 added base app tests, fixed Elasticsearch config #248

Closed erickpeirson closed 5 years ago

erickpeirson commented 5 years ago

Per #247 (ARXIVNG-2224), this adds app tests from arxiv.base to help catch potential configuration issues in the search app.

To address #246 (ARXIVNG-2225) I changed the naming of the Elasticsearch config params in config.py and search/services/index so that the same names are used in os.environ and in the app config. This allows us to continue to instantiate the Flask app at import time, which is kind of an important pattern since we are starting to add start-up checks to the app factory in other apps (and should start to do so here, as well).