Limit the webpack stats to what we need (#2132)

* Limit the webpack stats to what we need * Change an assertion
GoogleChrome · Jul 19, 2019 · b0825d7 · b0825d7
1 parent c626898
commit b0825d7
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/packages/workbox-webpack-plugin/src/lib/get-manifest-entries-from-compilation.js b/packages/workbox-webpack-plugin/src/lib/get-manifest-entries-from-compilation.js
@@ -86,7 +86,12 @@ function assetToChunkNameMapping(stats) {
  */
 function filterAssets(compilation, config) {
   const filteredAssets = new Set();
-  const stats = compilation.getStats().toJson();
+  // See https://webpack.js.org/configuration/stats/#stats
+  // We only need assets and chunkGroups here.
+  const stats = compilation.getStats().toJson({
+    assets: true,
+    chunkGroups: true,
+  });
 
   const assetNameToChunkNames = assetToChunkNameMapping(stats);
 

diff --git a/test/workbox-precaching/sw/test-PrecacheController.mjs b/test/workbox-precaching/sw/test-PrecacheController.mjs
@@ -614,7 +614,7 @@ describe(`PrecacheController`, function() {
       }
 
       const cleanupDetailsTwo = await precacheControllerTwo.activate();
-      expect(cleanupDetailsTwo.deletedURLs).to.deep.equal([
+      expect(cleanupDetailsTwo.deletedURLs).to.have.members([
         `${location.origin}/index.1234.html`,
         `${location.origin}/scripts/stress.js?test=search&foo=bar&__WB_REVISION__=1234`,
       ]);