Default random integer generation to maximun safe value

Signed-off-by: Jérôme Benoit <jerome.benoit@sap.com>

diff --git a/benchmarks/internal/benchmark-utils.js b/benchmarks/internal/benchmark-utils.js
index a7e4ddecded212d3f226af05d44e0a5d47441cf9..60cecbe5c8b565314c59549244e72db6c7178220 100644 (file)
--- a/benchmarks/internal/benchmark-utils.js
+++ b/benchmarks/internal/benchmark-utils.js
@@ -34,7 +34,7 @@ function jsonIntegerSerialization (n) {
   }
 }
 
-function generateRandomInteger (max, min = 0) {
+function generateRandomInteger (max = Number.MAX_SAFE_INTEGER, min = 0) {
   max = Math.floor(max)
   if (min) {
     min = Math.ceil(min)

diff --git a/tests/pools/selection-strategies/weighted-round-robin-worker-choice-strategy.test.js b/tests/pools/selection-strategies/weighted-round-robin-worker-choice-strategy.test.js
index 41c7751cc61d75eabc1762c7f3296f0f08945e9d..9d87c69970e6f371a8987483d1c06c090bb85a5f 100644 (file)
--- a/tests/pools/selection-strategies/weighted-round-robin-worker-choice-strategy.test.js
+++ b/tests/pools/selection-strategies/weighted-round-robin-worker-choice-strategy.test.js
@@ -25,12 +25,8 @@ describe('Weighted round robin strategy worker choice strategy test suite', () =
 
   it('Verify that reset() resets internals', () => {
     const strategy = new WeightedRoundRobinWorkerChoiceStrategy(pool)
-    strategy.previousWorkerIndex = TestUtils.generateRandomInteger(
-      Number.MAX_SAFE_INTEGER
-    )
-    strategy.currentWorkerIndex = TestUtils.generateRandomInteger(
-      Number.MAX_SAFE_INTEGER
-    )
+    strategy.previousWorkerIndex = TestUtils.generateRandomInteger()
+    strategy.currentWorkerIndex = TestUtils.generateRandomInteger()
     const workersTaskRunTimeClearStub = sinon
       .stub(strategy.workersTaskRunTime, 'clear')
       .returns()

diff --git a/tests/test-utils.js b/tests/test-utils.js
index a3affb61c91899d187dd91e97f2693cfa7340d0d..9a49c822991c2629ac45280776604cb3be188a04 100644 (file)
--- a/tests/test-utils.js
+++ b/tests/test-utils.js
@@ -36,7 +36,7 @@ class TestUtils {
     })
   }
 
-  static generateRandomInteger (max, min = 0) {
+  static generateRandomInteger (max = Number.MAX_SAFE_INTEGER, min = 0) {
     max = Math.floor(max)
     if (min) {
       min = Math.ceil(min)