Make FailException extend AssertionError, fixes #215

junit-interface/src/main/java/munit/internal/junitinterface/TestFilter.java

@@ -45,7 +45,6 @@ public boolean shouldRun(Description d)
       if(Pattern.matches(p, testName)) return true;
 
     ignored.add(testName);
-    ed.testIgnored(d);
     return false;
   }
 }

munit/js/src/main/scala/java/io/File.scala

@@ -5,8 +5,6 @@ import java.nio.file.Path
 
 import munit.internal.{JSIO, JSPath, NodeNIOPath}
 
-import munit.internal._
-
 // obtained implementation by experimentation on the JDK.
 class File(path: String) {
   def this(parent: String, child: String) =

munit/shared/src/main/scala/munit/FailException.scala

@@ -5,7 +5,7 @@ class FailException(
     val cause: Throwable,
     val isStackTracesEnabled: Boolean,
     val location: Location
-) extends Exception(message, cause)
+) extends AssertionError(message, cause)
     with Serializable {
   def this(message: String, location: Location) =
     this(message, null, true, location)

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

@@ -252,12 +252,12 @@ class MUnitRunner(val cls: Class[_ <: Suite], newInstance: () => Suite)
           case e => e
         }
         val failure = new Failure(description, cause)
-        ex match {
-          case _: AssumptionViolatedException =>
-            notifier.fireTestAssumptionFailed(failure)
+        cause match {
           case _: FailSuiteException =>
             suiteAborted = true
             notifier.fireTestFailure(failure)
+          case _: AssumptionViolatedException =>
+            notifier.fireTestAssumptionFailed(failure)
           case _ =>
             notifier.fireTestFailure(failure)
         }

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

@@ -0,0 +1,7 @@
+package munit
+
+class FailExceptionSuite extends BaseSuite {
+  test("assertion-error") {
+    val error: AssertionError = new FailException("", Location.generate)
+  }
+}