mirror of
https://github.com/nodejs/node.git
synced 2024-11-21 21:19:50 +01:00
2c4ebe0426
This makes sure that all async functions finish as expected. PR-URL: https://github.com/nodejs/node/pull/34363 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
56 lines
1.5 KiB
JavaScript
56 lines
1.5 KiB
JavaScript
'use strict';
|
|
|
|
// This test confirms that `undefined`, `null`, and `[]`
|
|
// can be used as a placeholder for the second argument (`args`) of `spawn()`.
|
|
// 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 { spawn } = require('child_process');
|
|
|
|
tmpdir.refresh();
|
|
|
|
const command = common.isWindows ? 'cd' : 'pwd';
|
|
const options = { cwd: tmpdir.path };
|
|
|
|
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();
|
|
|
|
(async () => {
|
|
const results = await Promise.all(
|
|
testCases.map((testCase) => {
|
|
return new Promise((resolve) => {
|
|
const subprocess = spawn(command, testCase, options);
|
|
|
|
let accumulatedData = Buffer.alloc(0);
|
|
|
|
subprocess.stdout.on('data', common.mustCall((data) => {
|
|
accumulatedData = Buffer.concat([accumulatedData, data]);
|
|
}));
|
|
|
|
subprocess.stdout.on('end', () => {
|
|
resolve(accumulatedData.toString().trim().toLowerCase());
|
|
});
|
|
});
|
|
})
|
|
);
|
|
|
|
assert.deepStrictEqual([...new Set(results)], [expectedResult]);
|
|
})().then(common.mustCall());
|