You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe
N/A
Describe the solution you'd like
Currently, the Docker Compose setup is untested. Unrelatedly, the CI tests for the code must mimic the environment created by docker/docker compose in order to test the code. We could run docker compose up and run the tests inside the docker compose setup during CI to test both at the same time and avoid duplicating the environments
Describe alternatives you've considered
We could also just test that the docker compose stack comes up and call it done, but I think the main suggestion is the better long term solution
Is your feature request related to a problem? Please describe
N/A
Describe the solution you'd like
Currently, the Docker Compose setup is untested. Unrelatedly, the CI tests for the code must mimic the environment created by docker/docker compose in order to test the code. We could run
docker compose up
and run the tests inside the docker compose setup during CI to test both at the same time and avoid duplicating the environmentsDescribe alternatives you've considered
We could also just test that the docker compose stack comes up and call it done, but I think the main suggestion is the better long term solution
Additional context
https://github.com/marketplace/actions/docker-compose-with-tests-action
The text was updated successfully, but these errors were encountered: