Skip to content

Commit ada344f

Browse files
committed
Smarter finding of the root "test" folder
1 parent 53e4639 commit ada344f

File tree

3 files changed

+17
-10
lines changed

3 files changed

+17
-10
lines changed

test/integration-tests/WorkspaceContext.test.ts

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,11 @@ import { FolderOperation, WorkspaceContext } from "../../src/WorkspaceContext";
2020
import { createBuildAllTask } from "../../src/tasks/SwiftTaskProvider";
2121
import { Version } from "../../src/utilities/version";
2222
import { SwiftExecution } from "../../src/tasks/SwiftExecution";
23-
import { activateExtensionForSuite, updateSettings } from "./utilities/testutilities";
23+
import {
24+
activateExtensionForSuite,
25+
getRootWorkspaceFolder,
26+
updateSettings,
27+
} from "./utilities/testutilities";
2428
import { FolderContext } from "../../src/FolderContext";
2529
import { assertContains } from "./testexplorer/utilities";
2630

@@ -60,7 +64,7 @@ suite("WorkspaceContext Test Suite", () => {
6064
recordedFolders.push(changedFolderRecord);
6165
});
6266

63-
const workspaceFolder = vscode.workspace.workspaceFolders?.values().next().value;
67+
const workspaceFolder = getRootWorkspaceFolder();
6468

6569
assert.ok(workspaceFolder, "No workspace folders found in workspace");
6670

@@ -102,7 +106,7 @@ suite("WorkspaceContext Test Suite", () => {
102106
});
103107

104108
// Was hitting a timeout in suiteSetup during CI build once in a while
105-
this.timeout(5000);
109+
this.timeout(15000);
106110

107111
test("Default Task values", async () => {
108112
const folder = workspaceContext.folders.find(
@@ -244,4 +248,4 @@ suite("WorkspaceContext Test Suite", () => {
244248
});
245249
}).timeout(1000);
246250
});
247-
}).timeout(10000);
251+
}).timeout(15000);

test/integration-tests/configuration.test.ts

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,9 +12,12 @@
1212
//
1313
//===----------------------------------------------------------------------===//
1414

15-
import * as vscode from "vscode";
1615
import * as path from "path";
17-
import { activateExtensionForSuite, updateSettings } from "./utilities/testutilities";
16+
import {
17+
activateExtensionForSuite,
18+
getRootWorkspaceFolder,
19+
updateSettings,
20+
} from "./utilities/testutilities";
1821
import { expect } from "chai";
1922
import { afterEach } from "mocha";
2023
import configuration from "../../src/configuration";
@@ -47,7 +50,7 @@ suite("Configuration Test Suite", function () {
4750
expect(task.definition.args).to.not.be.undefined;
4851
const index = task.definition.args.indexOf("--scratch-path");
4952
expect(task.definition.args[index + 1]).to.equal(
50-
vscode.workspace.workspaceFolders?.at(0)?.uri.fsPath + "/somepath"
53+
getRootWorkspaceFolder()?.uri.fsPath + "/somepath"
5154
);
5255
});
5356

@@ -56,7 +59,7 @@ suite("Configuration Test Suite", function () {
5659
"swift.buildPath": "${workspaceFolder}${pathSeparator}${workspaceFolderBasename}",
5760
});
5861

59-
const basePath = vscode.workspace.workspaceFolders?.at(0)?.uri.fsPath;
62+
const basePath = getRootWorkspaceFolder()?.uri.fsPath;
6063
const baseName = path.basename(basePath ?? "");
6164
const sep = path.sep;
6265
expect(configuration.buildPath).to.equal(`${basePath}${sep}${baseName}`);

test/integration-tests/utilities/testutilities.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,8 @@ import { resetBuildAllTaskCache } from "../../../src/tasks/SwiftTaskProvider";
2929

3030
const codeWorkspaceFolders = ["defaultPackage", "diagnostics", "command-plugin"];
3131

32-
function getRootWorkspaceFolder(): vscode.WorkspaceFolder {
33-
const result = vscode.workspace.workspaceFolders?.at(0);
32+
export function getRootWorkspaceFolder(): vscode.WorkspaceFolder {
33+
const result = vscode.workspace.workspaceFolders?.find(f => f.name === "test");
3434
assert(result, "No workspace folders were opened for the tests to use");
3535
return result;
3636
}

0 commit comments

Comments
 (0)