mirror of
https://github.com/nodejs/node.git
synced 2025-08-16 22:28:51 +02:00
bae14b7914
Our CI already run test files in parallel, having `node:test` spawns child processes concurrently could lead to oversubscribing the CI machine. This commit sets the `concurrency` depending on the presence of `TEST_PARALLEL` in the env, so running the test file individually still spawns child processes concurrently, and running the whole test suite does not oversubscribe the machine. PR-URL: https://github.com/nodejs/node/pull/52177 Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
48 lines
1.8 KiB
JavaScript
48 lines
1.8 KiB
JavaScript
import { mustCall, spawnPromisified } from '../common/index.mjs';
|
|
import { ok, match, notStrictEqual } from 'node:assert';
|
|
import { spawn as spawnAsync } from 'node:child_process';
|
|
import { execPath } from 'node:process';
|
|
import { describe, it } from 'node:test';
|
|
|
|
|
|
describe('ESM: http import via CLI', { concurrency: !process.env.TEST_PARALLEL }, () => {
|
|
const disallowedSpecifier = 'http://example.com';
|
|
|
|
it('should throw disallowed error for insecure protocol', async () => {
|
|
const { code, stderr } = await spawnPromisified(execPath, [
|
|
'--experimental-network-imports',
|
|
'--input-type=module',
|
|
'--eval',
|
|
`import ${JSON.stringify(disallowedSpecifier)}`,
|
|
]);
|
|
|
|
notStrictEqual(code, 0);
|
|
|
|
// [ERR_NETWORK_IMPORT_DISALLOWED]: import of 'http://example.com/' by
|
|
// …/[eval1] is not supported: http can only be used to load local
|
|
// resources (use https instead).
|
|
match(stderr, /ERR_NETWORK_IMPORT_DISALLOWED/);
|
|
ok(stderr.includes(disallowedSpecifier));
|
|
});
|
|
|
|
it('should throw disallowed error for insecure protocol in REPL', () => {
|
|
const child = spawnAsync(execPath, [
|
|
'--experimental-network-imports',
|
|
'--input-type=module',
|
|
]);
|
|
child.stdin.end(`import ${JSON.stringify(disallowedSpecifier)}`);
|
|
|
|
let stderr = '';
|
|
child.stderr.setEncoding('utf8');
|
|
child.stderr.on('data', (data) => stderr += data);
|
|
child.on('close', mustCall((code, _signal) => {
|
|
notStrictEqual(code, 0);
|
|
|
|
// [ERR_NETWORK_IMPORT_DISALLOWED]: import of 'http://example.com/' by
|
|
// …/[stdin] is not supported: http can only be used to load local
|
|
// resources (use https instead).
|
|
match(stderr, /\[ERR_NETWORK_IMPORT_DISALLOWED\]/);
|
|
ok(stderr.includes(disallowedSpecifier));
|
|
}));
|
|
});
|
|
});
|