fix: test

Author: mengnankkkk

integration_tests/graalvm_tests/pom.xml

@@ -63,12 +63,6 @@
       <artifactId>fory-graalvm-feature</artifactId>
       <version>${project.version}</version>
     </dependency>
-    <dependency>
-      <groupId>org.graalvm.sdk</groupId>
-      <artifactId>graal-sdk</artifactId>
-      <version>23.0.0</version>
-      <scope>provided</scope>
-    </dependency>
     <dependency>
       <groupId>org.testng</groupId>
       <artifactId>testng</artifactId>
@@ -164,20 +158,14 @@
             <version>${native.maven.plugin.version}</version>
             <extensions>true</extensions>
             <executions>
+              <!-- Build native image (no fork) to align with CI behavior -->
               <execution>
                 <id>build-native</id>
                 <goals>
-                  <goal>build</goal>
+                  <goal>compile-no-fork</goal>
                 </goals>
                 <phase>package</phase>
               </execution>
-              <execution>
-                <id>test-native</id>
-                <goals>
-                  <goal>test</goal>
-                </goals>
-                <phase>test</phase>
-              </execution>
             </executions>
             <configuration>
               <fallback>false</fallback>

java/fory-core/src/test/java/org/apache/fory/util/GraalvmSupportRecordTest.java renamed to integration_tests/graalvm_tests/src/test/java/org/apache/fory/util/GraalvmSupportRecordTest.java

@@ -66,10 +66,11 @@ public void testIsRecordConstructorPublicAccessible_WithString() {
   }
 
   @Test
-  public void testObjectCreators_IsProblematicForCreation_WithRegularClass() {
+  public void testObjectCreators_ReflectiveInstantiationFlag_WithRegularClass() {
     boolean result = GraalvmSupport.needReflectionRegisterForCreation(RegularClass.class);
     Assert.assertFalse(
-        result, "RegularClass with no-arg constructor should not be problematic for creation");
+        result,
+        "RegularClass with no-arg constructor does not require reflective instantiation registration");
   }
 
   @Test
@@ -109,3 +110,4 @@ public void testRecordUtilsIntegration() {
     Assert.assertFalse(GraalvmSupport.isRecordConstructorPublicAccessible(String.class));
   }
 }
+

java/fory-core/src/main/java/org/apache/fory/resolver/ClassResolver.java

