Add test coverage going from webpack stats JSON to end result

webpack-contrib · Aug 18, 2022 · 5b080fa · 5b080fa
1 parent 7a5e10f
commit 5b080fa
Show file tree

Hide file tree

Showing 5 changed files with 5,457 additions and 51 deletions.
diff --git a/src/analyzer.js b/src/analyzer.js
@@ -13,7 +13,6 @@ const FILENAME_QUERY_REGEXP = /\?.*$/u;
 const FILENAME_EXTENSIONS = /\.(js|mjs)$/iu;
 
 module.exports = {
-  getChunkToInitialByEntrypoint,
   getViewerData,
   readStatsFromFile
 };

diff --git a/test/analyzer.js b/test/analyzer.js
@@ -7,7 +7,6 @@ const path = require('path');
 const del = require('del');
 const childProcess = require('child_process');
 const puppeteer = require('puppeteer');
-const {getChunkToInitialByEntrypoint} = require('../src/analyzer');
 
 let browser;
 
@@ -196,6 +195,20 @@ describe('Analyzer', function () {
     await expectValidReport({bundleLabel: 'bundle.js'});
   });
 
+  it('should map chunks correctly to entrypoints', async function () {
+    generateReportFrom('with-multiple-entrypoints/stats.json');
+    const chartData = await getChartData();
+    expect(chartData).to.containSubset(
+      require('./stats/with-multiple-entrypoints/expected-chart-data')
+    );
+  });
+
+  it('should return empty chartData if there are no entrypoints', async function () {
+    generateReportFrom('with-no-entrypoints/stats.json');
+    const chartData = await getChartData();
+    expect(chartData).to.be.empty;
+  });
+
   describe('options', function () {
     describe('title', function () {
       it('should take the --title option', async function () {
@@ -226,55 +239,6 @@ describe('Analyzer', function () {
   });
 });
 
-
-describe('getChunkToInitialByEntrypoint', function () {
-  it('should map chunks correctly to entrypoints', function () {
-    const bundleStats = {
-      entrypoints: {
-        'A': {
-          name: 'A',
-          assets: [
-            {
-              name: 'chunkA.js'
-            }
-          ]
-        },
-        'B': {
-          name: 'B',
-          assets: [
-            {
-              name: 'chunkA.js'
-            },
-            {
-              name: 'chunkB.js'
-            }
-          ]
-        }
-      }
-    };
-    expect(JSON.stringify(getChunkToInitialByEntrypoint(bundleStats))).to.equal(JSON.stringify({
-      'chunkA.js': {'A': true, 'B': true},
-      'chunkB.js': {'B': true}
-    }));
-  });
-
-  it('should handle when bundlestats is null or undefined ', function () {
-    expect(JSON.stringify(getChunkToInitialByEntrypoint(null))).to.equal(JSON.stringify({}));
-    expect(JSON.stringify(getChunkToInitialByEntrypoint(undefined))).to.equal(JSON.stringify({}));
-  });
-
-  it('should handle when bundlestats is emoty', function () {
-    const bundleStatsWithoutEntryPoints = {};
-    expect(JSON.stringify(getChunkToInitialByEntrypoint(bundleStatsWithoutEntryPoints))).to.equal(JSON.stringify({}));
-  });
-
-  it('should handle when entrypoints is empty', function () {
-    const bundleStatsEmptyEntryPoint = {entrypoints: {}};
-    expect(JSON.stringify(getChunkToInitialByEntrypoint(bundleStatsEmptyEntryPoint))).to.equal(JSON.stringify({}));
-  });
-
-});
-
 function generateJSONReportFrom(statsFilename) {
   childProcess.execSync(`../lib/bin/analyzer.js -m json -r output/report.json stats/${statsFilename}`, {
     cwd: __dirname