Skip to content

Tests#50

Merged
rutgerfarry merged 16 commits intomasterfrom
tests
Feb 15, 2018
Merged

Tests#50
rutgerfarry merged 16 commits intomasterfrom
tests

Conversation

@rutgerfarry
Copy link
Copy Markdown
Contributor

closes #28, closes #38

Didn’t realize that child_process functions don’t return promises
Run some tests in the future and ensure gzip is worth it for smaller responses
This makes it easier to run inside docker, since we are no longer attempting to run docker inside docker
This is sad, because now we are now dependent on mongorestore for tests to function. Using the node driver to load mongoexport data was just too troublesome however (which had no dependencies beyond what npm installed)
Test usually fails since mongo text index hasn’t finished building. Should add a `/ready` endpoint and then poll that until the app is ready.
@rutgerfarry rutgerfarry merged commit 94a3eed into master Feb 15, 2018
@rutgerfarry rutgerfarry deleted the tests branch February 15, 2018 01:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove database-specific fields from api response Decide on testing framework and create tests

1 participant