warn when name is imported that is not exported

sokra · sokra · commit c9143d49c7b3 · 2016-06-24T01:52:28.000+02:00
diff --git a/lib/dependencies/HarmonyImportSpecifierDependency.js b/lib/dependencies/HarmonyImportSpecifierDependency.js
@@ -3,6 +3,7 @@
 	Author Tobias Koppers @sokra
 */
 var NullDependency = require("./NullDependency");
+var HarmonyModulesHelpers = require("./HarmonyModulesHelpers");
 
 function HarmonyImportSpecifierDependency(importDependency, importedVar, id, name, range) {
 	NullDependency.call(this);
@@ -24,7 +25,20 @@ HarmonyImportSpecifierDependency.prototype.getReference = function() {
 		module: this.importDependency.module,
 		importedNames: this.id ? [this.id] : true
 	};
-}
+};
+
+HarmonyImportSpecifierDependency.prototype.getWarnings = function() {
+	var importedModule = this.importDependency.module;
+	if(importedModule && importedModule.meta && importedModule.meta.harmonyModule) {
+		if(this.id && !HarmonyModulesHelpers.isExportedByHarmony(importedModule, this.id)) {
+			return [
+				new Error("export '" + this.id + "'" +
+					(this.id !== this.name ? " (imported as '" + this.name + "')" : "") +
+					" was not found in '" + this.importDependency.userRequest + "'")
+			];
+		}
+	}
+};
 
 HarmonyImportSpecifierDependency.prototype.updateHash = function(hash) {
 	NullDependency.prototype.updateHash.call(this, hash);
diff --git a/test/cases/parsing/harmony-info/commonjs.js b/test/cases/parsing/harmony-info/commonjs.js
diff --git a/test/cases/parsing/harmony-info/index.js b/test/cases/parsing/harmony-info/index.js
@@ -0,0 +1,6 @@
+import def, { a as aa, b, c, d, e } from "./module";
+import f, { g } from "./commonjs";
+
+it("should emit the correct warnings", function() {
+	def, aa, b, c, d, e, f, g
+});
diff --git a/test/cases/parsing/harmony-info/module.js b/test/cases/parsing/harmony-info/module.js
@@ -0,0 +1,9 @@
+function b() {}
+
+export {
+	b
+}
+
+export var c = 123;
+
+export function d() {}
diff --git a/test/cases/parsing/harmony-info/warnings.js b/test/cases/parsing/harmony-info/warnings.js
@@ -0,0 +1,5 @@
+module.exports = [
+	[/export 'default' \(imported as 'def'\) was not found in '\.\/module'/],
+	[/export 'e' was not found in '\.\/module'/],
+	[/export 'a' \(imported as 'aa'\) was not found in '\.\/module'/]
+];
