perf: use a single array to store pool workers and their related data

[poolifier.git] / tests / test-utils.js

diff --git a/tests/test-utils.js b/tests/test-utils.js
index 9a49c822991c2629ac45280776604cb3be188a04..6d3592c1b5d2e2798c00b95fe213c1c630370c96 100644 (file)
--- a/tests/test-utils.js
+++ b/tests/test-utils.js
@@ -2,16 +2,16 @@ const { WorkerFunctions } = require('./test-types')
 
 class TestUtils {
   static async waitExits (pool, numberOfExitEventsToWait) {
-    let exitEvents = 0
     return new Promise(resolve => {
-      pool.workers.forEach(w => {
-        w.on('exit', () => {
-          exitEvents++
+      let exitEvents = 0
+      for (const workerItem of pool.workers) {
+        workerItem.worker.on('exit', () => {
+          ++exitEvents
           if (exitEvents === numberOfExitEventsToWait) {
             resolve(exitEvents)
           }
         })
-      })
+      }
     })
   }
 
@@ -37,8 +37,11 @@ class TestUtils {
   }
 
   static generateRandomInteger (max = Number.MAX_SAFE_INTEGER, min = 0) {
+    if (max < min || max < 0 || min < 0) {
+      throw new RangeError('Invalid interval')
+    }
     max = Math.floor(max)
-    if (min) {
+    if (min != null && min !== 0) {
       min = Math.ceil(min)
       return Math.floor(Math.random() * (max - min + 1)) + min
     }