From c182319039822346e9bc3f31a796fd45dcc35450 Mon Sep 17 00:00:00 2001 From: Michael Kerrin Date: Wed, 29 May 2013 12:43:53 +0000 Subject: [PATCH] Fix colorizier thowing exception when a test fails This failure in colorizier was hidding the actual error in the tests. If we ran enough tests it also became difficult to see which test had actually failed. Fixes: bug 1183602 Change-Id: I806da8f13040b38258de7f3e0fa7b62700bdcead --- tools/colorizer.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/colorizer.py b/tools/colorizer.py index aa7427e406c7..a49abb141763 100755 --- a/tools/colorizer.py +++ b/tools/colorizer.py @@ -232,6 +232,8 @@ class NovaTestResult(testtools.TestResult): self._addResult(test, 'OK', 'green', '.', True) def addFailure(self, test, err): + if test.id() == 'process-returncode': + return super(NovaTestResult, self).addFailure(test, err) self._addResult(test, 'FAIL', 'red', 'F', False)