-
Notifications
You must be signed in to change notification settings - Fork 2
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
Integrate website into tests #235
Conversation
@patrick-5546 @jahn18 Is the idea that running the test task should also populate some data at I have a successful connection to the url when I run the test task, but there is no data. |
Yes once the to do's in the PR description is complete. That's why I created this PR as a draft (not ready for review) |
Decided to change the common database name from |
54083f9
to
ee33008
Compare
15eb53c
to
031baa3
Compare
@jahn18 can you switch the website to read from |
Description
To integrate the website into our tests, we will be enforcing these conventions:
sailbot_db
DBTestDB
test
This PR enables the website container to run by default in the dev container and in our CI.
To Do
sailbot_db
before running unit testssailbot_db
by defaultsailbot_workspace/.devcontainer/website/website.Dockerfile
Line 4 in 8d15221
Verification