more tests and fixes for webworker

sokra · sokra · commit b018bc7ac5e2 · 2018-03-26T14:34:54.000+02:00
diff --git a/lib/webworker/WebWorkerMainTemplatePlugin.js b/lib/webworker/WebWorkerMainTemplatePlugin.js
@@ -51,27 +51,27 @@ class WebWorkerMainTemplatePlugin {
 											length
 										)} + "`,
 									chunk: {
-										id: '" + chunkId + "'
-									},
-									contentHash: {
-										javascript: `" + ${JSON.stringify(
-											chunkMaps.contentHash.javascript
-										)}[chunkId] + "`
-									},
-									contentHashWithLength: {
-										javascript: length => {
-											const shortContentHashMap = {};
-											const contentHash = chunkMaps.contentHash.javascript;
-											for (const chunkId of Object.keys(contentHash)) {
-												if (typeof contentHash[chunkId] === "string") {
-													shortContentHashMap[chunkId] = contentHash[
-														chunkId
-													].substr(0, length);
+										id: '" + chunkId + "',
+										contentHash: {
+											javascript: `" + ${JSON.stringify(
+												chunkMaps.contentHash.javascript
+											)}[chunkId] + "`
+										},
+										contentHashWithLength: {
+											javascript: length => {
+												const shortContentHashMap = {};
+												const contentHash = chunkMaps.contentHash.javascript;
+												for (const chunkId of Object.keys(contentHash)) {
+													if (typeof contentHash[chunkId] === "string") {
+														shortContentHashMap[chunkId] = contentHash[
+															chunkId
+														].substr(0, length);
+													}
 												}
+												return `" + ${JSON.stringify(
+													shortContentHashMap
+												)}[chunkId] + "`;
 											}
-											return `" + ${JSON.stringify(
-												shortContentHashMap
-											)}[chunkId] + "`;
 										}
 									},
 									contentHashType: "javascript"
diff --git a/test/configCases/hash-length/output-filename/test.config.js b/test/configCases/hash-length/output-filename/test.config.js
@@ -47,5 +47,6 @@ module.exports = {
 	},
 	afterExecute: () => {
 		delete global.webpackJsonp;
+		delete global.webpackChunk;
 	}
 };
diff --git a/test/configCases/hash-length/output-filename/webpack.config.js b/test/configCases/hash-length/output-filename/webpack.config.js
@@ -191,6 +191,34 @@ module.exports = [
 			expectedFilenameLength: 9 + 7 + 3,
 			expectedChunkFilenameLength: 2 + 9 + 7 + 3
 		}
+	},
+	{
+		name: "contenthash in webpack",
+		entry: "./no-async",
+		output: {
+			filename: "bundle16.[contenthash].js",
+			chunkFilename: "[id].bundle16.[contenthash].js",
+			globalObject: "this"
+		},
+		target: "webworker",
+		amd: {
+			expectedFilenameLength: 32,
+			expectedChunkFilenameLength: 34
+		}
+	},
+	{
+		name: "contenthash in async-node with length",
+		entry: "./no-async",
+		output: {
+			filename: "bundle17.[contenthash:7].js",
+			chunkFilename: "[id].bundle17.[contenthash:7].js",
+			globalObject: "this"
+		},
+		target: "webworker",
+		amd: {
+			expectedFilenameLength: 9 + 7 + 3,
+			expectedChunkFilenameLength: 2 + 9 + 7 + 3
+		}
 	}
 ];
 

Original file line number	Diff line number	Diff line change
`@@ -47,5 +47,6 @@ module.exports = {`
`47`	`47`	`},`
`48`	`48`	`afterExecute: () => {`
`49`	`49`	`delete global.webpackJsonp;`
	`50`	`+ delete global.webpackChunk;`
`50`	`51`	`}`
`51`	`52`	`};`