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
tests: run postgres via pytest #15365
Conversation
Instead of running a separate container, allow the pytest plugin to bootstrap its own postgres process. Increases the development image by adding the postgres server package. Changes the overall Docker runtime to run under a non-root user. Signed-off-by: Mike Fiedler <miketheman@gmail.com>
Now that pytest controls the server behavior during tests, remove from CI steps as well. Signed-off-by: Mike Fiedler <miketheman@gmail.com>
Further isolates development from testing by using `app_config` fixture instead of pulling the configuration from the environment. Signed-off-by: Mike Fiedler <miketheman@gmail.com>
Signed-off-by: Mike Fiedler <miketheman@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
now that it's nobody
i'm 👍🏼
just had to configure coverage to write to a directory owned by the user.
confirmed creating migrations works.
Looks like I'll look for toggles that move that file |
With the change to have the main container run as `nobody`, move some more application-root directories to `dev/`. Refs: pypi#15365 Signed-off-by: Mike Fiedler <miketheman@gmail.com>
With the change to have the main container run as `nobody`, move some more application-root directories to `dev/`. Refs: pypi#15365 Signed-off-by: Mike Fiedler <miketheman@gmail.com>
This reverts commit 51d7c35.
Instead of running a separate container, allow the pytest plugin to
bootstrap its own postgres process.
Increases the development image by adding the postgres server package.
Changes the overall Docker runtime to run under a non-root user.
Signed-off-by: Mike Fiedler miketheman@gmail.com