From edbc0fadd13ccfe4c4791cc6077d93ea58930ff7 Mon Sep 17 00:00:00 2001
From: Mike McQuaid <mike@mikemcquaid.com>
Date: Sat, 29 Jun 2013 01:31:43 +0100
Subject: [PATCH] brew-test-bot: improve JUnit output for Jenkins.

---
 Library/Contributions/cmd/brew-test-bot.xml.erb | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/Library/Contributions/cmd/brew-test-bot.xml.erb b/Library/Contributions/cmd/brew-test-bot.xml.erb
index a835740f37..f5fb4079b7 100644
--- a/Library/Contributions/cmd/brew-test-bot.xml.erb
+++ b/Library/Contributions/cmd/brew-test-bot.xml.erb
@@ -3,11 +3,14 @@
   <% tests.each do |test| %>
   <testsuite name="brew-test-bot.<%= test.name %>" tests="<%= test.steps.count %>">
     <% test.steps.each do |step| %>
-    <testcase name="<%= step.command %>" status="<%= step.status %>" time="<%= step.time %>">
+    <testcase name="<%= step.command_short %>" status="<%= step.status %>" time="<%= step.time %>">
       <% if step.has_output? %>
+      <% if step.failed? %>
+      <failure message="<%= step.status %>: <%= step.command %>"><![CDATA[<%= step.output %>]]></failure>
+      <% else %>
       <system-out><![CDATA[<%= step.output %>]]></system-out>
       <% end %>
-      <% if step.failed? %>
+      <% elsif step.failed? %>
       <failure />
       <% end %>
     </testcase>