Harden Copilot Electron test temp-dir cleanup on Windows

[Copilot]

Fixes #314897

Description

Electron tests were intermittently failing after the extension host crashed because Windows still held Crashpad dump files open during temp-dir teardown, surfacing as EBUSY from fs.rm. This updates the Copilot Electron test runner cleanup path to tolerate transient file locks instead of failing the run during best-effort cleanup.

• Problem

  • Cleanup in runTest.ts removed the test user-data dir once, without retries.
  • On Windows, post-crash Crashpad artifacts can still be locked briefly, so teardown could fail even after the real test failure had already been handled.

• Changes

  • Added retry-enabled temp-dir cleanup directly in runTest.ts.
  • Switched cleanup to fs.rm(..., { recursive: true, force: true, maxRetries, retryDelay }) so transient EBUSY/EPERM locks are retried by Node.
  • Applied the same cleanup path to both temporary directories created by the runner:
    • --user-data-dir
    • temporary workspace folder
  • Made cleanup failures non-fatal after logging, so teardown does not mask the actual test outcome.

• Example

  await fs.rm(dir, {
  	recursive: true,
  	force: true,
  	maxRetries: 10,
  	retryDelay: 200
  });

Testing

• cd extensions/copilot && npm run typecheck
• node --experimental-strip-types build/hygiene.ts extensions/copilot/src/extension/completions-core/vscode-node/extension/test/runTest.ts
• Secret scan passed for the updated file

[dmitrivMS]

@copilot See and fix comments

[Copilot]

@copilot See and fix comments

Addressed the review comments in 7dc10e0.

[Copilot]

Pull request overview

Hardens the Copilot Electron test runner teardown path so transient Windows file locks (e.g., Crashpad artifacts after an extension host crash) don’t cause the overall test run to fail during best-effort cleanup.

Changes:

• Added a cleanupTestDirectory helper that deletes directories using fs.rm with force plus retry options (maxRetries, retryDelay) and logs failures without throwing.
• Applied the improved cleanup to both the temporary --user-data-dir and the temporary workspace folder in the runner’s finally block.

Show a summary per file

File	Description
extensions/copilot/src/extension/completions-core/vscode-node/extension/test/runTest.ts	Makes temp-dir cleanup resilient to transient Windows locks by adding retry-enabled, non-fatal teardown for both created temp directories.

Copilot's findings

• Files reviewed: 1/1 changed files
• Comments generated: 0