diff options
author | Thibault Saunier <tsaunier@igalia.com> | 2018-05-15 14:40:45 -0400 |
---|---|---|
committer | Thibault Saunier <tsaunier@igalia.com> | 2018-05-15 14:45:54 -0400 |
commit | bb71fd994479ddc32bcf28e5891ec4f78f97e198 (patch) | |
tree | 1b98f0b8e692928b1b2cda444906e84b67e3ce53 | |
parent | 1055540d55dc819a96699fce339c1e8a09f31f5f (diff) |
validate:launcher: Error out loudly if the testlist changes
When --fail-on-testlist-change is set.
-rw-r--r-- | validate/launcher/baseclasses.py | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/validate/launcher/baseclasses.py b/validate/launcher/baseclasses.py index 7068716..944aad0 100644 --- a/validate/launcher/baseclasses.py +++ b/validate/launcher/baseclasses.py @@ -1594,7 +1594,8 @@ class _TestsLauncher(Loggable): testlist_file.write("%s\n" % (tname)) if tname and tname not in know_tests: printc("Test %s is NEW in testsuite %s" - % (tname, testsuite.__file__), Colors.OKGREEN) + % (tname, testsuite.__file__), + Colors.FAIL if self.options.fail_on_testlist_change else Colors.OKGREEN) testlist_changed = True testlist_file.close() @@ -1610,7 +1611,7 @@ class _TestsLauncher(Loggable): tests = tester.list_tests() if self._check_defined_tests(tester, tests) and \ self.options.fail_on_testlist_change: - return -1 + raise RuntimeError("Unexpected new test in testsuite.") self.tests.extend(tests) return sorted(list(self.tests), key=lambda t: t.classname) @@ -1689,8 +1690,6 @@ class _TestsLauncher(Loggable): if not self.all_tests: all_tests = self.list_tests() - if all_tests == -1: - return False self.all_tests = all_tests self.total_num_tests = len(self.all_tests) |