Missing timestamps

[cpuguy83]

I was looking at importing test results into an Azure DevOps pipline and noticed that the test results don't have timestamps for the testsuites even though test2json includes this in even stream.

What ends up happening is the importer makes the assumption the test run is for time.Now() instead of when the test was actually run.
Would you be opposed a change to add this?

[cpuguy83]

I think even adding a timestamp with time.Now() here would be better than omitting it:

gotestsum/internal/junitxml/report.go

Lines 88 to 95 in bdcb1a9

	junitpkg := JUnitTestSuite{
	Name: cfg.FormatTestSuiteName(pkgname),
	Tests: pkg.Total,
	Time: formatDurationAsSeconds(pkg.Elapsed()),
	Properties: packageProperties(version),
	TestCases: packageTestCases(pkg, cfg.FormatTestCaseClassname),
	Failures: len(pkg.Failed),
	}

[dnephin]

I think this would be a good addition, thanks for opening the issue! The TestEvent struct has a timestamp field, but it never gets copied over to the TestCase struct. That would let us use the real timestamp from the TestEvent.

Do you know where this data is supposed to show up in the Junit XML? Do we need to add a new tag or is it an attribute on TestSuite ?

The current time attribute is being set to the elapsed time not wall clock time, so hopefully it's not that same attribute.

[cpuguy83]

According to this schema, there is a timestamp attribute on testsuite

[cpuguy83]

#201