Revert isTest check when reporting test finishing from JUnitPlatformRunnerListener

mannodermaus · mannodermaus · commit 600cd50ae7d4 · 2024-04-10T19:38:24.000+09:00
diff --git a/instrumentation/runner/src/main/kotlin/de/mannodermaus/junit5/internal/runners/AndroidJUnitPlatformRunnerListener.kt b/instrumentation/runner/src/main/kotlin/de/mannodermaus/junit5/internal/runners/AndroidJUnitPlatformRunnerListener.kt
@@ -67,9 +67,9 @@ internal class AndroidJUnitPlatformRunnerListener(
             notifier.fireTestAssumptionFailed(toFailure(testExecutionResult, description))
         } else if (status == TestExecutionResult.Status.FAILED) {
             notifier.fireTestFailure(toFailure(testExecutionResult, description))
+        } else if (testIdentifier.isTest) {
+            notifier.fireTestFinished(description)
         }
-
-        notifier.fireTestFinished(description)
     }
 
     override fun testPlanExecutionFinished(testPlan: TestPlan) {

Original file line number	Diff line number	Diff line change
`@@ -67,9 +67,9 @@ internal class AndroidJUnitPlatformRunnerListener(`
`67`	`67`	`notifier.fireTestAssumptionFailed(toFailure(testExecutionResult, description))`
`68`	`68`	`} else if (status == TestExecutionResult.Status.FAILED) {`
`69`	`69`	`notifier.fireTestFailure(toFailure(testExecutionResult, description))`
	`70`	`+ } else if (testIdentifier.isTest) {`
	`71`	`+ notifier.fireTestFinished(description)`
`70`	`72`	`}`
`71`		`-`
`72`		`- notifier.fireTestFinished(description)`
`73`	`73`	`}`
`74`	`74`
`75`	`75`	`override fun testPlanExecutionFinished(testPlan: TestPlan) {`