Merge pull request #8083 from webpack/bugfix/amd-hash

fixes #8081

lib/AmdMainTemplatePlugin.js

@@ -79,7 +79,9 @@ class AmdMainTemplatePlugin {
 
 		mainTemplate.hooks.hash.tap("AmdMainTemplatePlugin", hash => {
 			hash.update("exports amd");
-			hash.update(this.name);
+			if (this.name) {
+				hash.update(this.name);
+			}
 		});
 	}
 }

test/configCases/target/amd-named/index.js

@@ -0,0 +1,10 @@
+it("should run", function() {
+
+});
+
+it("should name define", function() {
+	var fs = require("fs");
+	var source = fs.readFileSync(__filename, "utf-8");
+
+	expect(source).toMatch("define(\"NamedLibrary\",");
+});

test/configCases/target/amd-named/webpack.config.js

@@ -0,0 +1,17 @@
+const webpack = require("../../../../");
+module.exports = {
+	output: {
+		library: "NamedLibrary",
+		libraryTarget: "amd"
+	},
+	node: {
+		__dirname: false,
+		__filename: false
+	},
+	plugins: [
+		new webpack.BannerPlugin({
+			raw: true,
+			banner: "function define(name, deps, fn) { fn(); }\n"
+		})
+	]
+};

test/configCases/target/amd-unnamed/index.js

@@ -0,0 +1,10 @@
+it("should run", function() {
+
+});
+
+it("should name define", function() {
+	var fs = require("fs");
+	var source = fs.readFileSync(__filename, "utf-8");
+
+	expect(source).toMatch("define(function(");
+});

test/configCases/target/amd-unnamed/webpack.config.js

@@ -0,0 +1,16 @@
+const webpack = require("../../../../");
+module.exports = {
+	output: {
+		libraryTarget: "amd"
+	},
+	node: {
+		__dirname: false,
+		__filename: false
+	},
+	plugins: [
+		new webpack.BannerPlugin({
+			raw: true,
+			banner: "function define(deps, fn) { fn(); }\n"
+		})
+	]
+};