Remove obsoleted tests and platformDir from Paparazzi config

paparazzi-gradle-plugin/api/paparazzi-gradle-plugin.api

@@ -14,7 +14,6 @@ public abstract class app/cash/paparazzi/gradle/PrepareResourcesTask : org/gradl
 	public abstract fun getAarAssetDirs ()Lorg/gradle/api/provider/ListProperty;
 	public abstract fun getAarExplodedDirs ()Lorg/gradle/api/provider/ListProperty;
 	public abstract fun getArtifactFiles ()Lorg/gradle/api/file/ConfigurableFileCollection;
-	public abstract fun getCompileSdkVersion ()Lorg/gradle/api/provider/Property;
 	public abstract fun getModuleResourceDirs ()Lorg/gradle/api/provider/ListProperty;
 	public abstract fun getNonTransitiveRClassEnabled ()Lorg/gradle/api/provider/Property;
 	public abstract fun getPackageName ()Lorg/gradle/api/provider/Property;

paparazzi-gradle-plugin/src/main/java/app/cash/paparazzi/gradle/PaparazziPlugin.kt

@@ -152,7 +152,6 @@ public class PaparazziPlugin : Plugin<Project> {
         task.artifactFiles.from(packageAwareArtifactFiles)
         task.nonTransitiveRClassEnabled.set(nonTransitiveRClassEnabled)
         task.targetSdkVersion.set(android.targetSdkVersion())
-        task.compileSdkVersion.set(android.compileSdkVersion())
 
         val localResourcePaths = localResourceDirs
           .map { layers -> layers.flatten() }
@@ -343,10 +342,6 @@ public class PaparazziPlugin : Plugin<Project> {
 
   private fun BaseExtension.packageName(): String = namespace ?: ""
 
-  private fun BaseExtension.compileSdkVersion(): String {
-    return compileSdkVersion!!.substringAfter("android-", DEFAULT_COMPILE_SDK_VERSION.toString())
-  }
-
   private fun BaseExtension.targetSdkVersion(): String {
     return defaultConfig.targetSdkVersion?.apiLevel?.toString()
       ?: DEFAULT_COMPILE_SDK_VERSION.toString()

paparazzi-gradle-plugin/src/main/java/app/cash/paparazzi/gradle/PrepareResourcesTask.kt

@@ -38,9 +38,6 @@ public abstract class PrepareResourcesTask : DefaultTask() {
   @get:Input
   public abstract val targetSdkVersion: Property<String>
 
-  @get:Input
-  public abstract val compileSdkVersion: Property<String>
-
   @get:Input
   public abstract val projectResourceDirs: ListProperty<String>
 
@@ -86,8 +83,6 @@ public abstract class PrepareResourcesTask : DefaultTask() {
     val config = Config(
       mainPackage = mainPackage,
       targetSdkVersion = targetSdkVersion.get(),
-      // Use compileSdkVersion for system framework resources.
-      platformDir = "platforms/android-${compileSdkVersion.get()}/",
       resourcePackageNames = resourcePackageNames,
       projectResourceDirs = projectResourceDirs.get(),
       moduleResourceDirs = moduleResourceDirs.get(),
@@ -103,7 +98,6 @@ public abstract class PrepareResourcesTask : DefaultTask() {
   internal data class Config(
     val mainPackage: String,
     val targetSdkVersion: String,
-    val platformDir: String,
     val resourcePackageNames: List<String>,
     val projectResourceDirs: List<String>,
     val moduleResourceDirs: List<String>,

paparazzi-gradle-plugin/src/test/java/app/cash/paparazzi/gradle/PaparazziPluginTest.kt

@@ -238,35 +238,6 @@ class PaparazziPluginTest {
     assertThat(snapshotsDir.exists()).isFalse()
   }
 
-  @Test
-  fun buildClassNextSdkAccess() {
-    val fixtureRoot = File("src/test/projects/build-class-next-sdk")
-
-    // Paparazzi detects Android platform dir contents to be static. Therefore, it re-runs only on
-    // compileSdk changes.  Sandbox previews are an exception, so let's disable caching for this
-    // test task.
-    gradleRunner
-      .withArguments("testDebug", "-Dorg.gradle.caching=false", "--stacktrace")
-      .runFixture(fixtureRoot) { build() }
-
-    val snapshotsDir = File(fixtureRoot, "custom/reports/paparazzi/debug/images")
-    assertThat(snapshotsDir.exists()).isFalse()
-  }
-
-  @Test
-  fun missingPlatformDirTest() {
-    val fixtureRoot = File("src/test/projects/missing-platform-dir")
-
-    val result = gradleRunner
-      .withArguments("testDebug", "--stacktrace")
-      .forwardOutput()
-      .runFixture(fixtureRoot) { buildAndFail() }
-
-    assertThat(result.task(":testDebug")).isNull()
-    assertThat(result.output).contains("java.io.FileNotFoundException")
-    assertThat(result.output).contains("Missing platform version oops")
-  }
-
   @Test
   fun flagDebugLinkedObjectsIsOff() {
     val fixtureRoot = File("src/test/projects/flag-debug-linked-objects-off")
@@ -1233,42 +1204,6 @@ class PaparazziPluginTest {
       .containsExactly("^caches/transforms-4/[0-9a-f]{32}/transformed/external2/assets\$")
   }
 
-  @Test
-  fun verifyTargetSdkIsSameAsCompileSdk() {
-    val fixtureRoot = File("src/test/projects/verify-resources-java")
-
-    val result = gradleRunner
-      .withArguments(":consumer:compileDebugUnitTestJavaWithJavac", "--stacktrace")
-      .runFixture(fixtureRoot) { build() }
-
-    assertThat(result.task(":consumer:preparePaparazziDebugResources")).isNotNull()
-
-    val resourcesFile = File(fixtureRoot, "consumer/build/intermediates/paparazzi/debug/resources.json")
-    assertThat(resourcesFile.exists()).isTrue()
-
-    val config = resourcesFile.loadConfig()
-    assertThat(config.targetSdkVersion).isEqualTo("34")
-    assertThat(config.platformDir).isEqualTo("platforms/android-34/")
-  }
-
-  @Test
-  fun verifyTargetSdkIsDifferentFromCompileSdk() {
-    val fixtureRoot = File("src/test/projects/different-target-sdk")
-
-    val result = gradleRunner
-      .withArguments("compileDebugUnitTestJavaWithJavac", "--stacktrace")
-      .runFixture(fixtureRoot) { build() }
-
-    assertThat(result.task(":preparePaparazziDebugResources")).isNotNull()
-
-    val resourcesFile = File(fixtureRoot, "build/intermediates/paparazzi/debug/resources.json")
-    assertThat(resourcesFile.exists()).isTrue()
-
-    val config = resourcesFile.loadConfig()
-    assertThat(config.targetSdkVersion).isEqualTo("29")
-    assertThat(config.platformDir).isEqualTo("platforms/android-34/")
-  }
-
   @Test
   fun verifyOpenAssets() {
     val fixtureRoot = File("src/test/projects/open-assets")

paparazzi/api/paparazzi.api

@@ -62,9 +62,9 @@ public final class app/cash/paparazzi/DeviceConfig$Companion {
 
 public final class app/cash/paparazzi/Environment {
 	public static final field $stable I
-	public fun <init> (Ljava/lang/String;Ljava/lang/String;Ljava/lang/String;ILjava/util/List;Ljava/util/List;Ljava/util/List;Ljava/util/List;Ljava/util/List;Ljava/util/List;)V
-	public final fun copy (Ljava/lang/String;Ljava/lang/String;Ljava/lang/String;ILjava/util/List;Ljava/util/List;Ljava/util/List;Ljava/util/List;Ljava/util/List;Ljava/util/List;)Lapp/cash/paparazzi/Environment;
-	public static synthetic fun copy$default (Lapp/cash/paparazzi/Environment;Ljava/lang/String;Ljava/lang/String;Ljava/lang/String;ILjava/util/List;Ljava/util/List;Ljava/util/List;Ljava/util/List;Ljava/util/List;Ljava/util/List;ILjava/lang/Object;)Lapp/cash/paparazzi/Environment;
+	public fun <init> (Ljava/lang/String;Ljava/lang/String;ILjava/util/List;Ljava/util/List;Ljava/util/List;Ljava/util/List;Ljava/util/List;Ljava/util/List;)V
+	public final fun copy (Ljava/lang/String;Ljava/lang/String;ILjava/util/List;Ljava/util/List;Ljava/util/List;Ljava/util/List;Ljava/util/List;Ljava/util/List;)Lapp/cash/paparazzi/Environment;
+	public static synthetic fun copy$default (Lapp/cash/paparazzi/Environment;Ljava/lang/String;Ljava/lang/String;ILjava/util/List;Ljava/util/List;Ljava/util/List;Ljava/util/List;Ljava/util/List;Ljava/util/List;ILjava/lang/Object;)Lapp/cash/paparazzi/Environment;
 	public fun equals (Ljava/lang/Object;)Z
 	public final fun getAllModuleAssetDirs ()Ljava/util/List;
 	public final fun getAppTestDir ()Ljava/lang/String;
@@ -74,14 +74,12 @@ public final class app/cash/paparazzi/Environment {
 	public final fun getLocalResourceDirs ()Ljava/util/List;
 	public final fun getModuleResourceDirs ()Ljava/util/List;
 	public final fun getPackageName ()Ljava/lang/String;
-	public final fun getPlatformDir ()Ljava/lang/String;
 	public final fun getResourcePackageNames ()Ljava/util/List;
 	public fun hashCode ()I
 	public fun toString ()Ljava/lang/String;
 }
 
 public final class app/cash/paparazzi/EnvironmentKt {
-	public static final fun androidHome ()Ljava/lang/String;
 	public static final fun detectEnvironment ()Lapp/cash/paparazzi/Environment;
 }
 

paparazzi/build.gradle

@@ -104,7 +104,6 @@ def generateTestConfig = tasks.register("generateTestConfig") {
       "mainPackage": "app.cash.paparazzi",
       "mergeResourcesOutputDir": ".",
       "targetSdkVersion": "34",
-      "platformDir": "platforms/android-34/",
       "mergeAssetsOutputDir": ".",
       "resourcePackageNames": ["app.cash.paparazzi"],
       "projectResourceDirs": [],
@@ -150,6 +149,7 @@ tasks.withType(Test).configureEach {
     "paparazzi.layoutlib.resources.root",
     configurations.layoutlibResources.singleFile.absolutePath
   )
+
   // Uncomment to debug JNI issues in layoutlib
   // jvmArgs '-Xcheck:jni'
   testLogging {