fix(reuse): disable tracing

src/reusedBrowser.ts

@@ -239,6 +239,11 @@ export class ReusedBrowser implements vscodeTypes.Disposable {
   browserServerEnv(debug: boolean): NodeJS.ProcessEnv | undefined {
     return (debug || this._shouldReuseBrowserForTests) && this._browserServerWS ? {
       PW_TEST_REUSE_CONTEXT: this._shouldReuseBrowserForTests ? '1' : undefined,
+      // "Show browser" mode forces context reuse that survives over multiple test runs.
+      // Playwright Test sets up `tracesDir` inside the `test-results` folder, so it will be removed between runs.
+      // When context is reused, its ongoing tracing will fail with ENOENT because trace files
+      // were suddenly removed. So we disable tracing in this case.
+      PW_TEST_DISABLE_TRACING: this._shouldReuseBrowserForTests ? '1' : undefined,
       PW_TEST_CONNECT_WS_ENDPOINT: this._browserServerWS,
     } : undefined;
   }

tests/run-tests.spec.ts

@@ -16,6 +16,7 @@
 
 import { expect, test } from './utils';
 import { TestRun } from './mock/vscode';
+import fs from 'fs';
 
 test('should run all tests', async ({ activate }) => {
   const { vscode, testController } = await activate({
@@ -1002,3 +1003,21 @@ test('should produce output twice', async ({ activate }) => {
 
   `);
 });
+
+test('should disable tracing when reusing context', async ({ activate, mode }) => {
+  test.skip(mode !== 'reuse');
+
+  const { testController } = await activate({
+    'playwright.config.js': `module.exports = { testDir: 'tests', use: { trace: 'on' } }`,
+    'tests/test.spec.ts': `
+      import { test } from '@playwright/test';
+      test('one', async ({ page }) => {});
+    `,
+  });
+
+  const testItems = testController.findTestItems(/test.spec.ts/);
+  expect(testItems.length).toBe(1);
+  await testController.run(testItems);
+
+  expect(fs.existsSync(test.info().outputPath('test-results', 'test-one', 'trace.zip'))).toBe(false);
+});