Merge pull request #601 from rpalcolea/issue-600/do-not-invoke-fireTe…

…stStarted-for-ignored-tests MUnitRunner: avoid invoking fireTestStarted if a test is skipped.
scalameta · Nov 10, 2022 · 47961eb · 47961eb
2 parents 0ec2259 + fa98de4
commit 47961eb
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/munit/shared/src/main/scala/munit/MUnitRunner.scala b/munit/shared/src/main/scala/munit/MUnitRunner.scala
@@ -279,11 +279,12 @@ class MUnitRunner(val cls: Class[_ <: Suite], newInstance: () => Suite)
       return Future.successful(false)
     }
 
-    notifier.fireTestStarted(description)
     if (test.tags(Ignore)) {
       notifier.fireTestIgnored(description)
       return Future.successful(false)
     }
+
+    notifier.fireTestStarted(description)
     val onError: PartialFunction[Throwable, Future[Unit]] = {
       case ex: AssumptionViolatedException =>
         trimStackTrace(ex)