fix: optimize dynamic class filtering logic (arextest#156)

Author: YongwuHe

arex-instrumentation/dynamic/arex-dynamic-common/src/main/java/io/arex/inst/dynamic/common/DynamicClassExtractor.java

@@ -22,6 +22,7 @@
 import java.lang.reflect.Method;
 import java.util.Collection;
 import java.util.Map;
+import java.util.Objects;
 import java.util.concurrent.CompletableFuture;
 import java.util.concurrent.Future;
 import org.slf4j.Logger;
@@ -162,7 +163,7 @@ String buildResultClazz(String resultClazz) {
 
         // @ArexMock actualType
         if (actualType != null && Object.class != actualType) {
-            return resultClazz + TypeUtil.HORIZONTAL_LINE + TypeUtil.getName(actualType.getName());
+            return resultClazz + TypeUtil.HORIZONTAL_LINE + actualType.getName();
         }
 
         if (Config.get() == null || Config.get().getDynamicEntityMap().isEmpty()) {
@@ -292,10 +293,10 @@ private boolean needRecord() {
     }
 
     private String buildDuplicateMethodKey() {
-        if (StringUtil.isEmpty(serializedResult)) {
+        if (Objects.isNull(result)) {
             return String.format("%s_%s_%s_no_result", clazzName, methodName, methodKey);
         }
-        return String.format("%s_%s_%s_has_result", clazzName, methodName, methodKey);
+        return String.format("%s_%s_%s_has_result_%s", clazzName, methodName, methodKey, result.getClass().getName());
     }
 
     /**

arex-instrumentation/dynamic/arex-dynamic-common/src/test/java/io/arex/inst/dynamic/common/DynamicClassExtractorTest.java

@@ -17,6 +17,7 @@
 import io.arex.inst.runtime.util.TypeUtil;
 import java.lang.reflect.Method;
 import java.lang.reflect.Type;
+import java.time.LocalDateTime;
 import java.util.concurrent.CompletableFuture;
 import java.util.concurrent.ExecutionException;
 import org.junit.jupiter.api.AfterAll;
@@ -165,7 +166,7 @@ static Stream<Arguments> replayCase() {
     void testSetFutureResponse() throws NoSuchMethodException {
         List<Integer> methodSignatureHashList = new ArrayList<>();
         methodSignatureHashList.add(StringUtil.encodeAndHash(
-            "io.arex.inst.dynamic.common.DynamicClassExtractorTest_testReturnListenableFuture_mock_no_result"
+            "io.arex.inst.dynamic.common.DynamicClassExtractorTest_testReturnListenableFuture_mock_has_result_java.lang.String"
         ));
         ArexContext context = Mockito.mock(ArexContext.class);
         Mockito.when(ContextManager.currentContext()).thenReturn(context);
@@ -219,7 +220,7 @@ void restoreResponseTest() throws NoSuchMethodException, ExecutionException, Int
     @Test
     void testBuildResultClazz() throws NoSuchMethodException {
         Method testWithArexMock = DynamicClassExtractorTest.class.getDeclaredMethod("testWithArexMock", String.class);
-        DynamicClassExtractor extractor = new DynamicClassExtractor(testWithArexMock, new Object[]{"mock"}, "#val", String.class);
+        DynamicClassExtractor extractor = new DynamicClassExtractor(testWithArexMock, new Object[]{"mock"}, "#val", LocalDateTime.class);
 
         // result clazz is emtpy
         String actualResult = extractor.buildResultClazz("");
@@ -231,7 +232,7 @@ void testBuildResultClazz() throws NoSuchMethodException {
 
         // @ArexMock actualType not null
         actualResult = extractor.buildResultClazz("Java.util.List");
-        assertEquals("Java.util.List-java.lang.String", actualResult);
+        assertEquals("Java.util.List-java.time.LocalDateTime", actualResult);
 
         ConfigBuilder.create("mock-service").build();
         extractor = new DynamicClassExtractor(testWithArexMock, new Object[]{"mock"}, "#val", null);