Previously a test would be marked with a note if the returncode was non-zero. This isn't very useful as no one looks at the note field. Instead, this patch marks tests that pass and return a non-zero status as warn, so a developer will actually see that.
Signed-off-by: Dylan Baker <[email protected]> --- framework/exectest.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/framework/exectest.py b/framework/exectest.py index 37d352b..effe12e 100644 --- a/framework/exectest.py +++ b/framework/exectest.py @@ -187,8 +187,8 @@ class Test(object): if returncode in crash_codes: results['result'] = 'crash' - elif returncode != 0: - results['note'] = 'Returncode was {0}'.format(returncode) + elif returncode != 0 and results['result'] == 'pass': + results['result'] = 'warn' if self.ENV.valgrind: # If the underlying test failed, simply report -- 1.9.2 _______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
