Run CI on JDK 21 (#1173)

line · Dec 1, 2023 · 08900e2 · 08900e2
1 parent c5c2a80
commit 08900e2
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 2 deletions.
diff --git a/.github/workflows/gradle.yml b/.github/workflows/gradle.yml
@@ -21,7 +21,9 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        java: [ '17' ] # We want to test on LTSs.
+        java: # We want to test on LTSs.
+          - '17'
+          - '21'
     steps:
       - name: actions/setup-java@v3 (JDK ${{ matrix.java }})
         uses: actions/setup-java@v4

diff --git a/...src/test/java/com/linecorp/bot/client/base/exception/AbstractLineClientExceptionTest.java b/...src/test/java/com/linecorp/bot/client/base/exception/AbstractLineClientExceptionTest.java
@@ -20,6 +20,7 @@
 import static org.mockito.Mockito.when;
 
 import java.io.IOException;
+import java.net.URI;
 import java.net.URL;
 import java.util.Map;
 
@@ -71,7 +72,7 @@ void constructorNoException() {
     @Test
     void getRequestUrl() throws Exception {
         AbstractLineClientException exception = new AbstractLineClientException(response, "test message");
-        assertThat(exception.getRequestUrl()).isEqualTo(new URL("https://example.com/"));
+        assertThat(exception.getRequestUrl()).isEqualTo(new URI("https://example.com/").toURL());
     }
 
     @Test

diff --git a/...src/main/java/com/linecorp/bot/spring/boot/handler/support/LineMessageHandlerSupport.java b/...src/main/java/com/linecorp/bot/spring/boot/handler/support/LineMessageHandlerSupport.java
@@ -20,6 +20,8 @@
 import java.util.Collection;
 import java.util.List;
 
+import jakarta.annotation.PostConstruct;
+
 import org.slf4j.Logger;
 import org.springframework.boot.autoconfigure.condition.ConditionalOnProperty;
 import org.springframework.boot.context.properties.EnableConfigurationProperties;
@@ -69,7 +71,10 @@ public LineMessageHandlerSupport(
             final ConfigurableApplicationContext applicationContext) {
         this.returnValueConsumerFactory = returnValueConsumerFactory;
         this.applicationContext = applicationContext;
+    }
 
+    @PostConstruct
+    public void initialize() {
         applicationContext.addApplicationListener(event -> {
             if (event instanceof ContextRefreshedEvent) {
                 refresh();