test: overhaul automated compatibility coverage

Author: nobodyiam

.github/workflows/build.yml

@@ -13,8 +13,6 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-# This workflow will build a Java project with Maven
-# For more information see: https://help.github.com/actions/language-and-framework-guides/building-and-testing-java-with-maven
 
 name: build
 
@@ -25,40 +23,170 @@ on:
     branches: [ main ]
 
 jobs:
-  build:
+  compile-matrix:
     runs-on: ubuntu-latest
     strategy:
       matrix:
         jdk: [8, 11, 17]
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v4
     - name: Set up JDK
-      uses: actions/setup-java@v1
+      uses: actions/setup-java@v4
       with:
+        distribution: temurin
         java-version: ${{ matrix.jdk }}
     - name: Cache Maven packages
       uses: actions/cache@v4
       with:
         path: ~/.m2/repository
-        key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
+        key: ${{ runner.os }}-maven-compile-${{ matrix.jdk }}-${{ hashFiles('**/pom.xml') }}
         restore-keys: |
+          ${{ runner.os }}-maven-compile-${{ matrix.jdk }}-
           ${{ runner.os }}-maven-
-    - name: JDK 8
-      if: matrix.jdk == '8'
-      uses: nick-fields/retry@v3
-      with:
-        timeout_minutes: 3
-        max_attempts: 3
-        retry_wait_seconds: 1
-        command: mvn -B clean package -P travis jacoco:report -Dmaven.gitcommitid.skip=true
-    - name: JDK 11
-      if: matrix.jdk == '11'
-      run: mvn -B clean compile -Dmaven.gitcommitid.skip=true
-    - name: JDK 17
-      if: matrix.jdk == '17'
+    - name: Compile
       run: mvn -B clean compile -Dmaven.gitcommitid.skip=true
+
+  unit-integration-pr:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4
+    - name: Set up JDK 8
+      uses: actions/setup-java@v4
+      with:
+        distribution: temurin
+        java-version: 8
+    - name: Cache Maven packages
+      uses: actions/cache@v4
+      with:
+        path: ~/.m2/repository
+        key: ${{ runner.os }}-maven-unit-integration-${{ hashFiles('**/pom.xml') }}
+        restore-keys: |
+          ${{ runner.os }}-maven-unit-integration-
+          ${{ runner.os }}-maven-
+    - name: Run unit and integration tests
+      run: |
+        mvn -B clean test -P travis jacoco:report \
+          -Dmaven.gitcommitid.skip=true
     - name: Upload coverage to Codecov
