esm: use correct URL for error decoration

PR-URL: #37854 Reviewed-By: Guy Bedford <[email protected]> Reviewed-By: Jan Krems <[email protected]>
nodejs · Apr 12, 2021 · 8dd0685 · 8dd0685
1 parent c8bbd83
commit 8dd0685
Show file tree

Hide file tree

Showing 4 changed files with 37 additions and 1 deletion.
diff --git a/lib/internal/modules/esm/module_job.js b/lib/internal/modules/esm/module_job.js
@@ -109,7 +109,11 @@ class ModuleJob {
       if (StringPrototypeIncludes(e.message,
                                   ' does not provide an export named')) {
         const splitStack = StringPrototypeSplit(e.stack, '\n');
-        const parentFileUrl = splitStack[0];
+        const parentFileUrl = StringPrototypeReplace(
+          splitStack[0],
+          /:\d+$/,
+          ''
+        );
         const { 1: childSpecifier, 2: name } = StringPrototypeMatch(
           e.message,
           /module '(.*)' does not provide an export named '(.+)'/);

diff --git a/test/fixtures/policy/bad-main.mjs b/test/fixtures/policy/bad-main.mjs
@@ -0,0 +1 @@
+import {doesNotExist} from './dep.js';
diff --git a/test/fixtures/policy/dependencies/dependencies-missing-export-policy.json b/test/fixtures/policy/dependencies/dependencies-missing-export-policy.json
@@ -0,0 +1,11 @@
+{
+  "resources": {
+    "../bad-main.mjs": {
+      "integrity": true,
+      "dependencies": true
+    },
+    "../dep.js": {
+      "integrity": true
+    }
+  }
+}
diff --git a/test/parallel/test-policy-dependencies.js b/test/parallel/test-policy-dependencies.js
@@ -89,3 +89,23 @@ const dep = fixtures.path('policy', 'parent.js');
   );
   assert.strictEqual(status, 1);
 }
+{
+  // Regression test for https://github.com/nodejs/node/issues/37812
+  const depPolicy = fixtures.path(
+    'policy',
+    'dependencies',
+    'dependencies-missing-export-policy.json');
+  const { status, stderr } = spawnSync(
+    process.execPath,
+    [
+      '--experimental-policy',
+      depPolicy,
+      fixtures.path('policy', 'bad-main.mjs'),
+    ]
+  );
+  assert.strictEqual(status, 1);
+  assert.match(
+    `${stderr}`,
+    /SyntaxError: Named export 'doesNotExist' not found\./,
+    'Should give the real SyntaxError and position');
+}