fix(vscode): retry temp directory cleanup in the e2e fixture

The per-test `tempDir` fixture removed the directory immediately after the
test body finished. The extension host is often still writing into it at that
point - the language server is shutting down, or uv is finishing a virtual
environment the test created there - so `fs.remove` intermittently failed
with `ENOTEMPTY` and failed an otherwise passing test, e.g.

    Error: ENOTEMPTY: directory not empty, rmdir
      '.../vscode-test-temp-Ay5Ma1/.venv/lib/python3.11/site-packages/tenacity'

Retry the removal with a backoff and, if it still cannot be removed, warn
instead of failing the test. Cleaning a temporary directory is not what the
test is asserting.

Signed-off-by: Adegbite Ayoade <tripleaceme@gmail.com>

Author: tripleaceme

vscode/extension/tests/fixtures.ts

@@ -8,6 +8,33 @@ import {
   CodeServerContext,
 } from './utils_code_server'
 
+const REMOVE_RETRY_DELAYS_MS = [100, 250, 500, 1000, 2000]
+
+/**
+ * Remove a directory, retrying with a backoff.
+ *
+ * When a test finishes, the extension host may still be writing into its
+ * temporary directory - the language server is shutting down, or uv is
+ * finishing off a virtual environment it created there. A single `fs.remove`
+ * then fails with `ENOTEMPTY`, which fails an otherwise passing test.
+ */
+const removeWithRetry = async (dir: string): Promise<void> => {
+  for (let attempt = 0; ; attempt++) {
+    try {
+      await fs.remove(dir)
+      return
+    } catch (error) {
+      if (attempt >= REMOVE_RETRY_DELAYS_MS.length) {
+        console.warn(`Failed to remove directory ${dir}:`, error)
+        return
+      }
+      await new Promise(resolve =>
+        setTimeout(resolve, REMOVE_RETRY_DELAYS_MS[attempt]),
+      )
+    }
+  }
+}
+
 // Worker-scoped fixture to start/stop VS Code server once per worker
 export const test = base.extend<
   // eslint-disable-next-line @typescript-eslint/no-empty-object-type
@@ -52,7 +79,7 @@ export const test = base.extend<
 
       // Clean up after each test
       console.log(`Cleaning up temporary directory: ${tempDir}`)
-      await fs.remove(tempDir)
+      await removeWithRetry(tempDir)
     },
     // eslint-disable-next-line @typescript-eslint/ban-ts-comment
     // @ts-expect-error