From fcef10a9e1abc12f37cf508554f69fbcdcabe8c9 Mon Sep 17 00:00:00 2001
From: Alan Agius <alan.agius4@gmail.com>
Date: Fri, 13 Mar 2020 12:24:58 +0100
Subject: [PATCH] fix(@angular-devkit/build-angular): add defaults to reduce
 methods in bundle calculator

Fixes #17215
---
 .../utilities/bundle-calculator.ts            | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/packages/angular_devkit/build_angular/src/angular-cli-files/utilities/bundle-calculator.ts b/packages/angular_devkit/build_angular/src/angular-cli-files/utilities/bundle-calculator.ts
index 08b464775138..8e380442a5c4 100644
--- a/packages/angular_devkit/build_angular/src/angular-cli-files/utilities/bundle-calculator.ts
+++ b/packages/angular_devkit/build_angular/src/angular-cli-files/utilities/bundle-calculator.ts
@@ -167,17 +167,16 @@ abstract class Calculator {
     } else {
       // No differential builds, get the chunk size by summing its assets.
       return chunk.files
-          .filter((file) => !file.endsWith('.map'))
-          .map((file: string) => {
+          .filter(file => !file.endsWith('.map'))
+          .map(file => {
             const asset = this.assets.find((asset) => asset.name === file);
             if (!asset) {
               throw new Error(`Could not find asset for file: ${file}`);
             }
 
-            return asset;
+            return asset.size;
           })
-          .map((asset) => asset.size)
-          .reduce((l, r) => l + r);
+          .reduce((l, r) => l + r, 0);
     }
   }
 }
@@ -197,8 +196,8 @@ class BundleCalculator extends Calculator {
     const buildSizes = Object.values(DifferentialBuildType).map((buildType) => {
       const size = this.chunks
           .filter(chunk => chunk.names.indexOf(budgetName) !== -1)
-          .map((chunk) => this.calculateChunkSize(chunk, buildType))
-          .reduce((l, r) => l + r);
+          .map(chunk => this.calculateChunkSize(chunk, buildType))
+          .reduce((l, r) => l + r, 0);
 
       return {size, label: `${this.budget.name}-${buildType}`};
     });
@@ -223,8 +222,8 @@ class InitialCalculator extends Calculator {
         label: `initial-${buildType}`,
         size: this.chunks
             .filter(chunk => chunk.initial)
-            .map((chunk) => this.calculateChunkSize(chunk, buildType))
-            .reduce((l, r) => l + r),
+            .map(chunk => this.calculateChunkSize(chunk, buildType))
+            .reduce((l, r) => l + r, 0),
       };
     });
 
@@ -244,7 +243,7 @@ class InitialCalculator extends Calculator {
 class AllScriptCalculator extends Calculator {
   calculate() {
     const size = this.assets
-      .filter((asset) => asset.name.endsWith('.js'))
+      .filter(asset => asset.name.endsWith('.js'))
       .map(asset => asset.size)
       .reduce((total: number, size: number) => total + size, 0);