Skip to content
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.

Sign up for GitHub

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

gh-82054: allow test runner to split test_asyncio to execute in parallel by sharding. #103927

Merged
merged 5 commits into from
Apr 30, 2023
Merged

gh-82054: allow test runner to split test_asyncio to execute in parallel by sharding. #103927

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
557e6b7
This allows the test runner to execute test_multiprocessing and test_…
zitterbewegung Apr 27, 2023
345d12f
revert whitespace changes
carljm Apr 28, 2023
9b71c20
multiprocessing, not compiler
carljm Apr 28, 2023
19c6c73
This removes an extraneous test package that doesn't exist and also c…
zitterbewegung Apr 29, 2023
b409e2c
simplify logic: no len()s, tight f-expression.
gpshead Apr 29, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
revert whitespace changes
  • Loading branch information
@carljm
carljm committed Apr 28, 2023
commit 345d12f94283d8e3430242ae52a964de646b188f
66 changes: 33 additions & 33 deletions Lib/test/libregrtest/runtest.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,10 +19,10 @@

class TestResult:
def __init__(
self,
name: str,
duration_sec: float = 0.0,
xml_data: list[str] | None = None,
self,
name: str,
duration_sec: float = 0.0,
xml_data: list[str] | None = None,
) -> None:
self.name = name
self.duration_sec = duration_sec
Expand All @@ -39,12 +39,12 @@ def __str__(self) -> str:

class Failed(TestResult):
def __init__(
self,
name: str,
duration_sec: float = 0.0,
xml_data: list[str] | None = None,
errors: list[tuple[str, str]] | None = None,
failures: list[tuple[str, str]] | None = None,
self,
name: str,
duration_sec: float = 0.0,
xml_data: list[str] | None = None,
errors: list[tuple[str, str]] | None = None,
failures: list[tuple[str, str]] | None = None,
) -> None:
super().__init__(name, duration_sec=duration_sec, xml_data=xml_data)
self.errors = errors
Expand Down Expand Up @@ -128,16 +128,16 @@ def __str__(self) -> str:
# small set of tests to determine if we have a basically functioning interpreter
# (i.e. if any of these fail, then anything else is likely to follow)
STDTESTS = [
'test_grammar',
'test_opcodes',
'test_dict',
'test_builtin',
'test_exceptions',
'test_types',
'test_unittest',
'test_doctest',
'test_doctest2',
'test_support'
'test_grammar',
'test_opcodes',
'test_dict',
'test_builtin',
'test_exceptions',
'test_types',
'test_unittest',
'test_doctest',
'test_doctest2',
'test_support'
]

# set of tests that we don't want to be executed when using regrtest
Expand All @@ -149,8 +149,8 @@ def __str__(self) -> str:
# __init__.py may do things which alter what tests are to be run.

SPLITTESTDIRS = {
"test_asyncio",
"test_compiler",
"test_asyncio",
"test_compiler",
}

# Storage of uncollectable objects
Expand Down Expand Up @@ -203,7 +203,7 @@ def _runtest(ns: Namespace, test_name: str) -> TestResult:
output_on_failure = ns.verbose3

use_timeout = (
ns.timeout is not None and threading_helper.can_start_thread
ns.timeout is not None and threading_helper.can_start_thread
)
if use_timeout:
faulthandler.dump_traceback_later(ns.timeout, exit=True)
Expand Down Expand Up @@ -234,7 +234,7 @@ def _runtest(ns: Namespace, test_name: str) -> TestResult:
print_warning.orig_stderr = stream

result = _runtest_inner(ns, test_name,
display_failure=False)
display_failure=False)
if not isinstance(result, Passed):
output = stream.getvalue()
finally:
Expand All @@ -250,13 +250,13 @@ def _runtest(ns: Namespace, test_name: str) -> TestResult:
support.verbose = ns.verbose

result = _runtest_inner(ns, test_name,
display_failure=not ns.verbose)
display_failure=not ns.verbose)

if xml_list:
import xml.etree.ElementTree as ET
result.xml_data = [
ET.tostring(x).decode('us-ascii')
for x in xml_list
ET.tostring(x).decode('us-ascii')
for x in xml_list
]

result.duration_sec = time.perf_counter() - start_time
Expand Down Expand Up @@ -284,7 +284,7 @@ def runtest(ns: Namespace, test_name: str) -> TestResult:
if not ns.pgo:
msg = traceback.format_exc()
print(f"test {test_name} crashed -- {msg}",
file=sys.stderr, flush=True)
file=sys.stderr, flush=True)
return Failed(test_name)


Expand Down Expand Up @@ -345,7 +345,7 @@ def _runtest_inner2(ns: Namespace, test_name: str) -> bool:
if gc.garbage:
support.environment_altered = True
print_warning(f"{test_name} created {len(gc.garbage)} "
f"uncollectable object(s).")
f"uncollectable object(s).")

# move the uncollectable objects somewhere,
# so we don't see them again
Expand All @@ -358,7 +358,7 @@ def _runtest_inner2(ns: Namespace, test_name: str) -> bool:


def _runtest_inner(
ns: Namespace, test_name: str, display_failure: bool = True
ns: Namespace, test_name: str, display_failure: bool = True
) -> TestResult:
# Detect environment changes, handle exceptions.

Expand Down Expand Up @@ -404,7 +404,7 @@ def _runtest_inner(
if not ns.pgo:
msg = traceback.format_exc()
print(f"test {test_name} crashed -- {msg}",
file=sys.stderr, flush=True)
file=sys.stderr, flush=True)
return UncaughtException(test_name)

if refleak:
Expand Down Expand Up @@ -432,7 +432,7 @@ def cleanup_test_droppings(test_name: str, verbose: int) -> None:
kind, nuker = "file", os.unlink
else:
raise RuntimeError(f"os.path says {name!r} exists but is neither "
f"directory nor file")
f"directory nor file")

if verbose:
print_warning(f"{test_name} left behind {kind} {name!r}")
Expand All @@ -445,4 +445,4 @@ def cleanup_test_droppings(test_name: str, verbose: int) -> None:
nuker(name)
except Exception as exc:
print_warning(f"{test_name} left behind {kind} {name!r} "
f"and it couldn't be removed: {exc}")
f"and it couldn't be removed: {exc}")