Use innerhtml instead of getting text which doesn't work when we have <![CDATA[..]> sections inside the failure/error tags

Author: Anirudh Ramanathan

ruby/bin/annotate

@@ -1,6 +1,36 @@
 #!/usr/bin/env ruby
 
 require 'rexml/document'
+require "rexml/element"
+
+# Adapted from https://stackoverflow.com/a/25888239/759019
+class REXML::Element
+  def innerHTML=(xml)
+    require "rexml/document"
+    self.to_a.each do |e|
+      self.delete e
+    end
+    d = REXML::Document.new "<root>#{xml}</root>"
+    d.root.to_a.each do |e|
+      case e
+      when REXML::Text
+        self.add_text e
+      when REXML::Element
+        self.add_element e
+      else
+        puts "ERROR"
+      end
+    end
+    xml
+  end
+  def inner_html
+    ret = ''
+    self.to_a.each do |e|
+      ret += e.to_s
+    end
+    ret
+  end
+end
 
 # Reads a list of junit files and returns a nice Buildkite build annotation on
 # STDOUT that summarizes any failures.
@@ -33,10 +63,10 @@ junit_report_files.sort.each do |file|
     name = testcase.attributes['name'].to_s
     failed_test = testcase.attributes[failure_format].to_s
     testcase.elements.each("failure") do |failure|
-      failures << Failure.new(name, failed_test, failure.text, job, :failure)
+      failures << Failure.new(name, failed_test, failure.inner_html, job, :failure)
     end
     testcase.elements.each("error") do |error|
-      failures << Failure.new(name, failed_test, error.text, job, :error)
+      failures << Failure.new(name, failed_test, error.inner_html, job, :error)
     end
   end
 end