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

Individual migration tests define sesh_with_large_data but never use it #201

Open
rod-glover opened this issue Jan 15, 2024 · 0 comments
Open

Comments

@rod-glover
Copy link
Contributor

In tests/alembic_migrations/versions, we define tests of single migrations. These occasionally need data to test migrations that include a data migration component, but they are relatively rare. However, the fixture sesh_with_large_data is defined in the conftest for each one. This is likely unnecessary; in fact I think that defining the fixture at a higher level could replace all of them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant