feat: show minimized assets in stats for webpack@5 (#289)

webpack-contrib · Aug 3, 2020 · d59eae2 · d59eae2
1 parent 37cc813
commit d59eae2
Show file tree

Hide file tree

Showing 5 changed files with 46 additions and 4 deletions.
diff --git a/src/index.js b/src/index.js
@@ -709,6 +709,15 @@ class TerserPlugin {
           plugin,
           optimizeFn.bind(this, compilation)
         );
+
+        compilation.hooks.statsPrinter.tap(plugin, (stats) => {
+          stats.hooks.print
+            .for('asset.info.minimized')
+            .tap('terser-webpack-plugin', (minimized, { green, formatFlag }) =>
+              // eslint-disable-next-line no-undefined
+              minimized ? green(formatFlag('minimized')) : undefined
+            );
+        });
       }
     });
   }

diff --git a/test/TerserPlugin.test.js b/test/TerserPlugin.test.js
@@ -852,4 +852,19 @@ describe('TerserPlugin', () => {
     expect(getErrors(stats)).toMatchSnapshot('errors');
     expect(getWarnings(stats)).toMatchSnapshot('warnings');
   });
+
+  it('should work and show minimized assets in stats', async () => {
+    const compiler = getCompiler();
+
+    new TerserPlugin().apply(compiler);
+
+    const stats = await compile(compiler);
+
+    expect(stats.toString().indexOf('[minimized]') !== -1).toBe(
+      !getCompiler.isWebpack4()
+    );
+    expect(readsAssets(compiler, stats)).toMatchSnapshot('assets');
+    expect(getErrors(stats)).toMatchSnapshot('errors');
+    expect(getWarnings(stats)).toMatchSnapshot('warnings');
+  });
 });
diff --git a/test/__snapshots__/TerserPlugin.test.js.snap.webpack4 b/test/__snapshots__/TerserPlugin.test.js.snap.webpack4
@@ -128,6 +128,16 @@ Unexpected token name «Object», expected punc «,» [main.js:1,8]",
 
 exports[`TerserPlugin should work and respect "terser" errors (the "parallel" option is "true"): warnings 1`] = `Array []`;
 
+exports[`TerserPlugin should work and show minimized assets in stats: assets 1`] = `
+Object {
+  "main.js": "!function(e){var t={};function n(r){if(t[r])return t[r].exports;var o=t[r]={i:r,l:!1,exports:{}};return e[r].call(o.exports,o,o.exports,n),o.l=!0,o.exports}n.m=e,n.c=t,n.d=function(e,t,r){n.o(e,t)||Object.defineProperty(e,t,{enumerable:!0,get:r})},n.r=function(e){\\"undefined\\"!=typeof Symbol&&Symbol.toStringTag&&Object.defineProperty(e,Symbol.toStringTag,{value:\\"Module\\"}),Object.defineProperty(e,\\"__esModule\\",{value:!0})},n.t=function(e,t){if(1&t&&(e=n(e)),8&t)return e;if(4&t&&\\"object\\"==typeof e&&e&&e.__esModule)return e;var r=Object.create(null);if(n.r(r),Object.defineProperty(r,\\"default\\",{enumerable:!0,value:e}),2&t&&\\"string\\"!=typeof e)for(var o in e)n.d(r,o,function(t){return e[t]}.bind(null,o));return r},n.n=function(e){var t=e&&e.__esModule?function(){return e.default}:function(){return e};return n.d(t,\\"a\\",t),t},n.o=function(e,t){return Object.prototype.hasOwnProperty.call(e,t)},n.p=\\"\\",n(n.s=0)}([function(e,t){e.exports=function(){console.log(7)}}]);",
+}
+`;
+
+exports[`TerserPlugin should work and show minimized assets in stats: errors 1`] = `Array []`;
+
+exports[`TerserPlugin should work and show minimized assets in stats: warnings 1`] = `Array []`;
+
 exports[`TerserPlugin should work as a minimizer: assets 1`] = `
 Object {
   "main.js": "!function(e){var t={};function n(r){if(t[r])return t[r].exports;var o=t[r]={i:r,l:!1,exports:{}};return e[r].call(o.exports,o,o.exports,n),o.l=!0,o.exports}n.m=e,n.c=t,n.d=function(e,t,r){n.o(e,t)||Object.defineProperty(e,t,{enumerable:!0,get:r})},n.r=function(e){\\"undefined\\"!=typeof Symbol&&Symbol.toStringTag&&Object.defineProperty(e,Symbol.toStringTag,{value:\\"Module\\"}),Object.defineProperty(e,\\"__esModule\\",{value:!0})},n.t=function(e,t){if(1&t&&(e=n(e)),8&t)return e;if(4&t&&\\"object\\"==typeof e&&e&&e.__esModule)return e;var r=Object.create(null);if(n.r(r),Object.defineProperty(r,\\"default\\",{enumerable:!0,value:e}),2&t&&\\"string\\"!=typeof e)for(var o in e)n.d(r,o,function(t){return e[t]}.bind(null,o));return r},n.n=function(e){var t=e&&e.__esModule?function(){return e.default}:function(){return e};return n.d(t,\\"a\\",t),t},n.o=function(e,t){return Object.prototype.hasOwnProperty.call(e,t)},n.p=\\"\\",n(n.s=0)}([function(e,t){e.exports=function(){console.log(7)}}]);",

diff --git a/test/__snapshots__/TerserPlugin.test.js.snap.webpack5 b/test/__snapshots__/TerserPlugin.test.js.snap.webpack5
@@ -128,6 +128,16 @@ Unexpected token name «Object», expected punc «,» [main.js:1,8]",
 
 exports[`TerserPlugin should work and respect "terser" errors (the "parallel" option is "true"): warnings 1`] = `Array []`;
 
+exports[`TerserPlugin should work and show minimized assets in stats: assets 1`] = `
+Object {
+  "main.js": "(()=>{var r={791:r=>{r.exports=function(){console.log(7)}}},o={};!function t(e){if(o[e])return o[e].exports;var n=o[e]={exports:{}};return r[e](n,n.exports,t),n.exports}(791)})();",
+}
+`;
+
+exports[`TerserPlugin should work and show minimized assets in stats: errors 1`] = `Array []`;
+
+exports[`TerserPlugin should work and show minimized assets in stats: warnings 1`] = `Array []`;
+
 exports[`TerserPlugin should work as a minimizer: assets 1`] = `
 Object {
   "main.js": "(()=>{var r={791:r=>{r.exports=function(){console.log(7)}}},o={};!function t(e){if(o[e])return o[e].exports;var n=o[e]={exports:{}};return r[e](n,n.exports,t),n.exports}(791)})();",

diff --git a/test/helpers/BrokenCodePlugin.js b/test/helpers/BrokenCodePlugin.js
@@ -4,10 +4,8 @@ export default class BrokenCodePlugin {
   apply(compiler) {
     const plugin = { name: this.constructor.name };
 
-    compiler.hooks.compilation.tap(plugin, (compilation) => {
-      compilation.hooks.additionalChunkAssets.tap(plugin, () => {
-        compilation.additionalChunkAssets.push('broken.js');
-
+    compiler.hooks.thisCompilation.tap(plugin, (compilation) => {
+      compilation.hooks.additionalAssets.tap(plugin, () => {
         // eslint-disable-next-line no-param-reassign
         compilation.assets['broken.js'] = new RawSource('`Broken===');
       });