Skip to content

Commit 893ac74

Browse files
committed
benchmark: move non-present deps down the list
In previous version of this fix, I've simply added a check if the tested tool is available or not. Unfortuntelly, this fails when only the first tool is to be run as part of the test-benchmark-misc, and it doesn't exist. benchmark/test-benchmark-misc ... AssertionError [ERR_ASSERTION]: benchmark file not running exactly one configuration in test: ... misc/startup-cli-version.js ... The solution is to move the tool that is not present in a tarball down the list. Refs: #51146
1 parent 544cfc5 commit 893ac74

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

benchmark/misc/startup-cli-version.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,10 @@ const path = require('path');
1111
// indispensible part of the CLI.
1212
const bench = common.createBenchmark(main, {
1313
cli: [
14-
'tools/node_modules/eslint/bin/eslint.js',
1514
'deps/npm/bin/npx-cli.js',
1615
'deps/npm/bin/npm-cli.js',
1716
'deps/corepack/dist/corepack.js',
17+
'tools/node_modules/eslint/bin/eslint.js',
1818
],
1919
count: [30],
2020
});

0 commit comments

Comments
 (0)