-      if: matrix.jdk == '8'
-      uses: codecov/codecov-action@v1
+      uses: codecov/codecov-action@v4
+      with:
+        files: ${{ github.workspace }}/apollo-*/target/site/jacoco/jacoco.xml
+
+  compat-api:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4
+    - name: Set up JDK 8
+      uses: actions/setup-java@v4
+      with:
+        distribution: temurin
+        java-version: 8
+    - name: Cache Maven packages
+      uses: actions/cache@v4
+      with:
+        path: ~/.m2/repository
+        key: ${{ runner.os }}-maven-compat-api-${{ hashFiles('**/pom.xml') }}
+        restore-keys: |
+          ${{ runner.os }}-maven-compat-api-
+          ${{ runner.os }}-maven-
+    - name: Build local artifacts for api compatibility
+      run: |
+        mvn -B -pl apollo-core,apollo-client,apollo-mockserver -am \
+          -DskipTests install -Dmaven.gitcommitid.skip=true
+    - name: Run api compatibility tests
+      run: |
+        mvn -B -f apollo-compat-tests/pom.xml -pl apollo-api-compat-it \
+          test -Dmaven.gitcommitid.skip=true
+
+  compat-spring:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        include:
+          - name: spring-3.1.1-jdk8
+            java: 8
+            spring_framework: 3.1.1.RELEASE
+            java_version_prop: 1.8
+          - name: spring-6.1-jdk17
+            java: 17
+            spring_framework: 6.1.18
+            java_version_prop: 17
+    name: compat-spring-${{ matrix.name }}
+    steps:
+    - uses: actions/checkout@v4
+    - name: Set up JDK
+      uses: actions/setup-java@v4
+      with:
+        distribution: temurin
+        java-version: ${{ matrix.java }}
+    - name: Cache Maven packages
+      uses: actions/cache@v4
+      with:
+        path: ~/.m2/repository
+        key: ${{ runner.os }}-maven-compat-spring-${{ matrix.name }}-${{ hashFiles('**/pom.xml') }}
+        restore-keys: |
+          ${{ runner.os }}-maven-compat-spring-${{ matrix.name }}-
+          ${{ runner.os }}-maven-
+    - name: Build local artifacts for compat tests
+      run: |
+        mvn -B -pl apollo-core,apollo-client,apollo-mockserver -am \
+          -DskipTests install -Dmaven.gitcommitid.skip=true
+    - name: Run spring compatibility tests
+      run: |
+        mvn -B -f apollo-compat-tests/pom.xml -pl apollo-spring-compat-it \
+          -Dspring.framework.version=${{ matrix.spring_framework }} \
+          -Djava.version=${{ matrix.java_version_prop }} \
+          test -Dmaven.gitcommitid.skip=true
+
+  compat-spring-boot:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        include:
+          - name: spring-boot-2.7-jdk8
+            java: 8
+            spring_boot: 2.7.18
+            java_version_prop: 1.8
+            compat_slf4j: 1.7.36
+            compat_vintage: 5.7.0
+          - name: spring-boot-3.3-jdk17
+            java: 17
+            spring_boot: 3.3.10
+            java_version_prop: 17
+            compat_slf4j: 2.0.17
+            compat_vintage: 5.10.5
+          - name: spring-boot-4.0-jdk17
+            java: 17
+            spring_boot: 4.0.0
+            java_version_prop: 17
+            compat_slf4j: 2.0.17
+            compat_vintage: 6.0.1
+    name: compat-spring-boot-${{ matrix.name }}
+    steps:
+    - uses: actions/checkout@v4
+    - name: Set up JDK
+      uses: actions/setup-java@v4
       with:
-        file: ${{ github.workspace }}/apollo-*/target/site/jacoco/jacoco.xml
+        distribution: temurin
+        java-version: ${{ matrix.java }}
+    - name: Cache Maven packages
+      uses: actions/cache@v4
+      with:
+        path: ~/.m2/repository
+        key: ${{ runner.os }}-maven-compat-spring-boot-${{ matrix.name }}-${{ hashFiles('**/pom.xml') }}
+        restore-keys: |
+          ${{ runner.os }}-maven-compat-spring-boot-${{ matrix.name }}-
+          ${{ runner.os }}-maven-
+    - name: Build local artifacts for spring boot compatibility
+      run: |
+        mvn -B -pl apollo-core,apollo-client,apollo-mockserver,apollo-client-config-data -am \
+          -DskipTests \
+          install -Dmaven.gitcommitid.skip=true
+    - name: Run spring boot compatibility tests
+      run: |
+        mvn -B -f apollo-compat-tests/pom.xml -pl apollo-spring-boot-compat-it \
+          -Dspring-boot.version=${{ matrix.spring_boot }} \
+          -Djava.version=${{ matrix.java_version_prop }} \
+          -Dcompat.slf4j.version=${{ matrix.compat_slf4j }} \
+          -Dcompat.junit.vintage.version=${{ matrix.compat_vintage }} \
+          test -Dmaven.gitcommitid.skip=true

apollo-client-config-data/pom.xml

@@ -73,5 +73,15 @@
                 </exclusion>
             </exclusions>
         </dependency>
+        <dependency>
+            <groupId>com.ctrip.framework.apollo</groupId>
+            <artifactId>apollo-mockserver</artifactId>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>io.projectreactor.netty</groupId>
+            <artifactId>reactor-netty-http</artifactId>
+            <scope>test</scope>
+        </dependency>
     </dependencies>
 </project>

