chore: flaky tests, seems setup was running per file causing it to remove dist mid run of other tests

Author: bra1nDump

src/test-setup.ts

@@ -1,24 +1,27 @@
 /**
  * Test setup file for vitest
- * 
- * Configures the test environment before running tests
+ *
+ * Global setup that runs ONCE before all tests
  */
 
-// Make sure to build the project before running tests
-// We rely on the dist files to spawn our CLI in integration tests
 import { spawnSync } from 'node:child_process'
 
-// Extend test timeout for integration tests
-process.env.VITEST_POOL_TIMEOUT = '60000'
+export function setup() {
+  // Extend test timeout for integration tests
+  process.env.VITEST_POOL_TIMEOUT = '60000'
 
-const buildResult = spawnSync('yarn', ['build'], { stdio: 'pipe' })
-if (buildResult.stderr && buildResult.stderr.length > 0) {
-  const errorOutput = buildResult.stderr.toString()
-  console.error(`Build stderr (could be debugger output): ${errorOutput}`)
-  const stdout = buildResult.stdout.toString()
-  console.log(`Build stdout: ${stdout}`)
+  // Make sure to build the project before running tests
+  // We rely on the dist files to spawn our CLI in integration tests
+  const buildResult = spawnSync('yarn', ['build'], { stdio: 'pipe' })
 
-  if (errorOutput.includes('Command failed with exit code')) {
-    throw new Error(`Build failed STDERR: ${errorOutput}`)
+  if (buildResult.stderr && buildResult.stderr.length > 0) {
+    const errorOutput = buildResult.stderr.toString()
+    console.error(`Build stderr (could be debugger output): ${errorOutput}`)
+    const stdout = buildResult.stdout.toString()
+    console.log(`Build stdout: ${stdout}`)
+
+    if (errorOutput.includes('Command failed with exit code')) {
+      throw new Error(`Build failed STDERR: ${errorOutput}`)
+    }
   }
 }

vitest.config.ts

@@ -12,7 +12,7 @@ export default defineConfig({
         globals: false,
         environment: 'node',
         include: ['src/**/*.test.ts'],
-        setupFiles: ['./src/test-setup.ts'],
+        globalSetup: ['./src/test-setup.ts'],
         coverage: {
             provider: 'v8',
             reporter: ['text', 'json', 'html'],