Merge pull request #70 from linyimin0812/testcase/20230807_add_test_case_1

linyimin0812 · web-flow · commit 35b3a7a2782d · 2023-08-14T02:37:48.000+08:00
testcase: add unit test case
diff --git a/spring-async-bean-core/pom.xml b/spring-async-bean-core/pom.xml
@@ -35,6 +35,16 @@
             <artifactId>javax.annotation-api</artifactId>
             <version>1.3.2</version>
         </dependency>
+        <dependency>
+            <groupId>junit</groupId>
+            <artifactId>junit</artifactId>
+        </dependency>
+
+        <dependency>
+            <groupId>com.alibaba</groupId>
+            <artifactId>fastjson</artifactId>
+            <scope>test</scope>
+        </dependency>
 
     </dependencies>
 
diff --git a/spring-async-bean-core/src/main/java/io/github/linyimin0812/async/bean/AsyncInitBeanFinder.java b/spring-async-bean-core/src/main/java/io/github/linyimin0812/async/bean/AsyncInitBeanFinder.java
@@ -17,7 +17,7 @@
 import java.util.*;
 
 /**
- * @author yiminlin
+ * @author linyimin
  **/
 public class AsyncInitBeanFinder {
 
diff --git a/spring-async-bean-core/src/main/java/io/github/linyimin0812/async/config/AsyncBeanProperties.java b/spring-async-bean-core/src/main/java/io/github/linyimin0812/async/config/AsyncBeanProperties.java
@@ -7,7 +7,7 @@
 import java.util.List;
 
 /**
- * @author yiminlin
+ * @author linyimin
  **/
 public class AsyncBeanProperties {
 
diff --git a/spring-async-bean-core/src/main/java/io/github/linyimin0812/async/config/AsyncConfig.java b/spring-async-bean-core/src/main/java/io/github/linyimin0812/async/config/AsyncConfig.java
@@ -1,7 +1,7 @@
 package io.github.linyimin0812.async.config;
 
 /**
- * @author yiminlin
+ * @author linyimin
  **/
 public class AsyncConfig {
     private static final AsyncConfig INSTANCE = new AsyncConfig();
diff --git a/spring-async-bean-core/src/main/java/io/github/linyimin0812/async/executor/AsyncTaskExecutor.java b/spring-async-bean-core/src/main/java/io/github/linyimin0812/async/executor/AsyncTaskExecutor.java
@@ -9,7 +9,7 @@
 import java.util.concurrent.*;
 
 /**
- * @author yiminlin
+ * @author linyimin
  **/
 public class AsyncTaskExecutor {
 
diff --git a/spring-async-bean-core/src/main/java/io/github/linyimin0812/async/executor/NamedThreadFactory.java b/spring-async-bean-core/src/main/java/io/github/linyimin0812/async/executor/NamedThreadFactory.java
@@ -4,7 +4,7 @@
 import java.util.concurrent.atomic.AtomicInteger;
 
 /**
- * @author yiminlin
+ * @author linyimin
  **/
 public class NamedThreadFactory implements ThreadFactory {
 
diff --git a/spring-async-bean-core/src/main/java/io/github/linyimin0812/async/listener/AsyncTaskExecutionListener.java b/spring-async-bean-core/src/main/java/io/github/linyimin0812/async/listener/AsyncTaskExecutionListener.java
@@ -10,7 +10,7 @@
 import org.springframework.core.PriorityOrdered;
 
 /**
- * @author yiminlin
+ * @author linyimin
  **/
 public class AsyncTaskExecutionListener implements ApplicationListener<ContextRefreshedEvent>, ApplicationContextAware, PriorityOrdered {
 
diff --git a/spring-async-bean-core/src/main/java/io/github/linyimin0812/async/processor/AsyncBeanPriorityLoadPostProcessor.java b/spring-async-bean-core/src/main/java/io/github/linyimin0812/async/processor/AsyncBeanPriorityLoadPostProcessor.java
@@ -13,7 +13,7 @@
 
 
 /**
- * @author yiminlin
+ * @author linyimin
  **/
 public class AsyncBeanPriorityLoadPostProcessor extends InstantiationAwareBeanPostProcessorAdapter implements BeanFactoryAware {
 
diff --git a/spring-async-bean-core/src/main/java/io/github/linyimin0812/async/processor/AsyncProxyBeanPostProcessor.java b/spring-async-bean-core/src/main/java/io/github/linyimin0812/async/processor/AsyncProxyBeanPostProcessor.java
@@ -23,7 +23,7 @@
 
 
 /**
- * @author yiminlin
+ * @author linyimin
  **/
 public class AsyncProxyBeanPostProcessor implements BeanPostProcessor, ApplicationContextAware, PriorityOrdered {
 
diff --git a/spring-async-bean-core/src/main/java/io/github/linyimin0812/async/utils/BeanDefinitionUtil.java b/spring-async-bean-core/src/main/java/io/github/linyimin0812/async/utils/BeanDefinitionUtil.java
@@ -10,7 +10,7 @@
 import org.springframework.util.StringUtils;
 
 /**
- * @author yiminlin
+ * @author linyimin
  **/
 public class BeanDefinitionUtil {
 
diff --git a/spring-async-bean-core/src/test/java/io/github/linyimin0812/async/bean/AsyncInitBeanFinderTest.java b/spring-async-bean-core/src/test/java/io/github/linyimin0812/async/bean/AsyncInitBeanFinderTest.java
@@ -0,0 +1,21 @@
+package io.github.linyimin0812.async.bean;
+
+import org.springframework.beans.factory.support.RootBeanDefinition;
+
+import static org.junit.Assert.*;
+
+/**
+ * @author linyimin
+ **/
+public class AsyncInitBeanFinderTest {
+
+    @org.junit.Test
+    public void getAsyncInitMethodName() {
+//        String definition = "{\"autowireCandidate\":true,\"autowireMode\":0,\"constructorArgumentValues\":{\"genericArgumentValues\":[],\"indexedArgumentValues\":{}},\"dependencyCheck\":0,\"enforceDestroyMethod\":true,\"enforceInitMethod\":true,\"externallyManagedConfigMembers\":[],\"externallyManagedDestroyMethods\":[],\"externallyManagedInitMethods\":[],\"lenientConstructorResolution\":true,\"methodOverrides\":{\"overrides\":[]},\"nonPublicAccessAllowed\":true,\"primary\":false,\"propertyValues\":{\"converted\":false,\"propertyValueList\":[{\"converted\":false,\"name\":\"order\",\"optional\":false,\"value\":-2147483648},{\"converted\":false,\"name\":\"proxyTargetClass\",\"optional\":false,\"value\":true}]},\"role\":2,\"scope\":\"\",\"synthetic\":false}";
+        RootBeanDefinition beanDefinition = new RootBeanDefinition();
+
+        String beanName = "testBean";
+
+        assertNull(AsyncInitBeanFinder.getAsyncInitMethodName(beanName, beanDefinition));
+    }
+}
diff --git a/spring-async-bean-starter/src/main/java/io/github/linyimin0812/async/AsyncBeanAutoConfiguration.java b/spring-async-bean-starter/src/main/java/io/github/linyimin0812/async/AsyncBeanAutoConfiguration.java
@@ -9,7 +9,7 @@
 import org.springframework.context.annotation.Configuration;
 
 /**
- * @author yiminlin
+ * @author linyimin
  **/
 @Configuration
 @EnableConfigurationProperties(AsyncSpringBeanProperties.class)
diff --git a/spring-async-bean-starter/src/main/java/io/github/linyimin0812/async/AsyncSpringBeanProperties.java b/spring-async-bean-starter/src/main/java/io/github/linyimin0812/async/AsyncSpringBeanProperties.java
@@ -4,7 +4,7 @@
 import org.springframework.boot.context.properties.ConfigurationProperties;
 
 /**
- * @author yiminlin
+ * @author linyimin
  **/
 @ConfigurationProperties(prefix = AsyncBeanProperties.PREFIX)
 public class AsyncSpringBeanProperties extends AsyncBeanProperties {
diff --git a/spring-profiler-agent/src/main/java/io/github/linyimin0812/profiler/agent/ProfilerAgentClassLoader.java b/spring-profiler-agent/src/main/java/io/github/linyimin0812/profiler/agent/ProfilerAgentClassLoader.java
@@ -4,7 +4,7 @@
 import java.net.URLClassLoader;
 
 /**
- * @author yiminlin
+ * @author linyimin
  **/
 public class ProfilerAgentClassLoader extends URLClassLoader {
 
diff --git a/spring-profiler-agent/src/test/java/io/github/linyimin0812/profiler/agent/ProfilerAgentClassLoaderTest.java b/spring-profiler-agent/src/test/java/io/github/linyimin0812/profiler/agent/ProfilerAgentClassLoaderTest.java
@@ -6,7 +6,7 @@
 import java.net.URL;
 
 /**
- * @author banzhe
+ * @author linyimin
  **/
 public class ProfilerAgentClassLoaderTest {
 
diff --git a/spring-profiler-common/src/main/java/io/github/linyimin0812/profiler/common/settings/ProfilerSettings.java b/spring-profiler-common/src/main/java/io/github/linyimin0812/profiler/common/settings/ProfilerSettings.java
@@ -54,6 +54,8 @@ public static boolean isNotBlank(String text) {
 
     public static void loadProperties(String path) {
 
+        clear();
+
         try (FileInputStream fileInputStream = new FileInputStream(path)) {
             properties.load(fileInputStream);
             logger.info(ProfilerSettings.class, "loaded settings from {}", path);
@@ -62,4 +64,8 @@ public static void loadProperties(String path) {
         }
     }
 
+    public static void clear() {
+        properties.clear();
+    }
+
 }
diff --git a/spring-profiler-common/src/main/java/io/github/linyimin0812/profiler/common/utils/AgentHomeUtil.java b/spring-profiler-common/src/main/java/io/github/linyimin0812/profiler/common/utils/AgentHomeUtil.java
@@ -3,7 +3,7 @@
 import java.io.File;
 
 /**
- * @author yiminlin
+ * @author linyimin
  **/
 public class AgentHomeUtil {
 
diff --git a/spring-profiler-common/src/test/java/io/github/linyimin0812/profiler/common/logger/LogFactoryTest.java b/spring-profiler-common/src/test/java/io/github/linyimin0812/profiler/common/logger/LogFactoryTest.java
@@ -0,0 +1,46 @@
+package io.github.linyimin0812.profiler.common.logger;
+
+import org.junit.Assert;
+import org.junit.Test;
+
+import java.net.URISyntaxException;
+import java.net.URL;
+import java.nio.file.Path;
+import java.nio.file.Paths;
+
+
+/**
+ * @author linyimin
+ **/
+public class LogFactoryTest {
+
+    @Test
+    public void getStartupLogger() {
+        Logger logger = LogFactory.getStartupLogger();
+        Assert.assertNotNull(logger);
+    }
+
+    @Test
+    public void getTransFormLogger() {
+        Logger logger = LogFactory.getTransFormLogger();
+        Assert.assertNotNull(logger);
+    }
+
+    @Test
+    public void close() {
+        LogFactory.close();
+    }
+
+    @Test
+    public void createLogger() throws URISyntaxException {
+        Logger logger = LogFactory.getStartupLogger();
+        URL url = LogFactoryTest.class.getClassLoader().getResource("spring-startup-analyzer.properties");
+        assert url != null;
+
+        Path path = Paths.get(url.toURI());
+        LogFactory.createLogger(LoggerName.startup, path.getParent().toUri().getPath());
+
+        Assert.assertNotEquals(logger, LogFactory.getStartupLogger());
+
+    }
+}
diff --git a/spring-profiler-common/src/test/java/io/github/linyimin0812/profiler/common/logger/LoggerTest.java b/spring-profiler-common/src/test/java/io/github/linyimin0812/profiler/common/logger/LoggerTest.java
@@ -0,0 +1,80 @@
+package io.github.linyimin0812.profiler.common.logger;
+
+import org.junit.Assert;
+import org.junit.BeforeClass;
+import org.junit.Test;
+
+import java.net.URISyntaxException;
+import java.net.URL;
+import java.nio.file.Files;
+import java.nio.file.Paths;
+
+/**
+ * @author linyimin
+ **/
+public class LoggerTest {
+
+    private static Logger logger;
+
+    @BeforeClass
+    public static void init() throws URISyntaxException {
+        URL url = LogFactoryTest.class.getClassLoader().getResource("spring-startup-analyzer.properties");
+        assert url != null;
+
+        String path = Paths.get(url.toURI()).getParent().toUri().getPath();
+        logger = new Logger(LoggerName.startup, path);
+
+        Assert.assertTrue(Files.exists(Paths.get(path, LoggerName.startup + ".log")));
+
+    }
+
+    @Test
+    public void debug() {
+        logger.debug(LoggerTest.class, "debug");
+    }
+
+    @Test
+    public void testDebug() {
+        logger.debug(LoggerTest.class, "debug: {}", "params");
+    }
+
+    @Test
+    public void warn() {
+        logger.warn(LoggerTest.class, "warn");
+    }
+
+    @Test
+    public void testWarn() {
+        logger.warn(LoggerTest.class, "warn: {}", "params");
+    }
+
+    @Test
+    public void info() {
+        logger.info(LoggerTest.class, "info");
+    }
+
+    @Test
+    public void testInfo() {
+        logger.info(LoggerTest.class, "info: {}", "params");
+    }
+
+    @Test
+    public void error() {
+        logger.error(LoggerTest.class, "error");
+    }
+
+    @Test
+    public void testError() {
+        logger.error(LoggerTest.class, "error: {}", "params");
+    }
+
+    @Test
+    public void testError1() {
+        logger.error(LoggerTest.class, "error: {}, {}", "params", new RuntimeException(""));
+    }
+
+    @Test
+    public void close() {
+        logger.close();
+    }
+}
diff --git a/spring-profiler-common/src/test/java/io/github/linyimin0812/profiler/common/settings/ProfilerSettingsTest.java b/spring-profiler-common/src/test/java/io/github/linyimin0812/profiler/common/settings/ProfilerSettingsTest.java
@@ -7,7 +7,7 @@
 import java.net.URL;
 
 /**
- * @author banzhe
+ * @author linyimin
  * @date 2023/07/27 16:15
  **/
 public class ProfilerSettingsTest {
@@ -45,6 +45,6 @@ public void isNotBlank() {
 
     @Test
     public void loadProperties() {
-        ProfilerSettings.loadProperties("");
+        init();
     }
 }
diff --git a/spring-profiler-common/src/test/java/io/github/linyimin0812/profiler/common/utils/IpUtilTest.java b/spring-profiler-common/src/test/java/io/github/linyimin0812/profiler/common/utils/IpUtilTest.java
@@ -8,7 +8,7 @@
 
 
 /**
- * @author banzhe
+ * @author linyimin
  **/
 public class IpUtilTest {
 
diff --git a/spring-profiler-core/src/main/java/io/github/linyimin0812/profiler/core/container/IocContainer.java b/spring-profiler-core/src/main/java/io/github/linyimin0812/profiler/core/container/IocContainer.java
@@ -150,4 +150,12 @@ public  static void copyFile(String sourceFilePath, String targetFilePath) throw
 
         Files.copy(sourcePath, targetPath);
     }
+
+    public static boolean isStarted() {
+        return started.get();
+    }
+
+    public static boolean isStopped() {
+        return started.get();
+    }
 }
diff --git a/spring-profiler-core/src/main/java/io/github/linyimin0812/profiler/core/http/SimpleHttpServer.java b/spring-profiler-core/src/main/java/io/github/linyimin0812/profiler/core/http/SimpleHttpServer.java
@@ -43,6 +43,14 @@ public static void start() {
         }
     }
 
+    public static void stop() {
+        if (server == null) {
+            return;
+        }
+
+        server.stop(1);
+    }
+
     private static class RootHandler implements HttpHandler {
 
         @Override
@@ -56,7 +64,10 @@ public void handle(HttpExchange exchange) throws IOException {
                 content = Files.readAllBytes(path);
                 exchange.getResponseHeaders().set("Content-Type", "text/javascript");
             } else {
-                if ("/stop".equals(url)) {
+
+                if ("/hello".equals(url)) {
+                    content = "Hello".getBytes(StandardCharsets.UTF_8);
+                } else if ("/stop".equals(url)) {
                     content = "Agent stop.".getBytes(StandardCharsets.UTF_8);
                     IocContainer.stop();
                 } else if (url.endsWith("flame-graph.html")) {
diff --git a/spring-profiler-core/src/main/java/io/github/linyimin0812/profiler/core/monitor/ApplicationRunMonitor.java b/spring-profiler-core/src/main/java/io/github/linyimin0812/profiler/core/monitor/ApplicationRunMonitor.java
@@ -12,7 +12,7 @@
 import java.util.List;
 
 /**
- * @author yiminlin
+ * @author linyimin
  **/
 @MetaInfServices
 public class ApplicationRunMonitor implements EventListener {
diff --git a/spring-profiler-core/src/test/java/io/github/linyimin0812/profiler/core/container/EventListenerTest.java b/spring-profiler-core/src/test/java/io/github/linyimin0812/profiler/core/container/EventListenerTest.java
diff --git a/spring-profiler-core/src/test/java/io/github/linyimin0812/profiler/core/container/IocContainerTest.java b/spring-profiler-core/src/test/java/io/github/linyimin0812/profiler/core/container/IocContainerTest.java
diff --git a/spring-profiler-core/src/test/java/io/github/linyimin0812/profiler/core/container/LifecycleTest.java b/spring-profiler-core/src/test/java/io/github/linyimin0812/profiler/core/container/LifecycleTest.java
diff --git a/spring-profiler-core/src/test/java/io/github/linyimin0812/profiler/core/enhance/EventDispatcherTest.java b/spring-profiler-core/src/test/java/io/github/linyimin0812/profiler/core/enhance/EventDispatcherTest.java
diff --git a/spring-profiler-core/src/test/java/io/github/linyimin0812/profiler/core/http/SimpleHttpServerTest.java b/spring-profiler-core/src/test/java/io/github/linyimin0812/profiler/core/http/SimpleHttpServerTest.java
diff --git a/spring-profiler-core/src/test/resources/spring-startup-analyzer.properties b/spring-profiler-core/src/test/resources/spring-startup-analyzer.properties
diff --git a/spring-profiler-extension/src/main/java/io/github/linyimin0812/profiler/extension/enhance/invoke/InvokeDetailListener.java b/spring-profiler-extension/src/main/java/io/github/linyimin0812/profiler/extension/enhance/invoke/InvokeDetailListener.java
diff --git a/spring-profiler-extension/src/main/java/io/github/linyimin0812/profiler/extension/enhance/sample/Profiler.java b/spring-profiler-extension/src/main/java/io/github/linyimin0812/profiler/extension/enhance/sample/Profiler.java
diff --git a/spring-profiler-extension/src/main/java/io/github/linyimin0812/profiler/extension/enhance/sample/asyncprofiler/AsyncProfiler.java b/spring-profiler-extension/src/main/java/io/github/linyimin0812/profiler/extension/enhance/sample/asyncprofiler/AsyncProfiler.java
diff --git a/spring-profiler-extension/src/test/java/io/github/linyimin0812/profiler/extension/enhance/invoke/InvokeDetailListenerTest.java b/spring-profiler-extension/src/test/java/io/github/linyimin0812/profiler/extension/enhance/invoke/InvokeDetailListenerTest.java
diff --git a/spring-profiler-extension/src/test/java/io/github/linyimin0812/profiler/extension/enhance/sample/jvmprofiler/FlameGraphTest.java b/spring-profiler-extension/src/test/java/io/github/linyimin0812/profiler/extension/enhance/sample/jvmprofiler/FlameGraphTest.java
