Update Gradle Wrapper from 8.14.3 to 9.4.1 & also run tests with JDK 25

.github/workflows/build.yml

@@ -8,7 +8,7 @@ on:
   pull_request:
 
 env:
-  build_java_version: 21
+  build_java_version: 25
 
 jobs:
   build:
@@ -48,6 +48,7 @@ jobs:
           - 11
           - 17
           - 21
+          - 25
     runs-on: ${{ matrix.os }}
     steps:
       - name: Checkout
@@ -97,6 +98,7 @@ jobs:
           - 11
           - 17
           - 21
+          - 25
     runs-on: ${{ matrix.os }}
     steps:
       - name: Checkout

.github/workflows/update-gradle-wrapper.yml

@@ -15,7 +15,7 @@ jobs:
         uses: actions/setup-java@v5.2.0
         with:
           distribution: 'zulu'
-          java-version: 17
+          java-version: 25
 
       - name: Update Gradle Wrapper
         uses: gradle-update/update-gradle-wrapper-action@v2.1.0

archunit/build.gradle

@@ -92,7 +92,7 @@ compileJdk9mainJava {
     dependsOn(compileJava)
     ext.minimumJavaVersion = JavaVersion.VERSION_1_9
 
-    destinationDir = compileJava.destinationDir
+    destinationDirectory.set(compileJava.destinationDirectory.get())
 }
 javadoc.dependsOn(compileJdk9mainJava)
 spotbugsMain.dependsOn(compileJdk9mainJava)

archunit/src/test/java/com/tngtech/archunit/core/importer/ClassFileImporterAccessesTest.java

@@ -459,7 +459,10 @@ String covariantlyOverriddenCausingBridgeMethod() {
         JavaMethodCall callFromBridgeMethodToOverriddenOne = getOnlyElement(
                 new ClassFileImporter().importClasses(Parent.class, Child.class)
                         .get(Child.class)
-                        .getMethodCallsFromSelf());
+                        .getMethodCallsFromSelf()
+                        .stream()
+                        .filter(call -> call.getOrigin().isMethod())
+                        .collect(toSet()));
         JavaCodeUnit bridgeMethod = callFromBridgeMethodToOverriddenOne.getOrigin();
 
         assertThat(bridgeMethod.getName()).isEqualTo("covariantlyOverriddenCausingBridgeMethod");
@@ -1146,10 +1149,11 @@ void call(Target target) {
             }
         }
 
-        JavaMethodCall call = getOnlyElement(new ClassFileImporter()
-                .importClasses(Origin.class, Target.class)
-                .get(Origin.class)
-                .getMethodCallsFromSelf());
+        JavaMethodCall call = getOnlyElement(
+                new ClassFileImporter().importClasses(Origin.class, Target.class)
+                        .get(Origin.class)
+                        .getMethod("call", Target.class)
+                        .getMethodCallsFromSelf());
 
         assertThat(call.getTarget().getParameterTypes())
                 .isEqualTo(call.getTarget().getRawParameterTypes());
@@ -1193,10 +1197,13 @@ void call(Target target) {
             }
         }
 
-        JavaClass origin = new ClassFileImporter().importClasses(Origin.class, Target.class).get(Origin.class);
-
-        assertThat(origin.getMethodCallsFromSelf()).hasSize(2);
-        for (JavaMethodCall call : origin.getMethodCallsFromSelf()) {
+        Set<JavaMethodCall> methodCallsFromSelf =
+                new ClassFileImporter().importClasses(Origin.class, Target.class)
+                        .get(Origin.class)
+                        .getMethod("call", Target.class)
+                        .getMethodCallsFromSelf();
+        assertThat(methodCallsFromSelf).hasSize(2);
+        for (JavaMethodCall call : methodCallsFromSelf) {
             assertThatCall(call).isTo("callMe");
         }
     }

archunit/src/test/java/com/tngtech/archunit/core/importer/ClassFileImporterAutomaticResolutionTest.java

@@ -71,6 +71,7 @@
 import static com.tngtech.java.junit.dataprovider.DataProviders.$;
 import static com.tngtech.java.junit.dataprovider.DataProviders.$$;
 import static com.tngtech.java.junit.dataprovider.DataProviders.testForEach;
