Merge pull request #11499 from webpack/bugfix/snapshot-iterable

fix incorrect iteration

lib/FileSystemInfo.js

@@ -316,8 +316,8 @@ class Snapshot {
 									const result = it.next();
 									if (!result.done) return result;
 									state = 0;
+									break;
 								}
-								/* falls through */
 								case 2: {
 									const children = snapshot.children;
 									if (children !== undefined) {

test/configCases/cache-dependencies/managed-items/index.js

@@ -0,0 +1,3 @@
+import "./loader!package";
+
+it("should compile and run the test in config", () => {});

test/configCases/cache-dependencies/managed-items/loader.js

@@ -0,0 +1,6 @@
+const path = require("path");
+module.exports = function (source) {
+	this.addDependency(path.resolve(__dirname, "node_modules/package/extra.js"));
+	this.addDependency(path.resolve(__dirname, "extra.js"));
+	return source;
+};

test/configCases/cache-dependencies/managed-items/node_modules/package/package.json

@@ -0,0 +1,4 @@
+{
+  "name": "package",
+  "version": "1.0.0"
+}

test/configCases/cache-dependencies/managed-items/webpack.config.js

@@ -0,0 +1,27 @@
+const path = require("path");
+
+/** @type {import("../../../../").Configuration} */
+module.exports = {
+	snapshot: {
+		managedPaths: [path.resolve(__dirname, "node_modules")]
+	},
+	plugins: [
+		compiler => {
+			compiler.hooks.done.tap("Test", ({ compilation }) => {
+				const fileDeps = Array.from(compilation.fileDependencies);
+				expect(fileDeps).toContain(
+					path.resolve(__dirname, "node_modules/package/index.js")
+				);
+				expect(fileDeps).toContain(
+					path.resolve(__dirname, "node_modules/package/extra.js")
+				);
+				expect(fileDeps).toContain(
+					path.resolve(__dirname, "node_modules/package/package.json")
+				);
+				expect(fileDeps).toContain(path.resolve(__dirname, "extra.js"));
+				expect(fileDeps).toContain(path.resolve(__dirname, "loader.js"));
+				expect(fileDeps).toContain(path.resolve(__dirname, "index.js"));
+			});
+		}
+	]
+};