mirror of
https://github.com/nodejs/node.git
synced 2025-04-28 13:40:37 +00:00
a4e7ea8a3a
* Let users provide more than one pattern by repeating the flag * Add new flag --exclude to exclude patterns * Add tests for --filter * Document --filter This commit also fixes a bug where things like `compare.js --new --old binary --new binary` was acceptable (now the script will exit and print the usage message). PR-URL: https://github.com/nodejs/node/pull/29987 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Denys Otrishko <shishugi@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
39 lines
1.2 KiB
JavaScript
39 lines
1.2 KiB
JavaScript
'use strict';
|
|
|
|
require('../common');
|
|
|
|
// This tests the CLI parser for our benchmark suite.
|
|
|
|
const assert = require('assert');
|
|
|
|
const CLI = require('../../benchmark/_cli.js');
|
|
|
|
const originalArgv = process.argv;
|
|
|
|
function testFilterPattern(filters, excludes, filename, expectedResult) {
|
|
process.argv = process.argv.concat(...filters.map((p) => ['--filter', p]));
|
|
process.argv = process.argv.concat(...excludes.map((p) => ['--exclude', p]));
|
|
process.argv = process.argv.concat(['bench']);
|
|
|
|
const cli = new CLI('', { 'arrayArgs': ['filter', 'exclude'] });
|
|
assert.deepStrictEqual(cli.shouldSkip(filename), expectedResult);
|
|
|
|
process.argv = originalArgv;
|
|
}
|
|
|
|
|
|
testFilterPattern([], [], 'foo', false);
|
|
|
|
testFilterPattern(['foo'], [], 'foo', false);
|
|
testFilterPattern(['foo'], [], 'bar', true);
|
|
testFilterPattern(['foo', 'bar'], [], 'foo', false);
|
|
testFilterPattern(['foo', 'bar'], [], 'bar', false);
|
|
|
|
testFilterPattern([], ['foo'], 'foo', true);
|
|
testFilterPattern([], ['foo'], 'bar', false);
|
|
testFilterPattern([], ['foo', 'bar'], 'foo', true);
|
|
testFilterPattern([], ['foo', 'bar'], 'bar', true);
|
|
|
|
testFilterPattern(['foo'], ['bar'], 'foo', false);
|
|
testFilterPattern(['foo'], ['bar'], 'foo-bar', true);
|