+import static java.util.stream.Collectors.toSet;
 
 @RunWith(DataProviderRunner.class)
 public class ClassFileImporterAutomaticResolutionTest {
@@ -349,9 +350,13 @@ void resolvesMethodCallTargetOwner() {
             }
         }
 
-        JavaClass javaClass = ImporterWithAdjustedResolutionRuns.disableAllIterationsExcept(MAX_ITERATIONS_FOR_ACCESSES_TO_TYPES_PROPERTY_NAME)
-                .importClass(Origin.class);
-        JavaMethodCall call = getOnlyElement(javaClass.getMethodCallsFromSelf());
+        JavaMethodCall call = getOnlyElement(
+                ImporterWithAdjustedResolutionRuns.disableAllIterationsExcept(MAX_ITERATIONS_FOR_ACCESSES_TO_TYPES_PROPERTY_NAME)
+                        .importClass(Origin.class)
+                        .getMethodCallsFromSelf()
+                        .stream()
+                        .filter(c -> c.getOrigin().isMethod())
+                        .collect(toSet()));
 
         assertThat(call.getTargetOwner()).isFullyImported(true);
     }

archunit/src/test/java/com/tngtech/archunit/core/importer/ClassFileImporterLambdaDependenciesTest.java

@@ -67,8 +67,11 @@ Runnable call() {
             }
         }
 
-        JavaClasses classes = new ClassFileImporter().importClasses(Target.class, Caller.class);
-        JavaMethodCall call = getOnlyElement(classes.get(Caller.class).getMethodCallsFromSelf());
+        JavaMethodCall call = getOnlyElement(
+                new ClassFileImporter().importClasses(Target.class, Caller.class)
+                        .get(Caller.class)
+                        .getMethod("call")
+                        .getMethodCallsFromSelf());
 
         assertThatCall(call).isFrom("call").isTo(Target.class, "target");
     }
@@ -171,8 +174,11 @@ Consumer<String> call() {
             }
         }
 
-        JavaClasses classes = new ClassFileImporter().importClasses(Target.class, Caller.class);
-        JavaMethodCall call = getOnlyElement(classes.get(Caller.class).getMethodCallsFromSelf());
+        JavaMethodCall call = getOnlyElement(
+                new ClassFileImporter().importClasses(Target.class, Caller.class)
+                        .get(Caller.class)
+                        .getMethod("call")
+                        .getMethodCallsFromSelf());
 
         assertThatCall(call).isFrom("call").isTo(Target.class, "target", String.class);
     }
@@ -195,8 +201,11 @@ Supplier<String> call() {
             }
         }
 
-        JavaClasses classes = new ClassFileImporter().importClasses(Target.class, Caller.class);
-        JavaMethodCall call = getOnlyElement(classes.get(Caller.class).getMethodCallsFromSelf());
+        JavaMethodCall call = getOnlyElement(
+                new ClassFileImporter().importClasses(Target.class, Caller.class)
+                        .get(Caller.class)
+                        .getMethod("call")
+                        .getMethodCallsFromSelf());
 
         assertThatCall(call).isFrom("call").isTo(Target.class, "target");
     }
@@ -251,8 +260,10 @@ Runnable call() {
             }
         }
 
-        JavaClasses classes = new ClassFileImporter().importClasses(Target.class, Caller.class);
-        Set<JavaMethodCall> calls = classes.get(Caller.class).getMethodCallsFromSelf();
+        Set<JavaMethodCall> calls = new ClassFileImporter().importClasses(Target.class, Caller.class)
+                .get(Caller.class)
+                .getMethod("call")
+                .getMethodCallsFromSelf();
 
         assertThat(calls).hasSize(2);
         calls.forEach(call -> assertThatCall(call).isFrom("call").isTo(Target.class, "target"));
@@ -284,8 +295,12 @@ Function<String, Target> call2() {
             }
         }
 
-        JavaClasses classes = new ClassFileImporter().importClasses(Target.class, Caller.class);
-        Set<JavaMethodCall> calls = classes.get(Caller.class).getMethodCallsFromSelf();
+        Set<JavaMethodCall> calls = new ClassFileImporter().importClasses(Target.class, Caller.class)
+                .get(Caller.class)
+                .getMethodCallsFromSelf()
+                .stream()
+                .filter(call -> call.getOrigin().isMethod())
+                .collect(toSet());
         assertThat(calls).hasSize(5);
 
         assertThat(filterOriginByName(calls, "call1"))
