diff options
author | Hemmo Nieminen <hemmo.nieminen@iki.fi> | 2022-02-01 00:00:00 +0200 |
---|---|---|
committer | Eli Schwartz <eschwartz93@gmail.com> | 2022-06-09 18:58:33 -0400 |
commit | e3b4065b0c8ded0e1bfe38c75c6b7eac6fbc1963 (patch) | |
tree | fb7801a057899b1266f8901dc0b8b36c6e81ee0f | |
parent | 9f26eae541ddf3f3162009339e0c007d0008b45f (diff) | |
download | meson-e3b4065b0c8ded0e1bfe38c75c6b7eac6fbc1963.zip meson-e3b4065b0c8ded0e1bfe38c75c6b7eac6fbc1963.tar.gz meson-e3b4065b0c8ded0e1bfe38c75c6b7eac6fbc1963.tar.bz2 |
mtest: split TestRun's get_details() to multiple functions
These new functions will be used individually in later commits.
-rw-r--r-- | mesonbuild/mtest.py | 16 |
1 files changed, 11 insertions, 5 deletions
diff --git a/mesonbuild/mtest.py b/mesonbuild/mtest.py index 17fd108..dde6afc 100644 --- a/mesonbuild/mtest.py +++ b/mesonbuild/mtest.py @@ -900,11 +900,7 @@ class TestRun: def direct_stdout(self) -> bool: return self.verbose and not self.is_parallel and not self.needs_parsing - def get_details(self) -> str: - if self.res is TestResult.PENDING: - return '' - if self.returncode: - return returncode_to_status(self.returncode) + def get_results(self) -> str: if self.results: # running or succeeded passed = sum(x.result.is_ok() for x in self.results) @@ -915,6 +911,16 @@ class TestRun: return f'{passed}/{ran} subtests passed' return '' + def get_exit_status(self) -> str: + return returncode_to_status(self.returncode) + + def get_details(self) -> str: + if self.res is TestResult.PENDING: + return '' + if self.returncode: + return self.get_exit_status() + return self.get_results() + def _complete(self, returncode: int, res: TestResult, stdo: T.Optional[str], stde: T.Optional[str]) -> None: assert isinstance(res, TestResult) |