diff --git a/cmd/prune-junit-xml/prunexml.go b/cmd/prune-junit-xml/prunexml.go index 9baef9c5bea..6eaeba6fc84 100644 --- a/cmd/prune-junit-xml/prunexml.go +++ b/cmd/prune-junit-xml/prunexml.go @@ -65,15 +65,17 @@ func pruneXML(suites *junitxml.JUnitTestSuites, maxBytes int) { if testcase.SkipMessage != nil { if len(testcase.SkipMessage.Message) > maxBytes { fmt.Printf("clipping skip message in test case : %s\n", testcase.Name) - testcase.SkipMessage.Message = "[... clipped...]" + - testcase.SkipMessage.Message[len(testcase.SkipMessage.Message)-maxBytes:] + head := testcase.SkipMessage.Message[:maxBytes/2] + tail := testcase.SkipMessage.Message[len(testcase.SkipMessage.Message)-maxBytes/2:] + testcase.SkipMessage.Message = head + "[...clipped...]" + tail } } if testcase.Failure != nil { if len(testcase.Failure.Contents) > maxBytes { fmt.Printf("clipping failure message in test case : %s\n", testcase.Name) - testcase.Failure.Contents = "[... clipped...]" + - testcase.Failure.Contents[len(testcase.Failure.Contents)-maxBytes:] + head := testcase.Failure.Contents[:maxBytes/2] + tail := testcase.Failure.Contents[len(testcase.Failure.Contents)-maxBytes/2:] + testcase.Failure.Contents = head + "[...clipped...]" + tail } } } diff --git a/cmd/prune-junit-xml/prunexml_test.go b/cmd/prune-junit-xml/prunexml_test.go index 6f9aef6ee7b..343e2261c81 100644 --- a/cmd/prune-junit-xml/prunexml_test.go +++ b/cmd/prune-junit-xml/prunexml_test.go @@ -19,9 +19,10 @@ package main import ( "bufio" "bytes" - "github.com/stretchr/testify/assert" "strings" "testing" + + "github.com/stretchr/testify/assert" ) func TestPruneXML(t *testing.T) { @@ -49,10 +50,10 @@ func TestPruneXML(t *testing.T) { - + - [... clipped...]prometheus/client_metrics.go:160 + /home/prow/go/[...clipped...]t_metrics.go:160 `