From 85583c6355e7f33b4000102dd1463375b35873f6 Mon Sep 17 00:00:00 2001
From: Romain Gilles <58700468+romain-gilles-ultra@users.noreply.github.com>
Date: Wed, 22 May 2024 09:48:50 +0200
Subject: [PATCH] Fix timestamp attribute in junit xml report (#3172)

There is a bug in the way the `testsuite` element is generated.
### Actual
```
    <testsuite name="MyTest" tests="4" failures="0" errors="0" skipped="0"
               time="0.724">
        timestamp=2024-05-21T18:57:25
        ...
    </testsuite>
```
### Expected
```
    <testsuite name="MyTest" tests="4" failures="0" errors="0" skipped="0"
               time="0.724" timestamp="2024-05-21T18:57:25">
        ...
    </testsuite>
```

The `timestamp` attribute in not an attribute...

Pull request: https://github.com/com-lihaoyi/mill/pull/3172
---
 scalalib/src/mill/scalalib/TestModule.scala | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/scalalib/src/mill/scalalib/TestModule.scala b/scalalib/src/mill/scalalib/TestModule.scala
index c867a8115af..8b58e4c64df 100644
--- a/scalalib/src/mill/scalalib/TestModule.scala
+++ b/scalalib/src/mill/scalalib/TestModule.scala
@@ -387,8 +387,8 @@ object TestModule {
                  failures={testResults.count(_.status == Status.Failure.toString).toString}
                  errors={testResults.count(_.status == Status.Error.toString).toString}
                  skipped={testResults.count(_.status == Status.Skipped.toString).toString}
-                 time={(testResults.map(_.duration).sum / 1000.0).toString}>
-                 timestamp={timestamp}
+                 time={(testResults.map(_.duration).sum / 1000.0).toString}
+                 timestamp={timestamp}>
         {properties}
         {cases}
       </testsuite>