mirror of
https://github.com/nodejs/node.git
synced 2024-11-21 10:59:27 +00:00
06206af181
This unflags --experimental-require-module so require(esm) can be used without the flag. For now, when require() actually encounters an ESM, it will still emit an experimental warning. To opt out of the feature, --no-experimental-require-module can be used. There are some tests specifically testing ERR_REQUIRE_ESM. Some of them are repurposed to test --no-experimental-require-module. Some of them are modified to just expect loading require(esm) to work, when it's appropriate. PR-URL: https://github.com/nodejs/node/pull/55085 Refs: https://github.com/nodejs/node/issues/52697 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: LiviaMedeiros <livia@cirno.name> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Filip Skokan <panva.ip@gmail.com> Reviewed-By: Michael Dawson <midawson@redhat.com> Reviewed-By: Richard Lau <rlau@redhat.com>
17 lines
578 B
JavaScript
17 lines
578 B
JavaScript
// Flags: --no-experimental-require-module
|
|
// Previously, this tested that require(esm) throws ERR_REQUIRE_ESM, which is no longer applicable
|
|
// since require(esm) is now supported. The test has been repurposed to ensure that the old behavior
|
|
// is preserved when the --no-experimental-require-module flag is used.
|
|
|
|
'use strict';
|
|
require('../common');
|
|
const assert = require('assert');
|
|
|
|
assert.throws(
|
|
() => require('../fixtures/es-modules/test-esm-ok.mjs'),
|
|
{
|
|
message: /dynamic import\(\) which is available in all CommonJS modules/,
|
|
code: 'ERR_REQUIRE_ESM'
|
|
}
|
|
);
|