Skip to content

benchmark: move non-present deps down the list

Rodrigo Muino Tomonari requested to merge github/fork/AdamMajer/benchmark-fixes into main

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.

Fixes: https://github.com/nodejs/node/pull/51146 Refs: https://github.com/nodejs/node/pull/50684

Merge request reports