<div dir="ltr">This will solve the issue, and shouldn't have any side effects. I think you can actually just drop the super call (although, now that I look at it the code in Test probably actually belongs in PiglitTest, but I digress).</div><div class="gmail_extra"> <div class="gmail_quote">On Wed, Dec 16, 2015 at 8:03 AM, Daniel Vetter <<a href="mailto:daniel.vetter@ffwll.ch" target="_blank">daniel.vetter@ffwll.ch</a>> wrote: <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">In commit 5cbc1834fd47e7f475afc85e1638762e5b221a81 Author: Dylan Baker <<a href="mailto:baker.dylan.c@gmail.com">baker.dylan.c@gmail.com</a>> Date: Mon Dec 14 15:34:11 2015 -0800 framework/test/base.py: Handle fail cases for tests. the Test baseclass was made more robust in it's default behaviour, but that totally broke igt error code handling. CI is ... rather unhappy about it, so fix it asap. Cc: Dylan Baker <<a href="mailto:baker.dylan.c@gmail.com">baker.dylan.c@gmail.com</a>> Cc: Mark Janes <<a href="mailto:mark.a.janes@intel.com">mark.a.janes@intel.com</a>> Signed-off-by: Daniel Vetter <<a href="mailto:daniel.vetter@intel.com">daniel.vetter@intel.com</a>> --- tests/igt.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/igt.py b/tests/igt.py index 4998ab203a23..074df08cc88c 100644 --- a/tests/igt.py +++ b/tests/igt.py @@ -109,6 +109,8 @@ class IGTTest(Test): self.timeout = 600 def interpret_result(self): + super(IGTTest, self).interpret_result() + if self.result.returncode == 0: self.result.result = 'pass' elif self.result.returncode == 77: @@ -118,8 +120,6 @@ class IGTTest(Test): else: self.result.result = 'fail' - super(IGTTest, self).interpret_result() - def list_tests(listname): """Parse igt test list and return them as a list.""" -- 2.6.4 </blockquote></div> </div>