apollo-client-config-data/src/main/java/com/ctrip/framework/apollo/config/data/extension/initialize/ApolloClientPropertiesFactory.java

@@ -17,7 +17,6 @@
 package com.ctrip.framework.apollo.config.data.extension.initialize;
 
 import com.ctrip.framework.apollo.config.data.extension.properties.ApolloClientProperties;
-import org.springframework.boot.autoconfigure.security.oauth2.client.OAuth2ClientProperties;
 import org.springframework.boot.context.properties.bind.BindHandler;
 import org.springframework.boot.context.properties.bind.Bindable;
 import org.springframework.boot.context.properties.bind.Binder;
@@ -35,10 +34,4 @@ public ApolloClientProperties createApolloClientProperties(
     return binder.bind(PROPERTIES_PREFIX,
         Bindable.of(ApolloClientProperties.class), bindHandler).orElse(null);
   }
-
-  public OAuth2ClientProperties createOauth2ClientProperties(Binder binder,
-      BindHandler bindHandler) {
-    return binder.bind("spring.security.oauth2.client", Bindable.of(OAuth2ClientProperties.class),
-        bindHandler).orElse(null);
-  }
 }

apollo-client-config-data/src/main/java/com/ctrip/framework/apollo/config/data/extension/webclient/ApolloClientLongPollingExtensionInitializer.java

@@ -23,11 +23,11 @@
 import com.ctrip.framework.apollo.util.http.HttpClient;
 import com.ctrip.framework.foundation.internals.ServiceBootstrap;
 import java.util.List;
+import java.util.function.Consumer;
 import org.apache.commons.logging.Log;
 import org.springframework.boot.context.properties.bind.BindHandler;
 import org.springframework.boot.context.properties.bind.Binder;
 import org.springframework.boot.logging.DeferredLogFactory;
-import org.springframework.boot.web.reactive.function.client.WebClientCustomizer;
 import org.springframework.util.CollectionUtils;
 import org.springframework.web.reactive.function.client.WebClient;
 
