Skip to content
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

Make all PG-requiring tests feature-gated #1739

Open
nyurik opened this issue Mar 11, 2025 · 0 comments
Open

Make all PG-requiring tests feature-gated #1739

nyurik opened this issue Mar 11, 2025 · 0 comments
Labels
ci/cd help wanted pg Related to PostgreSQL sources

Comments

@nyurik
Copy link
Member

nyurik commented Mar 11, 2025

We need to simplify our testing process, and that means that cargo test should pass without any external dependencies, i.e. without requiring PG to be running, or some env var to be set.

I think PostgreSQL tests are the only complex ones, so lets mark them all as requiring some test-pg feature (name?). Note that cargo test --all-features should still include the postgreSQL tests, and thus require some connection string to be set. This will be done by CI, but will make development simpler.

@nyurik nyurik added ci/cd help wanted pg Related to PostgreSQL sources labels Mar 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/cd help wanted pg Related to PostgreSQL sources
Projects
None yet
Development

No branches or pull requests

1 participant