Re-introduce pytest-xdist in supported envs#5431
Conversation
7ed6246 to
392ebbb
Compare
392ebbb to
edec305
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## master #5431 +/- ##
=======================================
Coverage 98.67% 98.67%
=======================================
Files 117 117
Lines 35824 35891 +67
Branches 4252 4258 +6
=======================================
+ Hits 35348 35415 +67
Misses 320 320
Partials 156 156
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
7dd330f to
18dd594
Compare
18dd594 to
c4eab87
Compare
ef79513 to
e1f51b9
Compare
9cc0f30 to
93dc6bd
Compare
26d0b16 to
763a477
Compare
b6a6440 to
1989754
Compare
|
@Dreamsorcerer @bdraco any ideas why the http parser tests might be failing in with pytest-xdist? |
92f0b5a to
7333846
Compare
I was under the impression this should not be needed on master (v4) anymore, so shouldn't be a factor in these tests. Unless it is something different from: https://docs.aiohttp.org/en/stable/client_advanced.html#graceful-shutdown |
|
Perhaps, somebody will be able to pair with me on the sprint day. Or just take care of rebasing+researching the solution. |
7333846 to
581d118
Compare
595ba15 to
469b882
Compare
CodSpeed Performance ReportMerging #5431 will not alter performanceComparing Summary
|
469b882 to
af785e2
Compare
3f26209 to
5fb8848
Compare
5fb8848 to
013eeaf
Compare
Co-authored-by: J. Nick Koston <[email protected]>
013eeaf to
1e79149
Compare
Backport to 3.10: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 751c3c4 on top of patchback/backports/3.10/751c3c4e17d621ffc91b79dccb4e2a5ef8b42888/pr-5431 Backporting merged PR #5431 into master
🤖 @patchback |
Backport to 3.11: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 751c3c4 on top of patchback/backports/3.11/751c3c4e17d621ffc91b79dccb4e2a5ef8b42888/pr-5431 Backporting merged PR #5431 into master
🤖 @patchback |
Co-authored-by: J. Nick Koston <[email protected]> (cherry picked from commit 751c3c4)
Co-authored-by: J. Nick Koston <[email protected]> (cherry picked from commit 751c3c4)
…orted envs (#9747) Co-authored-by: J. Nick Koston <[email protected]> Co-authored-by: Sviatoslav Sydorenko (Святослав Сидоренко) <[email protected]>
…orted envs (#9746) Co-authored-by: J. Nick Koston <[email protected]> Co-authored-by: Sviatoslav Sydorenko (Святослав Сидоренко) <[email protected]>
What do these changes do?
This change resurrects parallel test execution in most of the test envs. On my laptop, the full test run is almost 3x faster: 36.04s vs 105.53s (0:01:45).
Refs:
pytest-xdistbreaks asyncio-based code expecting to be run in the main thread pytest-dev/pytest-xdist#620Are there changes in behavior for the user?
Nope.
Related issue number
#3419 + #3450
Checklist
CONTRIBUTORS.txtCHANGESfolder<issue_id>.<type>for example (588.bugfix)issue_idchange it to the pr id after creating the pr.feature: Signifying a new feature..bugfix: Signifying a bug fix..doc: Signifying a documentation improvement..removal: Signifying a deprecation or removal of public API..misc: A ticket has been closed, but it is not of interest to users.