Skip to content

Commit a221126

Browse files
aduh95aduh95
committed
test: do not set concurrency on parallelized runs
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.
1 parent 0b67673 commit a221126

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

47 files changed

+67
-67
lines changed

Diff for: ‎test/es-module/test-cjs-esm-warn.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ const pjson = path.resolve(
1515
);
1616

1717

18-
describe('CJS ↔︎ ESM interop warnings', { concurrency: true }, () => {
18+
describe('CJS ↔︎ ESM interop warnings', { concurrency: !process.env.TEST_PARALLEL }, () => {
1919

2020
it(async () => {
2121
const required = path.resolve(

Diff for: ‎test/es-module/test-esm-cjs-exports.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ const { execPath } = require('node:process');
77
const { describe, it } = require('node:test');
88

99

10-
describe('ESM: importing CJS', { concurrency: true }, () => {
10+
describe('ESM: importing CJS', { concurrency: !process.env.TEST_PARALLEL }, () => {
1111
it('should support valid CJS exports', async () => {
1212
const validEntry = fixtures.path('/es-modules/cjs-exports.mjs');
1313
const { code, signal, stdout } = await spawnPromisified(execPath, [validEntry]);

Diff for: ‎test/es-module/test-esm-cjs-load-error-note.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ const mustNotIncludeMessage = {
1919
includeNote: false,
2020
};
2121

22-
describe('ESM: Errors for unexpected exports', { concurrency: true }, () => {
22+
describe('ESM: Errors for unexpected exports', { concurrency: !process.env.TEST_PARALLEL }, () => {
2323
for (
2424
const { errorNeedle, filePath, getMessage, includeNote }
2525
of [

Diff for: ‎test/es-module/test-esm-detect-ambiguous.mjs

+7-7
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@ import { spawn } from 'node:child_process';
44
import { describe, it } from 'node:test';
55
import { strictEqual, match } from 'node:assert';
66

7-
describe('--experimental-detect-module', { concurrency: true }, () => {
8-
describe('string input', { concurrency: true }, () => {
7+
describe('--experimental-detect-module', { concurrency: !process.env.TEST_PARALLEL }, () => {
8+
describe('string input', { concurrency: !process.env.TEST_PARALLEL }, () => {
99
it('permits ESM syntax in --eval input without requiring --input-type=module', async () => {
1010
const { stdout, stderr, code, signal } = await spawnPromisified(process.execPath, [
1111
'--experimental-detect-module',
@@ -72,7 +72,7 @@ describe('--experimental-detect-module', { concurrency: true }, () => {
7272
});
7373
});
7474

75-
describe('.js file input in a typeless package', { concurrency: true }, () => {
75+
describe('.js file input in a typeless package', { concurrency: !process.env.TEST_PARALLEL }, () => {
7676
for (const { testName, entryPath } of [
7777
{
7878
testName: 'permits CommonJS syntax in a .js entry point',
@@ -113,7 +113,7 @@ describe('--experimental-detect-module', { concurrency: true }, () => {
113113
}
114114
});
115115

116-
describe('extensionless file input in a typeless package', { concurrency: true }, () => {
116+
describe('extensionless file input in a typeless package', { concurrency: !process.env.TEST_PARALLEL }, () => {
117117
for (const { testName, entryPath } of [
118118
{
119119
testName: 'permits CommonJS syntax in an extensionless entry point',
@@ -177,7 +177,7 @@ describe('--experimental-detect-module', { concurrency: true }, () => {
177177
});
178178
});
179179

180-
describe('file input in a "type": "commonjs" package', { concurrency: true }, () => {
180+
describe('file input in a "type": "commonjs" package', { concurrency: !process.env.TEST_PARALLEL }, () => {
181181
for (const { testName, entryPath } of [
182182
{
183183
testName: 'disallows ESM syntax in a .js entry point',
@@ -206,7 +206,7 @@ describe('--experimental-detect-module', { concurrency: true }, () => {
206206
}
207207
});
208208

209-
describe('file input in a "type": "module" package', { concurrency: true }, () => {
209+
describe('file input in a "type": "module" package', { concurrency: !process.env.TEST_PARALLEL }, () => {
210210
for (const { testName, entryPath } of [
211211
{
212212
testName: 'disallows CommonJS syntax in a .js entry point',
@@ -236,7 +236,7 @@ describe('--experimental-detect-module', { concurrency: true }, () => {
236236
});
237237

238238
// https://github.com/nodejs/node/issues/50917
239-
describe('syntax that errors in CommonJS but works in ESM', { concurrency: true }, () => {
239+
describe('syntax that errors in CommonJS but works in ESM', { concurrency: !process.env.TEST_PARALLEL }, () => {
240240
it('permits top-level `await`', async () => {
241241
const { stdout, stderr, code, signal } = await spawnPromisified(process.execPath, [
242242
'--experimental-detect-module',

Diff for: ‎test/es-module/test-esm-experimental-warnings.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import { execPath } from 'node:process';
55
import { describe, it } from 'node:test';
66

77

8-
describe('ESM: warn for obsolete hooks provided', { concurrency: true }, () => {
8+
describe('ESM: warn for obsolete hooks provided', { concurrency: !process.env.TEST_PARALLEL }, () => {
99
it('should not print warnings when no experimental features are enabled or used', async () => {
1010
const { code, signal, stderr } = await spawnPromisified(execPath, [
1111
'--input-type=module',

Diff for: ‎test/es-module/test-esm-export-not-found.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ const importStatementMultiline = `import {
1212
} from './module-named-exports.mjs';
1313
`;
1414

15-
describe('ESM: nonexistent exports', { concurrency: true }, () => {
15+
describe('ESM: nonexistent exports', { concurrency: !process.env.TEST_PARALLEL }, () => {
1616
for (
1717
const { name, input }
1818
of [

Diff for: ‎test/es-module/test-esm-extension-lookup-deprecation.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import * as path from 'node:path';
77
import { execPath } from 'node:process';
88
import { describe, it, before } from 'node:test';
99

10-
describe('ESM in main field', { concurrency: true }, () => {
10+
describe('ESM in main field', { concurrency: !process.env.TEST_PARALLEL }, () => {
1111
before(() => tmpdir.refresh());
1212

1313
it('should handle fully-specified relative path without any warning', async () => {

Diff for: ‎test/es-module/test-esm-extensionless-esm-and-wasm.mjs

+4-4
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import * as fixtures from '../common/fixtures.mjs';
44
import { describe, it } from 'node:test';
55
import { match, ok, strictEqual } from 'node:assert';
66

7-
describe('extensionless ES modules within a "type": "module" package scope', { concurrency: true }, () => {
7+
describe('extensionless ES modules within a "type": "module" package scope', { concurrency: !process.env.TEST_PARALLEL }, () => {
88
it('should run as the entry point', async () => {
99
const { code, signal, stdout, stderr } = await spawnPromisified(process.execPath, [
1010
fixtures.path('es-modules/package-type-module/noext-esm'),
@@ -29,7 +29,7 @@ describe('extensionless ES modules within a "type": "module" package scope', { c
2929
strictEqual(defaultExport, 'module');
3030
});
3131
});
32-
describe('extensionless Wasm modules within a "type": "module" package scope', { concurrency: true }, () => {
32+
describe('extensionless Wasm modules within a "type": "module" package scope', { concurrency: !process.env.TEST_PARALLEL }, () => {
3333
it('should run as the entry point', async () => {
3434
const { code, signal, stdout, stderr } = await spawnPromisified(process.execPath, [
3535
'--experimental-wasm-modules',
@@ -55,7 +55,7 @@ describe('extensionless Wasm modules within a "type": "module" package scope', {
5555
});
5656
});
5757

58-
describe('extensionless ES modules within no package scope', { concurrency: true }, () => {
58+
describe('extensionless ES modules within no package scope', { concurrency: !process.env.TEST_PARALLEL }, () => {
5959
// This succeeds with `--experimental-default-type=module`
6060
it('should error as the entry point', async () => {
6161
const { code, signal, stdout, stderr } = await spawnPromisified(process.execPath, [
@@ -79,7 +79,7 @@ describe('extensionless ES modules within no package scope', { concurrency: true
7979
});
8080
});
8181

82-
describe('extensionless Wasm within no package scope', { concurrency: true }, () => {
82+
describe('extensionless Wasm within no package scope', { concurrency: !process.env.TEST_PARALLEL }, () => {
8383
// This succeeds with `--experimental-default-type=module`
8484
it('should error as the entry point', async () => {
8585
const { code, signal, stdout, stderr } = await spawnPromisified(process.execPath, [

Diff for: ‎test/es-module/test-esm-import-flag.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ const mjsEntry = fixtures.path('es-modules', 'mjs-file.mjs');
1010
const mjsImport = fixtures.fileURL('es-modules', 'mjs-file.mjs');
1111

1212

13-
describe('import modules using --import', { concurrency: true }, () => {
13+
describe('import modules using --import', { concurrency: !process.env.TEST_PARALLEL }, () => {
1414
it('should import when using --eval', async () => {
1515
const { code, signal, stderr, stdout } = await spawnPromisified(
1616
execPath,

Diff for: ‎test/es-module/test-esm-import-json-named-export.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import { execPath } from 'node:process';
55
import { describe, it } from 'node:test';
66

77

8-
describe('ESM: named JSON exports', { concurrency: true }, () => {
8+
describe('ESM: named JSON exports', { concurrency: !process.env.TEST_PARALLEL }, () => {
99
it('should throw, citing named import', async () => {
1010
const { code, stderr } = await spawnPromisified(execPath, [
1111
fixtures.path('es-modules', 'import-json-named-export.mjs'),

Diff for: ‎test/es-module/test-esm-initialization.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import { execPath } from 'node:process';
55
import { describe, it } from 'node:test';
66

77

8-
describe('ESM: ensure initialization happens only once', { concurrency: true }, () => {
8+
describe('ESM: ensure initialization happens only once', { concurrency: !process.env.TEST_PARALLEL }, () => {
99
it(async () => {
1010
const { code, stderr, stdout } = await spawnPromisified(execPath, [
1111
'--experimental-import-meta-resolve',

Diff for: ‎test/es-module/test-esm-invalid-pjson.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ const { execPath } = require('node:process');
77
const { describe, it } = require('node:test');
88

99

10-
describe('ESM: Package.json', { concurrency: true }, () => {
10+
describe('ESM: Package.json', { concurrency: !process.env.TEST_PARALLEL }, () => {
1111
it('should throw on invalid pson', async () => {
1212
const entry = fixtures.path('/es-modules/import-invalid-pjson.mjs');
1313
const invalidJson = fixtures.path('/node_modules/invalid-pjson/package.json');

Diff for: ‎test/es-module/test-esm-loader-chaining.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ const commonArgs = [
1515
commonInput,
1616
];
1717

18-
describe('ESM: loader chaining', { concurrency: true }, () => {
18+
describe('ESM: loader chaining', { concurrency: !process.env.TEST_PARALLEL }, () => {
1919
it('should load unadulterated source when there are no loaders', async () => {
2020
const { code, stderr, stdout } = await spawnPromisified(
2121
execPath,

Diff for: ‎test/es-module/test-esm-loader-hooks.mjs

+3-3
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import assert from 'node:assert';
44
import { execPath } from 'node:process';
55
import { describe, it } from 'node:test';
66

7-
describe('Loader hooks', { concurrency: true }, () => {
7+
describe('Loader hooks', { concurrency: !process.env.TEST_PARALLEL }, () => {
88
it('are called with all expected arguments', async () => {
99
const { code, signal, stdout, stderr } = await spawnPromisified(execPath, [
1010
'--no-warnings',
@@ -50,7 +50,7 @@ describe('Loader hooks', { concurrency: true }, () => {
5050
assert.strictEqual(lines.length, 5);
5151
});
5252

53-
describe('should handle never-settling hooks in ESM files', { concurrency: true }, () => {
53+
describe('should handle never-settling hooks in ESM files', { concurrency: !process.env.TEST_PARALLEL }, () => {
5454
it('top-level await of a never-settling resolve without warning', async () => {
5555
const { code, signal, stdout, stderr } = await spawnPromisified(execPath, [
5656
'--no-warnings',
@@ -134,7 +134,7 @@ describe('Loader hooks', { concurrency: true }, () => {
134134
});
135135
});
136136

137-
describe('should handle never-settling hooks in CJS files', { concurrency: true }, () => {
137+
describe('should handle never-settling hooks in CJS files', { concurrency: !process.env.TEST_PARALLEL }, () => {
138138
it('never-settling resolve', async () => {
139139
const { code, signal, stdout, stderr } = await spawnPromisified(execPath, [
140140
'--no-warnings',

Diff for: ‎test/es-module/test-esm-loader-http-imports.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ const {
4646
port,
4747
} = server.address();
4848

49-
describe('ESM: http import via loader', { concurrency: true }, () => {
49+
describe('ESM: http import via loader', { concurrency: !process.env.TEST_PARALLEL }, () => {
5050
it('should load using --import flag', async () => {
5151
// ! MUST NOT use spawnSync to avoid blocking the event loop
5252
const { code, signal, stderr, stdout } = await spawnPromisified(

Diff for: ‎test/es-module/test-esm-loader-programmatically.mjs

+2-2
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ const commonEvals = {
2020
staticImport: (module) => `import ${JSON.stringify(`data:text/javascript,${encodeURIComponent(module)}`)};`,
2121
};
2222

23-
describe('ESM: programmatically register loaders', { concurrency: true }, () => {
23+
describe('ESM: programmatically register loaders', { concurrency: !process.env.TEST_PARALLEL }, () => {
2424
it('works with only a dummy CLI argument', async () => {
2525
const parentURL = fixtures.fileURL('es-module-loaders', 'loader-resolve-passthru.mjs');
2626
const { code, signal, stdout, stderr } = await spawnPromisified(execPath, [
@@ -49,7 +49,7 @@ describe('ESM: programmatically register loaders', { concurrency: true }, () =>
4949
assert.strictEqual(lines[5], '');
5050
});
5151

52-
describe('registering via --import', { concurrency: true }, () => {
52+
describe('registering via --import', { concurrency: !process.env.TEST_PARALLEL }, () => {
5353
for (const moduleType of ['mjs', 'cjs']) {
5454
it(`should programmatically register a loader from a ${moduleType.toUpperCase()} file`, async () => {
5555
const { code, signal, stdout, stderr } = await spawnPromisified(execPath, [

Diff for: ‎test/es-module/test-esm-loader-spawn-promisified.mjs

+2-2
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import assert from 'node:assert';
44
import { execPath } from 'node:process';
55
import { describe, it } from 'node:test';
66

7-
describe('Loader hooks throwing errors', { concurrency: true }, () => {
7+
describe('Loader hooks throwing errors', { concurrency: !process.env.TEST_PARALLEL }, () => {
88
it('throws on nonexistent modules', async () => {
99
const { code, signal, stdout, stderr } = await spawnPromisified(execPath, [
1010
'--no-warnings',
@@ -161,7 +161,7 @@ describe('Loader hooks throwing errors', { concurrency: true }, () => {
161161
});
162162
});
163163

164-
describe('Loader hooks parsing modules', { concurrency: true }, () => {
164+
describe('Loader hooks parsing modules', { concurrency: !process.env.TEST_PARALLEL }, () => {
165165
it('can parse .js files as ESM', async () => {
166166
const { code, signal, stdout, stderr } = await spawnPromisified(execPath, [
167167
'--no-warnings',

Diff for: ‎test/es-module/test-esm-loader-thenable.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import { execPath } from 'node:process';
55
import { describe, it } from 'node:test';
66

77

8-
describe('ESM: thenable loader hooks', { concurrency: true }, () => {
8+
describe('ESM: thenable loader hooks', { concurrency: !process.env.TEST_PARALLEL }, () => {
99
it('should behave as a normal promise resolution', async () => {
1010
const { code, stderr } = await spawnPromisified(execPath, [
1111
'--experimental-loader',

Diff for: ‎test/es-module/test-esm-loader-with-syntax-error.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import { execPath } from 'node:process';
55
import { describe, it } from 'node:test';
66

77

8-
describe('ESM: loader with syntax error', { concurrency: true }, () => {
8+
describe('ESM: loader with syntax error', { concurrency: !process.env.TEST_PARALLEL }, () => {
99
it('should crash the node process', async () => {
1010
const { code, stderr } = await spawnPromisified(execPath, [
1111
'--experimental-loader',

Diff for: ‎test/es-module/test-esm-module-not-found-commonjs-hint.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import { execPath } from 'node:process';
55
import { describe, it } from 'node:test';
66

77

8-
describe('ESM: module not found hint', { concurrency: true }, () => {
8+
describe('ESM: module not found hint', { concurrency: !process.env.TEST_PARALLEL }, () => {
99
for (
1010
const { input, expected, cwd = fixturesDir }
1111
of [

Diff for: ‎test/es-module/test-esm-non-js.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import { execPath } from 'node:process';
55
import { describe, it } from 'node:test';
66

77

8-
describe('ESM: non-js extensions fail', { concurrency: true }, () => {
8+
describe('ESM: non-js extensions fail', { concurrency: !process.env.TEST_PARALLEL }, () => {
99
it(async () => {
1010
const { code, stderr, signal } = await spawnPromisified(execPath, [
1111
'--input-type=module',

Diff for: ‎test/es-module/test-esm-nowarn-exports.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import { execPath } from 'node:process';
55
import { describe, it } from 'node:test';
66

77

8-
describe('ESM: experiemental warning for import.meta.resolve', { concurrency: true }, () => {
8+
describe('ESM: experiemental warning for import.meta.resolve', { concurrency: !process.env.TEST_PARALLEL }, () => {
99
it('should not warn when caught', async () => {
1010
const { code, signal, stderr } = await spawnPromisified(execPath, [
1111
'--experimental-import-meta-resolve',

Diff for: ‎test/es-module/test-esm-preserve-symlinks-main.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ try {
4141
skip('insufficient privileges for symlinks');
4242
}
4343

44-
describe('Invoke the main file via a symlink.', { concurrency: true }, () => {
44+
describe('Invoke the main file via a symlink.', { concurrency: !process.env.TEST_PARALLEL }, () => {
4545
it('should resolve relative imports in the main file', async () => {
4646
const { code } = await spawnPromisified(execPath, [
4747
'--preserve-symlinks',

Diff for: ‎test/es-module/test-esm-repl-imports.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ const { execPath } = require('node:process');
88
const { describe, it } = require('node:test');
99

1010

11-
describe('ESM: REPL runs', { concurrency: true }, () => {
11+
describe('ESM: REPL runs', { concurrency: !process.env.TEST_PARALLEL }, () => {
1212
it((t, done) => {
1313
const child = spawn(execPath, [
1414
'--interactive',

Diff for: ‎test/es-module/test-esm-source-map.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import assert from 'node:assert';
44
import { execPath } from 'node:process';
55
import { describe, it } from 'node:test';
66

7-
describe('esm source-map', { concurrency: true }, () => {
7+
describe('esm source-map', { concurrency: !process.env.TEST_PARALLEL }, () => {
88
// Issue: https://github.com/nodejs/node/issues/51522
99

1010
[

Diff for: ‎test/es-module/test-esm-syntax-error.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import { execPath } from 'node:process';
55
import { describe, it } from 'node:test';
66

77

8-
describe('ESM: importing a module with syntax error(s)', { concurrency: true }, () => {
8+
describe('ESM: importing a module with syntax error(s)', { concurrency: !process.env.TEST_PARALLEL }, () => {
99
it('should throw', async () => {
1010
const { code, stderr } = await spawnPromisified(execPath, [
1111
path('es-module-loaders', 'syntax-error.mjs'),

Diff for: ‎test/es-module/test-esm-tla-unfinished.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ const commonArgs = [
1010
'--eval',
1111
];
1212

13-
describe('ESM: unsettled and rejected promises', { concurrency: true }, () => {
13+
describe('ESM: unsettled and rejected promises', { concurrency: !process.env.TEST_PARALLEL }, () => {
1414
it('should exit for an unsettled TLA promise via --eval with a warning', async () => {
1515
const { code, stderr, stdout } = await spawnPromisified(execPath, [
1616
...commonArgs,

Diff for: ‎test/es-module/test-esm-type-flag-cli-entry.mjs

+2-2
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import * as fixtures from '../common/fixtures.mjs';
33
import { describe, it } from 'node:test';
44
import { match, strictEqual } from 'node:assert';
55

6-
describe('--experimental-default-type=module should not support extension searching', { concurrency: true }, () => {
6+
describe('--experimental-default-type=module should not support extension searching', { concurrency: !process.env.TEST_PARALLEL }, () => {
77
it('should support extension searching under --experimental-default-type=commonjs', async () => {
88
const { code, signal, stdout, stderr } = await spawnPromisified(process.execPath, [
99
'--experimental-default-type=commonjs',
@@ -33,7 +33,7 @@ describe('--experimental-default-type=module should not support extension search
3333
});
3434
});
3535

36-
describe('--experimental-default-type=module should not parse paths as URLs', { concurrency: true }, () => {
36+
describe('--experimental-default-type=module should not parse paths as URLs', { concurrency: !process.env.TEST_PARALLEL }, () => {
3737
it('should not parse a `?` in a filename as starting a query string', async () => {
3838
const { code, signal, stdout, stderr } = await spawnPromisified(process.execPath, [
3939
'--experimental-default-type=module',

Diff for: ‎test/es-module/test-esm-type-flag-errors.mjs

+2-2
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@ import * as fixtures from '../common/fixtures.mjs';
33
import { describe, it } from 'node:test';
44
import { deepStrictEqual, match, strictEqual } from 'node:assert';
55

6-
describe('--experimental-default-type=module', { concurrency: true }, () => {
7-
describe('should not affect the interpretation of files with unknown extensions', { concurrency: true }, () => {
6+
describe('--experimental-default-type=module', { concurrency: !process.env.TEST_PARALLEL }, () => {
7+
describe('should not affect the interpretation of files with unknown extensions', { concurrency: !process.env.TEST_PARALLEL }, () => {
88
it('should error on an entry point with an unknown extension', async () => {
99
const { code, signal, stdout, stderr } = await spawnPromisified(process.execPath, [
1010
'--experimental-default-type=module',

Diff for: ‎test/es-module/test-esm-type-flag-loose-files.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import { describe, it } from 'node:test';
55
import { strictEqual } from 'node:assert';
66

77
describe('the type flag should change the interpretation of certain files outside of any package scope',
8-
{ concurrency: true }, () => {
8+
{ concurrency: !process.env.TEST_PARALLEL }, () => {
99
it('should run as ESM a .js file that is outside of any package scope', async () => {
1010
const { code, signal, stdout, stderr } = await spawnPromisified(process.execPath, [
1111
'--experimental-default-type=module',

0 commit comments

Comments
 (0)