mirror of
https://github.com/nodejs/node.git
synced 2024-11-21 10:59:27 +00: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>
97 lines
3.0 KiB
JavaScript
97 lines
3.0 KiB
JavaScript
import { spawnPromisified } from '../common/index.mjs';
|
|
import * as fixtures from '../common/fixtures.mjs';
|
|
import assert from 'node:assert';
|
|
import { execPath } from 'node:process';
|
|
import { describe, it } from 'node:test';
|
|
|
|
describe('esm source-map', { concurrency: !process.env.TEST_PARALLEL }, () => {
|
|
// Issue: https://github.com/nodejs/node/issues/51522
|
|
|
|
[
|
|
[
|
|
'in middle from esm',
|
|
'source-map/extract-url/esm-url-in-middle.mjs',
|
|
true,
|
|
],
|
|
[
|
|
'inside string from esm',
|
|
'source-map/extract-url/esm-url-in-string.mjs',
|
|
false,
|
|
],
|
|
].forEach(([name, path, shouldExtract]) => {
|
|
it((shouldExtract ? 'should extract source map url' : 'should not extract source map url') + name, async () => {
|
|
const { code, signal, stdout, stderr } = await spawnPromisified(execPath, [
|
|
'--no-warnings',
|
|
'--enable-source-maps',
|
|
fixtures.path(path),
|
|
]);
|
|
|
|
assert.strictEqual(stdout, '');
|
|
if (shouldExtract) {
|
|
assert.match(stderr, /index\.ts/);
|
|
} else {
|
|
assert.doesNotMatch(stderr, /index\.ts/);
|
|
}
|
|
assert.strictEqual(code, 1);
|
|
assert.strictEqual(signal, null);
|
|
});
|
|
});
|
|
|
|
[
|
|
[
|
|
'in middle from esm imported by esm',
|
|
`import ${JSON.stringify(fixtures.fileURL('source-map/extract-url/esm-url-in-middle.mjs'))}`,
|
|
true,
|
|
],
|
|
[
|
|
'in middle from cjs imported by esm',
|
|
`import ${JSON.stringify(fixtures.fileURL('source-map/extract-url/cjs-url-in-middle.js'))}`,
|
|
true,
|
|
],
|
|
[
|
|
'in middle from cjs required by esm',
|
|
"import { createRequire } from 'module';" +
|
|
'const require = createRequire(import.meta.url);' +
|
|
`require(${JSON.stringify(fixtures.path('source-map/extract-url/cjs-url-in-middle.js'))})`,
|
|
true,
|
|
],
|
|
|
|
[
|
|
'inside string from esm imported by esm',
|
|
`import ${JSON.stringify(fixtures.fileURL('source-map/extract-url/esm-url-in-string.mjs'))}`,
|
|
false,
|
|
],
|
|
[
|
|
'inside string from cjs imported by esm',
|
|
`import ${JSON.stringify(fixtures.fileURL('source-map/extract-url/cjs-url-in-string.js'))}`,
|
|
false,
|
|
],
|
|
[
|
|
'inside string from cjs required by esm',
|
|
"import { createRequire } from 'module';" +
|
|
'const require = createRequire(import.meta.url);' +
|
|
`require(${JSON.stringify(fixtures.path('source-map/extract-url/cjs-url-in-string.js'))})`,
|
|
false,
|
|
],
|
|
].forEach(([name, evalCode, shouldExtract]) => {
|
|
it((shouldExtract ? 'should extract source map url' : 'should not extract source map url') + name, async () => {
|
|
const { code, signal, stdout, stderr } = await spawnPromisified(execPath, [
|
|
'--no-warnings',
|
|
'--enable-source-maps',
|
|
'--input-type=module',
|
|
'--eval',
|
|
evalCode,
|
|
]);
|
|
|
|
assert.strictEqual(stdout, '');
|
|
if (shouldExtract) {
|
|
assert.match(stderr, /index\.ts/);
|
|
} else {
|
|
assert.doesNotMatch(stderr, /index\.ts/);
|
|
}
|
|
assert.strictEqual(code, 1);
|
|
assert.strictEqual(signal, null);
|
|
});
|
|
});
|
|
});
|