@@ -361,10 +376,14 @@ Supplier<String> call() {
 
         JavaClasses classes = new ClassFileImporter().importClasses(Target.class, Caller1.class, Caller2.class);
 
-        JavaMethodCall call1 = getOnlyElement(classes.get(Caller1.class).getMethodCallsFromSelf());
+        JavaMethodCall call1 = getOnlyElement(classes.get(Caller1.class)
+                .getMethod("call")
+                .getMethodCallsFromSelf());
         assertThatCall(call1).isFrom("call").isTo(Target.class, "target");
 
-        JavaMethodCall call2 = getOnlyElement(classes.get(Caller2.class).getMethodCallsFromSelf());
+        JavaMethodCall call2 = getOnlyElement(classes.get(Caller2.class)
+                .getMethod("call")
+                .getMethodCallsFromSelf());
         assertThatCall(call2).isFrom("call").isTo(Target.class, "target");
     }
 

build.gradle

@@ -46,7 +46,7 @@ ext {
     ]
 
     minSupportedJavaVersion = JavaVersion.VERSION_1_8
-    maxSupportedJavaVersion = JavaVersion.VERSION_21
+    maxSupportedJavaVersion = JavaVersion.VERSION_25
     isTestBuild = project.hasProperty('testJavaVersion')
     configuredTestJavaVersion = project.findProperty('testJavaVersion')?.toString()?.with { JavaVersion.toVersion(it) }
     assert configuredTestJavaVersion <= maxSupportedJavaVersion:

buildSrc/src/main/groovy/archunit.java-release-publishing-conventions.gradle

@@ -28,7 +28,7 @@ tasks.withType(AbstractPublishToMaven) {
 publishing {
     publications {
         mavenJava(MavenPublication) {
-            artifactId = project.archivesBaseName
+            artifactId = project.name
             from components.java
             pom {
                 name = app.name

buildSrc/src/main/groovy/archunit.java-version-conventions.gradle

@@ -3,7 +3,8 @@ class VersionSpec {
             JavaVersion.VERSION_1_8,
             JavaVersion.VERSION_11,
             JavaVersion.VERSION_17,
-            JavaVersion.VERSION_21
+            JavaVersion.VERSION_21,
+            JavaVersion.VERSION_25,
     ] as SortedSet
 
     JavaVersion desiredJdkVersion
@@ -43,11 +44,10 @@ afterEvaluate {
         toolchain {
             languageVersion = versionSpec.desiredJavaLanguageVersion
         }
+        sourceCompatibility = versionSpec.desiredSourceCompatibility
+        targetCompatibility = versionSpec.desiredSourceCompatibility
     }
 
-    sourceCompatibility = versionSpec.desiredSourceCompatibility
-    targetCompatibility = versionSpec.desiredSourceCompatibility
-
     tasks.withType(JavaCompile) { Task task ->
         VersionSpec taskVersionSpec = versionSpec.withLowerLtsVersionBoundCompatibleWith(getMinTaskJavaVersion(task))
         println "Task ${task.name} configured: ${taskVersionSpec.describe()}"

buildSrc/src/main/groovy/archunit.license-conventions.gradle

@@ -29,7 +29,7 @@ dependencies {
 def parseLicenseInfoFrom = { config ->
     def pom = config.resolve().find { it.name.endsWith('.pom') }
 
-    def projectNode = new XmlParser().parse(pom)
+    def projectNode = new groovy.xml.XmlParser().parse(pom)
     def licenses = projectNode.licenses.license
     assert licenses.size() == 1: 'Can only handle one declared license at the moment'
 

gradle/wrapper/gradle-wrapper.properties

@@ -1,7 +1,6 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionSha256Sum=bd71102213493060956ec229d946beee57158dbd89d0e62b91bca0fa2c5f3531
-distributionUrl=https\://services.gradle.org/distributions/gradle-8.14.3-bin.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-9.4.1-bin.zip
 networkTimeout=10000
 validateDistributionUrl=true
 zipStoreBase=GRADLE_USER_HOME