fix(browser): cleanup timeout on resolve and give more information in the error (#7487)

sheremet-va · web-flow · commit 5a45a7ca6ba6 · 2025-02-13T18:00:08.000+01:00
diff --git a/packages/vitest/src/node/browser/sessions.ts b/packages/vitest/src/node/browser/sessions.ts
@@ -1,6 +1,7 @@
 import type { TestProject } from '../project'
 import type { BrowserServerStateSession } from '../types/browser'
 import { createDefer } from '@vitest/utils'
+import { relative } from 'pathe'
 
 export class BrowserSessions {
   private sessions = new Map<string, BrowserServerStateSession>()
@@ -13,7 +14,8 @@ export class BrowserSessions {
     const defer = createDefer<void>()
 
     const timeout = setTimeout(() => {
-      defer.reject(new Error(`Failed to connect to the browser session "${sessionId}" within the timeout.`))
+      const tests = files.map(file => relative(project.config.root, file)).join('", "')
+      defer.reject(new Error(`Failed to connect to the browser session "${sessionId}" [${project.name}] for "${tests}" within the timeout.`))
     }, project.vitest.config.browser.connectTimeout ?? 60_000).unref()
 
     this.sessions.set(sessionId, {
@@ -25,6 +27,7 @@ export class BrowserSessions {
       },
       resolve: () => {
         defer.resolve()
+        clearTimeout(timeout)
         this.sessions.delete(sessionId)
       },
       reject: defer.reject,
