diff options
author | Christian Tismer <tismer@stackless.com> | 2017-12-05 14:17:18 +0100 |
---|---|---|
committer | Christian Tismer <tismer@stackless.com> | 2017-12-21 15:04:10 +0000 |
commit | 5f59ce344ffb1086b74e34fa24257b87d9325cd2 (patch) | |
tree | 80e46e8ef36257672678045dd459496dac4affbb | |
parent | b066cee55b290261e55ec4a5b11a87c417c2a4f0 (diff) |
testrunner 4: Improve the error listing
A simple change that makes the "FAIL!" entries easy to find.
Task-number: PYSIDE-578
Change-Id: I953bf94912f101208c1cddb7772aa0c34e9ca1a8
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
-rw-r--r-- | testing/command.py | 4 | ||||
-rw-r--r-- | testing/parser.py | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/testing/command.py b/testing/command.py index 9f91cdbfb..ec40c2ef8 100644 --- a/testing/command.py +++ b/testing/command.py @@ -192,7 +192,7 @@ def main(): for test, res in result.iter_blacklist(bl): print("%-6s" % res, ":", decorate(test) + "()") r[0] += 1 if res == "PASS" else 0 - r[1] += 1 if res == "FAIL" else 0 + r[1] += 1 if res == "FAIL!" else 0 r[2] += 1 if res == "SKIPPED" else 0 # not yet supported r[3] += 1 if res == "BFAIL" else 0 r[4] += 1 if res == "BPASS" else 0 @@ -219,7 +219,7 @@ def main(): runner = TestRunner(builds.selected, project) result = TestParser(runner.logfile) for test, res in result.iter_blacklist(bl): - if res == "FAIL": + if res == "FAIL!": raise ValueError("At least one failure was not blacklisted") # the makefile does run, although it does not find any tests. # We simply check if any tests were found. diff --git a/testing/parser.py b/testing/parser.py index e4b1bc013..6c0d2af6f 100644 --- a/testing/parser.py +++ b/testing/parser.py @@ -143,7 +143,7 @@ class TestParser(object): if match: res = "BFAIL" else: - res = "FAIL" + res = "FAIL!" else: if match: res = "BPASS" |