* 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>
145 lines
4.6 KiB
TypeScript
145 lines
4.6 KiB
TypeScript
import { expect, test } from "bun:test";
|
|
import { getEffectiveSettings, type SyncOptions } from "../src/core/conf.ts";
|
|
|
|
// =============================================================================
|
|
// CONF.TS BRANCH OVERRIDE TESTS
|
|
// Tests for getEffectiveSettings with branchOverride parameter
|
|
// =============================================================================
|
|
|
|
test("getEffectiveSettings: applies branch overrides when branchOverride is provided", async () => {
|
|
const config: SyncOptions = {
|
|
defaultTs: "bun",
|
|
includes: ["f/**"],
|
|
gitBranches: {
|
|
staging: {
|
|
overrides: {
|
|
includes: ["staging/**"],
|
|
skipVariables: true,
|
|
},
|
|
},
|
|
production: {
|
|
overrides: {
|
|
includes: ["prod/**"],
|
|
skipSecrets: true,
|
|
},
|
|
},
|
|
},
|
|
};
|
|
|
|
// Test with staging branch override
|
|
const stagingSettings = await getEffectiveSettings(config, undefined, true, true, "staging");
|
|
expect(stagingSettings.includes).toEqual(["staging/**"]);
|
|
expect(stagingSettings.skipVariables).toEqual(true);
|
|
expect(stagingSettings.skipSecrets).toEqual(undefined);
|
|
|
|
// Test with production branch override
|
|
const prodSettings = await getEffectiveSettings(config, undefined, true, true, "production");
|
|
expect(prodSettings.includes).toEqual(["prod/**"]);
|
|
expect(prodSettings.skipSecrets).toEqual(true);
|
|
expect(prodSettings.skipVariables).toEqual(undefined);
|
|
});
|
|
|
|
test("getEffectiveSettings: uses top-level settings when branchOverride has no overrides", async () => {
|
|
const config: SyncOptions = {
|
|
defaultTs: "bun",
|
|
includes: ["f/**"],
|
|
skipVariables: true,
|
|
gitBranches: {
|
|
staging: {
|
|
// No overrides defined
|
|
},
|
|
},
|
|
};
|
|
|
|
const settings = await getEffectiveSettings(config, undefined, true, true, "staging");
|
|
expect(settings.includes).toEqual(["f/**"]);
|
|
expect(settings.skipVariables).toEqual(true);
|
|
expect(settings.defaultTs).toEqual("bun");
|
|
});
|
|
|
|
test("getEffectiveSettings: uses top-level settings for unknown branch", async () => {
|
|
const config: SyncOptions = {
|
|
defaultTs: "bun",
|
|
includes: ["f/**"],
|
|
gitBranches: {
|
|
staging: {
|
|
overrides: {
|
|
includes: ["staging/**"],
|
|
},
|
|
},
|
|
},
|
|
};
|
|
|
|
const settings = await getEffectiveSettings(config, undefined, true, true, "nonexistent");
|
|
expect(settings.includes).toEqual(["f/**"]);
|
|
expect(settings.defaultTs).toEqual("bun");
|
|
});
|
|
|
|
test("getEffectiveSettings: promotionOverrides take precedence when promotion specified", async () => {
|
|
const config: SyncOptions = {
|
|
defaultTs: "bun",
|
|
includes: ["f/**"],
|
|
gitBranches: {
|
|
production: {
|
|
overrides: {
|
|
includes: ["prod/**"],
|
|
},
|
|
promotionOverrides: {
|
|
includes: ["promoted/**"],
|
|
skipVariables: true,
|
|
},
|
|
},
|
|
},
|
|
};
|
|
|
|
// Test without promotion flag - should use regular overrides
|
|
const normalSettings = await getEffectiveSettings(config, undefined, true, true, "production");
|
|
expect(normalSettings.includes).toEqual(["prod/**"]);
|
|
expect(normalSettings.skipVariables).toEqual(undefined);
|
|
|
|
// Test with promotion flag - should use promotionOverrides
|
|
const promoSettings = await getEffectiveSettings(config, "production", true, true);
|
|
expect(promoSettings.includes).toEqual(["promoted/**"]);
|
|
expect(promoSettings.skipVariables).toEqual(true);
|
|
});
|
|
|
|
test("getEffectiveSettings: branchOverride works without gitBranches config", async () => {
|
|
const config: SyncOptions = {
|
|
defaultTs: "bun",
|
|
includes: ["f/**"],
|
|
};
|
|
|
|
// Should not throw even with branchOverride but no gitBranches
|
|
const settings = await getEffectiveSettings(config, undefined, true, true, "staging");
|
|
expect(settings.includes).toEqual(["f/**"]);
|
|
expect(settings.defaultTs).toEqual("bun");
|
|
});
|
|
|
|
test("getEffectiveSettings: preserves all top-level settings in merged result", async () => {
|
|
const config: SyncOptions = {
|
|
defaultTs: "bun",
|
|
includes: ["f/**"],
|
|
excludes: ["*.test.ts"],
|
|
skipVariables: false,
|
|
skipResources: false,
|
|
skipFlows: false,
|
|
parallel: 4,
|
|
gitBranches: {
|
|
staging: {
|
|
overrides: {
|
|
skipVariables: true, // Override just this one
|
|
},
|
|
},
|
|
},
|
|
};
|
|
|
|
const settings = await getEffectiveSettings(config, undefined, true, true, "staging");
|
|
expect(settings.defaultTs).toEqual("bun");
|
|
expect(settings.includes).toEqual(["f/**"]);
|
|
expect(settings.excludes).toEqual(["*.test.ts"]);
|
|
expect(settings.skipVariables).toEqual(true); // Overridden
|
|
expect(settings.skipResources).toEqual(false);
|
|
expect(settings.skipFlows).toEqual(false);
|
|
expect(settings.parallel).toEqual(4);
|
|
});
|