Add tests for namedChunkGroups

slavab89 · slavab89 · commit 71a12c8bc1fc · 2018-04-24T19:01:02.000+03:00
diff --git a/lib/Stats.js b/lib/Stats.js
@@ -135,7 +135,10 @@ class Stats {
 		const showBuiltAt = optionOrLocalFallback(options.builtAt, true);
 		const showAssets = optionOrLocalFallback(options.assets, true);
 		const showEntrypoints = optionOrLocalFallback(options.entrypoints, true);
-		const showChunkGroups = optionOrLocalFallback(options.chunkGroups, true);
+		const showChunkGroups = optionOrLocalFallback(
+			options.chunkGroups,
+			!forToString
+		);
 		const showChunks = optionOrLocalFallback(options.chunks, !forToString);
 		const showChunkModules = optionOrLocalFallback(options.chunkModules, true);
 		const showChunkOrigins = optionOrLocalFallback(
@@ -877,22 +880,23 @@ class Stats {
 			colors.normal(obj.filteredAssets !== 1 ? " assets" : " asset");
 			newline();
 		}
-		if (obj.entrypoints) {
-			for (const name of Object.keys(obj.entrypoints)) {
-				const ep = obj.entrypoints[name];
-				colors.normal("Entrypoint ");
+
+		const processChunkGroups = (namedGroups, prefix) => {
+			for (const name of Object.keys(namedGroups)) {
+				const cg = namedGroups[name];
+				colors.normal(`${prefix} `);
 				colors.bold(name);
-				if (ep.isOverSizeLimit) {
+				if (cg.isOverSizeLimit) {
 					colors.normal(" ");
 					colors.yellow("[big]");
 				}
 				colors.normal(" =");
-				for (const asset of ep.assets) {
+				for (const asset of cg.assets) {
 					colors.normal(" ");
 					colors.green(asset);
 				}
-				for (const name of Object.keys(ep.childAssets)) {
-					const assets = ep.childAssets[name];
+				for (const name of Object.keys(cg.childAssets)) {
+					const assets = cg.childAssets[name];
 					if (assets && assets.length > 0) {
 						colors.normal(" ");
 						colors.magenta(`(${name}:`);
@@ -905,7 +909,25 @@ class Stats {
 				}
 				newline();
 			}
+		};
+
+		if (obj.entrypoints) {
+			processChunkGroups(obj.entrypoints, "Entrypoint");
 		}
+
+		if (obj.namedChunkGroups) {
+			let outputChunkGroups = obj.namedChunkGroups;
+			if (obj.entrypoints) {
+				outputChunkGroups = Object.keys(outputChunkGroups)
+					.filter(name => !obj.entrypoints[name])
+					.reduce((result, name) => {
+						result[name] = obj.namedChunkGroups[name];
+						return result;
+					}, {});
+			}
+			processChunkGroups(outputChunkGroups, "Chunk Group");
+		}
+
 		const modulesByIdentifier = {};
 		if (obj.modules) {
 			for (const module of obj.modules) {
@@ -1271,6 +1293,7 @@ class Stats {
 			case "verbose":
 				return {
 					entrypoints: true,
+					chunkGroups: true,
 					modules: false,
 					chunks: true,
 					chunkModules: true,
@@ -1289,6 +1312,7 @@ class Stats {
 			case "detailed":
 				return {
 					entrypoints: true,
+					chunkGroups: true,
 					chunks: true,
 					chunkModules: false,
 					chunkOrigins: true,
diff --git a/schemas/WebpackOptions.json b/schemas/WebpackOptions.json
@@ -1853,6 +1853,10 @@
               "type": "boolean",
               "description": "Display the entry points with the corresponding bundles"
             },
+            "chunkGroups": {
+              "type": "boolean",
+              "description": "Display all chunk groups with the corresponding bundles"
+            },
             "errorDetails": {
               "type": "boolean",
               "description": "add details to errors (like resolving log)"
diff --git a/test/statsCases/named-chunk-groups/a.js b/test/statsCases/named-chunk-groups/a.js
@@ -0,0 +1,3 @@
+import "./shared";
+
+export default "a";
diff --git a/test/statsCases/named-chunk-groups/b.js b/test/statsCases/named-chunk-groups/b.js
@@ -0,0 +1,3 @@
+import "./shared";
+
+export default "b";
diff --git a/test/statsCases/named-chunk-groups/c.js b/test/statsCases/named-chunk-groups/c.js
@@ -0,0 +1,4 @@
+import "x";
+import "y";
+
+export default "c";
diff --git a/test/statsCases/named-chunk-groups/expected.txt b/test/statsCases/named-chunk-groups/expected.txt
@@ -0,0 +1,24 @@
+Chunk Group main = main.js
+Chunk Group async-a = async-a~async-b.js async-a.js
+Chunk Group async-b = async-a~async-b.js async-b.js
+Chunk Group async-c = vendors.js async-c.js
+chunk    {0} async-a~async-b.js (async-a~async-b) 133 bytes <{5}> ={1}= ={2}= [rendered] split chunk (cache group: default) (name: async-a~async-b)
+    > ./a [6] ./index.js 1:0-47
+    > ./b [6] ./index.js 2:0-47
+ [0] ./shared.js 133 bytes {0} [built]
+chunk    {1} async-a.js (async-a) 40 bytes <{5}> ={0}= [rendered]
+    > ./a [6] ./index.js 1:0-47
+ [3] ./a.js 40 bytes {1} [built]
+chunk    {2} async-b.js (async-b) 40 bytes <{5}> ={0}= [rendered]
+    > ./b [6] ./index.js 2:0-47
+ [4] ./b.js 40 bytes {2} [built]
+chunk    {3} async-c.js (async-c) 45 bytes <{5}> ={4}= [rendered]
+    > ./c [6] ./index.js 3:0-47
+ [5] ./c.js 45 bytes {3} [built]
+chunk    {4} vendors.js (vendors) 40 bytes <{5}> ={3}= [rendered] split chunk (cache group: vendors) (name: vendors)
+    > ./c [6] ./index.js 3:0-47
+ [1] ./node_modules/x.js 20 bytes {4} [built]
+ [2] ./node_modules/y.js 20 bytes {4} [built]
+chunk    {5} main.js (main) 146 bytes >{0}< >{1}< >{2}< >{3}< >{4}< [entry] [rendered]
+    > ./ main
+ [6] ./index.js 146 bytes {5} [built]
diff --git a/test/statsCases/named-chunk-groups/index.js b/test/statsCases/named-chunk-groups/index.js
@@ -0,0 +1,3 @@
+import(/* webpackChunkName: "async-a" */ "./a");
+import(/* webpackChunkName: "async-b" */ "./b");
+import(/* webpackChunkName: "async-c" */ "./c");
diff --git a/test/statsCases/named-chunk-groups/node_modules/x.js b/test/statsCases/named-chunk-groups/node_modules/x.js
diff --git a/test/statsCases/named-chunk-groups/node_modules/y.js b/test/statsCases/named-chunk-groups/node_modules/y.js
diff --git a/test/statsCases/named-chunk-groups/shared.js b/test/statsCases/named-chunk-groups/shared.js
@@ -0,0 +1,2 @@
+// content content content content content content content content
+// content content content content content content content content
diff --git a/test/statsCases/named-chunk-groups/webpack.config.js b/test/statsCases/named-chunk-groups/webpack.config.js
@@ -0,0 +1,34 @@
+const stats = {
+	hash: false,
+	timings: false,
+	builtAt: false,
+	assets: false,
+	chunks: true,
+	chunkOrigins: true,
+	entrypoints: false,
+	chunkGroups: true,
+	modules: false
+};
+module.exports = {
+	mode: "production",
+	entry: {
+		main: "./"
+	},
+	output: {
+		filename: "[name].js"
+	},
+	optimization: {
+		splitChunks: {
+			minSize: 100,
+			cacheGroups: {
+				vendors: {
+					test: /[\\/]node_modules[\\/]/,
+					chunks: "async",
+					enforce: true,
+					name: "vendors"
+				}
+			}
+		}
+	},
+	stats
+};

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+import "./shared";`
	`2`	`+`
	`3`	`+export default "a";`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+import "./shared";`
	`2`	`+`
	`3`	`+export default "b";`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+import(/* webpackChunkName: "async-a" */ "./a");`
	`2`	`+import(/* webpackChunkName: "async-b" */ "./b");`
	`3`	`+import(/* webpackChunkName: "async-c" */ "./c");`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+// content content content content content content content content`
	`2`	`+// content content content content content content content content`