module: fix e.stack error when throwing undefined or null

Adds a test for module loading when throwing undefined or null.

PR-URL: nodejs#19282
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Jan Krems <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Tiancheng "Timothy" Gu <[email protected]>

Author: zhanzhenzhen

lib/internal/modules/esm/module_job.js

@@ -105,7 +105,6 @@ class ModuleJob {
     try {
       module.evaluate();
     } catch (e) {
-      e.stack;
       this.hadError = true;
       this.error = e;
       throw e;

test/es-module/test-esm-throw-undefined.mjs

@@ -0,0 +1,16 @@
+// Flags: --experimental-modules
+/* eslint-disable node-core/required-modules */
+import common from '../common/index.js';
+import assert from 'assert';
+
+async function doTest() {
+  await assert.rejects(
+    async () => {
+      await import('../fixtures/es-module-loaders/throw-undefined');
+    },
+    (e) => e === undefined
+  );
+}
+
+common.crashOnUnhandledRejection();
+doTest();

test/fixtures/es-module-loaders/throw-undefined.mjs

@@ -0,0 +1,3 @@
+'use strict';
+
+throw undefined;