diff options
author | Hemmo Nieminen <hemmo.nieminen@iki.fi> | 2022-02-01 00:00:00 +0200 |
---|---|---|
committer | Hemmo Nieminen <hemmo.nieminen@iki.fi> | 2022-03-23 00:00:00 +0200 |
commit | 7f0767289dc7beaa0e744427e3e6a3dbc5b9561d (patch) | |
tree | d0ff562cb8bf6ab61399ef93b1b7aed5c88a5fc1 | |
parent | df6cd8da500b7cb249e6aef90f6fd62d2dddaff8 (diff) | |
download | meson-7f0767289dc7beaa0e744427e3e6a3dbc5b9561d.zip meson-7f0767289dc7beaa0e744427e3e6a3dbc5b9561d.tar.gz meson-7f0767289dc7beaa0e744427e3e6a3dbc5b9561d.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 74b68ac..ac118ae 100644 --- a/mesonbuild/mtest.py +++ b/mesonbuild/mtest.py @@ -905,11 +905,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) @@ -920,6 +916,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) |