From 15928f8b67c2b1a39933062f6169a43f484f79cf Mon Sep 17 00:00:00 2001 From: Paul LeMarquand Date: Thu, 10 Jul 2025 13:21:18 -0400 Subject: [PATCH] Always clear enqueued test state This was only clearing in tests but could erroniously mark tests/suites as skipped in production. --- src/TestExplorer/TestRunner.ts | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/src/TestExplorer/TestRunner.ts b/src/TestExplorer/TestRunner.ts index e64a4066f..8c9621c39 100644 --- a/src/TestExplorer/TestRunner.ts +++ b/src/TestExplorer/TestRunner.ts @@ -18,13 +18,7 @@ import * as stream from "stream"; import * as os from "os"; import * as asyncfs from "fs/promises"; import { FolderContext } from "../FolderContext"; -import { - compactMap, - execFile, - getErrorDescription, - IS_PRODUCTION_BUILD, - IS_RUNNING_UNDER_TEST, -} from "../utilities/utilities"; +import { compactMap, execFile, getErrorDescription } from "../utilities/utilities"; import { createSwiftTask } from "../tasks/SwiftTaskProvider"; import configuration from "../configuration"; import { WorkspaceContext } from "../WorkspaceContext"; @@ -245,11 +239,6 @@ export class TestRunProxy { } private clearEnqueuedTest(test: vscode.TestItem) { - if (IS_PRODUCTION_BUILD && !IS_RUNNING_UNDER_TEST) { - // `runState.enqueued` exists only for test validation purposes. - return; - } - this.runState.enqueued = this.runState.enqueued.filter(t => t !== test); if (!test.parent) {