bae14b7914Our 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>
68 lines
2.2 KiB
JavaScript
68 lines
2.2 KiB
JavaScript
'use strict';
|
|
|
|
const { spawnPromisified } = require('../common');
|
|
const fixtures = require('../common/fixtures.js');
|
|
const assert = require('node:assert');
|
|
const path = require('node:path');
|
|
const { execPath } = require('node:process');
|
|
const { describe, it } = require('node:test');
|
|
|
|
|
|
const requiringCjsAsEsm = path.resolve(fixtures.path('/es-modules/cjs-esm.js'));
|
|
const requiringEsm = path.resolve(fixtures.path('/es-modules/cjs-esm-esm.js'));
|
|
const pjson = path.resolve(
|
|
fixtures.path('/es-modules/package-type-module/package.json')
|
|
);
|
|
|
|
|
|
describe('CJS ↔︎ ESM interop warnings', { concurrency: !process.env.TEST_PARALLEL }, () => {
|
|
|
|
it(async () => {
|
|
const required = path.resolve(
|
|
fixtures.path('/es-modules/package-type-module/cjs.js')
|
|
);
|
|
const basename = 'cjs.js';
|
|
const { code, signal, stderr } = await spawnPromisified(execPath, [requiringCjsAsEsm]);
|
|
|
|
assert.ok(
|
|
stderr.replaceAll('\r', '').includes(
|
|
`Error [ERR_REQUIRE_ESM]: require() of ES Module ${required} from ${requiringCjsAsEsm} not supported.\n`
|
|
)
|
|
);
|
|
assert.ok(
|
|
stderr.replaceAll('\r', '').includes(
|
|
`Instead either rename ${basename} to end in .cjs, change the requiring ` +
|
|
'code to use dynamic import() which is available in all CommonJS ' +
|
|
`modules, or change "type": "module" to "type": "commonjs" in ${pjson} to ` +
|
|
'treat all .js files as CommonJS (using .mjs for all ES modules ' +
|
|
'instead).\n'
|
|
)
|
|
);
|
|
|
|
assert.strictEqual(code, 1);
|
|
assert.strictEqual(signal, null);
|
|
});
|
|
|
|
it(async () => {
|
|
const required = path.resolve(
|
|
fixtures.path('/es-modules/package-type-module/esm.js')
|
|
);
|
|
const basename = 'esm.js';
|
|
const { code, signal, stderr } = await spawnPromisified(execPath, [requiringEsm]);
|
|
|
|
assert.ok(
|
|
stderr.replace(/\r/g, '').includes(
|
|
`Error [ERR_REQUIRE_ESM]: require() of ES Module ${required} from ${requiringEsm} not supported.\n`
|
|
)
|
|
);
|
|
assert.ok(
|
|
stderr.replace(/\r/g, '').includes(
|
|
`Instead change the require of ${basename} in ${requiringEsm} to` +
|
|
' a dynamic import() which is available in all CommonJS modules.\n'
|
|
)
|
|
);
|
|
|
|
assert.strictEqual(code, 1);
|
|
assert.strictEqual(signal, null);
|
|
});
|
|
});
|