@@ -55,11 +55,11 @@ public void initialize(ApolloClientProperties apolloClientProperties, Binder bin
         .loadAllOrdered(ApolloClientWebClientCustomizerFactory.class);
     if (!CollectionUtils.isEmpty(factories)) {
       for (ApolloClientWebClientCustomizerFactory factory : factories) {
-        WebClientCustomizer webClientCustomizer = factory
+        Consumer<WebClient.Builder> webClientCustomizer = factory
             .createWebClientCustomizer(apolloClientProperties, binder, bindHandler, this.log,
                 this.bootstrapContext);
         if (webClientCustomizer != null) {
-          webClientCustomizer.customize(webClientBuilder);
+          webClientCustomizer.accept(webClientBuilder);
         }
       }
     }

apollo-client-config-data/src/main/java/com/ctrip/framework/apollo/config/data/extension/webclient/ApolloWebClientHttpClient.java

@@ -22,11 +22,13 @@
 import com.ctrip.framework.apollo.util.http.HttpRequest;
 import com.ctrip.framework.apollo.util.http.HttpResponse;
 import com.google.gson.Gson;
+import java.lang.reflect.Method;
 import java.lang.reflect.Type;
 import java.net.URI;
 import java.util.Map;
 import org.springframework.http.HttpStatus;
 import org.springframework.util.CollectionUtils;
+import org.springframework.web.reactive.function.client.ClientResponse;
 import org.springframework.web.reactive.function.client.WebClient;
 import reactor.core.publisher.Mono;
 
@@ -64,15 +66,16 @@ private <T> HttpResponse<T> doGetInternal(HttpRequest httpRequest, Type response
       }
     }
     return requestHeadersSpec.exchangeToMono(clientResponse -> {
-      if (HttpStatus.OK.equals(clientResponse.statusCode())) {
+      int statusCode = this.resolveStatusCode(clientResponse);
+      if (HttpStatus.OK.value() == statusCode) {
         return clientResponse.bodyToMono(String.class)
             .map(body -> new HttpResponse<T>(HttpStatus.OK.value(),
                 gson.fromJson(body, responseType)));
       }
-      if (HttpStatus.NOT_MODIFIED.equals(clientResponse.statusCode())) {
+      if (HttpStatus.NOT_MODIFIED.value() == statusCode) {
         return Mono.just(new HttpResponse<T>(HttpStatus.NOT_MODIFIED.value(), null));
       }
-      return Mono.error(new ApolloConfigStatusCodeException(clientResponse.rawStatusCode(),
+      return Mono.error(new ApolloConfigStatusCodeException(statusCode,
           String.format("Get operation failed for %s", httpRequest.getUrl())));
     }).block();
   }
@@ -82,4 +85,28 @@ public <T> HttpResponse<T> doGet(HttpRequest httpRequest, Type responseType)
       throws ApolloConfigException {
     return this.doGetInternal(httpRequest, responseType);
   }
+
+  /**
+   * Resolve HTTP status code across Spring WebFlux 5/6/7.
+   *
+   * <p>ClientResponse#statusCode has different return types across major versions
+   * (HttpStatus in Spring 5, HttpStatusCode in Spring 6/7). Calling it directly would bind
+   * to one method descriptor at compile time and could fail on another runtime version.
+   * Reflection keeps this bridge binary-compatible for Boot 2/3/4 compatibility tests.
+   */
+  private int resolveStatusCode(Object clientResponse) {
+    try {
+      Method statusCodeMethod = ClientResponse.class.getMethod("statusCode");
+      Object statusCode = statusCodeMethod.invoke(clientResponse);
+      if (statusCode == null) {
+        throw new ApolloConfigException("Failed to resolve response status code: statusCode is null");
+      }
+      // Both HttpStatus and HttpStatusCode expose value(), so resolve it reflectively.
+      Method valueMethod = statusCode.getClass().getMethod("value");
+      Object value = valueMethod.invoke(statusCode);
+      return ((Number) value).intValue();
+    } catch (Exception ex) {
+      throw new ApolloConfigException("Failed to resolve response status code", ex);
+    }
+  }
 }

apollo-client-config-data/src/main/java/com/ctrip/framework/apollo/config/data/extension/webclient/customizer/spi/ApolloClientWebClientCustomizerFactory.java

@@ -18,19 +18,20 @@
 
 import com.ctrip.framework.apollo.config.data.extension.properties.ApolloClientProperties;
 import com.ctrip.framework.apollo.core.spi.Ordered;
+import java.util.function.Consumer;
 import org.apache.commons.logging.Log;
 import org.springframework.boot.context.properties.bind.BindHandler;
 import org.springframework.boot.context.properties.bind.Binder;
-import org.springframework.boot.web.reactive.function.client.WebClientCustomizer;
 import org.springframework.lang.Nullable;
+import org.springframework.web.reactive.function.client.WebClient;
 
 /**
  * @author vdisk <vdisk@foxmail.com>
  */
 public interface ApolloClientWebClientCustomizerFactory extends Ordered {
 
   /**
-   * create a WebClientCustomizer instance
+   * create a webclient builder customizer
    *
    * @param apolloClientProperties apollo client binded properties
    * @param binder                 properties binder
@@ -41,10 +42,10 @@ public interface ApolloClientWebClientCustomizerFactory extends Ordered {
    *                               Spring Boot 3.x or
    *                               org.springframework.boot.bootstrap.ConfigurableBootstrapContext
    *                               for Spring Boot 4.x)
-   * @return WebClientCustomizer instance or null
+   * @return customizer instance or null
    */
   @Nullable
-  WebClientCustomizer createWebClientCustomizer(ApolloClientProperties apolloClientProperties,
+  Consumer<WebClient.Builder> createWebClientCustomizer(ApolloClientProperties apolloClientProperties,
       Binder binder, BindHandler bindHandler, Log log,
       Object bootstrapContext);
 }