@@ -1794,8 +1794,10 @@ public void ensureSerializersCompiled() {
     try {
       fory.getJITContext().lock();
       Serializers.newSerializer(fory, LambdaSerializer.STUB_LAMBDA_CLASS, LambdaSerializer.class);
-      Serializers.newSerializer(
-          fory, JdkProxySerializer.SUBT_PROXY.getClass(), JdkProxySerializer.class);
+      if (!GraalvmSupport.isGraalRuntime()) {
+        Serializers.newSerializer(
+            fory, JdkProxySerializer.SUBT_PROXY.getClass(), JdkProxySerializer.class);
+      }
       classInfoMap.forEach(
           (cls, classInfo) -> {
             GraalvmSupport.registerClassForGraalvm(cls, fory.getConfig().getConfigHash());

java/fory-core/src/main/java/org/apache/fory/serializer/JdkProxySerializer.java

@@ -28,6 +28,7 @@
 import org.apache.fory.memory.Platform;
 import org.apache.fory.reflect.ReflectionUtils;
 import org.apache.fory.resolver.RefResolver;
+import org.apache.fory.util.GraalvmSupport;
 import org.apache.fory.util.Preconditions;
 
 /** Serializer for jdk {@link Proxy}. */
@@ -62,7 +63,11 @@ private interface StubInterface {
   public JdkProxySerializer(Fory fory, Class cls) {
     super(fory, cls);
     if (cls != ReplaceStub.class) {
-      Preconditions.checkArgument(ReflectionUtils.isJdkProxy(cls), "Require a jdk proxy class");
+      // Skip proxy class validation in GraalVM native image runtime to avoid issues with proxy
+      // detection
+      if (!GraalvmSupport.isGraalRuntime()) {
+        Preconditions.checkArgument(ReflectionUtils.isJdkProxy(cls), "Require a jdk proxy class");
+      }
     }
   }
 

java/fory-core/src/main/java/org/apache/fory/util/GraalvmSupport.java

@@ -234,13 +234,13 @@ public static boolean isRecordConstructorPublicAccessible(Class<?> type) {
   }
 
   /**
-   * Checks if a class is problematic for object creation and requires special handling in GraalVM.
+   * Checks whether a class requires reflective instantiation handling in GraalVM.
    *
-   * <p>A class is considered problematic if it lacks a public no-arg constructor and would
-   * typically require ReflectionFactory or unsafe allocation for instantiation.
+   * <p>Returns true when the class does not expose an accessible no-arg constructor and therefore
+   * needs reflective registration for instantiation during native image builds.
    *
    * @param type the class to check
-   * @return true if the class is problematic for creation, false otherwise
+   * @return true if reflective instantiation handling is required, false otherwise
    */
   public static boolean needReflectionRegisterForCreation(Class<?> type) {
     if (type.isInterface()

java/fory-graalvm-feature/src/main/java/org/apache/fory/graalvm/feature/ForyGraalVMFeature.java

@@ -33,7 +33,8 @@
  * to ensure Fory serialization works correctly at runtime. It handles:
  *
  * <ul>
- *   <li>Registering problematic classes for unsafe allocation
+ *   <li>Registering classes that require reflective instantiation (no accessible no-arg
+ *       constructor)
  *   <li>Registering field reflection access for serialization
  *   <li>Registering proxy interfaces for dynamic proxy creation
  * </ul>
@@ -68,7 +69,7 @@ public void duringAnalysis(DuringAnalysisAccess access) {
   }
 
   public String getDescription() {
-    return "Fory GraalVM Feature: Registers classes for serialization, proxying, and unsafe allocation.";
+    return "Fory GraalVM Feature: Registers classes for serialization and proxy support.";
   }
 
   private void handleForyClass(Class<?> clazz) {

integration_tests/graalvm_tests/src/test/java/org/apache/fory/graalvm/feature/ForyGraalVMFeatureTest.java renamed to java/fory-graalvm-feature/src/test/java/org/apache/fory/graalvm/feature/ForyGraalVMFeatureTest.java

@@ -73,28 +73,28 @@ public void tearDown() {
   public void testGetDescription() {
     String description = feature.getDescription();
     assertEquals(
-        "Fory GraalVM Feature: Registers classes for serialization, proxying, and unsafe allocation.",
+        "Fory GraalVM Feature: Registers classes for serialization and proxy support.",
         description);
   }
 
   @Test
-  public void testObjectCreatorsProblematicDetection() {
+  public void testObjectCreatorsDetection() {
     assertTrue(
         GraalvmSupport.needReflectionRegisterForCreation(
             PrivateParameterizedConstructorClass.class),
-        "Class without no-arg constructor should be problematic");
+        "Class without no-arg constructor requires reflective instantiation registration");
 
     assertFalse(
         GraalvmSupport.needReflectionRegisterForCreation(PublicNoArgConstructorClass.class),
-        "Public no-arg constructor should not be problematic");
+        "Public no-arg constructor does not require reflective instantiation registration");
 
     assertFalse(
         GraalvmSupport.needReflectionRegisterForCreation(ProtectedNoArgConstructorClass.class),
-        "Protected no-arg constructor should not be problematic");
+        "Protected no-arg constructor does not require reflective instantiation registration");
 
     assertFalse(
         GraalvmSupport.needReflectionRegisterForCreation(SampleEnum.class),
-        "Enums should not be considered problematic");
+        "Enums do not require reflective instantiation registration");
   }
 
   @Test
@@ -165,3 +165,4 @@ public void testClearRegistrationsResetsState() {
     }
   }
 }
+