Skip to content

[Internal] Fix test name prefixes to match CI environment routing#5529

Open
tanmay-db wants to merge 2 commits intomainfrom
enable-test-filter
Open

[Internal] Fix test name prefixes to match CI environment routing#5529
tanmay-db wants to merge 2 commits intomainfrom
enable-test-filter

Conversation

@tanmay-db
Copy link
Copy Markdown
Contributor

@tanmay-db tanmay-db commented Mar 26, 2026

Changes

  • Fix 28 integration tests across 10 files that never run in CI due to mismatched test name prefixes
  • Tests were routed to wrong environments (e.g., TestAcc* prefix routed to workspace env, but test calls AccountLevel or UnityWorkspaceLevel), causing them
    to be silently skipped
  • Rename prefixes to match CI filter conventions: TestMwsAcc* for account-level, TestUcAcc* for Unity Catalog, TestAcc* for workspace-level
  • Fix 3 catalog metastore tests that also used wrong environment level functions (AccountLevelUnityAccountLevel, WorkspaceLevelUnityWorkspaceLevel)

Tests

Tests triggered in the CI in the right environment
NO_CHANGELOG=true

@tanmay-db tanmay-db requested review from a team as code owners March 26, 2026 13:23
@tanmay-db tanmay-db requested review from simonfaltum and removed request for a team and simonfaltum March 26, 2026 13:23
@github-actions
Copy link
Copy Markdown

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/terraform

Inputs:

  • PR number: 5529
  • Commit SHA: 8c7ef0df6f64bd17523b36d30736809ab82b6321

Checks will be approved automatically on success.

@tanmay-db tanmay-db temporarily deployed to test-trigger-is March 26, 2026 13:27 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant