Skip to content

Commit ddb8824

Browse files
BridgeARMylesBorins
authored andcommitted
benchmark: remove special test entries
This reverts commit 7cfbc9f. PR-URL: #31755 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Vladimir de Turckheim <[email protected]> Reviewed-By: Rich Trott <[email protected]>
1 parent 92cc406 commit ddb8824

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

41 files changed

+15
-81
lines changed

benchmark/assert/deepequal-buffer.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,12 +6,10 @@ const bench = common.createBenchmark(main, {
66
n: [2e4],
77
len: [1e2, 1e3],
88
strict: [0, 1],
9-
method: [ 'deepEqual', 'notDeepEqual' ],
9+
method: ['deepEqual', 'notDeepEqual'],
1010
});
1111

1212
function main({ len, n, method, strict }) {
13-
if (!method)
14-
method = 'deepEqual';
1513
const data = Buffer.allocUnsafe(len + 1);
1614
const actual = Buffer.alloc(len);
1715
const expected = Buffer.alloc(len);

benchmark/assert/deepequal-map.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,6 @@ function main({ n, len, method, strict }) {
3434
const array = Array(len).fill(1);
3535

3636
switch (method) {
37-
case '':
38-
// Empty string falls through to next line as default, mostly for tests.
3937
case 'deepEqual_primitiveOnly': {
4038
const values = array.map((_, i) => [`str_${i}`, 123]);
4139
benchmark(strict ? deepStrictEqual : deepEqual, n, values);

benchmark/assert/deepequal-object.js

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ const bench = common.createBenchmark(main, {
77
n: [5e3],
88
size: [1e2, 1e3, 5e4],
99
strict: [0, 1],
10-
method: [ 'deepEqual', 'notDeepEqual' ],
10+
method: ['deepEqual', 'notDeepEqual'],
1111
});
1212

1313
function createObj(source, add = '') {
@@ -27,9 +27,6 @@ function main({ size, n, method, strict }) {
2727
// TODO: Fix this "hack". `n` should not be manipulated.
2828
n = Math.min(Math.ceil(n / size), 20);
2929

30-
if (!method)
31-
method = 'deepEqual';
32-
3330
const source = Array.apply(null, Array(size));
3431
const actual = createObj(source);
3532
const expected = createObj(source);

benchmark/assert/deepequal-prims-and-objs-big-array-set.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,8 +52,6 @@ function main({ n, len, primitive, method, strict }) {
5252
const expectedWrongSet = new Set(expectedWrong);
5353

5454
switch (method) {
55-
// Empty string falls through to next line as default, mostly for tests.
56-
case '':
5755
case 'deepEqual_Array':
5856
run(strict ? deepStrictEqual : deepEqual, n, actual, expected);
5957
break;

benchmark/assert/deepequal-prims-and-objs-big-loop.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,12 +13,10 @@ const bench = common.createBenchmark(main, {
1313
primitive: Object.keys(primValues),
1414
n: [2e4],
1515
strict: [0, 1],
16-
method: [ 'deepEqual', 'notDeepEqual' ],
16+
method: ['deepEqual', 'notDeepEqual'],
1717
});
1818

1919
function main({ n, primitive, method, strict }) {
20-
if (!method)
21-
method = 'deepEqual';
2220
const prim = primValues[primitive];
2321
const actual = prim;
2422
const expected = prim;

benchmark/assert/deepequal-set.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,6 @@ function main({ n, len, method, strict }) {
3434
const array = Array(len).fill(1);
3535

3636
switch (method) {
37-
case '':
38-
// Empty string falls through to next line as default, mostly for tests.
3937
case 'deepEqual_primitiveOnly': {
4038
const values = array.map((_, i) => `str_${i}`);
4139
benchmark(strict ? deepStrictEqual : deepEqual, n, values);

benchmark/assert/deepequal-typedarrays.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,6 @@ const bench = common.createBenchmark(main, {
2020
});
2121

2222
function main({ type, n, len, method, strict }) {
23-
if (!method)
24-
method = 'deepEqual';
2523
const clazz = global[type];
2624
const actual = new clazz(len);
2725
const expected = new clazz(len);

benchmark/assert/throws.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,6 @@ function main({ n, method }) {
1515
const message = 'failure';
1616

1717
switch (method) {
18-
case '':
19-
// Empty string falls through to next line as default, mostly for tests.
2018
case 'doesNotThrow':
2119
bench.start();
2220
for (let i = 0; i < n; ++i) {

benchmark/buffers/buffer-bytelength.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,9 @@ const chars = [
1717

1818
function main({ n, len, encoding }) {
1919
let strings = [];
20-
let results = [ len * 16 ];
20+
let results = [len * 16];
2121
if (encoding === 'buffer') {
22-
strings = [ Buffer.alloc(len * 16, 'a') ];
22+
strings = [Buffer.alloc(len * 16, 'a')];
2323
} else {
2424
for (const string of chars) {
2525
// Strings must be built differently, depending on encoding

benchmark/buffers/buffer-creation.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@ const bench = common.createBenchmark(main, {
1616
function main({ len, n, type }) {
1717
let fn, i;
1818
switch (type) {
19-
case '':
2019
case 'fast-alloc':
2120
fn = Buffer.alloc;
2221
break;

0 commit comments

Comments
 (0)