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

Disable old migration tests #2376

Open
winged opened this issue Feb 18, 2025 · 0 comments
Open

Disable old migration tests #2376

winged opened this issue Feb 18, 2025 · 0 comments

Comments

@winged
Copy link
Contributor

winged commented Feb 18, 2025

We have a few old migration tests that are not relevant anymore. Let me elaborate:

We did quite a few refactorings of the data structure. However, for most (all) of these complex migrations, we don't have any installations anymore, and new ones will be migrated past those steps without any affected data.

Therefore, it doesn't make sense to run those slow tests in CI and local test envs anymore. They slow things down significantly, and test aspects that nobody cares about anymore.

I therefore suggest we remove those migration tests, or at least mark them with #pragma: no cover and @pytest.mark.skip(reason="nobody cares and it's slow")

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