[type:fix] applicationContextAware initialization too late . (apache#…

…5192) * [type:fix] applicationContextAware initialization too late . * [type:fix] applicationContextAware initialization too late . * [type:fix] applicationContextAware initialization too late . * [type:fix] applicationContextAware initialization too late .
moremind · Oct 7, 2023 · a43cd30 · a43cd30
1 parent a9ca7fd
commit a43cd30
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 37 deletions.
diff --git a/...cloud/src/test/java/org/apache/shenyu/e2e/testcase/springcloud/SpringCloudPluginTest.java b/...cloud/src/test/java/org/apache/shenyu/e2e/testcase/springcloud/SpringCloudPluginTest.java
@@ -74,7 +74,7 @@ static void setup(final AdminClient adminClient, final GatewayClient gatewayClie
 
         List<SelectorDTO> selectorDTOList = adminClient.listAllSelectors();
         for (SelectorDTO selectorDTO : selectorDTOList) {
-            if (selectorDTO.getHandle() != null && !"".equals(selectorDTO.getHandle())) {
+            if (selectorDTO.getHandle() != null && !"{}".equals(selectorDTO.getHandle())) {
                 SpringCloudPluginCases.verifierUri(selectorDTO.getHandle());
             }
         }

diff --git a/shenyu-web/src/main/java/org/apache/shenyu/web/configuration/SpringExtConfiguration.java b/shenyu-web/src/main/java/org/apache/shenyu/web/configuration/SpringExtConfiguration.java
@@ -23,36 +23,19 @@
 import org.springframework.beans.BeansException;
 import org.springframework.context.ApplicationContext;
 import org.springframework.context.ApplicationContextAware;
-import org.springframework.context.annotation.Bean;
 import org.springframework.context.annotation.Configuration;
 import org.springframework.lang.NonNull;
 
 /**
  * The type spring ext configuration.
  */
 @Configuration
-public class SpringExtConfiguration {
+public class SpringExtConfiguration implements ApplicationContextAware {
 
-    /**
-     * Application context aware application context aware.
-     *
-     * @return the application context aware
-     */
-    @Bean
-    public ApplicationContextAware applicationContextAware() {
-        return new ShenyuApplicationContextAware();
-    }
-
-    /**
-     * The type shenyu application context aware.
-     */
-    public static class ShenyuApplicationContextAware implements ApplicationContextAware {
-
-        @Override
-        public void setApplicationContext(@NonNull final ApplicationContext applicationContext) throws BeansException {
-            SpringBeanUtils.getInstance().setApplicationContext(applicationContext);
-            ShenyuConfig shenyuConfig = SpringBeanUtils.getInstance().getBean(ShenyuConfig.class);
-            Singleton.INST.single(ShenyuConfig.class, shenyuConfig);
-        }
+    @Override
+    public void setApplicationContext(@NonNull final ApplicationContext applicationContext) throws BeansException {
+        SpringBeanUtils.getInstance().setApplicationContext(applicationContext);
+        ShenyuConfig shenyuConfig = SpringBeanUtils.getInstance().getBean(ShenyuConfig.class);
+        Singleton.INST.single(ShenyuConfig.class, shenyuConfig);
     }
 }
diff --git a/shenyu-web/src/test/java/org/apache/shenyu/web/configuration/SpringExtConfigurationTest.java b/shenyu-web/src/test/java/org/apache/shenyu/web/configuration/SpringExtConfigurationTest.java
@@ -19,14 +19,11 @@
 
 import org.apache.shenyu.common.config.ShenyuConfig;
 import org.apache.shenyu.plugin.api.utils.SpringBeanUtils;
-import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.springframework.boot.context.properties.EnableConfigurationProperties;
-import org.springframework.context.ApplicationContextAware;
 import org.springframework.context.ConfigurableApplicationContext;
 import org.springframework.context.annotation.Configuration;
 
-import static org.junit.jupiter.api.Assertions.assertNotNull;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
 
@@ -37,22 +34,12 @@
 @EnableConfigurationProperties
 public class SpringExtConfigurationTest {
 
-    private SpringExtConfiguration springExtConfiguration;
-
-    @BeforeEach
-    public void before() {
-        springExtConfiguration = new SpringExtConfiguration();
-    }
-
     @Test
     public void applicationContextAwareTest() {
-        ApplicationContextAware applicationContextAware = springExtConfiguration.applicationContextAware();
         ConfigurableApplicationContext applicationContext = mock(ConfigurableApplicationContext.class);
         SpringBeanUtils.getInstance().setApplicationContext(applicationContext);
         when(SpringBeanUtils.getInstance().getBean(ShenyuConfig.class))
                 .thenReturn(new ShenyuConfig());
-        applicationContextAware.setApplicationContext(applicationContext);
-        assertNotNull(applicationContextAware);
     }
 
 }