-
Notifications
You must be signed in to change notification settings - Fork 79
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
Re-use baked project for all tests #353
Conversation
1c71525
to
792f7bf
Compare
Awesome! Will merge this after #347. The doc test was also fixed in that PR, so let's take that fix out of this PR. |
I rebased on main and removed the test fixes from the second commit. That left only the deprecated .project replacement, so I renamed the commit to reflect that. This resolves the merge conflicts. |
Ok, fun! Two errors detected :)
|
It seems like |
Did a fixup rebase of the first commit to fix the tests there. |
Thanks a lot for this! |
Thanks for fixing the merge conflict @egpbos! |
Description
Speed up the tests by re-using the baked project.
Also:
.project
with.project_path
--no-autodetect
when running prospector to avoid a warning about Django not being installedOutput of
pytest --durations=0
before (119 seconds):and after (25 seconds):