-
Notifications
You must be signed in to change notification settings - Fork 0
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
attempt to fix integration tests in CI #45
base: main
Are you sure you want to change the base?
Conversation
The integration tests don't work after all. Exit code 137 indicates that the docker container runs out of memory. |
After checking RAM usage in Actions with https://github.com/marketplace/actions/workflow-telemetry, no noticeable difference when running all tests or just one test, however in the latter case it doesn't crash. In any case RAM usage stays around 1GB and GitHub runners have 7GB of RAM allocated to them, of which all should be utilized by Docker by default. |
resolves #41
Mocha 4.0 version disabled the --exit flag by default, which leaves some tests hanging if there is a database connection still open. In our case the database connection is made before the tests are run and it's closed independently from the tests, so I don't see an issue with adding the --exit flag back.