combine printFileSizes and removeFileNameHash to fileSizeReporter

Author: viankakrisna

packages/react-dev-utils/printFileSizes.js renamed to packages/react-dev-utils/fileSizeReporter.js

@@ -5,7 +5,7 @@ var filesize = require('filesize');
 var stripAnsi = require('strip-ansi');
 var gzipSize = require('gzip-size').sync;
 // Print a detailed summary of build files.
-module.exports = function printFileSizes(appBuild, stats, previousSizeMap) {
+function printFileSizesAfterBuild(appBuild, stats, previousSizeMap) {
   var assets = stats
     .toJson()
     .assets.filter(asset => /\.(js|css)$/.test(asset.name))
@@ -66,4 +66,20 @@ function getDifferenceLabel(currentSize, previousSize) {
   }
 };
 
+function measureFileSizesBeforeBuild(appBuild, fileNames){
+  return (fileNames || [])
+    .filter(fileName => /\.(js|css)$/.test(fileName))
+    .reduce((memo, fileName) => {
+      var contents = fs.readFileSync(fileName);
+      var key = removeFileNameHash(appBuild, fileName);
+      memo[key] = gzipSize(contents);
+      return memo;
+    }, {});
+}
+
+module.exports = {
+  measureFileSizesBeforeBuild: measureFileSizesBeforeBuild,
+  printFileSizesAfterBuild: printFileSizesAfterBuild
+}
+
 

packages/react-dev-utils/removeFileNameHash.js

@@ -28,8 +28,7 @@ var webpack = require('webpack');
 var config = require('../config/webpack.config.prod');
 var paths = require('../config/paths');
 var checkRequiredFiles = require('react-dev-utils/checkRequiredFiles');
-var printFileSizes = require('react-dev-utils/printFileSizes');
-var removeFileNameHash = require('react-dev-utils/removeFileNameHash');
+var fileSizeReporter = require('react-dev-utils/fileSizeReporter');
 var recursive = require('recursive-readdir');
 var stripAnsi = require('strip-ansi');
 
@@ -43,14 +42,7 @@ if (!checkRequiredFiles([paths.appHtml, paths.appIndexJs])) {
 // First, read the current file sizes in build directory.
 // This lets us display how much they changed later.
 recursive(paths.appBuild, (err, fileNames) => {
-  var previousSizeMap = (fileNames || [])
-    .filter(fileName => /\.(js|css)$/.test(fileName))
-    .reduce((memo, fileName) => {
-      var contents = fs.readFileSync(fileName);
-      var key = removeFileNameHash(paths.appBuild, fileName);
-      memo[key] = gzipSize(contents);
-      return memo;
-    }, {});
+  var previousSizeMap = fileSizeReporter.measureFileSizesBeforeBuild(paths, fileNames);
 
   // Remove all content but keep the directory so that
   // if you're in it, you don't end up in Trash
@@ -110,7 +102,7 @@ function build(previousSizeMap) {
 
     console.log('File sizes after gzip:');
     console.log();
-    printFileSizes(paths.appBuild, stats, previousSizeMap);
+    fileSizeReporter.printFileSizesAfterBuild(paths.appBuild, stats, previousSizeMap);
     console.log();
 
     var openCommand = process.platform === 'win32' ? 'start' : 'open';