Skip to content

Commit 7574471

Browse files
authored
Try to fix package reset tests (#1621)
* Try to fix package reset tests Issue: #1316 * Don't stop server * Fail if dependency not found * Make sure tree provider picks up task to grab focus * Remove extra iterations * Remove console.log statements
1 parent 8d0f0c3 commit 7574471

File tree

1 file changed

+11
-9
lines changed

1 file changed

+11
-9
lines changed

test/integration-tests/commands/dependency.test.ts

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -19,11 +19,11 @@ import { testAssetUri } from "../../fixtures";
1919
import { FolderContext } from "../../../src/FolderContext";
2020
import { WorkspaceContext } from "../../../src/WorkspaceContext";
2121
import { Commands } from "../../../src/commands";
22-
import { activateExtensionForSuite, folderInRootWorkspace } from "../utilities/testutilities";
23-
import { executeTaskAndWaitForResult } from "../../utilities/tasks";
22+
import { activateExtensionForSuite, findWorkspaceFolder } from "../utilities/testutilities";
23+
import { executeTaskAndWaitForResult, waitForNoRunningTasks } from "../../utilities/tasks";
2424
import { createBuildAllTask } from "../../../src/tasks/SwiftTaskProvider";
2525

26-
suite("Dependency Commmands Test Suite", function () {
26+
suite("Dependency Commmands Test Suite @slow", function () {
2727
// full workflow's interaction with spm is longer than the default timeout
2828
// 3 minutes for each test should be more than enough
2929
this.timeout(3 * 60 * 1000);
@@ -34,7 +34,7 @@ suite("Dependency Commmands Test Suite", function () {
3434
activateExtensionForSuite({
3535
async setup(ctx) {
3636
workspaceContext = ctx;
37-
depsContext = await folderInRootWorkspace("dependencies", workspaceContext);
37+
depsContext = findWorkspaceFolder("dependencies", workspaceContext)!;
3838
},
3939
testAssets: ["dependencies"],
4040
});
@@ -53,14 +53,13 @@ suite("Dependency Commmands Test Suite", function () {
5353
expect(result).to.be.true;
5454
});
5555

56-
// Skipping: https://github.com/swiftlang/vscode-swift/issues/1316
5756
suite("Swift: Use Local Dependency", function () {
5857
let treeProvider: ProjectPanelProvider;
5958

6059
setup(async () => {
61-
await workspaceContext.focusFolder(depsContext);
62-
await executeTaskAndWaitForResult(await createBuildAllTask(depsContext));
60+
await waitForNoRunningTasks();
6361
treeProvider = new ProjectPanelProvider(workspaceContext);
62+
await executeTaskAndWaitForResult(await createBuildAllTask(depsContext));
6463
});
6564

6665
teardown(() => {
@@ -72,7 +71,9 @@ suite("Dependency Commmands Test Suite", function () {
7271
const header = headers.find(n => n.name === "Dependencies") as PackageNode;
7372
expect(header).to.not.be.undefined;
7473
const children = await header.getChildren();
75-
return children.find(n => n.name === "swift-markdown") as PackageNode;
74+
return children.find(
75+
n => n.name.toLocaleLowerCase() === "swift-markdown"
76+
) as PackageNode;
7677
}
7778

7879
// Wait for the dependency to switch to the expected state.
@@ -87,11 +88,12 @@ suite("Dependency Commmands Test Suite", function () {
8788
}
8889
await new Promise(resolve => setTimeout(resolve, 1000));
8990
}
91+
throw Error(`Could not find dependency with state "${state}"`);
9092
}
9193

9294
async function useLocalDependencyTest() {
9395
// spm edit with user supplied local version of dependency
94-
const item = await getDependency();
96+
const item = await getDependencyInState("remote");
9597
const localDep = testAssetUri("swift-markdown");
9698
const result = await vscode.commands.executeCommand(
9799
Commands.USE_LOCAL_DEPENDENCY,

0 commit comments

Comments
 (0)