Fix typos

project-builder/build-revision.sh

@@ -166,7 +166,7 @@ function buildForScalaVersion(){
   echo "started" > build-status.txt 
   # Mill
   # We check either for mill boostrap script or one of valid root build files
-  if [ -f "repo/mill" ] || [ -f "repo/build.mill" ] || [ -f "repo/build.mill.scala"] || [ -f "repo/build.sc"]; then
+  if [ -f "repo/mill" ] || [ -f "repo/build.mill" ] || [ -f "repo/build.mill.scala" ] || [ -f "repo/build.sc" ]; then
     echo "Mill project found: ${isMillProject}"
     echo "mill" > $buildToolFile
     $scriptDir/mill/prepare-project.sh "$project" repo "$scalaVersion" "$projectConfig"

project-builder/mill/MillCommunityBuild.sc

@@ -335,12 +335,9 @@ private def collectCompileResults(evalResult: EvalResult[CompilationResult]): Co
 
 private def collectTestResults(evalResult: EvalResult[Seq[TestResult]]): TestsResult = {
   import sbt.testing.Status
-  val empty = TestsResult(
+  val default = TestsResult(
     status = evalResult.toStatus,
-    passed = 0,
-    failed = 0,
-    ignored = 0,
-    skipped = 0,
+    overall = TestStats.empty,
     tookMs = evalResult.evalTime
   )
   evalResult match {
@@ -352,13 +349,15 @@ private def collectTestResults(evalResult: EvalResult[Seq[TestResult]]): TestsRe
 
       def countOf(selected: Status*) = selected.foldLeft(0)(_ + resultsStatus.getOrElse(_, 0))
 
-      empty.copy(
-        passed = countOf(Status.Success),
-        failed = countOf(Status.Error, Status.Failure, Status.Canceled),
-        ignored = countOf(Status.Ignored),
-        skipped = countOf(Status.Skipped)
+      default.copy(
+        overall = TestStats(
+          passed = countOf(Status.Success),
+          failed = countOf(Status.Error, Status.Failure, Status.Canceled),
+          ignored = countOf(Status.Ignored),
+          skipped = countOf(Status.Skipped)
+        )
       )
-    case _ => empty.copy(failureContext = evalResult.toBuildError)
+    case _ => default.copy(failureContext = evalResult.toBuildError)
   }
 }
 

project-builder/sbt/CommunityBuildPlugin.scala

@@ -710,8 +710,6 @@ object CommunityBuildPlugin extends AutoPlugin {
     val default = TestsResult(
       evalResult.toStatus,
       failureContext = evalResult.toBuildError,
-      overall = TestStats.empty,
-      byFramework = Map.empty,
       tookMs = evalResult.evalTime
     )
 

project-builder/scala-cli/build.scala

@@ -74,10 +74,7 @@ import os.CommandResult
     testsExecute = TestsResult(
       status = testsExecuteResults.toStatus,
       failureContext = testsExecuteResults.toBuildError,
-      passed = 0,
-      failed = 0,
-      ignored = 0,
-      skipped = 0,
+      overall = TestStats.empty,
       tookMs = testsExecuteResults.evalTime
     ),
     publish = publishResult

project-builder/shared/CommunityBuildCore.scala

@@ -136,7 +136,7 @@ object Scala3CommunityBuild {
   case class TestStats(passed: Int, failed: Int, ignored: Int, skipped: Int){
     val total: Int = passed + failed + ignored + skipped
     def toJson(inlined: Boolean = false) = {
-      val raw = s""""passed": ${passed}, "failed": ${failed}, "ignored": ${ignored}, "skipped": ${skipped}", "total": ${total}"""
+      val raw = s""""passed": ${passed}, "failed": ${failed}, "ignored": ${ignored}, "skipped": ${skipped}, "total": ${total}"""
       if (inlined) raw
       else s"{$raw}"
     }
@@ -146,10 +146,10 @@ object Scala3CommunityBuild {
   }
   case class TestsResult(
       status: Status,
+      tookMs: Int,
       failureContext: Option[FailureContext] = None,
-      overall: TestStats,
-      byFramework: Map[String, TestStats],
-      tookMs: Int
+      overall: TestStats = TestStats.empty,
+      byFramework: Map[String, TestStats] = Map.empty
   ) extends StepResult {
     def toJson = {
       val byFrameworkJson = byFramework.toSeq.sortBy(_._1).map{