Nodejs/node
1
0
Fork 0
mirror of https://github.com/nodejs/node.git synced 2024-11-21 10:59:27 +00:00
Code Issues Actions 25 Packages Projects Releases Wiki Activity

benchmark: remove dependency on unshipped tools

Browse Source 
tools/node_modules is removed from the tarball so it should not be used
as part of unit tests or benchmarks.

Fixes: https://github.com/nodejs/node/issues/51145
Refs: https://github.com/nodejs/node/pull/50684
PR-URL: https://github.com/nodejs/node/pull/51146
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
This commit is contained in:
Adam Majer 2023-12-13 14:34:39 +01:00 committed by James M Snell
parent aa87b776cd
commit 345f15e493
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
benchmark/misc/startup-cli-version.js
View File

@ -1,6 +1,7 @@
'use strict';
const common = require('../common.js');
const { spawnSync } = require('child_process');
const { existsSync } = require('fs');
const path = require('path');
// This benchmarks the startup of various CLI tools that are already
@ -11,6 +12,7 @@ const path = require('path');
const bench = common.createBenchmark(main, {
cli: [
'tools/node_modules/eslint/bin/eslint.js',
'deps/npm/bin/npx-cli.js',
'deps/npm/bin/npm-cli.js',
'deps/corepack/dist/corepack.js',
],
@ -45,6 +47,10 @@ function spawnProcess(cli, bench, state) {
function main({ count, cli }) {
cli = path.resolve(__dirname, '../../', cli);
if (!existsSync(cli)) {
return;
}
const warmup = 3;
const state = { count, finished: -warmup };
spawnProcess(cli, bench, state);