[PATCH] D46886: Escape ]]> in xunit xml output

[Alexander Richardson via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL332440: Escape ]]> in xunit xml output (authored by arichardson, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D46886?vs=146849&id=147016#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D46886

Files:
  llvm/trunk/utils/lit/lit/Test.py
  llvm/trunk/utils/lit/tests/Inputs/xunit-output/bad&name.ini
  llvm/trunk/utils/lit/tests/xunit-output.py


Index: llvm/trunk/utils/lit/tests/xunit-output.py
===================================================================
--- llvm/trunk/utils/lit/tests/xunit-output.py
+++ llvm/trunk/utils/lit/tests/xunit-output.py
@@ -1,12 +1,14 @@
 # Check xunit output
 # RUN: rm -rf %t.xunit.xml
 # RUN: not %{lit} --xunit-xml-output %t.xunit.xml %{inputs}/xunit-output
+# If xmllint is installed verify that the generated xml is well-formed
+# RUN: sh -c 'if command -v xmllint 2>/dev/null; then xmllint --noout %t.xunit.xml; fi'
 # RUN: FileCheck < %t.xunit.xml %s
 
 # CHECK: <?xml version="1.0" encoding="UTF-8" ?>
 # CHECK: <testsuites>
 # CHECK: <testsuite name='test-data' tests='1' failures='1' skipped='0'>
 # CHECK: <testcase classname='test-data.test-data' name='bad&name.ini' time='{{[0-1]}}.{{[0-9]+}}'>
-# CHECK-NEXT: <failure ><![CDATA[& < > "]]></failure>
+# CHECK-NEXT: <failure ><![CDATA[& < > ]]]]><![CDATA[> &"]]></failure>
 # CHECK: </testsuite>
 # CHECK: </testsuites>
Index: llvm/trunk/utils/lit/tests/Inputs/xunit-output/bad&name.ini
===================================================================
--- llvm/trunk/utils/lit/tests/Inputs/xunit-output/bad&name.ini
+++ llvm/trunk/utils/lit/tests/Inputs/xunit-output/bad&name.ini
@@ -1,7 +1,7 @@
 [global]
 result_code = FAIL
-result_output = & < > "
+result_output = & < > ]]> &"
 
 [results]
 value0 = 1
-value1 = 2.3456
\ No newline at end of file
+value1 = 2.3456
Index: llvm/trunk/utils/lit/lit/Test.py
===================================================================
--- llvm/trunk/utils/lit/lit/Test.py
+++ llvm/trunk/utils/lit/lit/Test.py
@@ -381,7 +381,9 @@
                 encoded_output = self.result.output.encode("utf-8", 'ignore')
             else:
                 encoded_output = self.result.output
-            fil.write(encoded_output)
+            # In the unlikely case that the output contains the CDATA terminator
+            # we wrap it by creating a new CDATA block
+            fil.write(encoded_output.replace("]]>", "]]]]><![CDATA[>"))
             fil.write("]]></failure>\n</testcase>")
         elif self.result.code == UNSUPPORTED:
             unsupported_features = self.getMissingRequiredFeatures()


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D46886.147016.patch
Type: text/x-patch
Size: 2203 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180516/4dd66148/attachment.bin>