Merge pull request webpack#7062 from webpack/bugfix/issue-6931

sokra · web-flow · commit 3a5fda909f2e · 2018-04-17T11:41:50.000+02:00
fix bug which prevented some chunks to show up in Chunk.getAllAsyncChunks
diff --git a/lib/Chunk.js b/lib/Chunk.js
@@ -6,6 +6,7 @@
 
 const util = require("util");
 const SortableSet = require("./util/SortableSet");
+const intersect = require("./util/SetHelpers").intersect;
 const GraphHelpers = require("./GraphHelpers");
 let debugId = 1000;
 const ERR_CHUNK_ENTRY = "Chunk.entry was removed. Use hasRuntime()";
@@ -321,12 +322,15 @@ class Chunk {
 	}
 
 	getAllAsyncChunks() {
-		const initialChunks = new Set();
-		const queue = new Set(this.groupsIterable);
+		const queue = new Set();
 		const chunks = new Set();
 
-		for (const chunkGroup of queue) {
-			for (const chunk of chunkGroup.chunks) initialChunks.add(chunk);
+		const initialChunks = intersect(
+			Array.from(this.groupsIterable, g => new Set(g.chunks))
+		);
+
+		for (const chunkGroup of this.groupsIterable) {
+			for (const child of chunkGroup.childrenIterable) queue.add(child);
 		}
 
 		for (const chunkGroup of queue) {
diff --git a/test/configCases/split-chunks/runtime-chunk/a.js b/test/configCases/split-chunks/runtime-chunk/a.js
@@ -0,0 +1,17 @@
+const should = require("should");
+const FakeDocument = require("../../../helpers/FakeDocument");
+
+beforeEach(() => {
+	global.document = new FakeDocument();
+});
+
+afterEach(() => {
+	delete global.document;
+})
+
+it("should be able to load the split chunk on demand", () => {
+	const promise = import(/* webpackChunkName: "shared" */ "./shared");
+
+	const script = document.head._children[0];
+	should(script.src).be.eql("dep~b~shared.js");
+});
diff --git a/test/configCases/split-chunks/runtime-chunk/b.js b/test/configCases/split-chunks/runtime-chunk/b.js
@@ -0,0 +1 @@
+import "./shared";
diff --git a/test/configCases/split-chunks/runtime-chunk/shared.js b/test/configCases/split-chunks/runtime-chunk/shared.js
diff --git a/test/configCases/split-chunks/runtime-chunk/test.config.js b/test/configCases/split-chunks/runtime-chunk/test.config.js
@@ -0,0 +1,5 @@
+module.exports = {
+	findBundle: function(i, options) {
+		return ["runtime.js", "a.js"];
+	}
+};
diff --git a/test/configCases/split-chunks/runtime-chunk/webpack.config.js b/test/configCases/split-chunks/runtime-chunk/webpack.config.js
@@ -0,0 +1,24 @@
+const path = require("path");
+
+module.exports = {
+	entry: {
+		a: "./a",
+		b: "./b"
+	},
+	target: "web",
+	output: {
+		filename: "[name].js"
+	},
+	optimization: {
+		runtimeChunk: "single",
+		splitChunks: {
+			cacheGroups: {
+				dep: {
+					chunks: "all",
+					test: path.resolve(__dirname, "shared.js"),
+					enforce: true
+				}
+			}
+		}
+	}
+};
