refactor(@angular/build): cleanup Vitest build options utilities

Extract zone testing strategy resolution to a separate function and reduce complexity of template literals.

Author: clydin

packages/angular/build/src/builders/unit-test/runners/vitest/build-options.ts

@@ -6,6 +6,11 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
+/**
+ * @fileoverview
+ * Provides Vitest-specific build options and virtual file contents for Angular unit testing.
+ */
+
 import { createRequire } from 'node:module';
 import path from 'node:path';
 import { toPosixPath } from '../../../../utils/path';
@@ -29,6 +34,17 @@ function createTestBedInitVirtualFile(
     providersImport = `import providers from './${importPath}';`;
   }
 
+  let zoneTestingSnippet = '';
+  if (zoneTestingStrategy === 'static') {
+    zoneTestingSnippet = `import 'zone.js/testing';`;
+  } else if (zoneTestingStrategy === 'dynamic') {
+    zoneTestingSnippet = `if (typeof Zone !== 'undefined') {
+      // 'zone.js/testing' is used to initialize the ZoneJS testing environment.
+      // It must be imported dynamically to avoid a static dependency on 'zone.js'.
+      await import('zone.js/testing');
+    }`;
+  }
+
   return `
     // Initialize the Angular testing environment
     import { NgModule, provideZoneChangeDetection } from '@angular/core';
@@ -37,18 +53,7 @@ function createTestBedInitVirtualFile(
     import { afterEach, beforeEach } from 'vitest';
     ${providersImport}
 
-    ${
-      zoneTestingStrategy === 'static'
-        ? `import 'zone.js/testing';`
-        : zoneTestingStrategy === 'dynamic'
-          ? `
-    if (typeof Zone !== 'undefined') {
-      // 'zone.js/testing' is used to initialize the ZoneJS testing environment.
-      // It must be imported dynamically to avoid a static dependency on 'zone.js'.
-      await import('zone.js/testing');
-    }`
-          : ''
-    }
+    ${zoneTestingSnippet}
 
     // The beforeEach and afterEach hooks are registered outside the globalThis guard.
     // This ensures that the hooks are always applied, even in non-isolated browser environments.
@@ -92,6 +97,28 @@ function adjustOutputHashing(hashing?: OutputHashing): OutputHashing {
   }
 }
 
+function getZoneTestingStrategy(
+  buildOptions: Partial<ApplicationBuilderInternalOptions>,
+  projectSourceRoot: string,
+): 'none' | 'static' | 'dynamic' {
+  if (buildOptions.polyfills?.includes('zone.js/testing')) {
+    return 'none';
+  }
+
+  if (buildOptions.polyfills?.includes('zone.js')) {
+    return 'static';
+  }
+
+  try {
+    const projectRequire = createRequire(path.join(projectSourceRoot, 'package.json'));
+    projectRequire.resolve('zone.js');
+
+    return 'dynamic';
+  } catch {
+    return 'none';
+  }
+}
+
 export async function getVitestBuildOptions(
   options: NormalizedUnitTestBuilderOptions,
   baseBuildOptions: Partial<ApplicationBuilderInternalOptions>,
@@ -162,20 +189,7 @@ export async function getVitestBuildOptions(
   };
 
   // Inject the zone.js testing polyfill if Zone.js is installed.
-  let zoneTestingStrategy: 'none' | 'static' | 'dynamic';
-  if (buildOptions.polyfills?.includes('zone.js/testing')) {
-    zoneTestingStrategy = 'none';
-  } else if (buildOptions.polyfills?.includes('zone.js')) {
-    zoneTestingStrategy = 'static';
-  } else {
-    try {
-      const projectRequire = createRequire(path.join(projectSourceRoot, 'package.json'));
-      projectRequire.resolve('zone.js');
-      zoneTestingStrategy = 'dynamic';
-    } catch {
-      zoneTestingStrategy = 'none';
-    }
-  }
+  const zoneTestingStrategy = getZoneTestingStrategy(buildOptions, projectSourceRoot);
 
   const testBedInitContents = createTestBedInitVirtualFile(
     providersFile,