fix: Check for Thread id instead of execution time

Author: lolgab

core/src/test/scala/com.codacy.analysis.core/utils/SetOpsSpec.scala

@@ -6,19 +6,16 @@ class SetOpsSpec extends Specification {
 
   "SetOps.mapInParallel" should {
     "actually run in parallel" in {
-      val sleepMillis = 100L
-      val parallelism = 20
+      val parallelism = 10
 
-      val initialTime = System.currentTimeMillis()
-      SetOps.mapInParallel(1.to(parallelism).toSet, Some(parallelism)) { _ =>
-        Thread.sleep(sleepMillis)
-      }
-      val finalTime = System.currentTimeMillis()
+      val threadsIds = SetOps.mapInParallel(1.to(parallelism).toSet, Some(parallelism)) { _ =>
+        // If the function terminates too fast the thread pool reuses the same threads
+        Thread.sleep(100)
 
-      val elapsedMillis = finalTime - initialTime
-      val expectedMaxMillis = (sleepMillis * parallelism * 0.95).toLong
+        Thread.currentThread().getId()
+      }
 
-      elapsedMillis should beLessThan(expectedMaxMillis)
+      threadsIds.distinct should haveSize(parallelism)
     }
   }
 }