-
Notifications
You must be signed in to change notification settings - Fork 63
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
Streamline CI test runtimes #459
Conversation
…significant loss in test coverage
Codecov Report
@@ Coverage Diff @@
## develop #459 +/- ##
========================================
Coverage 64.06% 64.06%
========================================
Files 11 11
Lines 4352 4352
========================================
Hits 2788 2788
Misses 1564 1564 Continue to review full report at Codecov.
|
After these changes (and after merging in PR #440
|
I think this has succeeded in shaving off a couple minutes. It's not apples to oranges to compare the timing from my local machine vs. the VMs that the CI runs in. But locally these changes have trimmed total test suite runtime from ~500 to ~400 s. (with substantial variation run to run depending on other local loads, I think). After these changes:
|
Let's try to make the test suite not take 15+ minutes to run, without compromising coverage substantially. Are the tests we're running efficient?
Method. Use
pytest --durations=0
to print out a list of the time consuming tests, then see if we can find efficiencies.Outputs from pytest durations prior to these changes: