fix(mjs): remove unused namespace build errors

webpack · Jun 12, 2019 · df5e118 · df5e118
1 parent acf2c2d
commit df5e118
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 15 deletions.
diff --git a/lib/dependencies/HarmonyExportImportedSpecifierDependency.js b/lib/dependencies/HarmonyExportImportedSpecifierDependency.js
@@ -355,12 +355,11 @@ class HarmonyExportImportedSpecifierDependency extends HarmonyImportDependency {
 			// It's not an harmony module
 			if (
 				this.originModule.buildMeta.strictHarmonyModule &&
+				this._id &&
 				this._id !== "default"
 			) {
 				// In strict harmony modules we only support the default export
-				const exportName = this._id
-					? `the named export '${this._id}'`
-					: "the namespace object";
+				const exportName = `the named export '${this._id}'`;
 				return [
 					new HarmonyLinkingError(
 						`Can't reexport ${exportName} from non EcmaScript module (only default export is available)`

diff --git a/lib/dependencies/HarmonyImportSpecifierDependency.js b/lib/dependencies/HarmonyImportSpecifierDependency.js
@@ -80,12 +80,11 @@ class HarmonyImportSpecifierDependency extends HarmonyImportDependency {
 			// It's not an harmony module
 			if (
 				this.originModule.buildMeta.strictHarmonyModule &&
+				this._id &&
 				this._id !== "default"
 			) {
 				// In strict harmony modules we only support the default export
-				const exportName = this._id
-					? `the named export '${this._id}'`
-					: "the namespace object";
+				const exportName = `the named export '${this._id}'`;
 				return [
 					new HarmonyLinkingError(
 						`Can't import ${exportName} from non EcmaScript module (only default export is available)`

diff --git a/test/cases/mjs/cjs-import-default/errors.js b/test/cases/mjs/cjs-import-default/errors.js
@@ -1,19 +1,10 @@
 module.exports = [
-	[
-		/Can't import the namespace object from non EcmaScript module \(only default export is available\)/
-	],
-	[
-		/Can't import the namespace object from non EcmaScript module \(only default export is available\)/
-	],
 	[
 		/Can't import the named export 'data' from non EcmaScript module \(only default export is available\)/
 	],
 	[
 		/Can't import the named export 'data' from non EcmaScript module \(only default export is available\)/
 	],
-	[
-		/Can't reexport the namespace object from non EcmaScript module \(only default export is available\)/
-	],
 	[
 		/Can't reexport the named export 'data' from non EcmaScript module \(only default export is available\)/
 	]