kalugin66/windmill
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

don't print success message when workspace add is cancelled (#8003)

Browse Source 
Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
centdix
2026-02-18 17:35:49 +01:00
committed by GitHub
parent d2d08f8817
commit 02e50c915e
2 changed files with 46 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

39
cli/test/workspace_conflicts.test.ts
View File

@@ -121,6 +121,45 @@ Deno.test("addWorkspace: allows same workspace (name, remote, workspaceId) with
});
});
Deno.test("addWorkspace: returns true on successful add", async () => {
await withTestConfig(async (testConfigDir) => {
await clearTestRemotes(testConfigDir);
const workspace = {
name: "return_test",
remote: "http://localhost:8001/",
workspaceId: "test",
token: "token1"
};
const result = await addWorkspace(workspace, { force: true, configDir: testConfigDir });
assertEquals(result, true);
});
});
Deno.test("addWorkspace: returns true when force-overwriting conflict", async () => {
await withTestConfig(async (testConfigDir) => {
await clearTestRemotes(testConfigDir);
const workspace1 = {
name: "force_test",
remote: "http://localhost:8001/",
workspaceId: "workspace1",
token: "token1"
};
await addWorkspace(workspace1, { force: true, configDir: testConfigDir });
const workspace2 = {
name: "force_test",
remote: "http://localhost:8002/",
workspaceId: "workspace2",
token: "token2"
};
const result = await addWorkspace(workspace2, { force: true, configDir: testConfigDir });
assertEquals(result, true);
});
});
Deno.test("addWorkspace: allows different workspaces on different remotes", async () => {
await withTestConfig(async (testConfigDir) => {
await clearTestRemotes(testConfigDir);