Files
windmill/cli/test/specific_items_unit.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

858 lines
35 KiB
TypeScript

import { expect, test } from "bun:test";
// =============================================================================
// SPECIFIC ITEMS UNIT TESTS
// Tests for workspace-specific file path functions (no Docker required)
// =============================================================================
// Import the functions we need to test
import {
isSpecificItem,
isItemTypeConfigured,
toWorkspaceSpecificPath,
fromWorkspaceSpecificPath,
isWorkspaceSpecificFile,
isCurrentWorkspaceFile,
getWorkspaceSpecificPath,
getSpecificItemsForCurrentBranch,
} from "../src/core/specific_items.ts";
import type { SpecificItemsConfig } from "../src/core/specific_items.ts";
// =============================================================================
// toWorkspaceSpecificPath TESTS
// =============================================================================
test("toWorkspaceSpecificPath: converts variable path to workspace-specific", () => {
const result = toWorkspaceSpecificPath("f/test.variable.yaml", "main");
expect(result).toEqual("f/test.main.variable.yaml");
});
test("toWorkspaceSpecificPath: converts resource path to workspace-specific", () => {
const result = toWorkspaceSpecificPath("u/admin/db.resource.yaml", "develop");
expect(result).toEqual("u/admin/db.develop.resource.yaml");
});
test("toWorkspaceSpecificPath: converts trigger path to workspace-specific", () => {
const result = toWorkspaceSpecificPath("f/my_trigger.http_trigger.yaml", "feature-x");
expect(result).toEqual("f/my_trigger.feature-x.http_trigger.yaml");
});
test("toWorkspaceSpecificPath: sanitizes branch names with slashes", () => {
const result = toWorkspaceSpecificPath("f/test.variable.yaml", "feature/my-feature");
expect(result).toEqual("f/test.feature_my-feature.variable.yaml");
});
test("toWorkspaceSpecificPath: sanitizes branch names with dots", () => {
const result = toWorkspaceSpecificPath("f/test.variable.yaml", "release.1.0");
expect(result).toEqual("f/test.release_1_0.variable.yaml");
});
test("toWorkspaceSpecificPath: leaves non-specific files unchanged", () => {
const result = toWorkspaceSpecificPath("f/script.ts", "main");
expect(result).toEqual("f/script.ts");
});
test("toWorkspaceSpecificPath: handles resource files with extensions", () => {
const result = toWorkspaceSpecificPath("f/config.resource.file.json", "main");
expect(result).toEqual("f/config.main.resource.file.json");
});
// =============================================================================
// fromWorkspaceSpecificPath TESTS
// =============================================================================
test("fromWorkspaceSpecificPath: converts workspace-specific variable back to base", () => {
const result = fromWorkspaceSpecificPath("f/test.main.variable.yaml", "main");
expect(result).toEqual("f/test.variable.yaml");
});
test("fromWorkspaceSpecificPath: converts workspace-specific resource back to base", () => {
const result = fromWorkspaceSpecificPath("u/admin/db.develop.resource.yaml", "develop");
expect(result).toEqual("u/admin/db.resource.yaml");
});
test("fromWorkspaceSpecificPath: converts workspace-specific trigger back to base", () => {
const result = fromWorkspaceSpecificPath("f/my_trigger.feature-x.http_trigger.yaml", "feature-x");
expect(result).toEqual("f/my_trigger.http_trigger.yaml");
});
test("fromWorkspaceSpecificPath: handles sanitized branch names", () => {
const result = fromWorkspaceSpecificPath("f/test.feature_my-feature.variable.yaml", "feature/my-feature");
expect(result).toEqual("f/test.variable.yaml");
});
test("fromWorkspaceSpecificPath: returns unchanged if not workspace-specific", () => {
const result = fromWorkspaceSpecificPath("f/test.variable.yaml", "main");
expect(result).toEqual("f/test.variable.yaml");
});
test("fromWorkspaceSpecificPath: handles resource files with extensions", () => {
const result = fromWorkspaceSpecificPath("f/config.main.resource.file.json", "main");
expect(result).toEqual("f/config.resource.file.json");
});
// =============================================================================
// isSpecificItem TESTS
// =============================================================================
test("isSpecificItem: returns false when specificItems is undefined", () => {
const result = isSpecificItem("f/test.variable.yaml", undefined);
expect(result).toEqual(false);
});
test("isSpecificItem: matches variable paths with glob pattern", () => {
const config: SpecificItemsConfig = {
variables: ["f/**"],
};
expect(isSpecificItem("f/test.variable.yaml", config)).toEqual(true);
expect(isSpecificItem("u/admin/test.variable.yaml", config)).toEqual(false);
});
test("isSpecificItem: matches resource paths with glob pattern", () => {
const config: SpecificItemsConfig = {
resources: ["u/admin/**"],
};
expect(isSpecificItem("u/admin/db.resource.yaml", config)).toEqual(true);
expect(isSpecificItem("f/db.resource.yaml", config)).toEqual(false);
});
test("isSpecificItem: matches trigger paths with glob pattern", () => {
const config: SpecificItemsConfig = {
triggers: ["f/triggers/**"],
};
expect(isSpecificItem("f/triggers/my.http_trigger.yaml", config)).toEqual(true);
expect(isSpecificItem("u/admin/my.http_trigger.yaml", config)).toEqual(false);
});
test("isSpecificItem: matches multiple patterns", () => {
const config: SpecificItemsConfig = {
variables: ["f/**", "g/**"],
};
expect(isSpecificItem("f/test.variable.yaml", config)).toEqual(true);
expect(isSpecificItem("g/test.variable.yaml", config)).toEqual(true);
expect(isSpecificItem("u/admin/test.variable.yaml", config)).toEqual(false);
});
test("isSpecificItem: handles exact path patterns", () => {
const config: SpecificItemsConfig = {
variables: ["f/specific.variable.yaml"],
};
expect(isSpecificItem("f/specific.variable.yaml", config)).toEqual(true);
expect(isSpecificItem("f/other.variable.yaml", config)).toEqual(false);
});
// =============================================================================
// isWorkspaceSpecificFile TESTS
// =============================================================================
test("isWorkspaceSpecificFile: detects workspace-specific variable files", () => {
expect(isWorkspaceSpecificFile("f/test.main.variable.yaml")).toEqual(true);
expect(isWorkspaceSpecificFile("f/test.develop.variable.yaml")).toEqual(true);
expect(isWorkspaceSpecificFile("f/test.feature_branch.variable.yaml")).toEqual(true);
});
test("isWorkspaceSpecificFile: detects workspace-specific resource files", () => {
expect(isWorkspaceSpecificFile("u/admin/db.main.resource.yaml")).toEqual(true);
expect(isWorkspaceSpecificFile("u/admin/db.staging.resource.yaml")).toEqual(true);
});
test("isWorkspaceSpecificFile: detects workspace-specific trigger files", () => {
expect(isWorkspaceSpecificFile("f/my.main.http_trigger.yaml")).toEqual(true);
expect(isWorkspaceSpecificFile("f/my.develop.kafka_trigger.yaml")).toEqual(true);
expect(isWorkspaceSpecificFile("f/my.main.websocket_trigger.yaml")).toEqual(true);
});
test("isWorkspaceSpecificFile: returns false for non-workspace-specific files", () => {
expect(isWorkspaceSpecificFile("f/test.variable.yaml")).toEqual(false);
expect(isWorkspaceSpecificFile("u/admin/db.resource.yaml")).toEqual(false);
expect(isWorkspaceSpecificFile("f/my.http_trigger.yaml")).toEqual(false);
expect(isWorkspaceSpecificFile("f/script.ts")).toEqual(false);
});
test("isWorkspaceSpecificFile: handles resource files with extensions", () => {
expect(isWorkspaceSpecificFile("f/config.main.resource.file.json")).toEqual(true);
expect(isWorkspaceSpecificFile("f/config.resource.file.json")).toEqual(false);
});
// =============================================================================
// ROUND-TRIP TESTS
// =============================================================================
test("round-trip: variable file path conversion", () => {
const original = "f/my/nested/config.variable.yaml";
const branch = "feature/test-branch";
const branchSpecific = toWorkspaceSpecificPath(original, branch);
const restored = fromWorkspaceSpecificPath(branchSpecific, branch);
expect(restored).toEqual(original);
});
test("round-trip: resource file path conversion", () => {
const original = "u/admin/database.resource.yaml";
const branch = "develop";
const branchSpecific = toWorkspaceSpecificPath(original, branch);
const restored = fromWorkspaceSpecificPath(branchSpecific, branch);
expect(restored).toEqual(original);
});
test("round-trip: trigger file path conversion", () => {
const original = "f/webhooks/handler.http_trigger.yaml";
const branch = "main";
const branchSpecific = toWorkspaceSpecificPath(original, branch);
const restored = fromWorkspaceSpecificPath(branchSpecific, branch);
expect(restored).toEqual(original);
});
test("round-trip: resource file with extension", () => {
const original = "f/configs/settings.resource.file.ini";
const branch = "release/v1.0";
const branchSpecific = toWorkspaceSpecificPath(original, branch);
const restored = fromWorkspaceSpecificPath(branchSpecific, branch);
expect(restored).toEqual(original);
});
// =============================================================================
// BRANCH OVERRIDE TESTS (for --branch flag functionality)
// These tests validate that functions work correctly with explicit branch override
// =============================================================================
test("branchOverride: getWorkspaceSpecificPath with override returns workspace-specific path", () => {
// This test verifies that when branchOverride is provided, the function uses it
// instead of detecting the current git branch
const config: SpecificItemsConfig = {
variables: ["f/**"],
};
// When override is provided, it should return the workspace-specific path even outside git repo
const result = getWorkspaceSpecificPath("f/test.variable.yaml", config, "staging");
expect(result).toEqual("f/test.staging.variable.yaml");
});
test("branchOverride: getWorkspaceSpecificPath without override and not in git repo returns undefined", () => {
const config: SpecificItemsConfig = {
variables: ["f/**"],
};
// Without override and outside git repo (or if git returns null), should return undefined
// Note: This test's behavior depends on whether we're in a git repo
const result = getWorkspaceSpecificPath("f/test.variable.yaml", config);
// In a git repo, this would return a workspace-specific path; outside, it would be undefined
// We test the override case above which is deterministic
});
test("branchOverride: isCurrentWorkspaceFile with override uses provided branch", () => {
// Test that isCurrentWorkspaceFile uses the override branch instead of git detection
const result = isCurrentWorkspaceFile("f/test.staging.variable.yaml", "staging");
expect(result).toEqual(true);
// Should return false for different branch
const resultOther = isCurrentWorkspaceFile("f/test.staging.variable.yaml", "production");
expect(resultOther).toEqual(false);
// Should return false for non-workspace-specific file
const resultNonSpecific = isCurrentWorkspaceFile("f/test.variable.yaml", "staging");
expect(resultNonSpecific).toEqual(false);
});
test("branchOverride: isCurrentWorkspaceFile with override handles sanitized branch names", () => {
// Test with branch names that get sanitized
const result = isCurrentWorkspaceFile("f/test.feature_my-branch.variable.yaml", "feature/my-branch");
expect(result).toEqual(true);
// Different sanitized branch should return false
const resultOther = isCurrentWorkspaceFile("f/test.feature_my-branch.variable.yaml", "feature/other-branch");
expect(resultOther).toEqual(false);
});
test("branchOverride: getSpecificItemsForCurrentBranch with override returns correct config", () => {
// Test that getSpecificItemsForCurrentBranch uses the override branch
const config = {
workspaces: {
staging: {
specificItems: {
variables: ["f/**"],
resources: ["u/admin/**"],
},
},
production: {
specificItems: {
variables: ["g/**"],
},
},
commonSpecificItems: {
triggers: ["f/webhooks/**"],
},
},
};
const stagingItems = getSpecificItemsForCurrentBranch(config as any, "staging");
expect(stagingItems?.variables).toEqual(["f/**"]);
expect(stagingItems?.resources).toEqual(["u/admin/**"]);
expect(stagingItems?.triggers).toEqual(["f/webhooks/**"]); // From common
const productionItems = getSpecificItemsForCurrentBranch(config as any, "production");
expect(productionItems?.variables).toEqual(["g/**"]);
expect(productionItems?.resources).toEqual(undefined);
expect(productionItems?.triggers).toEqual(["f/webhooks/**"]); // From common
});
test("branchOverride: getSpecificItemsForCurrentBranch with non-existent branch returns undefined", () => {
const config = {
workspaces: {
staging: {
specificItems: {
variables: ["f/**"],
},
},
},
};
// When the branch doesn't have specific items (and there's no common), should return undefined
const result = getSpecificItemsForCurrentBranch(config as any, "nonexistent");
expect(result).toEqual(undefined);
});
test("branchOverride: getSpecificItemsForCurrentBranch merges common and branch items", () => {
const config = {
workspaces: {
commonSpecificItems: {
variables: ["common/**"],
resources: ["shared/**"],
},
develop: {
specificItems: {
variables: ["dev/**"],
triggers: ["dev/triggers/**"],
},
},
},
};
const result = getSpecificItemsForCurrentBranch(config as any, "develop");
// Should merge common and workspace-specific
expect(result?.variables).toEqual(["common/**", "dev/**"]);
expect(result?.resources).toEqual(["shared/**"]);
expect(result?.triggers).toEqual(["dev/triggers/**"]);
});
// =============================================================================
// FOLDER BRANCH-SPECIFIC TESTS
// Format: f/folder/folder.branchName.meta.yaml
// =============================================================================
test("toWorkspaceSpecificPath: converts folder meta path to workspace-specific", () => {
// f/my_folder/folder.meta.yaml -> f/my_folder/folder.main.meta.yaml
const result = toWorkspaceSpecificPath("f/my_folder/folder.meta.yaml", "main");
expect(result).toEqual("f/my_folder/folder.main.meta.yaml");
});
test("toWorkspaceSpecificPath: converts nested folder meta path to workspace-specific", () => {
const result = toWorkspaceSpecificPath("f/parent/child/folder.meta.yaml", "develop");
expect(result).toEqual("f/parent/child/folder.develop.meta.yaml");
});
test("toWorkspaceSpecificPath: sanitizes branch name in folder path", () => {
const result = toWorkspaceSpecificPath("f/env/folder.meta.yaml", "feature/test");
expect(result).toEqual("f/env/folder.feature_test.meta.yaml");
});
test("fromWorkspaceSpecificPath: converts workspace-specific folder back to base", () => {
const result = fromWorkspaceSpecificPath("f/my_folder/folder.main.meta.yaml", "main");
expect(result).toEqual("f/my_folder/folder.meta.yaml");
});
test("fromWorkspaceSpecificPath: handles nested workspace-specific folder", () => {
const result = fromWorkspaceSpecificPath("f/parent/child/folder.develop.meta.yaml", "develop");
expect(result).toEqual("f/parent/child/folder.meta.yaml");
});
test("fromWorkspaceSpecificPath: handles sanitized branch names for folders", () => {
const result = fromWorkspaceSpecificPath("f/env/folder.feature_test.meta.yaml", "feature/test");
expect(result).toEqual("f/env/folder.meta.yaml");
});
test("isSpecificItem: matches folder paths with glob pattern", () => {
const config: SpecificItemsConfig = {
folders: ["f/env_*"],
};
expect(isSpecificItem("f/env_staging/folder.meta.yaml", config)).toEqual(true);
expect(isSpecificItem("f/env_production/folder.meta.yaml", config)).toEqual(true);
expect(isSpecificItem("f/other/folder.meta.yaml", config)).toEqual(false);
});
test("isSpecificItem: matches folder paths with exact pattern", () => {
const config: SpecificItemsConfig = {
folders: ["f/config"],
};
expect(isSpecificItem("f/config/folder.meta.yaml", config)).toEqual(true);
expect(isSpecificItem("f/other/folder.meta.yaml", config)).toEqual(false);
});
test("isWorkspaceSpecificFile: detects workspace-specific folder files", () => {
expect(isWorkspaceSpecificFile("f/my_folder/folder.main.meta.yaml")).toEqual(true);
expect(isWorkspaceSpecificFile("f/my_folder/folder.develop.meta.yaml")).toEqual(true);
expect(isWorkspaceSpecificFile("f/nested/path/folder.staging.meta.yaml")).toEqual(true);
});
test("isWorkspaceSpecificFile: returns false for non-workspace-specific folder files", () => {
expect(isWorkspaceSpecificFile("f/my_folder/folder.meta.yaml")).toEqual(false);
expect(isWorkspaceSpecificFile("f/nested/path/folder.meta.yaml")).toEqual(false);
});
test("isCurrentWorkspaceFile: detects workspace-specific folder for current branch", () => {
expect(isCurrentWorkspaceFile("f/my_folder/folder.staging.meta.yaml", "staging")).toEqual(true);
expect(isCurrentWorkspaceFile("f/my_folder/folder.staging.meta.yaml", "production")).toEqual(false);
expect(isCurrentWorkspaceFile("f/my_folder/folder.meta.yaml", "staging")).toEqual(false);
});
test("isCurrentWorkspaceFile: handles sanitized branch for folders", () => {
expect(isCurrentWorkspaceFile("f/env/folder.feature_test.meta.yaml", "feature/test")).toEqual(true);
expect(isCurrentWorkspaceFile("f/env/folder.feature_test.meta.yaml", "feature/other")).toEqual(false);
});
test("round-trip: folder meta path conversion", () => {
const original = "f/configs/env_folder/folder.meta.yaml";
const branch = "main";
const branchSpecific = toWorkspaceSpecificPath(original, branch);
expect(branchSpecific).toEqual("f/configs/env_folder/folder.main.meta.yaml");
const restored = fromWorkspaceSpecificPath(branchSpecific, branch);
expect(restored).toEqual(original);
});
test("round-trip: folder meta with sanitized branch", () => {
const original = "f/env/folder.meta.yaml";
const branch = "feature/new-env";
const branchSpecific = toWorkspaceSpecificPath(original, branch);
expect(branchSpecific).toEqual("f/env/folder.feature_new-env.meta.yaml");
const restored = fromWorkspaceSpecificPath(branchSpecific, branch);
expect(restored).toEqual(original);
});
// =============================================================================
// SETTINGS BRANCH-SPECIFIC TESTS
// =============================================================================
test("toWorkspaceSpecificPath: converts settings.yaml to workspace-specific", () => {
const result = toWorkspaceSpecificPath("settings.yaml", "main");
expect(result).toEqual("settings.main.yaml");
});
test("toWorkspaceSpecificPath: sanitizes branch name in settings path", () => {
const result = toWorkspaceSpecificPath("settings.yaml", "feature/test");
expect(result).toEqual("settings.feature_test.yaml");
});
test("fromWorkspaceSpecificPath: converts workspace-specific settings back to base", () => {
const result = fromWorkspaceSpecificPath("settings.main.yaml", "main");
expect(result).toEqual("settings.yaml");
});
test("fromWorkspaceSpecificPath: handles sanitized branch names for settings", () => {
const result = fromWorkspaceSpecificPath("settings.feature_test.yaml", "feature/test");
expect(result).toEqual("settings.yaml");
});
test("isSpecificItem: matches settings.yaml when settings is true", () => {
const config: SpecificItemsConfig = {
settings: true,
};
expect(isSpecificItem("settings.yaml", config)).toEqual(true);
});
test("isSpecificItem: does not match settings.yaml when settings is false", () => {
const config: SpecificItemsConfig = {
settings: false,
};
expect(isSpecificItem("settings.yaml", config)).toEqual(false);
});
test("isSpecificItem: does not match settings.yaml when settings is undefined", () => {
const config: SpecificItemsConfig = {
variables: ["f/**"],
};
expect(isSpecificItem("settings.yaml", config)).toEqual(false);
});
test("isWorkspaceSpecificFile: detects workspace-specific settings files", () => {
expect(isWorkspaceSpecificFile("settings.main.yaml")).toEqual(true);
expect(isWorkspaceSpecificFile("settings.develop.yaml")).toEqual(true);
expect(isWorkspaceSpecificFile("settings.feature_test.yaml")).toEqual(true);
});
test("isWorkspaceSpecificFile: returns false for non-workspace-specific settings", () => {
expect(isWorkspaceSpecificFile("settings.yaml")).toEqual(false);
});
test("isCurrentWorkspaceFile: detects workspace-specific settings for current branch", () => {
expect(isCurrentWorkspaceFile("settings.staging.yaml", "staging")).toEqual(true);
expect(isCurrentWorkspaceFile("settings.staging.yaml", "production")).toEqual(false);
expect(isCurrentWorkspaceFile("settings.yaml", "staging")).toEqual(false);
});
test("isCurrentWorkspaceFile: handles sanitized branch for settings", () => {
expect(isCurrentWorkspaceFile("settings.feature_test.yaml", "feature/test")).toEqual(true);
expect(isCurrentWorkspaceFile("settings.feature_test.yaml", "feature/other")).toEqual(false);
});
test("round-trip: settings path conversion", () => {
const original = "settings.yaml";
const branch = "main";
const branchSpecific = toWorkspaceSpecificPath(original, branch);
expect(branchSpecific).toEqual("settings.main.yaml");
const restored = fromWorkspaceSpecificPath(branchSpecific, branch);
expect(restored).toEqual(original);
});
test("round-trip: settings with sanitized branch", () => {
const original = "settings.yaml";
const branch = "release/v1.0";
const branchSpecific = toWorkspaceSpecificPath(original, branch);
expect(branchSpecific).toEqual("settings.release_v1_0.yaml");
const restored = fromWorkspaceSpecificPath(branchSpecific, branch);
expect(restored).toEqual(original);
});
// =============================================================================
// isItemTypeConfigured TESTS
// This function checks if the TYPE is configured, not whether it matches pattern.
// Used to determine if workspace-specific files should be used for this type.
// =============================================================================
test("isItemTypeConfigured: returns false when specificItems is undefined", () => {
expect(isItemTypeConfigured("f/test.variable.yaml", undefined)).toEqual(false);
expect(isItemTypeConfigured("f/test.resource.yaml", undefined)).toEqual(false);
expect(isItemTypeConfigured("f/folder/folder.meta.yaml", undefined)).toEqual(false);
expect(isItemTypeConfigured("settings.yaml", undefined)).toEqual(false);
});
test("isItemTypeConfigured: returns true for variables when variables is configured", () => {
const config: SpecificItemsConfig = {
variables: ["f/**"],
};
// Type is configured (even if path doesn't match the pattern)
expect(isItemTypeConfigured("f/test.variable.yaml", config)).toEqual(true);
expect(isItemTypeConfigured("g/other.variable.yaml", config)).toEqual(true);
});
test("isItemTypeConfigured: returns false for variables when variables is NOT configured", () => {
const config: SpecificItemsConfig = {
resources: ["f/**"],
};
expect(isItemTypeConfigured("f/test.variable.yaml", config)).toEqual(false);
});
test("isItemTypeConfigured: returns true for resources when resources is configured", () => {
const config: SpecificItemsConfig = {
resources: ["f/**"],
};
expect(isItemTypeConfigured("f/test.resource.yaml", config)).toEqual(true);
expect(isItemTypeConfigured("g/other.resource.yaml", config)).toEqual(true);
});
test("isItemTypeConfigured: returns false for resources when resources is NOT configured", () => {
const config: SpecificItemsConfig = {
variables: ["f/**"],
};
expect(isItemTypeConfigured("f/test.resource.yaml", config)).toEqual(false);
});
test("isItemTypeConfigured: returns true for triggers when triggers is configured", () => {
const config: SpecificItemsConfig = {
triggers: ["f/**"],
};
expect(isItemTypeConfigured("f/my.http_trigger.yaml", config)).toEqual(true);
expect(isItemTypeConfigured("f/my.kafka_trigger.yaml", config)).toEqual(true);
expect(isItemTypeConfigured("g/other.websocket_trigger.yaml", config)).toEqual(true);
});
test("isItemTypeConfigured: returns false for triggers when triggers is NOT configured", () => {
const config: SpecificItemsConfig = {
variables: ["f/**"],
};
expect(isItemTypeConfigured("f/my.http_trigger.yaml", config)).toEqual(false);
});
test("isItemTypeConfigured: returns true for folders when folders is configured", () => {
const config: SpecificItemsConfig = {
folders: ["f/env_*"],
};
// Type is configured (even if path doesn't match the pattern)
expect(isItemTypeConfigured("f/env_staging/folder.meta.yaml", config)).toEqual(true);
expect(isItemTypeConfigured("f/other/folder.meta.yaml", config)).toEqual(true);
});
test("isItemTypeConfigured: returns false for folders when folders is NOT configured", () => {
const config: SpecificItemsConfig = {
variables: ["f/**"],
};
expect(isItemTypeConfigured("f/my_folder/folder.meta.yaml", config)).toEqual(false);
});
test("isItemTypeConfigured: returns true for settings when settings is configured (true)", () => {
const config: SpecificItemsConfig = {
settings: true,
};
expect(isItemTypeConfigured("settings.yaml", config)).toEqual(true);
});
test("isItemTypeConfigured: returns true for settings when settings is configured (false)", () => {
// settings: false still means the type is "configured" (explicitly disabled)
const config: SpecificItemsConfig = {
settings: false,
};
expect(isItemTypeConfigured("settings.yaml", config)).toEqual(true);
});
test("isItemTypeConfigured: returns false for settings when settings is NOT configured", () => {
const config: SpecificItemsConfig = {
variables: ["f/**"],
};
expect(isItemTypeConfigured("settings.yaml", config)).toEqual(false);
});
test("isItemTypeConfigured: returns true for resource files (with extension) when resources is configured", () => {
const config: SpecificItemsConfig = {
resources: ["f/**"],
};
expect(isItemTypeConfigured("f/config.resource.file.json", config)).toEqual(true);
expect(isItemTypeConfigured("f/data.resource.file.ini", config)).toEqual(true);
});
test("isItemTypeConfigured: returns false for resource files when resources is NOT configured", () => {
const config: SpecificItemsConfig = {
variables: ["f/**"],
};
expect(isItemTypeConfigured("f/config.resource.file.json", config)).toEqual(false);
});
// =============================================================================
// BRANCH-SPECIFIC FILE FILTERING TESTS
// These tests verify the expected filtering behavior:
// - When type IS configured: use workspace-specific files, skip base files
// - When type is NOT configured: skip workspace-specific files, use base files
// =============================================================================
test("filtering logic: folders - when NOT configured, workspace-specific should be ignored", () => {
// Config has variables but NOT folders
const config: SpecificItemsConfig = {
variables: ["f/**"],
};
const basePath = "f/my_folder/folder.meta.yaml";
const branchSpecificPath = "f/my_folder/folder.main.meta.yaml";
// Folder type is NOT configured
expect(isItemTypeConfigured(basePath, config)).toEqual(false);
// Therefore, workspace-specific file detection should not apply to this type
// The sync logic should:
// 1. Skip workspace-specific folder files (isWorkspaceSpecificFile returns true)
// 2. Use the base file
expect(isWorkspaceSpecificFile(branchSpecificPath)).toEqual(true);
expect(isWorkspaceSpecificFile(basePath)).toEqual(false);
});
test("filtering logic: folders - when IS configured and matches, use workspace-specific", () => {
const config: SpecificItemsConfig = {
folders: ["f/my_folder"],
};
const basePath = "f/my_folder/folder.meta.yaml";
const branchSpecificPath = "f/my_folder/folder.main.meta.yaml";
// Folder type IS configured
expect(isItemTypeConfigured(basePath, config)).toEqual(true);
// And path matches the pattern
expect(isSpecificItem(basePath, config)).toEqual(true);
// The sync logic should:
// 1. Use workspace-specific folder file (map to base path)
// 2. Skip the base file
expect(isWorkspaceSpecificFile(branchSpecificPath)).toEqual(true);
expect(fromWorkspaceSpecificPath(branchSpecificPath, "main")).toEqual(basePath);
});
test("filtering logic: folders - when IS configured but doesn't match, skip workspace-specific", () => {
const config: SpecificItemsConfig = {
folders: ["f/env_*"], // Only env_ folders are workspace-specific
};
const basePath = "f/other_folder/folder.meta.yaml";
const branchSpecificPath = "f/other_folder/folder.main.meta.yaml";
// Folder type IS configured
expect(isItemTypeConfigured(basePath, config)).toEqual(true);
// But this path doesn't match the pattern
expect(isSpecificItem(basePath, config)).toEqual(false);
// The sync logic should:
// 1. Skip the workspace-specific file (type configured but doesn't match)
// 2. Use the base file
});
test("filtering logic: settings - when NOT configured, workspace-specific should be ignored", () => {
// Config has variables but NOT settings
const config: SpecificItemsConfig = {
variables: ["f/**"],
};
const basePath = "settings.yaml";
const branchSpecificPath = "settings.main.yaml";
// Settings type is NOT configured
expect(isItemTypeConfigured(basePath, config)).toEqual(false);
// Therefore, workspace-specific file detection should not apply to this type
expect(isWorkspaceSpecificFile(branchSpecificPath)).toEqual(true);
expect(isWorkspaceSpecificFile(basePath)).toEqual(false);
});
test("filtering logic: settings - when IS configured (true), use workspace-specific", () => {
const config: SpecificItemsConfig = {
settings: true,
};
const basePath = "settings.yaml";
const branchSpecificPath = "settings.main.yaml";
// Settings type IS configured
expect(isItemTypeConfigured(basePath, config)).toEqual(true);
// And settings: true means it matches
expect(isSpecificItem(basePath, config)).toEqual(true);
// The sync logic should use workspace-specific file
expect(isWorkspaceSpecificFile(branchSpecificPath)).toEqual(true);
expect(fromWorkspaceSpecificPath(branchSpecificPath, "main")).toEqual(basePath);
});
test("filtering logic: settings - when IS configured (false), skip workspace-specific", () => {
// settings: false means type is configured but explicitly disabled
const config: SpecificItemsConfig = {
settings: false,
};
const basePath = "settings.yaml";
const branchSpecificPath = "settings.main.yaml";
// Settings type IS configured (even though value is false)
expect(isItemTypeConfigured(basePath, config)).toEqual(true);
// But settings: false means it doesn't match (not a specific item)
expect(isSpecificItem(basePath, config)).toEqual(false);
// The sync logic should skip workspace-specific file and use base
});
test("filtering logic: variables - when NOT configured, workspace-specific should be ignored", () => {
// Config has folders but NOT variables
const config: SpecificItemsConfig = {
folders: ["f/env_*"],
};
const basePath = "f/test.variable.yaml";
const branchSpecificPath = "f/test.main.variable.yaml";
// Variable type is NOT configured
expect(isItemTypeConfigured(basePath, config)).toEqual(false);
// Branch-specific variable files should be ignored
expect(isWorkspaceSpecificFile(branchSpecificPath)).toEqual(true);
expect(isWorkspaceSpecificFile(basePath)).toEqual(false);
});
test("filtering logic: resources - when NOT configured, workspace-specific should be ignored", () => {
// Config has folders but NOT resources
const config: SpecificItemsConfig = {
folders: ["f/env_*"],
};
const basePath = "f/db.resource.yaml";
const branchSpecificPath = "f/db.main.resource.yaml";
// Resource type is NOT configured
expect(isItemTypeConfigured(basePath, config)).toEqual(false);
expect(isWorkspaceSpecificFile(branchSpecificPath)).toEqual(true);
expect(isWorkspaceSpecificFile(basePath)).toEqual(false);
});
test("filtering logic: triggers - when NOT configured, workspace-specific should be ignored", () => {
// Config has folders but NOT triggers
const config: SpecificItemsConfig = {
folders: ["f/env_*"],
};
const basePath = "f/webhook.http_trigger.yaml";
const branchSpecificPath = "f/webhook.main.http_trigger.yaml";
// Trigger type is NOT configured
expect(isItemTypeConfigured(basePath, config)).toEqual(false);
expect(isWorkspaceSpecificFile(branchSpecificPath)).toEqual(true);
expect(isWorkspaceSpecificFile(basePath)).toEqual(false);
});
// =============================================================================
// MIXED CONFIGURATION TESTS
// Tests for configs that have some types configured but not others
// =============================================================================
test("mixed config: only folders configured - other types use base files", () => {
const config: SpecificItemsConfig = {
folders: ["f/env_*"],
};
// Folders IS configured
expect(isItemTypeConfigured("f/env_staging/folder.meta.yaml", config)).toEqual(true);
expect(isSpecificItem("f/env_staging/folder.meta.yaml", config)).toEqual(true);
// Variables, resources, triggers, settings are NOT configured
expect(isItemTypeConfigured("f/test.variable.yaml", config)).toEqual(false);
expect(isItemTypeConfigured("f/db.resource.yaml", config)).toEqual(false);
expect(isItemTypeConfigured("f/hook.http_trigger.yaml", config)).toEqual(false);
expect(isItemTypeConfigured("settings.yaml", config)).toEqual(false);
});
test("mixed config: only settings configured - other types use base files", () => {
const config: SpecificItemsConfig = {
settings: true,
};
// Settings IS configured
expect(isItemTypeConfigured("settings.yaml", config)).toEqual(true);
expect(isSpecificItem("settings.yaml", config)).toEqual(true);
// Other types are NOT configured
expect(isItemTypeConfigured("f/test.variable.yaml", config)).toEqual(false);
expect(isItemTypeConfigured("f/db.resource.yaml", config)).toEqual(false);
expect(isItemTypeConfigured("f/hook.http_trigger.yaml", config)).toEqual(false);
expect(isItemTypeConfigured("f/my_folder/folder.meta.yaml", config)).toEqual(false);
});
test("mixed config: variables and folders configured - resources and triggers use base", () => {
const config: SpecificItemsConfig = {
variables: ["f/**"],
folders: ["f/env_*"],
};
// Variables IS configured
expect(isItemTypeConfigured("f/test.variable.yaml", config)).toEqual(true);
expect(isSpecificItem("f/test.variable.yaml", config)).toEqual(true);
// Folders IS configured (path matches)
expect(isItemTypeConfigured("f/env_staging/folder.meta.yaml", config)).toEqual(true);
expect(isSpecificItem("f/env_staging/folder.meta.yaml", config)).toEqual(true);
// Folders IS configured but path doesn't match
expect(isItemTypeConfigured("f/other/folder.meta.yaml", config)).toEqual(true);
expect(isSpecificItem("f/other/folder.meta.yaml", config)).toEqual(false);
// Resources and triggers are NOT configured
expect(isItemTypeConfigured("f/db.resource.yaml", config)).toEqual(false);
expect(isItemTypeConfigured("f/hook.http_trigger.yaml", config)).toEqual(false);
expect(isItemTypeConfigured("settings.yaml", config)).toEqual(false);
});