mirror of
https://github.com/nodejs/node.git
synced 2024-11-21 10:59:27 +00:00
dcc5e51e1c
PR-URL: https://github.com/nodejs/node/pull/27650 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
49 lines
1.2 KiB
JavaScript
49 lines
1.2 KiB
JavaScript
'use strict';
|
|
|
|
// This test confirms that `undefined`, `null`, and `[]` can be used
|
|
// as a placeholder for the second argument (`args`) of `spawnSync()`.
|
|
// Previously, there was a bug where using `undefined` for the second argument
|
|
// caused the third argument (`options`) to be ignored.
|
|
// See https://github.com/nodejs/node/issues/24912.
|
|
|
|
const common = require('../common');
|
|
const tmpdir = require('../common/tmpdir');
|
|
|
|
const assert = require('assert');
|
|
const { spawnSync } = require('child_process');
|
|
|
|
const command = common.isWindows ? 'cd' : 'pwd';
|
|
const options = { cwd: tmpdir.path };
|
|
|
|
tmpdir.refresh();
|
|
|
|
if (common.isWindows) {
|
|
// This test is not the case for Windows based systems
|
|
// unless the `shell` options equals to `true`
|
|
|
|
options.shell = true;
|
|
}
|
|
|
|
const testCases = [
|
|
undefined,
|
|
null,
|
|
[],
|
|
];
|
|
|
|
const expectedResult = tmpdir.path.trim().toLowerCase();
|
|
|
|
const results = testCases.map((testCase) => {
|
|
const { stdout, stderr, error } = spawnSync(
|
|
command,
|
|
testCase,
|
|
options
|
|
);
|
|
|
|
assert.ifError(error);
|
|
assert.deepStrictEqual(stderr, Buffer.alloc(0));
|
|
|
|
return stdout.toString().trim().toLowerCase();
|
|
});
|
|
|
|
assert.deepStrictEqual([...new Set(results)], [expectedResult]);
|