Skip to content

Commit 4c7a483

Browse files
authored
Always clear enqueued test state (#1711)
This was only clearing in tests but could erroniously mark tests/suites as skipped in production.
1 parent 323d996 commit 4c7a483

File tree

1 file changed

+1
-12
lines changed

1 file changed

+1
-12
lines changed

src/TestExplorer/TestRunner.ts

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -18,13 +18,7 @@ import * as stream from "stream";
1818
import * as os from "os";
1919
import * as asyncfs from "fs/promises";
2020
import { FolderContext } from "../FolderContext";
21-
import {
22-
compactMap,
23-
execFile,
24-
getErrorDescription,
25-
IS_PRODUCTION_BUILD,
26-
IS_RUNNING_UNDER_TEST,
27-
} from "../utilities/utilities";
21+
import { compactMap, execFile, getErrorDescription } from "../utilities/utilities";
2822
import { createSwiftTask } from "../tasks/SwiftTaskProvider";
2923
import configuration from "../configuration";
3024
import { WorkspaceContext } from "../WorkspaceContext";
@@ -245,11 +239,6 @@ export class TestRunProxy {
245239
}
246240

247241
private clearEnqueuedTest(test: vscode.TestItem) {
248-
if (IS_PRODUCTION_BUILD && !IS_RUNNING_UNDER_TEST) {
249-
// `runState.enqueued` exists only for test validation purposes.
250-
return;
251-
}
252-
253242
this.runState.enqueued = this.runState.enqueued.filter(t => t !== test);
254243

255244
if (!test.parent) {

0 commit comments

Comments
 (0)