refactor (jkube-kit/common) : Refactor IoUtil.getFreeRandomPort to …

…always generate a valid free random port (eclipse-jkube#2501)

+ Use ServerSocket to establish a connection instead of Socket which might
  also throw ConnectException for a socket already in use.
+ Modify IoUtilTes.findOpenPortWithSmallAttemptsCount to use
  `@RepeatedTest` annotation to run it 500 times

Signed-off-by: Rohan Kumar <[email protected]>

jkube-kit/common/src/main/java/org/eclipse/jkube/kit/common/util/IoUtil.java

@@ -18,6 +18,7 @@
 import java.io.InputStream;
 import java.io.OutputStream;
 import java.net.ConnectException;
+import java.net.ServerSocket;
 import java.net.Socket;
 import java.net.URL;
 import java.nio.file.Files;
@@ -128,12 +129,10 @@ public static int getFreeRandomPort() {
     public static int getFreeRandomPort(int min, int max, int attempts) {
         for (int i=0; i < attempts; i++) {
             int port = min + RANDOM.nextInt(max - min + 1);
-            try (Socket ignored = new Socket("localhost", port)) { // NOSONAR
-                // Port is open for communication, meaning it's used up, try again
-            } catch (ConnectException e) {
+            try (ServerSocket ignored = new ServerSocket(port)) {
                 return port;
-            } catch (IOException e) {
-                throw new IllegalStateException("Error while trying to check open ports", e);
+            } catch (Exception e) {
+                // NOOP
             }
         }
         throw new IllegalStateException("Cannot find a free random port in the range [" + min + ", " + max + "] after " + attempts + " attempts");

jkube-kit/common/src/test/java/org/eclipse/jkube/kit/common/util/IoUtilTest.java

@@ -22,6 +22,7 @@
 import org.eclipse.jkube.kit.common.TestHttpStaticServer;
 import org.eclipse.jkube.kit.common.assertj.FileAssertions;
 import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.RepeatedTest;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.io.TempDir;
 
@@ -64,7 +65,7 @@ void findOpenPortWhenPortsAreBusy() throws IOException {
         assertThat(port2).isGreaterThan(port);
     }
 
-    @Test
+    @RepeatedTest(500)
     void findOpenPortWithSmallAttemptsCount() throws IOException {
         int port = IoUtil.getFreeRandomPort(30000, 60000, 30);
         try (ServerSocket ss = new ServerSocket(port)) {