esm: use correct error arguments

ERR_UNKNOWN_FILE_EXTENSION expects a single argument. This
commit fixes the argument count.

PR-URL: #27763
Fixes: #27761
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Richard Lau <[email protected]>

Author: cjihrig

lib/internal/modules/esm/default_resolve.js

@@ -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 };
 }

test/es-module/test-esm-invalid-extension.js

@@ -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));

test/fixtures/es-modules/import-invalid-ext.mjs

@@ -0,0 +1 @@
+import './simple.wat';