diff options
author | Eli Schwartz <eschwartz@archlinux.org> | 2021-03-04 17:02:31 -0500 |
---|---|---|
committer | Eli Schwartz <eschwartz@archlinux.org> | 2021-03-04 17:11:26 -0500 |
commit | 4340bf34faca7eed8076ba4c388fbe15355f2183 (patch) | |
tree | 6082548a6cb79091d1059a73e7b3b9f59657f6d9 /mesonbuild/mtest.py | |
parent | 76df995ba69ef5d790462856b3edbd42b28b906a (diff) | |
download | meson-4340bf34faca7eed8076ba4c388fbe15355f2183.zip meson-4340bf34faca7eed8076ba4c388fbe15355f2183.tar.gz meson-4340bf34faca7eed8076ba4c388fbe15355f2183.tar.bz2 |
various python neatness cleanups
All changes were created by running
"pyupgrade --py3-only --keep-percent-format"
and committing the results. I have not touched string formatting for
now.
- use set literals
- simplify .format() parameter naming
- remove __future__
- remove default "r" mode for open()
- use OSError rather than compatibility aliases
- remove stray parentheses in function(generator) scopes
Diffstat (limited to 'mesonbuild/mtest.py')
-rw-r--r-- | mesonbuild/mtest.py | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/mesonbuild/mtest.py b/mesonbuild/mtest.py index af5ad6d..ac4f515 100644 --- a/mesonbuild/mtest.py +++ b/mesonbuild/mtest.py @@ -902,8 +902,8 @@ class TestRun: return returncode_to_status(self.returncode) if self.results: # running or succeeded - passed = sum((x.result.is_ok() for x in self.results)) - ran = sum((x.result is not TestResult.SKIP for x in self.results)) + passed = sum(x.result.is_ok() for x in self.results) + ran = sum(x.result is not TestResult.SKIP for x in self.results) if passed == ran: return '{} subtests passed'.format(passed) else: @@ -928,7 +928,7 @@ class TestRun: return None test_only_env = set(self.env.items()) - set(os.environ.items()) return env_tuple_to_str(test_only_env) + \ - ' '.join((sh_quote(x) for x in self.cmd)) + ' '.join(sh_quote(x) for x in self.cmd) def complete_skip(self, message: str) -> None: self.starttime = time.time() @@ -1634,13 +1634,13 @@ class TestHarness: os.chdir(self.options.wd) runners = [] # type: T.List[SingleTestRunner] for i in range(self.options.repeat): - runners.extend((self.get_test_runner(test) for test in tests)) + runners.extend(self.get_test_runner(test) for test in tests) if i == 0: self.duration_max_len = max([len(str(int(runner.timeout or 99))) for runner in runners]) # Disable the progress report if it gets in the way - self.need_console = any((runner.console_mode is not ConsoleUser.LOGGER - for runner in runners)) + self.need_console = any(runner.console_mode is not ConsoleUser.LOGGER + for runner in runners) self.test_count = len(runners) self.run_tests(runners) |