From 050e22e86b11ee0e0725acd8b76e45997101cdc2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jo=C3=ABl=20Stemmer?= <stemmertech@gmail.com>
Date: Tue, 8 Oct 2019 00:44:52 +0100
Subject: [PATCH] junit: Omit Testcase attributes if they're empty

---
 pkg/junit/junit.go | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/pkg/junit/junit.go b/pkg/junit/junit.go
index 69b45ad..c521075 100644
--- a/pkg/junit/junit.go
+++ b/pkg/junit/junit.go
@@ -82,8 +82,8 @@ type Testcase struct {
 	Classname string `xml:"classname,attr"`
 
 	// optional attributes
-	Time   string `xml:"time,attr"` // duration in seconds
-	Status string `xml:"status,attr"`
+	Time   string `xml:"time,attr,omitempty"` // duration in seconds
+	Status string `xml:"status,attr,omitempty"`
 
 	Skipped   *Result `xml:"skipped,omitempty"`
 	Error     *Result `xml:"error,omitempty"`
@@ -110,3 +110,9 @@ type Result struct {
 func FormatDuration(d time.Duration) string {
 	return fmt.Sprintf("%.3f", d.Seconds())
 }
+
+// FormatBenchmarkTime returns the JUnit string representation of the given
+// benchmark time.
+func FormatBenchmarkTime(d time.Duration) string {
+	return fmt.Sprintf("%.9f", d.Seconds())
+}