Add output for webworker tests

Author: armanbilge

webworker/src/main/scala/org/scalajs/macrotaskexecutor/MacrotaskExecutorSuiteRunner.scala

@@ -37,15 +37,39 @@ object MacrotaskExecutorSuiteRunner {
       classOf[MacrotaskExecutorSuite],
       () => new MacrotaskExecutorSuite
     ).runAsync(new RunNotifier {
+
+      var count = new MacrotaskExecutorSuite().munitTests().size
+      var overallSuccess = true
+      def reportTest(success: Boolean): Unit = {
+        overallSuccess &= success
+        count -= 1
+        if (count == 0) postMessage(overallSuccess)
+      }
+
       def fireTestStarted(description: Description): Unit = ()
-      def fireTestSuiteStarted(description: Description): Unit = ()
+
+      def fireTestSuiteStarted(description: Description): Unit =
+        postMessage(s"${classOf[MacrotaskExecutorSuite].getName}:")
+
+      // This doesn't account for async and fires before any tests are run!
       def fireTestSuiteFinished(description: Description): Unit = ()
+
       def fireTestIgnored(description: Description): Unit = ()
-      def fireTestFinished(description: Description): Unit = ()
-      def fireTestFailure(failure: Failure): Unit = postMessage(false)
-      def fireTestAssumptionFailed(failure: Failure): Unit = postMessage(false)
-    }).onComplete {
-      case util.Success(_) => postMessage(true)
-      case util.Failure(_) => postMessage(false)
-    }
+
+      def fireTestFinished(description: Description): Unit = {
+        postMessage(s"  + ${description.getMethodName}")
+        reportTest(success = true)
+      }
+
+      def fireTestFailure(failure: Failure): Unit = {
+        postMessage(
+          s"==> X ${classOf[MacrotaskExecutorSuite].getName}.${failure.description.getMethodName}"
+        )
+        reportTest(success = false)
+      }
+
+      def fireTestAssumptionFailed(failure: Failure): Unit =
+        reportTest(success = false)
+
+    })
 }

webworker/src/test/scala/org/scalajs/macrotaskexecutor/WebWorkerMacrotaskSuite.scala

@@ -26,7 +26,7 @@ import scala.util.Try
 class WebWorkerMacrotaskSuite extends FunSuite {
 
   import MacrotaskExecutor.Implicits._
-  
+
   def scalaVersion = if (BuildInfo.scalaVersion.startsWith("2"))
     BuildInfo.scalaVersion.split("\\.").init.mkString(".")
   else
@@ -37,10 +37,12 @@ class WebWorkerMacrotaskSuite extends FunSuite {
   Try(js.isUndefined(js.Dynamic.global.window.Worker)).toOption
     .filterNot(identity)
     .foreach { _ =>
-      test("macrotask executor should pass the suite on a webworker") {
+      test("pass the MacrotaskSuite in a web worker") {
         val p = Promise[Boolean]()
 
-        val worker = new Worker(s"file://${targetDir}/scala-js-macrotask-executor-webworker-fastopt/main.js")
+        val worker = new Worker(
+          s"file://${targetDir}/scala-js-macrotask-executor-webworker-fastopt/main.js"
+        )
 
         worker.onmessage = { event =>
           event.data match {