From 0b4efddae0194c5b03a314d632e4c294e608fbab Mon Sep 17 00:00:00 2001
From: Patrick Ohly <patrick.ohly@intel.com>
Date: Fri, 2 Jun 2023 13:53:16 +0200
Subject: [PATCH] verify: nicer failure message rendering in Prow

The "message" attribute is redundant. Adding line breaks ensures that the curly
brackets used by Prow are on separate lines.

Before:
    {Script Error ScriptError test/e2e/e2e.go:437:5: var `unused` is unused (unused)
    var unused = 1 // intentionally trigger a golangci-lint warning.
        ^}

After:
    {ScriptError
    test/e2e/e2e.go:437:5: var `unused` is unused (unused)
    var unused = 1 // intentionally trigger a golangci-lint warning.
        ^
    }
---
 third_party/forked/shell2junit/sh2ju.sh | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/third_party/forked/shell2junit/sh2ju.sh b/third_party/forked/shell2junit/sh2ju.sh
index 29a820ac9d1..82d17f8b92c 100755
--- a/third_party/forked/shell2junit/sh2ju.sh
+++ b/third_party/forked/shell2junit/sh2ju.sh
@@ -153,7 +153,9 @@ function juLog() {
           failureMsg="${errMsg}"
       fi
       failure="
-      <failure type=\"ScriptError\" message=\"Script Error\"><![CDATA[${failureMsg}]]></failure>
+      <failure type=\"ScriptError\"><![CDATA[
+${failureMsg}
+]]></failure>
   "
   fi
   ## testcase tag