From 4ce910564c2c604fc0f2a9ae9e70d817e4fc4646 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jo=C3=ABl=20Stemmer?= <stemmertech@gmail.com>
Date: Sat, 13 Aug 2022 21:59:50 +0100
Subject: [PATCH] parser/gotest: timestampFunc should always be set, skip nil
 check

---
 parser/gotest/report_builder.go | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/parser/gotest/report_builder.go b/parser/gotest/report_builder.go
index f69164b..388d122 100644
--- a/parser/gotest/report_builder.go
+++ b/parser/gotest/report_builder.go
@@ -187,12 +187,9 @@ func (b *reportBuilder) CreateBuildError(packageName string) {
 // so far. Afterwards all state is reset.
 func (b *reportBuilder) CreatePackage(name, result string, duration time.Duration, data string) {
 	pkg := gtr.Package{
-		Name:     name,
-		Duration: duration,
-	}
-
-	if b.timestampFunc != nil {
-		pkg.Timestamp = b.timestampFunc()
+		Name:      name,
+		Duration:  duration,
+		Timestamp: b.timestampFunc(),
 	}
 
 	// Build errors are treated somewhat differently. Rather than having a