fix(@angular/build): reverts the test output location to its original and removes unique UUID folders. closes #30713

Author: Bjorn S.

packages/angular/build/src/builders/karma/application_builder.ts

@@ -8,7 +8,6 @@
 
 import type { BuilderContext, BuilderOutput } from '@angular-devkit/architect';
 import type { Config, ConfigOptions, FilePattern, InlinePluginDef, Server } from 'karma';
-import { randomUUID } from 'node:crypto';
 import * as fs from 'node:fs/promises';
 import type { IncomingMessage, ServerResponse } from 'node:http';
 import { createRequire } from 'node:module';
@@ -24,9 +23,9 @@ import { ApplicationBuilderInternalOptions } from '../application/options';
 import { Result, ResultFile, ResultKind } from '../application/results';
 import { OutputHashing } from '../application/schema';
 import { findTests, getTestEntrypoints } from './find-tests';
+import type { KarmaBuilderTransformsOptions } from './index';
 import { NormalizedKarmaBuilderOptions, normalizeOptions } from './options';
 import { Schema as KarmaBuilderOptions } from './schema';
-import type { KarmaBuilderTransformsOptions } from './index';
 
 const localResolve = createRequire(__filename).resolve;
 const isWindows = process.platform === 'win32';
@@ -380,7 +379,8 @@ async function initializeApplication(
 ): Promise<
   [typeof import('karma'), Config & ConfigOptions, BuildOptions, AsyncIterator<Result> | null]
 > {
-  const outputPath = path.join(context.workspaceRoot, 'dist/test-out', randomUUID());
+  const testOutput = '.angular/cache/test-out'; // Could be extended later to optionally allow for a randomUUID or a custom outputPath.
+  const outputPath = path.join(context.workspaceRoot, testOutput);
   const projectSourceRoot = await getProjectSourceRoot(context);
 
   const [karma, entryPoints] = await Promise.all([

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

@@ -8,7 +8,6 @@
 
 import type { BuilderOutput } from '@angular-devkit/architect';
 import assert from 'node:assert';
-import { randomUUID } from 'node:crypto';
 import { createRequire } from 'node:module';
 import path from 'node:path';
 import type { InlineConfig, Vitest } from 'vitest/node';
@@ -255,10 +254,10 @@ function setupBrowserConfiguration(
 }
 
 function generateOutputPath(): string {
-  const datePrefix = new Date().toISOString().replaceAll(/[-:.]/g, '');
-  const uuidSuffix = randomUUID().slice(0, 8);
+  // Could be extended later to optionally allow for a randomUUID or a custom outputPath.
+  const testOutput = '.angular/cache/test-out';
 
-  return path.join('dist', 'test-out', `${datePrefix}-${uuidSuffix}`);
+  return path.join(testOutput);
 }
 
 function generateCoverageOption(

packages/angular_devkit/build_angular/src/builders/jest/index.ts

@@ -6,10 +6,9 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
-import { ResultKind, buildApplicationInternal } from '@angular/build/private';
 import { BuilderContext, BuilderOutput, createBuilder } from '@angular-devkit/architect';
+import { ResultKind, buildApplicationInternal } from '@angular/build/private';
 import { execFile as execFileCb } from 'node:child_process';
-import { randomUUID } from 'node:crypto';
 import * as fs from 'node:fs/promises';
 import * as path from 'node:path';
 import { promisify } from 'node:util';
@@ -30,7 +29,10 @@ export default createBuilder(
     );
 
     const options = normalizeOptions(schema);
-    const testOut = path.join(context.workspaceRoot, 'dist/test-out', randomUUID()); // TODO(dgp1130): Hide in temp directory.
+    // Could be extended later to optionally allow for a randomUUID or a custom outputPath.
+    const testOutput = '.angular/cache/test-out';
+
+    const testOut = path.join(context.workspaceRoot, testOutput);
 
     // Verify Jest installation and get the path to it's binary.
     // We need to `node_modules/.bin/jest`, but there is no means to resolve that directly. Fortunately Jest's `package.json` exports the

packages/angular_devkit/build_angular/src/builders/web-test-runner/index.ts

@@ -6,10 +6,9 @@
  * found in the LICENSE file at https://angular.dev/license
  */
 
-import { Result, ResultKind, buildApplicationInternal } from '@angular/build/private';
 import { BuilderContext, BuilderOutput, createBuilder } from '@angular-devkit/architect';
+import { Result, ResultKind, buildApplicationInternal } from '@angular/build/private';
 import type * as WebTestRunner from '@web/test-runner';
-import { randomUUID } from 'node:crypto';
 import fs from 'node:fs/promises';
 import { createRequire } from 'node:module';
 import path from 'node:path';
@@ -43,8 +42,10 @@ export default createBuilder(
     }
 
     const options = normalizeOptions(schema);
+    // Could be extended later to optionally allow for a randomUUID or a custom outputPath.
+    const testOutput = '.angular/cache/test-out';
 
-    const testDir = path.join(ctx.workspaceRoot, 'dist/test-out', randomUUID());
+    const testDir = path.join(ctx.workspaceRoot, testOutput);
 
     // Parallelize startup work.
     const [testFiles] = await Promise.all([