* fix: only enable EE features in test backend when license key is available Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: skip EE tests without license key and exclude test-skills from test discovery Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: unskip passing tests and add duplicate (remote, workspaceId) check in addWorkspace Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * refactor(cli): migrate from Deno APIs to Node.js/Bun-compatible APIs Replace Deno-specific APIs with Node.js equivalents across the entire CLI codebase to enable running on Node.js/Bun. Switch build system from dnt to bun, update imports from jsr:/npm: prefixed to bare specifiers, and add package.json/tsconfig.json for the Node.js ecosystem. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * all * test(cli): expand test coverage with new integration and unit tests Add standalone_commands.test.ts covering folder list, schedule list, resource-type list/push/update, script show/run/bootstrap, and user commands. Add unit tests for filePathExtensionFromContentType and removeExtensionToPath. Add git_unit, local_encryption_unit, resource_folders_unit, and settings_unit test files. Fix schedule cron expressions (6-field format), add includeSchedules flag, improve test setup with pre-build and auto-cleanup, and support TEST_CLI_RUNTIME=node. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(cli): replace Deno.readFile with node:fs in WASM loaders and add schema parsing tests Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * refactor(cli): switch WASM parsers from local files to npm packages Use published windmill-parser-wasm-* npm packages instead of local wasm/ files. A loadParser() helper uses createRequire to resolve the .wasm binary from node_modules and passes it to init() via readFileSync, avoiding fetch() and Deno.readFile() patches. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * test(cli): add coverage for --locks-required lint feature Add 15 tests covering the lock-checking functionality merged from main: - checkMissingLocks: standalone scripts (python, bun, bash), inline lock file resolution (valid, empty, missing), flow inline rawscripts (with/without locks, nested forloopflow), app inline scripts, raw apps without backend folder - runLint --locks-required integration: reports issues when locks missing, skips checks when flag absent, passes when locks exist Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * ci(cli): replace Deno with Bun in CI workflows - cli-tests.yml: remove Deno setup, use `bun test` instead of `deno test`, add `bun install` step for dependency installation - npm_on_release.yml: replace Deno setup with Bun setup for CLI publishing - build.sh: add `bun install` before building so CI has dependencies Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(cli): pre-start backend in test preload and remove Deno test leftovers Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(cli): normalize path separators for Windows compatibility Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * more tests + windows * ci(cli): use Blacksmith runner for Windows tests Switch test-windows job from windows-latest to blacksmith-16vcpu-windows-2025 for faster CI execution. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(cli): fix Windows path separator expectations in unit tests buildMetadataPath and extractResourceName normalize to forward slashes internally, so tests should not expect platform-specific separators in their output. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(cli): fix Windows CI test failures for dev_server and script_run Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(cli): set BUN_PATH and NODE_BIN_PATH for backend worker on Windows Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * ci(cli): add SSH debug step on Windows test failure Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(cli): use native path separators for ignore check in dev mode on Windows Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
188 lines
6.5 KiB
TypeScript
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("gitBranches: {}");
|
|
});
|
|
});
|
|
|
|
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("gitBranches: {}");
|
|
});
|
|
});
|