michaelh/zephyr
1
0
Fork 0
Code Releases Activity

tests: twister: robot: Fix setting testcases status

Browse source 
Currently, when a Robot test is run, the test case status value is
reported as `None`, which causes the XML to contain information about
the test being skipped due to misconfiguration.

This commit fixes the value reported in Twister XML test result by
assigning the test result to the `status` variable of a testcase
instance after the test has finished running.

Signed-off-by: Franciszek Pindel <fpindel@internships.antmicro.com>
Signed-off-by: Piotr Zierhoffer <pzierhoffer@antmicro.com>
This commit is contained in:
Franciszek Pindel 2023-09-01 13:34:09 +02:00 • committed by Anas Nashif
commit a50343cf62
1 changed files with 5 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

5
scripts/pylib/twister/twisterlib/harness.py
View file

@ -143,10 +143,15 @@ class Robot(Harness):
if cmake_proc.returncode == 0: if cmake_proc.returncode == 0:
self.instance.status = "passed" self.instance.status = "passed"
# all tests in one Robot file are treated as a single test case,
# so its status should be set accordingly to the instance status
# please note that there should be only one testcase in testcases list
self.instance.testcases[0].status = "passed"
else: else:
logger.error("Robot test failure: %s for %s" % logger.error("Robot test failure: %s for %s" %
(handler.sourcedir, self.instance.platform.name)) (handler.sourcedir, self.instance.platform.name))
self.instance.status = "failed" self.instance.status = "failed"
self.instance.testcases[0].status = "failed"
if out: if out:
with open(os.path.join(self.instance.build_dir, handler.log), "wt") as log: with open(os.path.join(self.instance.build_dir, handler.log), "wt") as log: