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

Add some test cases #3

Merged
merged 1 commit into from
Dec 15, 2020
Merged

Add some test cases #3

merged 1 commit into from
Dec 15, 2020

Conversation

sonallux
Copy link
Contributor

This PR adds some test cases. The tests are executed against a real postgres database running in a docker container. The tests are starting the database container themself.

I have also done the following minor changes:

  • Fix some console log statements
  • Add a close method to PostgresRepository

test/query.test.js Outdated Show resolved Hide resolved
Copy link
Member

@georg-schwarz georg-schwarz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@georg-schwarz georg-schwarz merged commit cb568bc into jvalue:main Dec 15, 2020
@sonallux sonallux deleted the first-tests branch December 15, 2020 13:58
georg-schwarz added a commit that referenced this pull request Feb 8, 2021
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.

2 participants