Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GH-0 Add integration test as a test containers #21

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Volland
Copy link
Owner

@Volland Volland commented Jun 14, 2018

So.

The test is a crucial part of any project. It better to have the test immediately on go.
I use an integration test. It is more preferable to use a contract testing like but as far as out API is just a
standard crud and we have no complex service relations Pact is overkill.

Why docker containers?

Yep it a bit harder to write and a bit tricky to run
Maybe https://github.com/One-com/node-mocha-docker-postgres could make life a bit easier.
Main benefit. We set up a real DB and real full tuned server with all middleware and no magic env variable toggle that change how a system behaves.
Only one - we need to tune connection and preferably to use separate DB with a different name

It is still in progress.
I have covered a creation of item the happy path.

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.

None yet

1 participant