benchmark: improve assert benchmarks

This reduces the runtime and makes sure the strict and loose options
can be tested individually.

Besides that a couple of redundant cases were removed.

PR-URL: https://github.com/nodejs/node/pull/22211
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

1 files changed, 12 insertions, 8 deletions

diff --git a/benchmark/assert/deepequal-object.js b/benchmark/assert/deepequal-object.js
index 124943b19e..1a9c826a00 100644
--- a/benchmark/assert/deepequal-object.js
+++ b/benchmark/assert/deepequal-object.js
@@ -4,13 +4,12 @@ const common = require('../common.js');
 const assert = require('assert');
 
 const bench = common.createBenchmark(main, {
-  n: [1e6],
-  size: [1e2, 1e3, 1e4],
+  n: [5e3],
+  size: [1e2, 1e3, 5e4],
+  strict: [0, 1],
   method: [
     'deepEqual',
-    'deepStrictEqual',
-    'notDeepEqual',
-    'notDeepStrictEqual'
+    'notDeepEqual'
   ]
 });
 
@@ -20,14 +19,16 @@ function createObj(source, add = '') {
     nope: {
       bar: `123${add}`,
       a: [1, 2, 3],
-      baz: n
+      baz: n,
+      c: {},
+      b: []
     }
   }));
 }
 
-function main({ size, n, method }) {
+function main({ size, n, method, strict }) {
   // TODO: Fix this "hack". `n` should not be manipulated.
-  n = n / size;
+  n = Math.min(Math.ceil(n / size), 20);
 
   if (!method)
     method = 'deepEqual';
@@ -37,6 +38,9 @@ function main({ size, n, method }) {
   const expected = createObj(source);
   const expectedWrong = createObj(source, '4');
 
+  if (strict) {
+    method = method.replace('eep', 'eepStrict');
+  }
   const fn = assert[method];
   const value2 = method.includes('not') ? expectedWrong : expected;