Skip to content
Snippets Groups Projects
Unverified Commit 7fa3a5a2 authored by Gregory P. Smith's avatar Gregory P. Smith Committed by GitHub
Browse files

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.

* :scroll::robot:

 Added by blurb_it.

Co-authored-by: default avatarblurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
parent 0ddc63b2
No related branches found
No related tags found
No related merge requests found
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment