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 andrewhsu committed Jul 1, 2022
1 parent 0bde4bd commit 59aa1a4
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 59aa1a4

Please sign in to comment.