Fix random integer generator

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

diff --git a/benchmarks/benchmarks-utils.js b/benchmarks/benchmarks-utils.js
index 62613c7e877308195871675021cc23cbba700a06..8bd137a34e3fdbc5a8d8418ac00e097ba251f64f 100644 (file)
--- a/benchmarks/benchmarks-utils.js
+++ b/benchmarks/benchmarks-utils.js
@@ -32,7 +32,7 @@ function jsonIntegerSerialization (n) {
 
 function generateRandomInteger (max = Number.MAX_SAFE_INTEGER, min = 0) {
   max = Math.floor(max)
-  if (min) {
+  if (min != null || min !== 0) {
     min = Math.ceil(min)
     return Math.floor(Math.random() * (max - min + 1)) + min
   }

diff --git a/benchmarks/worker-selection/lru.js b/benchmarks/worker-selection/lru.js
index 90628d3ad9d877d1002fa2df384c9f42cb890bb5..a4b2c69a330e0394cd82eff32e4f7768ed111a09 100644 (file)
--- a/benchmarks/worker-selection/lru.js
+++ b/benchmarks/worker-selection/lru.js
@@ -50,7 +50,7 @@ const defaultPivotIndexSelect = (leftIndex, rightIndex) => {
 }
 
 const randomPivotIndexSelect = (leftIndex, rightIndex) => {
-  return generateRandomInteger(leftIndex, rightIndex)
+  return generateRandomInteger(rightIndex, leftIndex)
 }
 
 function swap (array, index1, index2) {

diff --git a/tests/test-utils.js b/tests/test-utils.js
index 9a49c822991c2629ac45280776604cb3be188a04..76db35571578428b8c6a7a74a91ea4e42cff1dea 100644 (file)
--- a/tests/test-utils.js
+++ b/tests/test-utils.js
@@ -38,7 +38,7 @@ class TestUtils {
 
   static generateRandomInteger (max = Number.MAX_SAFE_INTEGER, min = 0) {
     max = Math.floor(max)
-    if (min) {
+    if (min != null || min !== 0) {
       min = Math.ceil(min)
       return Math.floor(Math.random() * (max - min + 1)) + min
     }