diff options
author | Kenneth Graunke <kenneth@whitecape.org> | 2012-01-27 14:08:42 -0800 |
---|---|---|
committer | Kenneth Graunke <kenneth@whitecape.org> | 2012-02-01 14:07:40 -0800 |
commit | 7df8948f9117c0d59f1131d5332e0dbeed0f7795 (patch) | |
tree | 59ea0499a27d8a1b7260339035a2c5bbc75f5aa7 /framework | |
parent | 797b5745073db912b64612fc9106e29f220f68f3 (diff) |
framework: Don't override result to 'fail' on returncode != 0.
We do want to override interpretResult() for crashes, but not for generic
failures. So default to 'fail', call interpretResult(), then check for
crashes.
Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Chad Versace <chad.versace@linux.intel.com>
Diffstat (limited to 'framework')
-rw-r--r-- | framework/exectest.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/framework/exectest.py b/framework/exectest.py index d783fa57..04b0d3e4 100644 --- a/framework/exectest.py +++ b/framework/exectest.py @@ -78,6 +78,7 @@ class ExecTest(Test): results = TestResult() + results['result'] = 'fail' out = self.interpretResult(out, results) if proc.returncode == -5: @@ -95,7 +96,6 @@ class ExecTest(Test): # Windows EXCEPTION_INT_DIVIDE_BY_ZERO results['result'] = 'crash' elif proc.returncode != 0: - results['result'] = 'fail' results['note'] = 'Returncode was %d' % (proc.returncode) env = '' |