Fixes #417 missing module chunks (#433)

* Fixes #417 missing module chunks * triger build * test fix * test fix
webpack-contrib · Apr 13, 2021 · e5b617e · e5b617e
1 parent 155edb7
commit e5b617e
Show file tree

Hide file tree

Showing 4 changed files with 521 additions and 1 deletion.
diff --git a/src/analyzer.js b/src/analyzer.js
@@ -190,7 +190,7 @@ function getBundleModules(bundleStats) {
 
 function assetHasModule(statAsset, statModule) {
   // Checking if this module is the part of asset chunks
-  return statModule.chunks.some(moduleChunk =>
+  return (statModule.chunks || []).some(moduleChunk =>
     statAsset.chunks.includes(moduleChunk)
   );
 }

diff --git a/test/analyzer.js b/test/analyzer.js
@@ -131,6 +131,23 @@ describe('Analyzer', function () {
     });
   });
 
+  it('should gracefully process missing chunks', async function () {
+    generateReportFrom('with-missing-module-chunks/stats.json');
+    const chartData = await getChartData();
+    const invalidChunk = _.find(chartData, {label: 'invalid-chunk.js'});
+    expect(invalidChunk).to.exist;
+    expect(invalidChunk.statSize).to.equal(568);
+    forEachChartItem([invalidChunk], item => {
+      expect(typeof item.statSize).to.equal('number');
+      expect(item.parsedSize).to.be.undefined;
+    });
+    const validChunk = _.find(chartData, {label: 'valid-chunk.js'});
+    forEachChartItem([validChunk], item => {
+      expect(typeof item.statSize).to.equal('number');
+      expect(typeof item.parsedSize).to.equal('number');
+    });
+  });
+
   it('should support stats files with js modules chunk', async function () {
     generateReportFrom('with-modules-chunk.json');
     await expectValidReport({bundleLabel: 'bundle.mjs'});