chore(vscode): improving the playwright setup/teardown steps (#4883)

Author: benfdking

vscode/extension/playwright.config.ts

@@ -7,8 +7,16 @@ export default defineConfig({
   retries: process.env.CI ? 2 : 0,
   workers: 4,
   reporter: [['html', { outputFolder: 'playwright-report' }], ['list']],
-  globalSetup: './tests/global-setup.ts',
   projects: [
+    {
+      name: 'setup',
+      testMatch: 'tests/extension.setup.ts',
+      teardown: 'cleanup',
+    },
+    {
+      name: 'cleanup',
+      testMatch: 'tests/extension.teardown.ts',
+    },
     {
       name: 'electron-vscode',
       use: {
@@ -20,6 +28,7 @@ export default defineConfig({
         viewport: { width: 1512, height: 944 },
         video: 'retain-on-failure',
       },
+      dependencies: ['setup'],
     },
   ],
 })

vscode/extension/tests/global-setup.ts renamed to vscode/extension/tests/extension.setup.ts

@@ -1,9 +1,10 @@
+import { test as setup } from '@playwright/test'
 import { execSync } from 'child_process'
 import path from 'path'
 import fs from 'fs-extra'
 import { createHash } from 'crypto'
 
-async function globalSetup() {
+setup('prepare extension', async () => {
   console.log('Setting up extension for Playwright tests...')
 
   const extensionDir = path.join(__dirname, '..')
@@ -70,11 +71,9 @@ async function globalSetup() {
     // Clean up temporary user data directory
     await fs.remove(tempUserDataDir)
   }
-}
+})
 
 async function hashFile(filePath: string): Promise<string> {
   const fileBuffer = await fs.readFile(filePath)
   return createHash('sha256').update(fileBuffer).digest('hex')
 }
-
-export default globalSetup

vscode/extension/tests/extension.teardown.ts

@@ -0,0 +1,16 @@
+import { test as teardown } from '@playwright/test'
+import path from 'path'
+import fs from 'fs-extra'
+
+teardown('cleanup extension', async () => {
+  console.log('Cleaning up extension test setup...')
+
+  const extensionDir = path.join(__dirname, '..')
+  const testSetupDir = path.join(extensionDir, '.test_setup')
+
+  // Clean up test setup directory
+  if (fs.existsSync(testSetupDir)) {
+    await fs.remove(testSetupDir)
+    console.log('Test setup directory cleaned up')
+  }
+})