parser/gotest: Don't ignore output when there were no tests

In some cases we may have captured some output for a package that didn't
have any tests. We should include it in the generated report as it may
contain useful information.

Author: jstemmer

parser/gotest/report_builder.go

@@ -210,6 +210,8 @@ func (b *reportBuilder) CreatePackage(name, result string, duration time.Duratio
 				Name:   name,
 				Output: b.output.Get(globalID),
 			}
+		} else if b.output.Contains(globalID) {
+			pkg.Output = b.output.Get(globalID)
 		}
 		b.packages = append(b.packages, pkg)
 		b.output.Clear(globalID)

testdata/015-report.xml

@@ -4,5 +4,6 @@
 		<properties>
 			<property name="go.version" value="1.0"></property>
 		</properties>
+		<system-out><![CDATA[testing: warning: no tests to run]]></system-out>
 	</testsuite>
 </testsuites>

testdata/034-report.xml

@@ -9,6 +9,7 @@
 		<properties>
 			<property name="go.version" value="1.0"></property>
 		</properties>
+		<system-out><![CDATA[testing: warning: no tests to run]]></system-out>
 	</testsuite>
 	<testsuite name="package/name" tests="0" failures="0" errors="0" id="2" hostname="hostname" time="0.000" timestamp="2022-01-01T00:00:00Z">
 		<properties>

testdata/111-report.xml

@@ -4,5 +4,6 @@
 		<properties>
 			<property name="go.version" value="1.0"></property>
 		</properties>
+		<system-out><![CDATA[testing: warning: no tests to run]]></system-out>
 	</testsuite>
 </testsuites>