Refactored to use ports 11000 to 14000 instead of random ports

Author: user

main/src/mill/main/PortManager.scala

@@ -9,26 +9,31 @@ object PortManager extends ExternalModule {
   var portsAllocated = Set.empty[Int]
   var portsByName : Map[String, Set[Int]] = Map.empty[String, Set[Int]]
 
+  val startingPort = 11000
+  val endingPort = 14000
+
   def getPorts(tartgetNumberOfPorts : Int): Option[Set[Int]] = {
     var i = 0
     var ports = Set.empty[Int]
 
     this.synchronized {
-      for (z <- 1 to 100) {
+      for (port <- startingPort to endingPort) {
 
         if (i >= tartgetNumberOfPorts) {
           return Some(ports)
         }
-        val socket = new ServerSocket(0)
-        try {
-          val port = socket.getLocalPort
-          if (!ports.contains(port) && !portsAllocated.contains(port)) {
-            ports = ports + port
-            i += 1
+        var portGood = false
+        if (!ports.contains(port) && !portsAllocated.contains(port)) {
+          try {
+            val socket = new ServerSocket(port)
+            portGood = true
+            socket.close()
           }
 
-        } finally {
-          socket.close()
+          if (portGood){
+            i = i+1
+            ports = ports + port
+          }
         }
       }
       portsAllocated = ports ++ portsAllocated