GEODE-10490: Upgrade Gradle from 7.3.3 to 7.6.6

This commit upgrades the Gradle wrapper and addresses all compatibility issues:

Core Changes:
- Updated gradle-wrapper.properties to use Gradle 7.6.6
- Updated gradlew script and wrapper jar to match 7.6.6 version

API Compatibility Fixes:
- ExecutionTrackingTestResultProcessor: Updated failure() method signature to match Gradle 7.6.6 API changes
- UncheckedUtilsTest: Fixed lambda expression to properly trigger ClassCastException with explicit String assignment

POM Validation Updates:
- Updated expected-pom.xml files across 29 modules to reflect Gradle 7.6.6's POM generation changes
- Changed schema URLs from http:// to https:// format as required by newer Gradle versions
- Preserved essential element ordering while minimizing diff noise for reviewers

Build and Test Fixes:
- Applied spotless formatting across entire codebase
- All builds and tests pass successfully with new Gradle version
- Verified compatibility with existing Java 8 environment

Author: JinwooHwang

build-tools/geode-repeat-test/src/main/java/org/apache/geode/gradle/testing/repeat/ExecutionTrackingTestResultProcessor.java

@@ -25,6 +25,7 @@
 import org.gradle.api.internal.tasks.testing.TestResultProcessor;
 import org.gradle.api.internal.tasks.testing.TestStartEvent;
 import org.gradle.api.internal.tasks.testing.worker.WorkerTestClassProcessor;
+import org.gradle.api.tasks.testing.TestFailure;
 import org.gradle.api.tasks.testing.TestOutputEvent;
 
 /**
@@ -68,7 +69,8 @@ public void output(Object testId, TestOutputEvent event) {
   }
 
   @Override
-  public void failure(Object testId, Throwable result) {
+  public void failure(Object testId, TestFailure result) {
+    // TestResultProcessor interface was updated in Gradle 7.6.6 to use TestFailure instead of Throwable
     processor.failure(testId, result);
   }
 

build-tools/geode-repeat-test/src/main/java/org/apache/geode/gradle/testing/repeat/RepeatTestExecuter.java

@@ -44,6 +44,7 @@
 import org.gradle.internal.actor.ActorFactory;
 import org.gradle.internal.time.Clock;
 import org.gradle.internal.work.WorkerLeaseRegistry;
+import org.gradle.internal.work.WorkerLeaseService;
 import org.gradle.process.internal.worker.WorkerProcessFactory;
 
 /**
@@ -72,21 +73,24 @@ public class RepeatTestExecuter implements TestExecuter<JvmTestExecutionSpec> {
   private final ActorFactory actorFactory;
   private final ModuleRegistry moduleRegistry;
   private final WorkerLeaseRegistry workerLeaseRegistry;
+  private final WorkerLeaseService workerLeaseService;
   private final int maxWorkerCount;
   private final Clock clock;
   private final DocumentationRegistry documentationRegistry;
   private final DefaultTestFilter testFilter;
   private final int iterationCount;
   private TestClassProcessor processor;
 
+  // Gradle 7.6.6: WorkerLeaseRegistry became private, use WorkerLeaseService instead
   public RepeatTestExecuter(WorkerProcessFactory workerFactory, ActorFactory actorFactory,
-      ModuleRegistry moduleRegistry, WorkerLeaseRegistry workerLeaseRegistry, int maxWorkerCount,
+      ModuleRegistry moduleRegistry, WorkerLeaseService workerLeaseService, int maxWorkerCount,
       Clock clock, DocumentationRegistry documentationRegistry, DefaultTestFilter testFilter,
       int iterationCount) {
     this.workerFactory = workerFactory;
     this.actorFactory = actorFactory;
     this.moduleRegistry = moduleRegistry;
-    this.workerLeaseRegistry = workerLeaseRegistry;
+    this.workerLeaseRegistry = null; // Keep for backward compatibility but not used
+    this.workerLeaseService = workerLeaseService;
     this.maxWorkerCount = maxWorkerCount;
     this.clock = clock;
     this.documentationRegistry = documentationRegistry;
@@ -99,17 +103,15 @@ public void execute(final JvmTestExecutionSpec testExecutionSpec,
       TestResultProcessor testResultProcessor) {
     final TestFramework testFramework = testExecutionSpec.getTestFramework();
     final WorkerTestClassProcessorFactory testInstanceFactory = testFramework.getProcessorFactory();
-    final WorkerLeaseRegistry.WorkerLease
-        currentWorkerLease =
-        workerLeaseRegistry.getCurrentWorkerLease();
     final Set<File> classpath = ImmutableSet.copyOf(testExecutionSpec.getClasspath());
     final Set<File> modulePath = ImmutableSet.copyOf(testExecutionSpec.getModulePath());
     final List<String>
         testWorkerImplementationModules =
         testFramework.getTestWorkerImplementationModules();
     final Factory<TestClassProcessor> forkingProcessorFactory = () -> {
+      // Gradle 7.6.6: Worker API changed to require WorkerLeaseService for thread management
       TestClassProcessor forkingTestClassProcessor =
-          new ForkingTestClassProcessor(currentWorkerLease, workerFactory, testInstanceFactory,
+          new ForkingTestClassProcessor(workerLeaseService, workerFactory, testInstanceFactory,
               testExecutionSpec.getJavaForkOptions(), classpath, modulePath,
               testWorkerImplementationModules, testFramework.getWorkerConfigurationAction(),
               moduleRegistry, documentationRegistry);
@@ -139,7 +141,8 @@ public void execute(final JvmTestExecutionSpec testExecutionSpec,
       detector = new DefaultTestClassScanner(testClassFiles, null, processor);
     }
 
-    new TestMainAction(detector, processor, testResultProcessor, clock, testExecutionSpec.getPath(),
+    // Gradle 7.6.6: TestMainAction constructor changed to accept WorkerLeaseService for resource management
+    new TestMainAction(detector, processor, testResultProcessor, workerLeaseService, clock, testExecutionSpec.getPath(),
         "Gradle Test Run " + testExecutionSpec.getIdentityPath()).run();
   }
 

build-tools/geode-testing-isolation/src/main/groovy/org/apache/geode/gradle/testing/Workers.groovy

@@ -32,6 +32,8 @@ class Workers {
             ProcessLauncher processLauncher,
             MessagingServer messagingServer) {
         def workerImplementationFactory = donor.workerImplementationFactory
+        // Gradle 7.5.1+: jvmVersionDetector moved from workerImplementationFactory to donor
+        def jvmVersionDetector = donor.jvmVersionDetector
         return new LauncherProxyWorkerProcessFactory(
                 donor.loggingManager,
                 messagingServer,
@@ -40,7 +42,7 @@ class Workers {
                 workerImplementationFactory.gradleUserHomeDir,
                 workerImplementationFactory.temporaryFileProvider,
                 donor.execHandleFactory,
-                workerImplementationFactory.jvmVersionDetector,
+                jvmVersionDetector,
                 donor.outputEventListener,
                 donor.memoryManager,
                 processLauncher)

build-tools/geode-testing-isolation/src/main/java/org/apache/geode/gradle/testing/process/LauncherProxyWorkerProcessBuilder.java

@@ -128,6 +128,11 @@ public void enableJvmMemoryInfoPublishing(boolean shouldPublish) {
     delegate.enableJvmMemoryInfoPublishing(shouldPublish);
   }
 
+  @Override
+  public WorkerProcessBuilder setUseLegacyAddOpens(boolean useLegacyAddOpens) {
+    return delegate.setUseLegacyAddOpens(useLegacyAddOpens);
+  }
+
   /**
    * Replaces the standard worker process's process launcher with this builder's launcher.
    */

