esm: use correct error arguments
ERR_UNKNOWN_FILE_EXTENSION expects a single argument. This commit fixes the argument count. PR-URL: https://github.com/nodejs/node/pull/27763 Fixes: https://github.com/nodejs/node/issues/27761 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com>
This commit is contained in:
parent
1b381d630a
commit
873c372e67
@ -91,8 +91,7 @@ function resolve(specifier, parentURL) {
|
||||
if (isMain)
|
||||
format = type === TYPE_MODULE ? 'module' : 'commonjs';
|
||||
else
|
||||
throw new ERR_UNKNOWN_FILE_EXTENSION(fileURLToPath(url),
|
||||
fileURLToPath(parentURL));
|
||||
throw new ERR_UNKNOWN_FILE_EXTENSION(fileURLToPath(url));
|
||||
}
|
||||
return { url: `${url}`, format };
|
||||
}
|
||||
|
13
test/es-module/test-esm-invalid-extension.js
Normal file
13
test/es-module/test-esm-invalid-extension.js
Normal file
@ -0,0 +1,13 @@
|
||||
'use strict';
|
||||
require('../common');
|
||||
const fixtures = require('../common/fixtures');
|
||||
const assert = require('assert');
|
||||
const { spawnSync } = require('child_process');
|
||||
const fixture = fixtures.path('/es-modules/import-invalid-ext.mjs');
|
||||
const child = spawnSync(process.execPath, ['--experimental-modules', fixture]);
|
||||
const errMsg = 'TypeError [ERR_UNKNOWN_FILE_EXTENSION]: Unknown file extension';
|
||||
|
||||
assert.strictEqual(child.status, 1);
|
||||
assert.strictEqual(child.signal, null);
|
||||
assert.strictEqual(child.stdout.toString().trim(), '');
|
||||
assert(child.stderr.toString().includes(errMsg));
|
1
test/fixtures/es-modules/import-invalid-ext.mjs
vendored
Normal file
1
test/fixtures/es-modules/import-invalid-ext.mjs
vendored
Normal file
@ -0,0 +1 @@
|
||||
import './simple.wat';
|
Loading…
x
Reference in New Issue
Block a user