Commit 15d48f9
lib: add parent to ERR_UNKNOWN_FILE_EXTENSION
- default_resolve updated to pass parentURL into error
- ERR_UNKNOWN_FILE_EXTENSION updated to include parentURL
- test added to check for import message in error
PR-URL: #30728
Fixes: #30721
Reviewed-By: Guy Bedford <guybedford@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>1 parent a1ccf07 commit 15d48f9
3 files changed
Lines changed: 9 additions & 2 deletions
File tree
- lib/internal
- modules/esm
- test/es-module
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1213,7 +1213,9 @@ E('ERR_UNHANDLED_ERROR', | |||
| 1213 | 1213 | E('ERR_UNKNOWN_BUILTIN_MODULE', 'No such built-in module: %s', Error); | |
| 1214 | 1214 | E('ERR_UNKNOWN_CREDENTIAL', '%s identifier does not exist: %s', Error); | |
| 1215 | 1215 | E('ERR_UNKNOWN_ENCODING', 'Unknown encoding: %s', TypeError); | |
| 1216 | - E('ERR_UNKNOWN_FILE_EXTENSION', 'Unknown file extension: %s', TypeError); | ||
| 1216 | + E('ERR_UNKNOWN_FILE_EXTENSION', | ||
| 1217 | + 'Unknown file extension "%s" for %s imported from %s', | ||
| 1218 | + TypeError); | ||
| 1217 | 1219 | E('ERR_UNKNOWN_MODULE_FORMAT', 'Unknown module format: %s', RangeError); | |
| 1218 | 1220 | E('ERR_UNKNOWN_SIGNAL', 'Unknown signal: %s', TypeError); | |
| 1219 | 1221 | ||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -114,7 +114,10 @@ function resolve(specifier, parentURL) { | |||
| 114 | 114 | 'ExperimentalWarning'); | |
| 115 | 115 | format = legacyExtensionFormatMap[ext]; | |
| 116 | 116 | } else { | |
| 117 | - throw new ERR_UNKNOWN_FILE_EXTENSION(fileURLToPath(url)); | ||
| 117 | + throw new ERR_UNKNOWN_FILE_EXTENSION( | ||
| 118 | + ext, | ||
| 119 | + fileURLToPath(url), | ||
| 120 | + fileURLToPath(parentURL)); | ||
| 118 | 121 | } | |
| 119 | 122 | } | |
| 120 | 123 | return { url: `${url}`, format }; | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -6,8 +6,10 @@ const { spawnSync } = require('child_process'); | |||
| 6 | 6 | const fixture = fixtures.path('/es-modules/import-invalid-ext.mjs'); | |
| 7 | 7 | const child = spawnSync(process.execPath, ['--experimental-modules', fixture]); | |
| 8 | 8 | const errMsg = 'TypeError [ERR_UNKNOWN_FILE_EXTENSION]: Unknown file extension'; | |
| 9 | + const importMsg = `imported from ${fixture}`; | ||
| 9 | 10 | ||
| 10 | 11 | assert.strictEqual(child.status, 1); | |
| 11 | 12 | assert.strictEqual(child.signal, null); | |
| 12 | 13 | assert.strictEqual(child.stdout.toString().trim(), ''); | |
| 13 | 14 | assert(child.stderr.toString().includes(errMsg)); | |
| 15 | + assert(child.stderr.toString().includes(importMsg)); | ||
0 commit comments