diff options
author | Thibault Saunier <thibault.saunier@collabora.com> | 2014-02-19 10:31:15 +0100 |
---|---|---|
committer | Thibault Saunier <thibault.saunier@collabora.com> | 2014-02-19 14:34:10 +0100 |
commit | f6bf92cbc96cb7f0f4d9f12ece36fa0275a37629 (patch) | |
tree | 40000450969534964f722f6726355804e5c8184c | |
parent | 33aae792f63bd377d54ba93a4c687813297125fe (diff) |
validate: Do not check result furthers if alredy set as passing
-rw-r--r-- | validate/tools/launcher/baseclasses.py | 6 |
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) |