sanitycheck: fix reporting of timeouts
Timeouts were not reported correctly in the XML file as failures, causing some confusion in the shippable results. Signed-off-by: Anas Nashif <anas.nashif@intel.com>
This commit is contained in:
parent
dbed9027ba
commit
e337882189
|
@ -3070,7 +3070,7 @@ class TestSuite:
|
||||||
for instance in self.instances.values():
|
for instance in self.instances.values():
|
||||||
handler_time = instance.metrics.get('handler_time', 0)
|
handler_time = instance.metrics.get('handler_time', 0)
|
||||||
duration += handler_time
|
duration += handler_time
|
||||||
if instance.status == "failed":
|
if instance.status in ["failed", "timeout"]:
|
||||||
if instance.reason in ['build_error', 'handler_crash']:
|
if instance.reason in ['build_error', 'handler_crash']:
|
||||||
errors += 1
|
errors += 1
|
||||||
else:
|
else:
|
||||||
|
@ -3118,7 +3118,7 @@ class TestSuite:
|
||||||
name="%s" % (instance.testcase.name),
|
name="%s" % (instance.testcase.name),
|
||||||
time="%f" % handler_time)
|
time="%f" % handler_time)
|
||||||
|
|
||||||
if instance.status == "failed":
|
if instance.status in ["failed", "timeout"]:
|
||||||
failure = ET.SubElement(
|
failure = ET.SubElement(
|
||||||
eleTestcase,
|
eleTestcase,
|
||||||
'failure',
|
'failure',
|
||||||
|
|
Loading…
Reference in a new issue