From 3ea364bfdc207c318633f5bb86479ba59ff7b958 Mon Sep 17 00:00:00 2001 From: John Spray Date: Thu, 14 Apr 2022 13:42:59 +0100 Subject: [PATCH] Include summary in XML output for OFAIL tests 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. --- ducktape/tests/reporter.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ducktape/tests/reporter.py b/ducktape/tests/reporter.py index 601f0fcba..82993a01b 100644 --- a/ducktape/tests/reporter.py +++ b/ducktape/tests/reporter.py @@ -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] ))