Improve random integer array generation

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

diff --git a/benchmark-utils.js b/benchmark-utils.js
index 56689d652dafd3f4bc92720c0f7cf0ef14f5bb8d..1e7914f95b78ec72175f5d5827bebd28980f1038 100644 (file)
--- a/benchmark-utils.js
+++ b/benchmark-utils.js
@@ -29,6 +29,19 @@ function generateRandomInteger (max, min = 0) {
   return Math.floor(secureRandom() * (max + 1))
 }
 
+/**
+ *
+ * @param size
+ * @param max
+ */
+function generateRandomIntegerArray (size, max = Number.MAX_SAFE_INTEGER) {
+  const integerArray = []
+  for (let i = 0; i < size; i++) {
+    integerArray.push(generateRandomInteger(max))
+  }
+  return integerArray
+}
+
 /**
  * @param ms
  * @returns
@@ -42,4 +55,10 @@ const LIST_FORMATTER = new Intl.ListFormat('en-US', {
   type: 'conjunction'
 })
 
-module.exports = { generateRandomInteger, sleep, secureRandom, LIST_FORMATTER }
+module.exports = {
+  generateRandomInteger,
+  generateRandomIntegerArray,
+  sleep,
+  secureRandom,
+  LIST_FORMATTER
+}

diff --git a/busy-wait.js b/busy-wait.js
index 46e5853ca0c1e1951e616cf984fae44a484e0e44..eece669424406485a677c5248337e335e4bee008 100644 (file)
--- a/busy-wait.js
+++ b/busy-wait.js
@@ -61,7 +61,7 @@ suite
     dummyTimeoutBusyWait(timeout)
   })
   .add('sleepTimeoutBusyWait', async () => {
-    sleepTimeoutBusyWait(timeout)
+    await sleepTimeoutBusyWait(timeout)
   })
   .add('divideAndConquerTimeoutBusyWait', async () => {
     await divideAndConquerTimeoutBusyWait(timeout)

diff --git a/empty-array.js b/empty-array.js
index e623b5ea88a66ff6ad8a6cfc214d83b81ac4cd9a..9ddc17d45b06d37b57c77129d0d1ed7b69620480 100644 (file)
--- a/empty-array.js
+++ b/empty-array.js
@@ -1,19 +1,7 @@
 const Benchmark = require('benny')
+const { generateRandomIntegerArray } = require('./benchmark-utils')
 
-let testArray = [
-  83, 93, 27, 29, 2828, 234, 23, 56, 32, 56, 67, 77, 32, 45, 93, 17, 28, 83, 62,
-  99, 36, 28, 93, 27, 29, 2828, 234, 23, 56, 32, 56, 67, 77, 32, 45, 93, 17, 28,
-  83, 62, 99, 36, 28, 93, 27, 29, 2828, 234, 23, 56, 32, 56, 67, 77, 32, 45, 93,
-  17, 28, 83, 62, 99, 36, 28, 93, 27, 29, 2828, 234, 23, 56, 32, 56, 67, 77, 32,
-  45, 93, 17, 28, 83, 62, 99, 36, 28, 93, 27, 29, 2828, 234, 23, 56, 32, 56, 67,
-  77, 32, 45, 93, 17, 28, 83, 62, 99, 36, 28, 93, 27, 29, 2828, 234, 23, 56, 32,
-  56, 67, 77, 32, 45, 93, 17, 28, 83, 62, 99, 36, 28, 93, 27, 29, 2828, 234, 23,
-  56, 32, 56, 67, 77, 32, 45, 93, 17, 28, 83, 62, 99, 36, 28, 93, 27, 29, 2828,
-  234, 23, 56, 32, 56, 67, 77, 32, 45, 93, 17, 28, 83, 62, 99, 36, 28, 93, 27,
-  29, 2828, 234, 23, 56, 32, 56, 67, 77, 32, 45, 93, 17, 28, 83, 62, 99, 36, 28,
-  93, 27, 29, 2828, 234, 23, 56, 32, 56, 67, 77, 32, 45, 93, 17, 28, 83, 62, 99,
-  36, 28
-]
+let testArray = generateRandomIntegerArray(10000)
 
 Benchmark.suite(
   'Empty array',

diff --git a/max.js b/max.js
index d0d1381154170fa72776fa506f33def2aa075c30..5525c9293451fd335924e88a15698b43cf2e31ae 100644 (file)
--- a/max.js
+++ b/max.js
@@ -1,19 +1,7 @@
 const Benchmark = require('benny')
+const { generateRandomIntegerArray } = require('./benchmark-utils')
 
-const testArray = [
-  83, 93, 27, 29, 2828, 234, 23, 56, 32, 56, 67, 77, 32, 45, 93, 17, 28, 83, 62,
-  99, 36, 28, 93, 27, 29, 2828, 234, 23, 56, 32, 56, 67, 77, 32, 45, 93, 17, 28,
-  83, 62, 99, 36, 28, 93, 27, 29, 2828, 234, 23, 56, 32, 56, 67, 77, 32, 45, 93,
-  17, 28, 83, 62, 99, 36, 28, 93, 27, 29, 2828, 234, 23, 56, 32, 56, 67, 77, 32,
-  45, 93, 17, 28, 83, 62, 99, 36, 28, 93, 27, 29, 2828, 234, 23, 56, 32, 56, 67,
-  77, 32, 45, 93, 17, 28, 83, 62, 99, 36, 28, 93, 27, 29, 2828, 234, 23, 56, 32,
-  56, 67, 77, 32, 45, 93, 17, 28, 83, 62, 99, 36, 28, 93, 27, 29, 2828, 234, 23,
-  56, 32, 56, 67, 77, 32, 45, 93, 17, 28, 83, 62, 99, 36, 28, 93, 27, 29, 2828,
-  234, 23, 56, 32, 56, 67, 77, 32, 45, 93, 17, 28, 83, 62, 99, 36, 28, 93, 27,
-  29, 2828, 234, 23, 56, 32, 56, 67, 77, 32, 45, 93, 17, 28, 83, 62, 99, 36, 28,
-  93, 27, 29, 2828, 234, 23, 56, 32, 56, 67, 77, 32, 45, 93, 17, 28, 83, 62, 99,
-  36, 28
-]
+const testArray = generateRandomIntegerArray(10000)
 
 /**
  *