[3.12] gh-119050: Add type hints to libregrtest/results.py (GH-119144) (#119157)

gh-119050: Add type hints to libregrtest/results.py (GH-119144)

Sort also 'omitted' in TestResults.display_result().
(cherry picked from commit 30b4e9f9c4)

Co-authored-by: Victor Stinner <vstinner@python.org>
This commit is contained in:
Miss Islington (bot) 2024-05-18 23:06:41 +02:00 committed by GitHub
parent bd1e9509a4
commit d76d95eacb
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -17,7 +17,7 @@ EXITCODE_INTERRUPTED = 130 # 128 + signal.SIGINT=2
class TestResults: class TestResults:
def __init__(self): def __init__(self) -> None:
self.bad: TestList = [] self.bad: TestList = []
self.good: TestList = [] self.good: TestList = []
self.rerun_bad: TestList = [] self.rerun_bad: TestList = []
@ -35,22 +35,22 @@ class TestResults:
# used by --junit-xml # used by --junit-xml
self.testsuite_xml: list = [] self.testsuite_xml: list = []
def is_all_good(self): def is_all_good(self) -> bool:
return (not self.bad return (not self.bad
and not self.skipped and not self.skipped
and not self.interrupted and not self.interrupted
and not self.worker_bug) and not self.worker_bug)
def get_executed(self): def get_executed(self) -> set[TestName]:
return (set(self.good) | set(self.bad) | set(self.skipped) return (set(self.good) | set(self.bad) | set(self.skipped)
| set(self.resource_denied) | set(self.env_changed) | set(self.resource_denied) | set(self.env_changed)
| set(self.run_no_tests)) | set(self.run_no_tests))
def no_tests_run(self): def no_tests_run(self) -> bool:
return not any((self.good, self.bad, self.skipped, self.interrupted, return not any((self.good, self.bad, self.skipped, self.interrupted,
self.env_changed)) self.env_changed))
def get_state(self, fail_env_changed): def get_state(self, fail_env_changed: bool) -> str:
state = [] state = []
if self.bad: if self.bad:
state.append("FAILURE") state.append("FAILURE")
@ -195,7 +195,7 @@ class TestResults:
omitted = set(tests) - self.get_executed() omitted = set(tests) - self.get_executed()
# less important # less important
all_tests.append((omitted, "test", "{} omitted:")) all_tests.append((sorted(omitted), "test", "{} omitted:"))
if not quiet: if not quiet:
all_tests.append((self.skipped, "test", "{} skipped:")) all_tests.append((self.skipped, "test", "{} skipped:"))
all_tests.append((self.resource_denied, "test", "{} skipped (resource denied):")) all_tests.append((self.resource_denied, "test", "{} skipped (resource denied):"))