Make basic testing working

.gitignore

@@ -46,3 +46,4 @@ website/i18n/*
 !website/i18n/en.json
 
 project/metals.sbt
+out/

build.sbt

@@ -4,7 +4,10 @@ inThisBuild(
   List(
     scalaVersion := scala213,
     crossScalaVersions := List(scala213, scala212),
-    fork := true
+    fork := true,
+    testFrameworks := List(
+      new TestFramework("com.geirsson.junit.JUnitFramework")
+    )
   )
 )
 

munit/src/main/scala/munit/Runner.scala

@@ -99,19 +99,23 @@ final class Runner(cls: Class[_ <: Suite]) extends org.junit.runner.Runner {
 
   def runAll(notifier: RunNotifier): Unit = {
     try {
-      val canStart = runBeforeAll(notifier)
-      if (canStart) {
+      val isContinue = runBeforeAll(notifier)
+      if (isContinue) {
         suite.tests.foreach { test =>
           val description = Description.createTestDescription(cls, test.name)
-          notifier.fireTestStarted(description)
-          try {
-            StackMarker.dropOutside(test.body())
-          } catch {
-            case ex: Throwable =>
-              StackMarker.trimStackTrace(ex)
-              notifier.fireTestFailure(new Failure(description, ex))
-          } finally {
-            notifier.fireTestFinished(description)
+          var isContinue = runBeforeEach(notifier, test)
+          if (isContinue) {
+            notifier.fireTestStarted(description)
+            try {
+              StackMarker.dropOutside(test.body())
+            } catch {
+              case ex: Throwable =>
+                StackMarker.trimStackTrace(ex)
+                notifier.fireTestFailure(new Failure(description, ex))
+            } finally {
+              notifier.fireTestFinished(description)
+              runAfterEach(notifier, test)
+            }
           }
         }
       }

munit/src/main/scala/munit/internal/StackMarker.scala

@@ -6,6 +6,7 @@ import java.{util => ju}
   * Wrapper-functions that can be used to mark parts of the callstack that are
   * meant to be filtered out later.
   */
+class StackMarker
 object StackMarker {
   // Ask Scalac/Scala.js nicely to try and avoid inlining these two marker methods,
   // to make sure they don't disappear from the stack traces
@@ -19,24 +20,27 @@ object StackMarker {
       new ju.IdentityHashMap[Throwable, java.lang.Boolean]()
     )
     def loop(e: Throwable): Unit = {
-      if (!isVisited.contains(e)) {
-        isVisited.add(e)
-        e.setStackTrace(filterCallStack(e.getStackTrace()))
+      if (e != null && isVisited.add(e)) {
+        val stack = e.getStackTrace()
+        if (stack != null) {
+          e.setStackTrace(filterCallStack(stack))
+        }
         loop(e.getCause())
       }
     }
     loop(ex)
   }
+  val className = classOf[StackMarker].getCanonicalName() + "$"
   def filterCallStack(
       stack: Array[StackTraceElement]
   ): Array[StackTraceElement] = {
     val droppedInside = stack.indexWhere(x =>
-      x.getClassName == "munit.internal.StackMarker$" &&
+      x.getClassName == className &&
         x.getMethodName == "dropInside"
     )
 
     val droppedOutside = stack.indexWhere(x =>
-      x.getClassName == "munit.internal.StackMarker$" &&
+      x.getClassName == className &&
         x.getMethodName == "dropOutside"
     )
 

munit/src/test/scala/munit/BasicSuite.scala

@@ -0,0 +1,30 @@
+package munit
+
+import org.junit.runner.RunWith
+
+@RunWith(classOf[Runner])
+class BasicSuite extends Suite {
+  override def beforeAll(context: BeforeAll): Unit = {
+    println("beforeAll")
+  }
+  override def afterAll(context: AfterAll): Unit = {
+    println("afterAll")
+  }
+  override def beforeEach(context: BeforeEach): Unit = {
+    println("beforeEach: " + context.test.name)
+  }
+  override def afterEach(context: AfterEach): Unit = {
+    println("afterEach: " + context.test.name)
+  }
+  test("not-implemented") {
+    println("not-implemented")
+    Thread.sleep(200)
+    ???
+  }
+  test("pass") {
+    Thread.sleep(400)
+    println("pass")
+    println("pass")
+    assert(1 == 1)
+  }
+}