Add more logging for running tests with the test sub-command

Author: Gedochao

modules/build/src/main/scala/scala/build/internal/Runner.scala

@@ -3,7 +3,7 @@ package scala.build.internal
 import coursier.jvm.Execve
 import org.scalajs.jsenv.jsdomnodejs.JSDOMNodeJSEnv
 import org.scalajs.jsenv.nodejs.NodeJSEnv
-import org.scalajs.jsenv.{Input, RunConfig}
+import org.scalajs.jsenv.{Input, JSEnv, RunConfig}
 import org.scalajs.testing.adapter.TestAdapter as ScalaJsTestAdapter
 import sbt.testing.{Framework, Status}
 
@@ -378,14 +378,27 @@ object Runner {
 
   def frameworkNames(
     classPath: Seq[Path],
-    parentInspector: AsmTestRunner.ParentInspector
+    parentInspector: AsmTestRunner.ParentInspector,
+    logger: Logger
   ): Either[NoTestFrameworkFoundError, Seq[String]] = {
-    val foundFrameworkServices = AsmTestRunner.findFrameworkServices(classPath)
+    logger.debug("Looking for test framework services on the classpath...")
+    val foundFrameworkServices =
+      AsmTestRunner.findFrameworkServices(classPath)
+        .map(_.replace('/', '.').replace('\\', '.'))
+    logger.debug(s"Found ${foundFrameworkServices.length} test framework services.")
+    if foundFrameworkServices.nonEmpty then
+      logger.debug(s"  - ${foundFrameworkServices.mkString("\n  - ")}")
+    logger.debug("Looking for more test frameworks on the classpath...")
     val foundFrameworks =
       AsmTestRunner.findFrameworks(classPath, TestRunner.commonTestFrameworks, parentInspector)
-    val frameworks: Seq[String] =
-      (foundFrameworkServices ++ foundFrameworks)
         .map(_.replace('/', '.').replace('\\', '.'))
+    logger.debug(s"Found ${foundFrameworks.length} additional test frameworks")
+    if foundFrameworks.nonEmpty then
+      logger.debug(s"  - ${foundFrameworks.mkString("\n  - ")}")
+    val frameworks: Seq[String] = foundFrameworkServices ++ foundFrameworks
+    logger.log(s"Found ${frameworks.length} test frameworks in total")
+    if frameworks.nonEmpty then
+      logger.debug(s"  - ${frameworks.mkString("\n  - ")}")
     if frameworks.nonEmpty then Right(frameworks) else Left(new NoTestFrameworkFoundError)
   }
 
@@ -401,23 +414,30 @@ object Runner {
   ): Either[TestError, Int] = either {
     import org.scalajs.jsenv.Input
     import org.scalajs.jsenv.nodejs.NodeJSEnv
+    logger.debug("Preparing to run tests with Scala.js...")
+    logger.debug(s"Scala.js tests class path: $classPath")
     val nodePath = findInPath("node").fold("node")(_.toString)
-    val jsEnv =
-      if jsDom then
+    logger.debug(s"Node found at $nodePath")
+    val jsEnv: JSEnv =
+      if jsDom then {
+        logger.log("Loading JS environment with JS DOM...")
         new JSDOMNodeJSEnv(
           JSDOMNodeJSEnv.Config()
             .withExecutable(nodePath)
             .withArgs(Nil)
             .withEnv(Map.empty)
         )
-      else
+      }
+      else {
+        logger.log("Loading JS environment with Node...")
         new NodeJSEnv(
           NodeJSEnv.Config()
             .withExecutable(nodePath)
             .withArgs(Nil)
             .withEnv(Map.empty)
             .withSourceMap(NodeJSEnv.SourceMap.Disable)
         )
+      }
     val adapterConfig = ScalaJsTestAdapter.Config().withLogger(logger.scalaJsLogger)
     val inputs =
       Seq(if esModule then Input.ESModule(entrypoint.toPath) else Input.Script(entrypoint.toPath))
@@ -428,7 +448,7 @@ object Runner {
     val parentInspector = new AsmTestRunner.ParentInspector(classPath)
     val foundFrameworkNames: List[String] = testFrameworkOpt match {
       case some @ Some(_) => some.toList
-      case None           => value(frameworkNames(classPath, parentInspector)).toList
+      case None           => value(frameworkNames(classPath, parentInspector, logger)).toList
     }
 
     val res =
@@ -470,12 +490,13 @@ object Runner {
     args: Seq[String],
     logger: Logger
   ): Either[TestError, Int] = either {
+    logger.debug("Preparing to run tests with Scala Native...")
     logger.debug(s"Native tests class path: $classPath")
 
     val parentInspector = new AsmTestRunner.ParentInspector(classPath)
     val foundFrameworkNames: List[String] = frameworkNameOpt match {
       case Some(fw) => List(fw)
-      case None     => value(frameworkNames(classPath, parentInspector)).toList
+      case None     => value(frameworkNames(classPath, parentInspector, logger)).toList
     }
 
     val config = ScalaNativeTestAdapter.Config()

modules/cli/src/main/scala/scala/cli/commands/test/Test.scala

@@ -273,7 +273,7 @@ object Test extends ScalaCommand[TestOptions] {
     if classPath0.exists(_.contains("zio-test")) && !classPath0.exists(_.contains("zio-test-sbt"))
     then {
       val parentInspector = new AsmTestRunner.ParentInspector(classPath)
-      Runner.frameworkNames(classPath, parentInspector) match {
+      Runner.frameworkNames(classPath, parentInspector, logger) match {
         case Right(f) => f.headOption
         case Left(_) =>
           logger.message(

modules/cli/src/main/scala/scala/cli/exportCmd/MillProjectDescriptor.scala

@@ -149,7 +149,7 @@ final case class MillProjectDescriptor(
     }
     val parentInspector = new AsmTestRunner.ParentInspector(testClassPath)
     val frameworkName0 = options.testOptions.frameworkOpt.orElse {
-      frameworkNames(testClassPath, parentInspector).toOption
+      frameworkNames(testClassPath, parentInspector, logger).toOption
         .flatMap(_.headOption) // TODO: handle multiple frameworks here
     }
 

modules/cli/src/main/scala/scala/cli/exportCmd/SbtProjectDescriptor.scala

@@ -262,7 +262,7 @@ final case class SbtProjectDescriptor(
 
     val parentInspector = new AsmTestRunner.ParentInspector(testClassPath)
     val frameworkName0 = options.testOptions.frameworkOpt.orElse {
-      frameworkNames(testClassPath, parentInspector).toOption
+      frameworkNames(testClassPath, parentInspector, logger).toOption
         .flatMap(_.headOption) // TODO: handle multiple frameworks here
     }
 

modules/integration/src/test/scala/scala/cli/integration/TestTestDefinitions.scala

@@ -936,7 +936,10 @@ abstract class TestTestDefinitions extends ScalaCliSuite with TestScalaVersionAr
                |}
                |""".stripMargin
         ).fromRoot { root =>
-          val r = os.proc(TestUtil.cli, "test", extraOptions, ".", platformOptions).call(cwd = root)
+          val r =
+            os.proc(TestUtil.cli, "test", extraOptions, ".", platformOptions, "-v", "-v").call(cwd =
+              root
+            )
           val output = r.out.trim()
           expect(output.nonEmpty)
           expectedMessages.foreach { expectedMessage =>