twister: fix valgrind failure handling
falures found during valgrind execution were treated as normal test failures, so the logger would display a link to handler.log and not to valgrind.log, leading to some confusion. the return code is now handled in a correct sequence. Signed-off-by: Mikkel Jakobsen <mikkel.aunsbjerg@escolifesciences.com>
This commit is contained in:
parent
78aa424f69
commit
646499c36e
|
@ -633,13 +633,14 @@ class BinaryHandler(Handler):
|
|||
self.instance.results = harness.tests
|
||||
|
||||
if not self.terminated and self.returncode != 0:
|
||||
# When a process is killed, the default handler returns 128 + SIGTERM
|
||||
# so in that case the return code itself is not meaningful
|
||||
self.set_state("failed", handler_time)
|
||||
self.instance.reason = "Failed"
|
||||
elif run_valgrind and self.returncode == 2:
|
||||
self.set_state("failed", handler_time)
|
||||
self.instance.reason = "Valgrind error"
|
||||
if run_valgrind and self.returncode == 2:
|
||||
self.set_state("failed", handler_time)
|
||||
self.instance.reason = "Valgrind error"
|
||||
else:
|
||||
# When a process is killed, the default handler returns 128 + SIGTERM
|
||||
# so in that case the return code itself is not meaningful
|
||||
self.set_state("failed", handler_time)
|
||||
self.instance.reason = "Failed"
|
||||
elif harness.state:
|
||||
self.set_state(harness.state, handler_time)
|
||||
if harness.state == "failed":
|
||||
|
|
Loading…
Reference in New Issue