build(@angular/build): update vitest to v4.0.0-beta

This commit updates the version of `vitest` and its related packages from `3.2.4` to `4.0.0-beta.17` within the unit testing system.

This major version update for Vitest introduces several changes that are addressed here:
- The `coverage.all` option has been removed in Vitest v4. Accordingly, the `coverageAll` option has been removed from the unit-test builders schema and its usage has been removed from the Karma and Vitest test runner implementations.
- The browser provider mechanism has been updated. Vitest now uses provider-specific packages (e.g., @vitest/browser-playwright) instead of the general @vitest/browser package. The browser configuration setup has been made asynchronous and now dynamically loads the appropriate provider.
- A workaround has been added to filter non-string values from the resolved coverage exclude patterns. This prevents an error caused by Vitest incorrectly adding boolean values into the array.

Author: clydin

goldens/public-api/angular/build/index.api.md

@@ -219,7 +219,6 @@ export type UnitTestBuilderOptions = {
     browsers?: string[];
     buildTarget?: string;
     coverage?: boolean;
-    coverageAll?: boolean;
     coverageExclude?: string[];
     coverageInclude?: string[];
     coverageReporters?: SchemaCoverageReporter[];

modules/testing/builder/package.json

@@ -4,9 +4,9 @@
     "@angular-devkit/architect": "workspace:*",
     "@angular/ssr": "workspace:*",
     "@angular-devkit/build-angular": "workspace:*",
-    "@vitest/coverage-v8": "3.2.4",
+    "@vitest/coverage-v8": "4.0.0-beta.17",
     "jsdom": "27.0.0",
     "rxjs": "7.8.2",
-    "vitest": "3.2.4"
+    "vitest": "4.0.0-beta.17"
   }
 }

packages/angular/build/package.json

@@ -56,7 +56,7 @@
     "ng-packagr": "21.0.0-next.4",
     "postcss": "8.5.6",
     "rxjs": "7.8.2",
