From f9f153439495aacb186e57eadec2ce20e413d71b Mon Sep 17 00:00:00 2001
From: kwoyke <krzysztofwoyke@gmail.com>
Date: Fri, 13 Dec 2019 11:38:28 +0100
Subject: [PATCH] BAEL-3253: Use a random port number for Tomcat (#8351)

* BAEL-3253: Use a random port number for Tomcat

* BAEL-3253: Validate a port availability
---
 .../baeldung/tomcat/ProgrammaticTomcat.java   | 30 ++++++++++++++++++-
 .../ProgrammaticTomcatIntegrationTest.java    |  3 +-
 2 files changed, 31 insertions(+), 2 deletions(-)

diff --git a/libraries-server/src/main/java/com/baeldung/tomcat/ProgrammaticTomcat.java b/libraries-server/src/main/java/com/baeldung/tomcat/ProgrammaticTomcat.java
index 6c4fed6d07..f42db2d3de 100644
--- a/libraries-server/src/main/java/com/baeldung/tomcat/ProgrammaticTomcat.java
+++ b/libraries-server/src/main/java/com/baeldung/tomcat/ProgrammaticTomcat.java
@@ -7,22 +7,50 @@ import org.apache.tomcat.util.descriptor.web.FilterDef;
 import org.apache.tomcat.util.descriptor.web.FilterMap;
 
 import java.io.File;
+import java.io.IOException;
+import java.net.ServerSocket;
+import java.util.Random;
 
 /**
  * Created by adi on 1/10/18.
  */
 public class ProgrammaticTomcat {
 
+    private static boolean isFree(int port) {
+        try {
+            new ServerSocket(port).close();
+            return true;
+        } catch (IOException e) {
+            return false;
+        }
+    }
+
     private Tomcat tomcat = null;
 
+    private int randomPort;
+
+    public ProgrammaticTomcat() {
+        // Get a random port number in range 6000 (inclusive) - 9000 (exclusive)
+        this.randomPort = new Random()
+                .ints(6000, 9000)
+                .filter(ProgrammaticTomcat::isFree)
+                .findFirst()
+                .orElse(8080);
+    }
+
     // uncomment for live test
     // public static void main(String[] args) throws LifecycleException, ServletException, URISyntaxException, IOException {
     // startTomcat();
     // }
 
+
+    public int getPort() {
+        return randomPort;
+    }
+
     public void startTomcat() throws LifecycleException {
         tomcat = new Tomcat();
-        tomcat.setPort(8080);
+        tomcat.setPort(randomPort);
         tomcat.setHostname("localhost");
         String appBase = ".";
         tomcat.getHost().setAppBase(appBase);
diff --git a/libraries-server/src/test/java/com/baeldung/tomcat/ProgrammaticTomcatIntegrationTest.java b/libraries-server/src/test/java/com/baeldung/tomcat/ProgrammaticTomcatIntegrationTest.java
index 9224561341..888fb8e366 100644
--- a/libraries-server/src/test/java/com/baeldung/tomcat/ProgrammaticTomcatIntegrationTest.java
+++ b/libraries-server/src/test/java/com/baeldung/tomcat/ProgrammaticTomcatIntegrationTest.java
@@ -37,7 +37,8 @@ public class ProgrammaticTomcatIntegrationTest {
     @Test
     public void givenTomcatStarted_whenAccessServlet_responseIsTestAndResponseHeaderIsSet() throws Exception {
         CloseableHttpClient httpClient = HttpClientBuilder.create().build();
-        HttpGet getServlet = new HttpGet("http://localhost:8080/my-servlet");
+        String uri = "http://localhost:" + tomcat.getPort() + "/my-servlet";
+        HttpGet getServlet = new HttpGet(uri);
 
         HttpResponse response = httpClient.execute(getServlet);
         assertEquals(HttpStatus.SC_OK, response.getStatusLine().getStatusCode());