Skip to content

Commit

Permalink
Include summary in XML output for OFAIL tests
Browse files Browse the repository at this point in the history
Previously this attribute was omitted for all but failures,
so on OFAIL we didn't get our one-line summary of a test's
failure reason (usually the exception).

Including this for OFAIL results makes it quicker to see what
went wrong.
  • Loading branch information
jcsp authored and savex committed Sep 12, 2023
1 parent 7074119 commit 3ea364b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ducktape/tests/reporter.py
Original file line number Diff line number Diff line change
Expand Up @@ -230,7 +230,7 @@ def report(self):
name=name, classname=test.cls_name, time=str(test.run_time_seconds),
status=str(test.test_status), assertions=""
))
if test.test_status == FAIL:
if test.test_status == FAIL or test.test_status == OFAIL:
xml_failure = ET.SubElement(xml_testcase, 'failure', attrib=dict(
message=test.summary.splitlines()[0]
))
Expand Down

0 comments on commit 3ea364b

Please sign in to comment.