-    "vitest": "3.2.4"
+    "vitest": "4.0.0-beta.17"
   },
   "peerDependencies": {
     "@angular/core": "0.0.0-ANGULAR-FW-PEER-DEP",
@@ -74,7 +74,7 @@
     "tailwindcss": "^2.0.0 || ^3.0.0 || ^4.0.0",
     "tslib": "^2.3.0",
     "typescript": ">=5.9 <6.0",
-    "vitest": "^3.1.1"
+    "vitest": "^4.0.0-beta.17"
   },
   "peerDependenciesMeta": {
     "@angular/core": {

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

@@ -85,7 +85,6 @@ export async function normalizeOptions(
     runnerName: runner,
     coverage: options.coverage
       ? {
-          all: options.coverageAll,
           exclude: options.coverageExclude,
           include: options.coverageInclude,
           reporters: normalizeReporterOption(options.coverageReporters),

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

@@ -39,12 +39,6 @@ export class KarmaExecutor implements TestExecutor {
       );
     }
 
-    if (unitTestOptions.coverage?.all) {
-      context.logger.warn(
-        'The "karma" test runner does not support the "coverageAll" option. The option will be ignored.',
-      );
-    }
-
     if (unitTestOptions.coverage?.include) {
       context.logger.warn(
         'The "karma" test runner does not support the "coverageInclude" option. The option will be ignored.',

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

@@ -7,6 +7,7 @@
  */
 
 import { createRequire } from 'node:module';
+import { assertIsError } from '../../../../utils/error';
 
 export interface BrowserConfiguration {
   browser?: import('vitest/node').BrowserConfigOptions;
@@ -39,45 +40,60 @@ function normalizeBrowserName(browserName: string): string {
   return normalized.replace(/headless$/, '');
 }
 
-export function setupBrowserConfiguration(
+export async function setupBrowserConfiguration(
   browsers: string[] | undefined,
   debug: boolean,
   projectSourceRoot: string,
   viewport: { width: number; height: number } | undefined,
-): BrowserConfiguration {
+): Promise<BrowserConfiguration> {
   if (browsers === undefined) {
     return {};
   }
 
   const projectResolver = createRequire(projectSourceRoot + '/').resolve;
   let errors: string[] | undefined;
 
-  try {
-    projectResolver('@vitest/browser');
-  } catch {
-    errors ??= [];
-    errors.push(
-      'The "browsers" option requires the "@vitest/browser" package to be installed within the project.' +
-        ' Please install this package and rerun the test command.',
-    );
-  }
-
-  const provider = findBrowserProvider(projectResolver);
-  if (!provider) {
+  const providerName = findBrowserProvider(projectResolver);
+  if (!providerName) {
     errors ??= [];
     errors.push(
       'The "browsers" option requires either "playwright" or "webdriverio" to be installed within the project.' +
         ' Please install one of these packages and rerun the test command.',
     );
   }
 
-  // Vitest current requires the playwright browser provider to use the inspect-brk option used by "debug"
-  if (debug && provider !== 'playwright') {
-    errors ??= [];
-    errors.push(
-      'Debugging browser mode tests currently requires the use of "playwright".' +
-        ' Please install this package and rerun the test command.',
-    );
+  let provider: import('vitest/node').BrowserProviderOption | undefined;
+  if (providerName) {
+    const providerPackage = `@vitest/browser-${providerName}`;
+    try {
+      const providerModule = await import(providerPackage);
+
+      // Validate that the imported module has the expected structure
+      const providerFactory = providerModule[providerName];
+      if (typeof providerFactory === 'function') {
+        provider = providerFactory();
+      } else {
+        errors ??= [];
+        errors.push(
+          `The "${providerPackage}" package does not have a valid browser provider export.`,
+        );
+      }
+    } catch (e) {
+      assertIsError(e);
+      errors ??= [];
+      // Check for a module not found error to provide a more specific message
+      if (e.code === 'ERR_MODULE_NOT_FOUND') {
+        errors.push(
+          `The "browsers" option with "${providerName}" requires the "${providerPackage}" package.` +
+            ' Please install this package and rerun the test command.',
+        );
+      } else {
+        // Handle other potential errors during import
+        errors.push(
+          `An error occurred while loading the "${providerPackage}" browser provider:\n  ${e.message}`,
+        );
+      }
+    }
   }
 
   if (errors) {

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

@@ -154,7 +154,7 @@ export class VitestExecutor implements TestExecutor {
     const { startVitest } = vitestNodeModule;
 
     // Setup vitest browser options if configured
-    const browserOptions = setupBrowserConfiguration(
+    const browserOptions = await setupBrowserConfiguration(
       browsers,
       debug,
       this.options.projectSourceRoot,
@@ -237,7 +237,6 @@ async function generateCoverageOption(
 
   return {
     enabled: true,
-    all: coverage.all,
     excludeAfterRemap: true,
     include: coverage.include,
     reportsDirectory: toPosixPath(path.join('coverage', projectName)),

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

@@ -23,7 +23,6 @@ const VitestTestRunner: TestRunner = {
     checker.check('vitest');
 
     if (options.browsers?.length) {
-      checker.check('@vitest/browser');
       checker.checkAny(
         ['playwright', 'webdriverio'],
         'The "browsers" option requires either "playwright" or "webdriverio" to be installed.',

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

@@ -45,7 +45,7 @@ export function createVitestPlugins(
         // Create a subproject that can be configured with plugins for browser mode.
         // Plugins defined directly in the vite overrides will not be present in the
         // browser specific Vite instance.
-        await context.injectTestProjects({
+        const [project] = await context.injectTestProjects({
           test: {
             name: projectName,
             root: workspaceRoot,
@@ -155,6 +155,13 @@ export function createVitestPlugins(
             },
           ],
         });
+
+        // Filter the resolved coverage exclude patterns to remove non-string values.
+        // This is a workaround for vitest adding `false` into the array when `startVitest` is
+        // used with `config: false`. This can be removed once corrected in Vitest.
+        project.config.coverage.exclude = project.config.coverage.exclude.filter(
+          (pattern) => typeof pattern === 'string',
+        );
       },
     },
   ];

packages/angular/build/src/builders/unit-test/schema.json

@@ -64,11 +64,6 @@
       "description": "Enables coverage reporting for tests.",
       "default": false
     },
-    "coverageAll": {
-      "type": "boolean",
-      "description": "Includes all files that match the `coverageInclude` pattern in the coverage report, not just those touched by tests.",
-      "default": true
-    },
     "coverageInclude": {
       "type": "array",
       "description": "Specifies glob patterns of files to include in the coverage report.",