build: Disable Android debug variants outside the sample app (#5825)

runningcode · claude · web-flow · commit 02e6bc88fab2 · 2026-07-23T16:02:02.000+02:00
Unit tests already target the release variant (testBuildType = release)
and CI disabled the debug variant. Disable the debug variant locally too
so local builds match CI and skip building the unused variant. The
sample app keeps both variants for manual runs and profiling.

Update the contributor docs and test skill to reference
testReleaseUnitTest instead of testDebugUnitTest.

Co-authored-by: Claude Opus 4.8 &lt;noreply@anthropic.com&gt;
diff --git a/.claude/skills/test/SKILL.md b/.claude/skills/test/SKILL.md
@@ -40,9 +40,9 @@ Determine the Gradle test task:
 
 | Module Pattern | Test Task |
 |---------------|-----------|
-| `sentry-android-*` | `testDebugUnitTest` |
-| `sentry-compose*` | `testDebugUnitTest` |
-| `*-android` | `testDebugUnitTest` |
+| `sentry-android-*` | `testReleaseUnitTest` |
+| `sentry-compose*` | `testReleaseUnitTest` |
+| `*-android` | `testReleaseUnitTest` |
 | Everything else | `test` |
 
 **Interactive mode:** Before running, read the test class file and use AskUserQuestion to ask:
diff --git a/.cursor/rules/coding.mdc b/.cursor/rules/coding.mdc
@@ -24,7 +24,7 @@ sentry-java is the Java and Android SDK for Sentry. This repository contains the
 ./gradlew check
 
 # Run unit tests for a specific file
-./gradlew ':<module>:testDebugUnitTest' --tests="*<file name>*" --info
+./gradlew ':<module>:testReleaseUnitTest' --tests="*<file name>*" --info
 ```
 
 ## Contributing Guidelines
diff --git a/AGENTS.md b/AGENTS.md
@@ -44,13 +44,13 @@ The project uses **Gradle** with Kotlin DSL. Key build files:
 ### Testing
 ```bash
 # Run unit tests for a specific file
-./gradlew ':<module>:testDebugUnitTest' --tests="*<file name>*" --info
+./gradlew ':<module>:testReleaseUnitTest' --tests="*<file name>*" --info
 
 # Run system tests (requires Python virtual env)
 make systemTest
 
 # Run specific test suites
-./gradlew :sentry-android-core:testDebugUnitTest
+./gradlew :sentry-android-core:testReleaseUnitTest
 ./gradlew :sentry:test
 ```
 
diff --git a/buildSrc/src/main/java/Config.kt b/buildSrc/src/main/java/Config.kt
@@ -12,8 +12,10 @@ object Config {
     object Android {
         val abiFilters = listOf("x86", "armeabi-v7a", "x86_64", "arm64-v8a")
 
+        // Debug variants are disabled everywhere. Unit tests run against the release
+        // variant, so building the debug variant would only add overhead.
         fun shouldSkipDebugVariant(name: String?): Boolean {
-            return System.getenv("CI")?.toBoolean() ?: false && name == "debug"
+            return name == "debug"
         }
     }
 
diff --git a/sentry-android-core/build.gradle.kts b/sentry-android-core/build.gradle.kts
@@ -34,8 +34,8 @@ android {
     getByName("release") { consumerProguardFiles("proguard-rules.pro") }
   }
 
-  // AGP 9 only generates unit tests for the testBuildType. CI disables the debug
-  // variant, so unit tests must target release to run at all.
+  // AGP 9 only generates unit tests for the testBuildType. The debug variant is
+  // disabled, so unit tests must target release to run at all.
   testBuildType = "release"
 
   kotlin { compilerOptions.jvmTarget = JVM_1_8 }
@@ -83,7 +83,7 @@ tasks.withType<JavaCompile>().configureEach {
 // outputs so Gradle's build cache restores them on cache hits (otherwise the CLI upload step
 // finds an empty directory).
 tasks
-  .matching { it.name == "testDebugUnitTest" || it.name == "testReleaseUnitTest" }
+  .matching { it.name == "testReleaseUnitTest" }
   .configureEach { outputs.dir(layout.buildDirectory.dir("test-snapshots")) }
 
 dependencies {
diff --git a/sentry-android-distribution/build.gradle.kts b/sentry-android-distribution/build.gradle.kts
@@ -13,8 +13,8 @@ android {
   defaultConfig { minSdk = libs.versions.minSdk.get().toInt() }
   buildFeatures { buildConfig = false }
 
-  // AGP 9 only generates unit tests for the testBuildType. CI disables the debug
-  // variant, so unit tests must target release to run at all.
+  // AGP 9 only generates unit tests for the testBuildType. The debug variant is
+  // disabled, so unit tests must target release to run at all.
   testBuildType = "release"
 
   testOptions {
diff --git a/sentry-android-fragment/build.gradle.kts b/sentry-android-fragment/build.gradle.kts
@@ -25,8 +25,8 @@ android {
     getByName("release") { consumerProguardFiles("proguard-rules.pro") }
   }
 
-  // AGP 9 only generates unit tests for the testBuildType. CI disables the debug
-  // variant, so unit tests must target release to run at all.
+  // AGP 9 only generates unit tests for the testBuildType. The debug variant is
+  // disabled, so unit tests must target release to run at all.
   testBuildType = "release"
 
   kotlin {
diff --git a/sentry-android-navigation/build.gradle.kts b/sentry-android-navigation/build.gradle.kts
@@ -25,8 +25,8 @@ android {
     getByName("release") { consumerProguardFiles("proguard-rules.pro") }
   }
 
-  // AGP 9 only generates unit tests for the testBuildType. CI disables the debug
-  // variant, so unit tests must target release to run at all.
+  // AGP 9 only generates unit tests for the testBuildType. The debug variant is
+  // disabled, so unit tests must target release to run at all.
   testBuildType = "release"
 
   kotlin {
diff --git a/sentry-android-ndk/build.gradle.kts b/sentry-android-ndk/build.gradle.kts
@@ -26,8 +26,8 @@ android {
     getByName("release") { consumerProguardFiles("proguard-rules.pro") }
   }
 
-  // AGP 9 only generates unit tests for the testBuildType. CI disables the debug
-  // variant, so unit tests must target release to run at all.
+  // AGP 9 only generates unit tests for the testBuildType. The debug variant is
+  // disabled, so unit tests must target release to run at all.
   testBuildType = "release"
 
   kotlin { compilerOptions.jvmTarget = org.jetbrains.kotlin.gradle.dsl.JvmTarget.JVM_1_8 }
diff --git a/sentry-android-replay/build.gradle.kts b/sentry-android-replay/build.gradle.kts
@@ -34,8 +34,8 @@ android {
     getByName("release") { consumerProguardFiles("proguard-rules.pro") }
   }
 
-  // AGP 9 only generates unit tests for the testBuildType. CI disables the debug
-  // variant, so unit tests must target release to run at all.
+  // AGP 9 only generates unit tests for the testBuildType. The debug variant is
+  // disabled, so unit tests must target release to run at all.
   testBuildType = "release"
 
   kotlin {
diff --git a/sentry-android-sqlite/build.gradle.kts b/sentry-android-sqlite/build.gradle.kts
@@ -25,8 +25,8 @@ android {
     getByName("release") { consumerProguardFiles("proguard-rules.pro") }
   }
 
-  // AGP 9 only generates unit tests for the testBuildType. CI disables the debug
-  // variant, so unit tests must target release to run at all.
+  // AGP 9 only generates unit tests for the testBuildType. The debug variant is
+  // disabled, so unit tests must target release to run at all.
   testBuildType = "release"
 
   kotlin {
diff --git a/sentry-android-timber/build.gradle.kts b/sentry-android-timber/build.gradle.kts
@@ -32,8 +32,8 @@ android {
     getByName("release") { consumerProguardFiles("proguard-rules.pro") }
   }
 
-  // AGP 9 only generates unit tests for the testBuildType. CI disables the debug
-  // variant, so unit tests must target release to run at all.
+  // AGP 9 only generates unit tests for the testBuildType. The debug variant is
+  // disabled, so unit tests must target release to run at all.
   testBuildType = "release"
 
   kotlin {
diff --git a/sentry-compose/build.gradle.kts b/sentry-compose/build.gradle.kts
@@ -92,8 +92,8 @@ android {
     getByName("release") { consumerProguardFiles("proguard-rules.pro") }
   }
 
-  // AGP 9 only generates unit tests for the testBuildType. CI disables the debug
-  // variant, so unit tests must target release to run at all.
+  // AGP 9 only generates unit tests for the testBuildType. The debug variant is
+  // disabled, so unit tests must target release to run at all.
   testBuildType = "release"
 
   testOptions {
diff --git a/sentry-launchdarkly-android/build.gradle.kts b/sentry-launchdarkly-android/build.gradle.kts
@@ -27,8 +27,8 @@ android {
     getByName("release") { consumerProguardFiles("proguard-rules.pro") }
   }
 
-  // AGP 9 only generates unit tests for the testBuildType. CI disables the debug
-  // variant, so unit tests must target release to run at all.
+  // AGP 9 only generates unit tests for the testBuildType. The debug variant is
+  // disabled, so unit tests must target release to run at all.
   testBuildType = "release"
 
   kotlin { compilerOptions.jvmTarget = org.jetbrains.kotlin.gradle.dsl.JvmTarget.JVM_1_8 }
diff --git a/sentry-samples/sentry-samples-android/build.gradle.kts b/sentry-samples/sentry-samples-android/build.gradle.kts
@@ -114,6 +114,9 @@ android {
       // Suffix the id so debug and release builds can be installed side by side.
       applicationIdSuffix = ".debug"
       addManifestPlaceholders(mapOf("sentryDebug" to true, "sentryEnvironment" to "debug"))
+      // The SDK modules only publish a release variant, so fall back to it for the
+      // debug build of the sample.
+      matchingFallbacks += "release"
     }
     getByName("release") {
       isMinifyEnabled = true
@@ -133,10 +136,6 @@ android {
 
   kotlin { compilerOptions.jvmTarget = org.jetbrains.kotlin.gradle.dsl.JvmTarget.JVM_11 }
 
-  androidComponents.beforeVariants {
-    it.enable = !Config.Android.shouldSkipDebugVariant(it.buildType)
-  }
-
   androidComponents.onVariants { variant ->
     variant.buildConfigFields?.put(
       "USE_SAGP",

Original file line number	Diff line number	Diff line change
`@@ -12,8 +12,10 @@ object Config {`
`12`	`12`	`object Android {`
`13`	`13`	`val abiFilters = listOf("x86", "armeabi-v7a", "x86_64", "arm64-v8a")`
`14`	`14`
	`15`	`+ // Debug variants are disabled everywhere. Unit tests run against the release`
	`16`	`+ // variant, so building the debug variant would only add overhead.`
`15`	`17`	`fun shouldSkipDebugVariant(name: String?): Boolean {`
`16`		`- return System.getenv("CI")?.toBoolean() ?: false && name == "debug"`
	`18`	`+ return name == "debug"`
`17`	`19`	`}`
`18`	`20`	`}`
`19`	`21`
Original file line number	Diff line number	Diff line change
`@@ -25,8 +25,8 @@ android {`
`25`	`25`	`getByName("release") { consumerProguardFiles("proguard-rules.pro") }`
`26`	`26`	`}`
`27`	`27`
`28`		`- // AGP 9 only generates unit tests for the testBuildType. CI disables the debug`
`29`		`- // variant, so unit tests must target release to run at all.`
	`28`	`+ // AGP 9 only generates unit tests for the testBuildType. The debug variant is`
	`29`	`+ // disabled, so unit tests must target release to run at all.`
`30`	`30`	`testBuildType = "release"`
`31`	`31`
`32`	`32`	`kotlin {`
Original file line number	Diff line number	Diff line change
`@@ -26,8 +26,8 @@ android {`
`26`	`26`	`getByName("release") { consumerProguardFiles("proguard-rules.pro") }`
`27`	`27`	`}`
`28`	`28`
`29`		`- // AGP 9 only generates unit tests for the testBuildType. CI disables the debug`
`30`		`- // variant, so unit tests must target release to run at all.`
	`29`	`+ // AGP 9 only generates unit tests for the testBuildType. The debug variant is`
	`30`	`+ // disabled, so unit tests must target release to run at all.`
`31`	`31`	`testBuildType = "release"`
`32`	`32`
`33`	`33`	`kotlin { compilerOptions.jvmTarget = org.jetbrains.kotlin.gradle.dsl.JvmTarget.JVM_1_8 }`
Original file line number	Diff line number	Diff line change
`@@ -34,8 +34,8 @@ android {`
`34`	`34`	`getByName("release") { consumerProguardFiles("proguard-rules.pro") }`
`35`	`35`	`}`
`36`	`36`
`37`		`- // AGP 9 only generates unit tests for the testBuildType. CI disables the debug`
`38`		`- // variant, so unit tests must target release to run at all.`
	`37`	`+ // AGP 9 only generates unit tests for the testBuildType. The debug variant is`
	`38`	`+ // disabled, so unit tests must target release to run at all.`
`39`	`39`	`testBuildType = "release"`
`40`	`40`
`41`	`41`	`kotlin {`