fix: stop debugging if running other profiles

src/debug/debugManager.ts

@@ -138,6 +138,10 @@ export class TestDebugManager extends vscode.Disposable {
   public async stopDebugging() {
     await Promise.allSettled([...this.sessions].map(([, s]) => vscode.debug.stopDebugging(s)))
   }
+
+  public get enabled() {
+    return this.port !== undefined
+  }
 }
 
 interface TestDebugConfiguration {

src/extension.ts

@@ -140,7 +140,10 @@ class VitestExtension {
         )
       }
       runProfile.tag = api.tag
-      runProfile.runHandler = (request, token) => runner.runTests(request, token)
+      runProfile.runHandler = async (request, token) => {
+        await runner.stopDebugRun()
+        await runner.runTests(request, token)
+      }
       this.runProfiles.set(`${api.id}:run`, runProfile)
       let debugProfile = previousRunProfiles.get(`${api.id}:debug`)
       if (!debugProfile) {
@@ -180,7 +183,10 @@ class VitestExtension {
           )
         }
         coverageProfile.tag = api.tag
-        coverageProfile.runHandler = (request, token) => runner.runCoverage(request, token)
+        coverageProfile.runHandler = async (request, token) => {
+          await runner.stopDebugRun()
+          await runner.runCoverage(request, token)
+        }
         coverageProfile.loadDetailedCoverage = coverageContext.loadDetailedCoverage
         this.runProfiles.set(`${api.id}:coverage`, coverageProfile)
       }

src/runner/runner.ts

@@ -143,7 +143,7 @@ export class TestRunner extends vscode.Disposable {
     // it's important to keep the same test items that were in the original request
     // because they dictate how format testNamePattern
     for (const [testFileData, testFileChildren] of testItems) {
-      if (token.isCancellationRequested)
+      if (token.isCancellationRequested || !this.debug.enabled)
         break
 
       const includedTests = testFileChildren.length
@@ -195,6 +195,16 @@ export class TestRunner extends vscode.Disposable {
     }
   }
 
+  public async stopDebugRun() {
+    // if debuggins session is active, users can still run tests
+    // there is no guarantee it will end correctly, so we nuke it
+    if (this.debug.enabled) {
+      await this.debug.disable(this.api)
+      this.simpleTestRunRequest = undefined
+      this.endTestRun()
+    }
+  }
+
   public async runCoverage(request: vscode.TestRunRequest, token: vscode.CancellationToken) {
     try {
       await this.api.enableCoverage()