loader: remove unused error code in module_job

PR-URL: #21354
Reviewed-By: Bradley Farias <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Guy Bedford <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>

Author: devsnek

lib/internal/modules/esm/module_job.js

@@ -14,8 +14,6 @@ class ModuleJob {
   // `moduleProvider` is a function
   constructor(loader, url, moduleProvider, isMain) {
     this.loader = loader;
-    this.error = null;
-    this.hadError = false;
     this.isMain = isMain;
 
     // This is a Promise<{ module, reflect }>, whose fields will be copied
@@ -72,15 +70,7 @@ class ModuleJob {
       const dependencyJobs = await moduleJob.linked;
       return Promise.all(dependencyJobs.map(addJobsToDependencyGraph));
     };
-    try {
-      await addJobsToDependencyGraph(this);
-    } catch (e) {
-      if (!this.hadError) {
-        this.error = e;
-        this.hadError = true;
-      }
-      throw e;
-    }
+    await addJobsToDependencyGraph(this);
     try {
       if (this.isMain && process._breakFirstLine) {
         delete process._breakFirstLine;
@@ -103,13 +93,7 @@ class ModuleJob {
 
   async run() {
     const module = await this.instantiate();
-    try {
-      module.evaluate(-1, false);
-    } catch (e) {
-      this.hadError = true;
-      this.error = e;
-      throw e;
-    }
+    module.evaluate(-1, false);
     return module;
   }
 }

test/es-module/test-esm-error-cache.js

@@ -0,0 +1,28 @@
+'use strict';
+
+// Flags: --experimental-modules
+
+const common = require('../common');
+const assert = require('assert');
+
+common.crashOnUnhandledRejection();
+
+const file = '../../fixtures/syntax/bad_syntax.js';
+
+let error;
+(async () => {
+  try {
+    await import(file);
+  } catch (e) {
+    assert.strictEqual(e.name, 'SyntaxError');
+    error = e;
+  }
+
+  assert(error);
+
+  try {
+    await import(file);
+  } catch (e) {
+    assert.strictEqual(error, e);
+  }
+})();