test: delete e2e directories after tests finish

Deletes the e2e directories after tests complete to ensure a clean state for subsequent test runs and to free up disk space.

Author: alan-agius4

tests/legacy-cli/e2e/utils/assets.ts

@@ -5,7 +5,7 @@ import { getGlobalVariable } from './env';
 import { resolve } from 'node:path';
 import { copyFile } from './fs';
 import { installWorkspacePackages, setRegistry } from './packages';
-import { useBuiltPackagesVersions } from './project';
+import { getTestProjectDir, useBuiltPackagesVersions } from './project';
 
 export function assetDir(assetName: string) {
   return join(__dirname, '../e2e/assets', assetName);
@@ -21,7 +21,7 @@ export function copyProjectAsset(assetName: string, to?: string) {
 
 export function copyAssets(assetName: string, to?: string) {
   const seed = +Date.now();
-  const tempRoot = join(getGlobalVariable('projects-root'), 'assets', assetName + '-' + seed);
+  const tempRoot = join(getTestAssetsDir(), assetName + '-' + seed);
   const root = assetDir(assetName);
 
   return Promise.resolve()
@@ -30,9 +30,7 @@ export function copyAssets(assetName: string, to?: string) {
 
       return allFiles.reduce((promise, filePath) => {
         const toPath =
-          to !== undefined
-            ? resolve(getGlobalVariable('projects-root'), 'test-project', to, filePath)
-            : join(tempRoot, filePath);
+          to !== undefined ? resolve(getTestProjectDir(), to, filePath) : join(tempRoot, filePath);
 
         return promise
           .then(() => copyFile(join(root, filePath), toPath))
@@ -65,3 +63,7 @@ export async function createProjectFromAsset(
 
   return () => setRegistry(true /** useTestRegistry */);
 }
+
+export function getTestAssetsDir(): string {
+  return join(getGlobalVariable('projects-root'), 'assets');
+}

tests/legacy-cli/e2e/utils/project.ts

@@ -7,6 +7,7 @@ import { gitCommit } from './git';
 import { findFreePort } from './network';
 import { installWorkspacePackages, PkgInfo } from './packages';
 import { execAndWaitForOutputToMatch, git, ng } from './process';
+import { join } from 'node:path';
 
 export function updateJsonFile(filePath: string, fn: (json: any) => any | void) {
   return readFile(filePath).then((tsConfigJson) => {
@@ -270,3 +271,7 @@ export function updateServerFileForEsbuild(filepath: string): Promise<void> {
     `,
   );
 }
+
+export function getTestProjectDir(): string {
+  return join(getGlobalVariable('projects-root'), 'test-project');
+}

tests/legacy-cli/e2e_runner.ts

@@ -4,6 +4,7 @@ import colors from 'ansi-colors';
 import glob from 'fast-glob';
 import * as path from 'node:path';
 import * as fs from 'node:fs';
+import { rm } from 'node:fs/promises';
 import { getGlobalVariable, setGlobalVariable } from './e2e/utils/env';
 import { gitClean } from './e2e/utils/git';
 import { createNpmRegistry } from './e2e/utils/registry';
@@ -13,7 +14,7 @@ import { findFreePort } from './e2e/utils/network';
 import { extractFile } from './e2e/utils/tar';
 import { realpathSync } from 'node:fs';
 import { PkgInfo } from './e2e/utils/packages';
-import { rm } from 'node:fs/promises';
+import { getTestProjectDir } from './e2e/utils/project';
 
 Error.stackTraceLimit = Infinity;
 
@@ -271,6 +272,14 @@ Promise.all([findFreePort(), findFreePort(), findPackageTars()])
     } finally {
       registryProcess.kill();
       secureRegistryProcess.kill();
+
+      await rm(getGlobalVariable('projects-root'), {
+        recursive: true,
+        force: true,
+        maxRetries: 3,
+      }).catch(() => {
+        // If this fails it is not fatal.
+      });
     }
   })
   .catch((err) => {
@@ -334,7 +343,7 @@ function runInitializer(absoluteName: string): Promise<void> {
  * Run a file from the main 'test-project' directory in a subprocess via launchTestProcess().
  */
 async function runTest(absoluteName: string): Promise<void> {
-  process.chdir(join(getGlobalVariable('projects-root'), 'test-project'));
+  process.chdir(getTestProjectDir());
 
   await launchTestProcess(absoluteName);
   await cleanTestProject();
@@ -343,7 +352,7 @@ async function runTest(absoluteName: string): Promise<void> {
 async function cleanTestProject() {
   await gitClean();
 
-  const testProject = join(getGlobalVariable('projects-root'), 'test-project');
+  const testProject = getTestProjectDir();
 
   // Note: Dist directory is not cleared between tests, as `git clean`
   // doesn't delete it.