Verify that the named export is not resolved before throwing a ERR_EX…

…PORT_MISSING. [closes #544]
standard-things · Aug 6, 2018 · 0ce89d1 · 0ce89d1
1 parent 9a3f1e5
commit 0ce89d1
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/src/module/esm/validate.js b/src/module/esm/validate.js
@@ -165,14 +165,15 @@ function validateExportedName(entry, exportedName, seen) {
 
   if (seen &&
       Reflect.has(seen, name)) {
-    if (exportedStars.indexOf(exportedSpecifiers[exportedName].specifier) === -1) {
+    const { specifier } =  exportedSpecifiers[exportedName]
+    const childEntry = dependencySpecifiers[specifier].entry
+
+    if (exportedStars.indexOf(specifier) === -1) {
       throw new ERR_EXPORT_CYCLE(mod, exportedName)
-    } else {
+    } else if (childEntry.compileData.exportedSpecifiers[exportedName] !== true) {
       throw new ERR_EXPORT_MISSING(mod, exportedName)
     }
-  }
-
-  if (Reflect.has(exportedSpecifiers, exportedName)) {
+  } else if (Reflect.has(exportedSpecifiers, exportedName)) {
     const exportedSpecifier = exportedSpecifiers[exportedName]
 
     if (exportedSpecifier) {