Merge pull request #115 from koral--/issue-112

Change additional classpath building
koral-- · Nov 12, 2023 · f53d0c3 · f53d0c3
2 parents 7230ce0 + 5fb80c8
commit f53d0c3
Show file tree

Hide file tree

Showing 6 changed files with 35 additions and 11 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,7 @@
+# 0.2.15 - 2023-11-12
+
+- Change additional classpath building [#112](https://github.com/koral--/gradle-pitest-plugin/issue/112)
+
 # 0.2.14 - 2023-11-06
 
 - Update dependency versions

diff --git a/README.md b/README.md
@@ -7,13 +7,13 @@ which supports Android gradle projects.
 
 ```groovy
 plugins {
-  id 'pl.droidsonroids.pitest' version '0.2.14'
+  id 'pl.droidsonroids.pitest' version '0.2.15'
 }
 ```
 
 ```kotlin
 plugins {
-  id("pl.droidsonroids.pitest") version "0.2.14"
+  id("pl.droidsonroids.pitest") version "0.2.15"
 }
 ```
 
@@ -25,7 +25,7 @@ buildscript {
     google()
   }
   dependencies {
-    classpath("pl.droidsonroids.gradle:gradle-pitest-plugin:0.2.14")
+    classpath("pl.droidsonroids.gradle:gradle-pitest-plugin:0.2.15")
   }
 }
 

diff --git a/gradle.properties b/gradle.properties
@@ -1,4 +1,4 @@
-VERSION_NAME=0.2.14
+VERSION_NAME=0.2.15
 GROUP=pl.droidsonroids.gradle
 POM_DESCRIPTION=Gradle plugin for PIT Mutation Testing in Android projects
 POM_URL=https://github.com/koral--/gradle-pitest-plugin

diff --git a/src/main/groovy/pl/droidsonroids/gradle/pitest/PitestPlugin.groovy b/src/main/groovy/pl/droidsonroids/gradle/pitest/PitestPlugin.groovy
@@ -211,11 +211,11 @@ class PitestPlugin implements Plugin<Project> {
 
             if (ANDROID_GRADLE_PLUGIN_VERSION_NUMBER.major == 3 && project.findProperty("android.enableJetifier") != "true") {
                 if (ANDROID_GRADLE_PLUGIN_VERSION_NUMBER.minor < 3) {
-                    from(project.configurations["${variant.name}CompileClasspath"].copyRecursive { configuration ->
-                        configuration.properties.dependencyProject == null
+                    from(project.configurations["${variant.name}CompileClasspath"].copyRecursive { dependency ->
+                        dependency.properties.dependencyProject == null
                     })
-                    from(project.configurations["${variant.name}UnitTestCompileClasspath"].copyRecursive { configuration ->
-                        configuration.properties.dependencyProject == null
+                    from(project.configurations["${variant.name}UnitTestCompileClasspath"].copyRecursive { dependency ->
+                        dependency.properties.dependencyProject == null
                     })
                 } else if (ANDROID_GRADLE_PLUGIN_VERSION_NUMBER.minor < 4) {
                     from(project.configurations["${variant.name}CompileClasspath"])
@@ -225,9 +225,10 @@ class PitestPlugin implements Plugin<Project> {
                 from(project.configurations["compile"])
                 from(project.configurations["testCompile"])
             } else if (ANDROID_GRADLE_PLUGIN_VERSION_NUMBER.major > 4 && project.findProperty("android.enableJetifier") != "true") {
-                from(project.configurations["${variant.name}UnitTestRuntimeClasspath"].copyRecursive { configuration ->
-                    configuration.properties.dependencyProject == null && configuration.version != null
-                })
+                Configuration unittestRuntimeConfig = project.configurations.getByName("${variant.name}UnitTestRuntimeClasspath")
+                from(unittestRuntimeConfig.copyRecursive { dependency ->
+                    dependency.properties.dependencyProject == null && dependency.version != null
+                }.shouldResolveConsistentlyWith(unittestRuntimeConfig))
             }
             from(project.configurations["pitestRuntimeOnly"])
             from(project.files("${project.buildDir}/intermediates/sourceFolderJavaResources/${variant.dirName}"))

diff --git a/src/test/groovy/pl/droidsonroids/gradle/pitest/AndroidUtils.groovy b/src/test/groovy/pl/droidsonroids/gradle/pitest/AndroidUtils.groovy
@@ -82,6 +82,14 @@ class AndroidUtils {
                 unitTests.returnDefaultValues = true
             }
         }
+        project.dependencies {
+            implementation("org.jetbrains.kotlin:kotlin-reflect:1.6.10")
+            testImplementation("io.mockk:mockk:1.11.0")
+        }
+        project.repositories {
+            mavenCentral()
+            google()
+        }
         project.apply(plugin: "pl.droidsonroids.pitest")
         return project
     }

diff --git a/src/test/groovy/pl/droidsonroids/gradle/pitest/PitestPluginTest.groovy b/src/test/groovy/pl/droidsonroids/gradle/pitest/PitestPluginTest.groovy
@@ -87,6 +87,17 @@ class PitestPluginTest extends Specification {
             assert classpath.find { it.toString().endsWith('sourceFolderJavaResources/test/freeBlue/release') }
     }
 
+    @SuppressWarnings("ImplicitClosureParameter")
+    void "combined task classpath contains dependencies in correct order"() {
+        when:
+            Project project = AndroidUtils.createSampleApplicationProject()
+            project.evaluate()
+        then:
+            Set<File> classpath = project.tasks["${PitestPlugin.PITEST_TASK_NAME}FreeBlueRelease"].additionalClasspath.files
+            assert classpath.find { it.toString().endsWith('kotlin-reflect-1.3.72.jar') } == null
+            assert classpath.find { it.toString().endsWith('kotlin-reflect-1.6.10.jar') }
+    }
+
     void "strange sdk versions get properly sanitized"() {
         when:
             String version = PitestPlugin.sanitizeSdkVersion('strange version (0)')