Fix e2e benchmarks: use beforeEachIteration instead of afterEachIteration

Replace afterEachIteration with beforeEachIteration to ensure files
are reset BEFORE each benchmark iteration runs, not after. This
ensures files are always in the expected location when the benchmark
starts, preventing "file not found" errors during warmup and actual
benchmark runs.

Co-authored-by: LayZeeDK <6364586+LayZeeDK@users.noreply.github.com>

Author: Copilot

packages/workspace-e2e/src/performance-benchmark.bench.ts

@@ -97,9 +97,15 @@ describe('Move-File Generator E2E Performance', () => {
       );
     });
 
-    afterEachIteration(() => {
-      // Move files back to lib1 for next iteration
-      [smallFileName, mediumFileName, largeFileName].forEach((fileName) => {
+    beforeEachIteration(() => {
+      // Ensure files are in lib1 before each iteration
+      const fileContentMap: Map<string, string> = new Map([
+        [smallFileName, smallFileContent],
+        [mediumFileName, mediumFileContent],
+        [largeFileName, largeFileContent],
+      ]);
+
+      fileContentMap.forEach((content, fileName) => {
         const lib2Path = join(
           projectDirectory,
           benchmarkLib2,
@@ -114,13 +120,9 @@ describe('Move-File Generator E2E Performance', () => {
           'lib',
           fileName,
         );
-        try {
-          const content = readFileSync(lib2Path, 'utf-8');
-          writeFileSync(lib1Path, content);
-          rmSync(lib2Path, { force: true });
-        } catch {
-          // File might not have been moved yet, ignore
-        }
+        // Always write the file to lib1 and remove from lib2
+        writeFileSync(lib1Path, content);
+        rmSync(lib2Path, { force: true });
       });
     });
 
@@ -174,8 +176,8 @@ describe('Move-File Generator E2E Performance', () => {
       }
     });
 
-    afterEachIteration(() => {
-      // Move files back to lib1
+    beforeEachIteration(() => {
+      // Ensure files are in lib1 before each iteration
       fileContents.forEach((content, fileName) => {
         const lib2Path = join(
           projectDirectory,
@@ -191,12 +193,9 @@ describe('Move-File Generator E2E Performance', () => {
           'lib',
           fileName,
         );
-        try {
-          writeFileSync(lib1Path, content);
-          rmSync(lib2Path, { force: true });
-        } catch {
-          // File might not have been moved yet, ignore
-        }
+        // Always write the file to lib1 and remove from lib2
+        writeFileSync(lib1Path, content);
+        rmSync(lib2Path, { force: true });
       });
     });
 

packages/workspace-e2e/src/performance-stress-test.bench.ts

@@ -114,9 +114,9 @@ describe('Move-File Generator E2E Stress Tests', () => {
       }
     });
 
-    afterEachIteration(() => {
+    beforeEachIteration(() => {
       const projectCount = libs.length;
-      // Move utility file back to first project
+      // Ensure utility file is in first project before each iteration
       const movedPath = join(
         projectDirectory,
         libs[projectCount - 1],
@@ -132,24 +132,20 @@ describe('Move-File Generator E2E Stress Tests', () => {
         utilityFile,
       );
 
-      try {
-        writeFileSync(originalPath, utilityContent);
-        rmSync(movedPath, { force: true });
+      writeFileSync(originalPath, utilityContent);
+      rmSync(movedPath, { force: true });
 
-        // Reset index file
-        const sourceIndexPath = join(
-          projectDirectory,
-          libs[0],
-          'src',
-          'index.ts',
-        );
-        writeFileSync(
-          sourceIndexPath,
-          `export * from './lib/${utilityFile.replace('.ts', '')}';\n`,
-        );
-      } catch {
-        // File might not have been moved yet, ignore
-      }
+      // Reset index file
+      const sourceIndexPath = join(
+        projectDirectory,
+        libs[0],
+        'src',
+        'index.ts',
+      );
+      writeFileSync(
+        sourceIndexPath,
+        `export * from './lib/${utilityFile.replace('.ts', '')}';\n`,
+      );
     });
 
     it('should efficiently move files across workspace with 10+ projects', () => {
@@ -241,8 +237,8 @@ describe('Move-File Generator E2E Stress Tests', () => {
       }
     });
 
-    afterEachIteration(() => {
-      // Move file back to source library
+    beforeEachIteration(() => {
+      // Ensure file is in source library before each iteration
       const movedPath = join(
         projectDirectory,
         targetLib,
@@ -258,24 +254,20 @@ describe('Move-File Generator E2E Stress Tests', () => {
         targetFile,
       );
 
-      try {
-        writeFileSync(originalPath, targetFileContent);
-        rmSync(movedPath, { force: true });
+      writeFileSync(originalPath, targetFileContent);
+      rmSync(movedPath, { force: true });
 
-        // Reset index file
-        const sourceIndexPath = join(
-          projectDirectory,
-          sourceLib,
-          'src',
-          'index.ts',
-        );
-        writeFileSync(
-          sourceIndexPath,
-          `export * from './lib/${targetFile.replace('.ts', '')}';\n`,
-        );
-      } catch {
-        // File might not have been moved yet, ignore
-      }
+      // Reset index file
+      const sourceIndexPath = join(
+        projectDirectory,
+        sourceLib,
+        'src',
+        'index.ts',
+      );
+      writeFileSync(
+        sourceIndexPath,
+        `export * from './lib/${targetFile.replace('.ts', '')}';\n`,
+      );
     });
 
     it('should efficiently process workspace with 100+ large files', () => {