Handle a StackOverflowError in addition to NonFatal errors

if they are happening inside a test

I only execute the test on the JVM, because
- on JS the error message is different
- on Native the runner just terminates

munit/shared/src/main/scala/munit/MUnitRunner.scala

@@ -285,25 +285,31 @@ class MUnitRunner(val cls: Class[_ <: Suite], newInstance: () => Suite)
     }
 
     notifier.fireTestStarted(description)
+    def handleNonFatalOrStackOverflow(ex: Throwable): Future[Unit] = {
+      trimStackTrace(ex)
+      val cause = Exceptions.rootCause(ex)
+      val failure = new Failure(description, cause)
+      cause match {
+        case _: AssumptionViolatedException =>
+          notifier.fireTestAssumptionFailed(failure)
+        case _: FailSuiteException =>
+          suiteAborted = true
+          notifier.fireTestFailure(failure)
+        case _ =>
+          notifier.fireTestFailure(failure)
+      }
+      Future.successful(())
+    }
+
     val onError: PartialFunction[Throwable, Future[Unit]] = {
       case ex: AssumptionViolatedException =>
         trimStackTrace(ex)
         notifier.fireTestAssumptionFailed(new Failure(description, ex))
         Future.successful(())
       case NonFatal(ex) =>
-        trimStackTrace(ex)
-        val cause = Exceptions.rootCause(ex)
-        val failure = new Failure(description, cause)
-        cause match {
-          case _: AssumptionViolatedException =>
-            notifier.fireTestAssumptionFailed(failure)
-          case _: FailSuiteException =>
-            suiteAborted = true
-            notifier.fireTestFailure(failure)
-          case _ =>
-            notifier.fireTestFailure(failure)
-        }
-        Future.successful(())
+        handleNonFatalOrStackOverflow(ex)
+      case ex: StackOverflowError =>
+        handleNonFatalOrStackOverflow(ex)
     }
     val result: Future[Unit] =
       try runTestBody(notifier, description, test).recoverWith(onError)

tests/shared/src/main/scala/munit/Issue583FrameworkSuite.scala

@@ -0,0 +1,25 @@
+package munit
+
+class Issue583FrameworkSuite extends FunSuite {
+  test("simple test") {
+    ()
+  }
+  test("infinite loop test") {
+    def loop(x: Int): Int = loop(x) + loop(x)
+
+    loop(0)
+  }
+  test("another test") {
+    ()
+  }
+}
+
+object Issue583FrameworkSuite
+    extends FrameworkTest(
+      classOf[Issue583FrameworkSuite],
+      """|==> success munit.Issue583FrameworkSuite.simple test
+         |==> failure munit.Issue583FrameworkSuite.infinite loop test - null
+         |==> success munit.Issue583FrameworkSuite.another test
+         |""".stripMargin,
+      //tags = Set(OnlyJVM)
+    )

tests/shared/src/test/scala/munit/FrameworkSuite.scala

@@ -30,6 +30,7 @@ class FrameworkSuite extends BaseFrameworkSuite {
     Issue179FrameworkSuite,
     Issue285FrameworkSuite,
     Issue497FrameworkSuite,
+    Issue583FrameworkSuite,
     ScalaCheckExceptionFrameworkSuite,
     BoxedFrameworkSuite,
     SkippedFrameworkSuite