-
Notifications
You must be signed in to change notification settings - Fork 70
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 integration tests to docker image #53
Comments
I think this could be complicated (time consuming at the beginning) but what about having a repository with one branch per scenario (under https://github.com/readthedocs/template for example), including a So, then in this repository a |
Perhaps, though I'd avoid manual review as a requirement. I suppose we can move the output files to a docker shared volume for review afterwards though. testinfra could be a useful addition here, and has a connector for docker: I'd say we could just include fixtures here and automate testing of the output, no need for a separate repo. |
This is already done in the https://github.com/readthedocs/test-builds repository and we can use here to make some simple builds on the just created docker images. We can just check the exit code is 0 in all the commands for now.
There are already done in #143. We can easily add the rest one once that PR gets merged and we are happy with the results. |
This would still be a manual process, as I don't know how we would automate the integration tests, but it would at least give maintainers and contributors a quick way of testing changes to the build image.
Tests would include:
Any more here?
Refs #48
The text was updated successfully, but these errors were encountered: