lib: fix checking syntax of esm module

Fixes: nodejs#41189 PR-URL: nodejs#41198 Refs: nodejs#37468 Reviewed-By: Derek Lewis <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
jacobtarango23 · Dec 19, 2021 · 3f7dabd · 3f7dabd
1 parent 0fcae7b
commit 3f7dabd
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 1 deletion.
diff --git a/lib/internal/main/check_syntax.js b/lib/internal/main/check_syntax.js
@@ -54,7 +54,7 @@ function checkSyntax(source, filename) {
     const { defaultResolve } = require('internal/modules/esm/resolve');
     const { defaultGetFormat } = require('internal/modules/esm/get_format');
     const { url } = defaultResolve(pathToFileURL(filename).toString());
-    const { format } = defaultGetFormat(url);
+    const format = defaultGetFormat(url);
     isModule = format === 'module';
   }
   if (isModule) {

diff --git a/test/fixtures/syntax/good_syntax.mjs b/test/fixtures/syntax/good_syntax.mjs
@@ -0,0 +1,3 @@
+export function testFunction(req, res) {
+  return 'PASS';
+}
diff --git a/test/sequential/test-cli-syntax-good.js b/test/sequential/test-cli-syntax-good.js
@@ -17,6 +17,7 @@ const syntaxArgs = [
 [
   'syntax/good_syntax.js',
   'syntax/good_syntax',
+  'syntax/good_syntax.mjs',
   'syntax/good_syntax_shebang.js',
   'syntax/good_syntax_shebang',
   'syntax/illegal_if_not_wrapped.js',