Adding test for shutdown

sashirestela · Jan 16, 2025 · 9064e44 · 9064e44
1 parent 9854f23
commit 9064e44
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 0 deletions.
diff --git a/src/test/java/io/github/sashirestela/cleverclient/http/HttpProcessorTest.java b/src/test/java/io/github/sashirestela/cleverclient/http/HttpProcessorTest.java
@@ -39,6 +39,8 @@ interface HttpProcessorTest {
 
     void setMocksForStreamWithError(Stream<String> result) throws IOException, URISyntaxException;
 
+    void testShutdown();
+
     @BeforeAll
     static void setup() {
         TestSupport.setupConfigurator();
@@ -343,4 +345,9 @@ default void shouldExecuteDefaultMethodWhenItIsCalled() {
         assertEquals(expectedValue, actualValue);
     }
 
+    @Test
+    default void shouldShutdownWithoutExceptions() {
+        testShutdown();
+    }
+
 }
diff --git a/src/test/java/io/github/sashirestela/cleverclient/http/JavaHttpProcessorTest.java b/src/test/java/io/github/sashirestela/cleverclient/http/JavaHttpProcessorTest.java
@@ -15,8 +15,10 @@
 import java.util.List;
 import java.util.Map;
 import java.util.concurrent.CompletableFuture;
+import java.util.concurrent.Executors;
 import java.util.stream.Stream;
 
+import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
@@ -146,4 +148,14 @@ public void setMocksForStreamWithError(Stream<String> result) throws IOException
         when(httpResponseStream.request()).thenReturn(httpRequest);
     }
 
+    @Override
+    public void testShutdown() {
+        var defaultAdapter = new JavaHttpClientAdapter();
+        assertDoesNotThrow(() -> defaultAdapter.shutdown());
+
+        var client = HttpClient.newBuilder().executor(Executors.newFixedThreadPool(2)).build();
+        var customAdapter = new JavaHttpClientAdapter(client);
+        assertDoesNotThrow(() -> customAdapter.shutdown());
+    }
+
 }
diff --git a/src/test/java/io/github/sashirestela/cleverclient/http/OkHttpProcessorTest.java b/src/test/java/io/github/sashirestela/cleverclient/http/OkHttpProcessorTest.java
@@ -20,6 +20,7 @@
 import java.util.stream.Collectors;
 import java.util.stream.Stream;
 
+import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.Mockito.doAnswer;
 import static org.mockito.Mockito.mock;
@@ -179,4 +180,14 @@ public void setMocksForStreamWithError(Stream<String> result) throws IOException
         when(okHttpResponse.request()).thenReturn(okHttpRequest);
     }
 
+    @Override
+    public void testShutdown() {
+        var defaultAdapter = new OkHttpClientAdapter();
+        assertDoesNotThrow(() -> defaultAdapter.shutdown());
+
+        var client = new OkHttpClient();
+        var customAdapter = new OkHttpClientAdapter(client);
+        assertDoesNotThrow(() -> customAdapter.shutdown());
+    }
+
 }