Change default unsupported device behavior to "throw exception"

Author: mannodermaus

instrumentation/runner/src/main/kotlin/de/mannodermaus/junit5/internal/runners/DummyJUnitFramework.kt

@@ -25,18 +25,14 @@ internal class DummyJUnitFramework(
 
     override fun run(notifier: RunNotifier) {
         when (behaviorForUnsupportedDevices) {
-            "skip" -> skipTests(notifier)
             "fail" -> failExecution(unsupportedDeviceMessage)
-            else -> {
-                Log.w(
-                    LOG_TAG,
+            "skip" -> skipTests(notifier)
+            else ->
+                failExecution(
                     "Unknown value found for configuration parameter " +
                         "'${ConfigurationParameters.BEHAVIOR_FOR_UNSUPPORTED_DEVICES}': " +
-                        "$behaviorForUnsupportedDevices. Apply default behavior " +
-                        "and skip tests for this class.",
+                        "$behaviorForUnsupportedDevices"
                 )
-                skipTests(notifier)
-            }
         }
     }
 

plugin/CHANGELOG.md

@@ -7,8 +7,8 @@ Change Log
 - Support instrumentation with JUnit 5 and 6 (the plugin will choose the correct runtime accordingly)
 - Introduce `de.mannodermaus.android-junit-framework` as the new plugin ID
 - New: Control behavior of test execution on unsupported devices via `instrumentationTests.behaviorForUnsupportedDevices`
-  - "Skip": Skip tests and mark them as ignored (default and equal to previous behavior)
-  - "Fail": Throw an exception and fail test execution
+  - "Fail": Throw an exception and fail test execution (**this is the new default**)
+  - "Skip": Skip tests and mark them as ignored (**this is the old behavior**)
 
 ## 1.14.0.0 (2025-10-10)
 - JUnit 5.14.0

plugin/android-junit5/src/main/kotlin/de/mannodermaus/gradle/plugins/junit5/dsl/AndroidJUnitPlatformExtension.kt

@@ -75,7 +75,7 @@ constructor(project: Project, private val objects: ObjectFactory) : GroovyObject
         objects.newInstance(InstrumentationTestOptions::class.java).apply {
             enabled.convention(true)
             version.convention(Libraries.Instrumentation.version)
-            behaviorForUnsupportedDevices.convention(UnsupportedDeviceBehavior.Skip)
+            behaviorForUnsupportedDevices.convention(UnsupportedDeviceBehavior.Fail)
             includeExtensions.convention(false)
             useConfigurationParameters.convention(true)
         }

plugin/android-junit5/src/test/kotlin/de/mannodermaus/gradle/plugins/junit5/plugin/InstrumentationSupportTests.kt

@@ -88,21 +88,21 @@ internal class InstrumentationSupportTests {
         val runnerArgs =
             project.android.defaultConfig.testInstrumentationRunnerArguments[
                     "configurationParameters"]
-        assertThat(runnerArgs).contains("de.mannodermaus.junit.unsupported.behavior=skip")
+        assertThat(runnerArgs).contains("de.mannodermaus.junit.unsupported.behavior=fail")
     }
 
     @Test
     fun `can configure the unsupported device behavior via dsl`() {
         project.addJUnit(JUnit5, "androidTest")
         project.junitPlatform.instrumentationTests.behaviorForUnsupportedDevices.set(
-            UnsupportedDeviceBehavior.Fail
+            UnsupportedDeviceBehavior.Skip
         )
         project.evaluate()
 
         val runnerArgs =
             project.android.defaultConfig.testInstrumentationRunnerArguments[
                     "configurationParameters"]
-        assertThat(runnerArgs).contains("de.mannodermaus.junit.unsupported.behavior=fail")
+        assertThat(runnerArgs).contains("de.mannodermaus.junit.unsupported.behavior=skip")
     }
 
     @Test