summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorThibault Saunier <thibault.saunier@collabora.com>2014-02-19 10:31:15 +0100
committerThibault Saunier <thibault.saunier@collabora.com>2014-02-19 14:34:10 +0100
commitf6bf92cbc96cb7f0f4d9f12ece36fa0275a37629 (patch)
tree40000450969534964f722f6726355804e5c8184c
parent33aae792f63bd377d54ba93a4c687813297125fe (diff)
validate: Do not check result furthers if alredy set as passing
-rw-r--r--validate/tools/launcher/baseclasses.py6
1 files changed, 4 insertions, 2 deletions
diff --git a/validate/tools/launcher/baseclasses.py b/validate/tools/launcher/baseclasses.py
index 0874dd5..f6a78df 100644
--- a/validate/tools/launcher/baseclasses.py
+++ b/validate/tools/launcher/baseclasses.py
@@ -89,6 +89,8 @@ class Test(Loggable):
pass
def set_result(self, result, message="", error=""):
+ self.debug("Setting result: %s (message: %s, error: %s", result,
+ message, error)
self.result = result
self.message = message
self.error_str = error
@@ -101,7 +103,7 @@ class Test(Loggable):
if self.result == Result.TIMEOUT:
self.set_result(Result.TIMEOUT, "Application timed out", "timeout")
elif self.process.returncode == 0:
- self.result = Result.PASSED
+ self.set_result(Result.PASSED)
else:
self.set_result(Result.FAILED,
"Application returned %d" % (
@@ -244,7 +246,7 @@ class GstValidateTest(Test):
return ret + "]"
def check_results(self):
- if self.result is Result.FAILED:
+ if self.result is Result.FAILED or self.result is Result.PASSED:
return
self.debug("%s returncode: %s", self, self.process.returncode)