build.gradle

@@ -21,7 +21,7 @@ plugins {
   id "base"
   id "idea"
   id "eclipse"
-  id "com.diffplug.spotless" version "6.4.1" apply false
+  id "com.diffplug.spotless" version "6.11.0" apply false
   id "com.github.ben-manes.versions" version "0.42.0" apply false
   id "nebula.lint" version "17.7.0" apply false
   id "com.palantir.docker" version "0.32.0" apply false

extensions/geode-modules-tomcat8/src/test/resources/expected-pom.xml

@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">
   <!--
   Licensed to the Apache Software Foundation (ASF) under one or more
   contributor license agreements.  See the NOTICE file distributed with

extensions/geode-modules-tomcat9/src/test/resources/expected-pom.xml

@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">
   <!--
   Licensed to the Apache Software Foundation (ASF) under one or more
   contributor license agreements.  See the NOTICE file distributed with

extensions/geode-modules/src/test/resources/expected-pom.xml

@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">
   <!--
   Licensed to the Apache Software Foundation (ASF) under one or more
   contributor license agreements.  See the NOTICE file distributed with

geode-assembly/build.gradle

@@ -409,8 +409,9 @@ tasks.register('docs', Javadoc) {
   doLast {
     rootProject.subprojects.each { project ->
       copy {
-        if (project.hasProperty('sourceSets')) {
-          from project.sourceSets.main.resources.srcDirs
+        if (project.hasProperty('sourceSets') && project.sourceSets.findByName('main')) {
+          // Prevent NullPointerException when 'main' source set doesn't exist
+          from project.sourceSets.getByName('main').resources.srcDirs
         }
         include 'javadoc-images/*'
         into docsDir

geode-common/src/test/java/org/apache/geode/util/internal/UncheckedUtilsTest.java

@@ -53,7 +53,10 @@ public void uncheckedCast_rawList_wrongTypes() {
     rawList.add(2);
     List<String> wrongType = uncheckedCast(rawList);
 
-    Throwable thrown = catchThrowable(() -> wrongType.get(0));
+    Throwable thrown = catchThrowable(() -> {
+      // Explicit assignment needed to trigger ClassCastException in newer Gradle versions
+      String str = wrongType.get(0);
+    });
 
     assertThat(thrown).isInstanceOf(ClassCastException.class);
   }