fix(@angular-devkit/build-angular) : fix incorrect budget calculation

Fixes angular#29040
santoshyadavdev · Dec 28, 2024 · 656defa · 656defa
1 parent 9b883fe
commit 656defa
Show file tree

Hide file tree

Showing 2 changed files with 39 additions and 6 deletions.
diff --git a/packages/angular/build/src/utils/bundle-calculator.ts b/packages/angular/build/src/utils/bundle-calculator.ts
@@ -12,6 +12,8 @@ import { formatSize } from './format-bytes';
 // Re-export to avoid direct schema importing throughout code
 export { type BudgetEntry, BudgetType };
 
+export const kB = 1000;
+
 interface Size {
   size: number;
   label?: string;
@@ -306,13 +308,13 @@ function calculateBytes(input: string, baseline?: string, factor: 1 | -1 = 1): n
       value = (baselineBytes * value) / 100;
       break;
     case 'kb':
-      value *= 1024;
+      value *= kB;
       break;
     case 'mb':
-      value *= 1024 * 1024;
+      value *= kB * kB;
       break;
     case 'gb':
-      value *= 1024 * 1024 * 1024;
+      value *= kB * kB * kB;
       break;
   }
 

diff --git a/packages/angular/build/src/utils/bundle-calculator_spec.ts b/packages/angular/build/src/utils/bundle-calculator_spec.ts
@@ -6,9 +6,7 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
-import { BudgetEntry, BudgetType, ThresholdSeverity, checkBudgets } from './bundle-calculator';
-
-const kB = 1000;
+import { BudgetEntry, BudgetType, ThresholdSeverity, checkBudgets, kB } from './bundle-calculator';
 
 describe('bundle-calculator', () => {
   describe('checkBudgets()', () => {
@@ -338,5 +336,38 @@ describe('bundle-calculator', () => {
         message: jasmine.stringMatching('foo.ext exceeded maximum budget.'),
       });
     });
+
+    it('yields exceeded individual file budget - 29040', () => {
+      const budgets: BudgetEntry[] = [
+        {
+          type: BudgetType.bundle,
+          maximumError: '1000kb',
+        },
+      ];
+      const stats = {
+        chunks: [
+          {
+            id: 0,
+            initial: true,
+            names: ['foo'],
+            files: ['foo.ext', 'bar.ext'],
+          },
+        ],
+        assets: [
+          {
+            name: 'foo.ext',
+            size: 1 * kB,
+          },
+          {
+            name: 'bar.ext',
+            size: 0.5 * kB,
+          },
+        ],
+      };
+
+      const failures = Array.from(checkBudgets(budgets, stats));
+
+      expect(failures.length).toBe(0);
+    });
   });
 });