diff options
| author | Gregory P. Smith <greg@krypto.org> | 2022-04-16 18:46:33 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-04-16 18:46:33 (GMT) |
| commit | 7fa3a5a2197896066e3fe53ee325ac6ab54c3414 (patch) | |
| tree | 38e63154e63bcfc3626277ef26afe7b80e229344 /Misc/NEWS.d | |
| parent | 0ddc63b240340a952692b11dfe0810973393ed11 (diff) | |
| download | cpython-7fa3a5a2197896066e3fe53ee325ac6ab54c3414.zip cpython-7fa3a5a2197896066e3fe53ee325ac6ab54c3414.tar.gz cpython-7fa3a5a2197896066e3fe53ee325ac6ab54c3414.tar.bz2 | |
gh-91607: Fix several test_concurrent_futures tests to actually test what they claim (#91600)
* Fix test_concurrent_futures to actually test what it says.
Many ProcessPoolExecutor based tests were ignoring the mp_context
and using the default instead. This meant we lacked proper test
coverage of all of them.
Also removes the old _prime_executor() worker delay seeding code
as it appears to have no point and causes 20-30 seconds extra
latency on this already long test. It also interfered with some
of the refactoring to fix the above to not needlessly create their
own executor when setUp has already created an appropriate one.
* Don't import the name from multiprocessing directly to avoid confusion.
* 📜🤖 Added by blurb_it.
Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
Diffstat (limited to 'Misc/NEWS.d')
| -rw-r--r-- | Misc/NEWS.d/next/Tests/2022-04-16-17-54-05.gh-issue-91607.FnXjtW.rst | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/Misc/NEWS.d/next/Tests/2022-04-16-17-54-05.gh-issue-91607.FnXjtW.rst b/Misc/NEWS.d/next/Tests/2022-04-16-17-54-05.gh-issue-91607.FnXjtW.rst new file mode 100644 index 0000000..32839a8 --- /dev/null +++ b/Misc/NEWS.d/next/Tests/2022-04-16-17-54-05.gh-issue-91607.FnXjtW.rst @@ -0,0 +1 @@ +Fix ``test_concurrent_futures`` to test the correct multiprocessing start method context in several cases where the test logic mixed this up. |