Files
windmill/cli/test/init_no_git_sync.test.ts
Ruben Fiszel 5b97092997 feat: unify CLI config to workspaces, deprecate gitBranches/environments (#8767)
* refactor: unify CLI config to workspaces, deprecate gitBranches/environments

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* chore: update frontend examples and regenerate system prompts for workspaces config

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* test: update test files to use workspaces config instead of gitBranches

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: handle --branch with --base-url correctly in sync pull/push

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* feat: warn when --workspace overrides auto-detected branch or misses config entry

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* feat: show reason why workspace was selected in log message

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* docs: clarify specificItems file naming uses gitBranch as suffix

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* refactor: rename branch-specific to workspace-specific, use workspace name as file suffix

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* refactor: rename branch-specific to workspace-specific, add comprehensive integration tests

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* refactor: simplify bind and init to be workspace-centric

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* refactor: make bind/unbind interactive with --workspace and --branch flags

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* refactor: make bind interactive with profile selection, workspace name, and optional branch

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* refactor: init offers to bind workspace using same flow as wmill workspace bind

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: skip backend git-sync check in init when no workspace was bound

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: skip all API calls in init when no workspace was bound

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* feat: log when RT namespace is skipped, offer to generate it after bind

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: warn when no workspace bound during init

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: init git-sync check uses bound workspace, not active profile

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: init uses selected profile directly, avoids re-resolving and duplicate prompt

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: init skips requireLogin, uses bound profile token directly

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* feat: auto-pick or prompt workspace from config when no branch matches

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: show configured workspaces list and bind hint in resolution messages

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: cache bound profile to avoid duplicate profile selection prompts in init

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: hoist boundProfile scope, add 2 comprehensive integration tests covering all flows

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: rt.d.ts prompt defaults to no when file exists, better description

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: remove empty overrides from generated config, add specificItems hint

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: add inline comments for non-trivial fields, add overrides/promotionOverrides hints to bound workspaces

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* chore: regenerate system prompts

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
2026-04-09 19:46:34 +00:00

188 lines
6.5 KiB
TypeScript

/**
* Test to verify that wmill init handles workspaces with no git-sync settings correctly
* This creates a unit test that directly tests the logic without needing a backend
*/
import { expect, test } from "bun:test";
import { readFile } from "node:fs/promises";
import { DEFAULT_SYNC_OPTIONS } from "../src/core/conf.ts";
import { withTestBackend } from "./test_backend.ts";
import { shouldSkipOnCI } from "./cargo_backend.ts";
import { addWorkspace } from "../workspace.ts";
// Mock the workspace object
const mockWorkspace = {
remote: 'https://app.windmill.dev/',
workspaceId: 'test-workspace',
name: 'test-workspace'
};
// Simulate the createWorkspaceProfile function logic for the "no repositories" case
function createWorkspaceProfileNoRepos(workspace: any): any {
const workspaceProfile: any = {
baseUrl: workspace.remote,
workspaceId: workspace.workspaceId,
};
// Simulate the case where listRepositories returns empty array
const repositories: any[] = [];
if (repositories.length === 0) {
console.log(`No git repositories found in workspace '${workspace.workspaceId}'`);
// This is the fix: include default sync settings when no repositories exist
Object.assign(workspaceProfile, DEFAULT_SYNC_OPTIONS);
return workspaceProfile;
}
return workspaceProfile;
}
test("Init: createWorkspaceProfile includes defaults when no repositories exist", () => {
console.log('Testing init logic for workspace with no git-sync repositories...');
const workspaceProfile = createWorkspaceProfileNoRepos(mockWorkspace);
console.log('Generated workspace profile:', JSON.stringify(workspaceProfile, null, 2));
// Verify basic workspace info
expect(workspaceProfile.baseUrl).toEqual('https://app.windmill.dev/');
expect(workspaceProfile.workspaceId).toEqual('test-workspace');
// Verify default sync settings are included
expect(Array.isArray(workspaceProfile.includes)).toBeTruthy();
expect(workspaceProfile.includes.length).toEqual(1);
expect(workspaceProfile.includes[0]).toEqual('f/**');
expect(Array.isArray(workspaceProfile.excludes)).toBeTruthy();
expect(workspaceProfile.excludes.length).toEqual(0);
expect(workspaceProfile.defaultTs).toEqual('bun');
console.log('Workspace profile correctly includes default sync settings when no repositories exist');
});
test("Init: verify DEFAULT_SYNC_OPTIONS has expected values", () => {
console.log('Verifying DEFAULT_SYNC_OPTIONS contains expected values...');
console.log('DEFAULT_SYNC_OPTIONS:', JSON.stringify(DEFAULT_SYNC_OPTIONS, null, 2));
// Verify the default options include the expected f/** pattern
expect(Array.isArray(DEFAULT_SYNC_OPTIONS.includes)).toBeTruthy();
expect(DEFAULT_SYNC_OPTIONS.includes.length).toEqual(1);
expect(DEFAULT_SYNC_OPTIONS.includes[0]).toEqual('f/**');
expect(Array.isArray(DEFAULT_SYNC_OPTIONS.excludes)).toBeTruthy();
expect(DEFAULT_SYNC_OPTIONS.excludes.length).toEqual(0);
expect(DEFAULT_SYNC_OPTIONS.defaultTs).toEqual('bun');
console.log('DEFAULT_SYNC_OPTIONS has expected values');
});
test.skipIf(shouldSkipOnCI())("Init: --use-backend flag applies git-sync settings", async () => {
await withTestBackend(async (backend, tempDir) => {
// Set up workspace
const testWorkspace = {
remote: backend.baseUrl,
workspaceId: backend.workspace,
name: backend.workspace,
token: backend.token
};
await addWorkspace(testWorkspace, { force: true, configDir: backend.testConfigDir });
// First create the git repository resource that the git-sync will reference
await backend.createAdditionalGitRepo("u/test/init_repo", "Test init repository");
// Configure backend with git-sync settings
await backend.updateGitSyncConfig({
git_sync_settings: {
repositories: [{
git_repo_resource_path: "u/test/init_repo",
script_path: "f/**",
group_by_folder: false,
use_individual_branch: false,
settings: {
include_path: ["f/backend/**"],
include_type: ["script", "flow"],
exclude_path: ["*.test.ts"],
extra_include_path: ["g/**"]
}
}]
}
});
// Run init with --use-backend flag
const result = await backend.runCLICommand([
'init',
'--use-backend',
'--repository', 'u/test/init_repo'
], tempDir);
expect(result.code).toEqual(0);
// Verify wmill.yaml was created with backend settings
const wmillYaml = await readFile(`${tempDir}/wmill.yaml`, "utf-8");
// Should have backend-applied settings written to top-level (not overrides)
expect(wmillYaml).toContain("f/backend/**");
expect(wmillYaml).toContain("*.test.ts");
expect(wmillYaml).toContain("g/**");
// Should have empty overrides section for consistency
expect(wmillYaml).toContain("workspaces:");
});
});
test.skipIf(shouldSkipOnCI())("Init: --use-default bypasses backend settings check", async () => {
await withTestBackend(async (backend, tempDir) => {
// Set up workspace
const testWorkspace = {
remote: backend.baseUrl,
workspaceId: backend.workspace,
name: backend.workspace,
token: backend.token
};
await addWorkspace(testWorkspace, { force: true, configDir: backend.testConfigDir });
// Create git repository resource
await backend.createAdditionalGitRepo("u/test/ignored_repo", "Test ignored repository");
// Configure backend with git-sync settings
await backend.updateGitSyncConfig({
git_sync_settings: {
repositories: [{
git_repo_resource_path: "u/test/ignored_repo",
script_path: "f/**",
group_by_folder: false,
use_individual_branch: false,
settings: {
include_path: ["f/should-be-ignored/**"],
include_type: ["script"],
exclude_path: [],
extra_include_path: []
}
}]
}
});
// Run init with --use-default (should ignore backend)
const result = await backend.runCLICommand([
'init',
'--use-default'
], tempDir);
expect(result.code).toEqual(0);
// Verify wmill.yaml was created with default settings only
const wmillYaml = await readFile(`${tempDir}/wmill.yaml`, "utf-8");
// Should have default settings, not backend settings
expect(wmillYaml).toContain("includes:\n - f/**");
expect(wmillYaml).toContain("defaultTs: bun");
// Should NOT have backend-specific settings
expect(wmillYaml.includes("f/should-be-ignored/**")).toEqual(false);
expect(wmillYaml).toContain("workspaces:");
});
});