Skip to content

Commit b141670

Browse files
committed
fix(@angular/build): inject testing polyfills in Karma unit-test executor
This commit ensures that testing polyfills (specifically `zone.js/testing`) are correctly injected when using the new unit-test builder with the Karma runner. Previously, only the application polyfills were included, which could lead to runtime errors in tests dependent on `zone.js/testing`. (cherry picked from commit 385165c)
1 parent 0fc7d57 commit b141670

File tree

1 file changed

+2
-2
lines changed
  • packages/angular/build/src/builders/unit-test/runners/karma

1 file changed

+2
-2
lines changed

packages/angular/build/src/builders/unit-test/runners/karma/executor.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ import fs from 'node:fs/promises';
1111
import path from 'node:path';
1212
import type { ApplicationBuilderInternalOptions } from '../../../application/options';
1313
import type { KarmaBuilderOptions, KarmaBuilderTransformsOptions } from '../../../karma';
14-
import { NormalizedUnitTestBuilderOptions } from '../../options';
14+
import { type NormalizedUnitTestBuilderOptions, injectTestingPolyfills } from '../../options';
1515
import type { TestExecutor } from '../api';
1616

1717
export class KarmaExecutor implements TestExecutor {
@@ -70,7 +70,7 @@ export class KarmaExecutor implements TestExecutor {
7070
const karmaOptions: KarmaBuilderOptions = {
7171
karmaConfig,
7272
tsConfig: unitTestOptions.tsConfig ?? buildTargetOptions.tsConfig,
73-
polyfills: buildTargetOptions.polyfills,
73+
polyfills: injectTestingPolyfills(buildTargetOptions.polyfills),
7474
assets: buildTargetOptions.assets,
7575
scripts: buildTargetOptions.scripts,
7676
styles: buildTargetOptions.styles,

0 commit comments

Comments
 (0)