Address the comments

apache · Oct 12, 2023 · 97d5131 · 97d5131
1 parent f17ab04
commit 97d5131
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/server-common/src/test/java/com/datastrato/graviton/server/web/TestJettyServer.java b/server-common/src/test/java/com/datastrato/graviton/server/web/TestJettyServer.java
@@ -9,6 +9,8 @@
 import static org.mockito.Mockito.mock;
 
 import com.datastrato.graviton.Config;
+import com.datastrato.graviton.rest.RESTUtils;
+import java.io.IOException;
 import javax.servlet.Filter;
 import javax.servlet.Servlet;
 import org.junit.jupiter.api.AfterEach;
@@ -32,17 +34,19 @@ public void tearDown() {
   }
 
   @Test
-  public void testInitialize() {
+  public void testInitialize() throws IOException {
     Config config = new Config(false) {};
+    config.set(JettyServerConfig.WEBSERVER_HTTP_PORT, RESTUtils.findAvailablePort(5000, 6000));
     JettyServerConfig serverConfig = JettyServerConfig.fromConfig(config);
     jettyServer.initialize(serverConfig, "test");
 
     // TODO might be nice to have an isInitalised method or similar?
   }
 
   @Test
-  public void testStartAndStop() throws RuntimeException {
+  public void testStartAndStop() throws RuntimeException, IOException {
     Config config = new Config(false) {};
+    config.set(JettyServerConfig.WEBSERVER_HTTP_PORT, RESTUtils.findAvailablePort(5000, 6000));
     JettyServerConfig serverConfig = JettyServerConfig.fromConfig(config);
     jettyServer.initialize(serverConfig, "test");
     jettyServer.start();
@@ -51,8 +55,9 @@ public void testStartAndStop() throws RuntimeException {
   }
 
   @Test
-  public void testAddServletAndFilter() throws RuntimeException {
+  public void testAddServletAndFilter() throws RuntimeException, IOException {
     Config config = new Config(false) {};
+    config.set(JettyServerConfig.WEBSERVER_HTTP_PORT, RESTUtils.findAvailablePort(5000, 6000));
     JettyServerConfig serverConfig = JettyServerConfig.fromConfig(config);
     jettyServer.initialize(serverConfig, "test");
     jettyServer.start();