Skip to content

Add test for modified_status in bundle summary - #3142

Merged
denik merged 4 commits into
mainfrom
denik/acc-modified_status
Jun 30, 2025
Merged

Add test for modified_status in bundle summary#3142
denik merged 4 commits into
mainfrom
denik/acc-modified_status

Conversation

@denik

Copy link
Copy Markdown
Contributor

Changes

Add an acceptance test covering modified_status lifecycle: "created" -> "" -> "deleted".

Why

To support refactoring such as #3134 and direct deployment method support in summary.

@eng-dev-ecosystem-bot

eng-dev-ecosystem-bot commented Jun 30, 2025

Copy link
Copy Markdown
Collaborator

Run: 15971401084

Env✅‌pass❌‌FAIL💥‌PANIC🙈‌skip🤯‌MISS
❌‌aws linux26518136710
❌‌aws windows2662013668
❌‌aws-ucws linux37226267
❌‌aws-ucws windows37027266
❌‌azure linux28412366
❌‌azure windows28512365
❌‌azure-ucws linux38020264
❌‌azure-ucws windows37922263
❌‌gcp linux28312368
❌‌gcp windows28412367
Test Nameaws linuxaws windowsaws-ucws linuxaws-ucws windowsazure linuxazure windowsazure-ucws linuxazure-ucws windowsgcp linuxgcp windows
TestAccept❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL
TestAccept/bundle/deploy/dashboard/detect-change❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL
TestAccept/bundle/deploy/dashboard/nested-folders✅‌pass✅‌pass❌‌FAIL❌‌FAIL✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestAccept/bundle/deploy/dashboard/simple✅‌pass✅‌pass❌‌FAIL❌‌FAIL✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestAccept/bundle/deploy/dashboard/simple_outside_bundle_root✅‌pass✅‌pass❌‌FAIL❌‌FAIL✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestAccept/bundle/deploy/dashboard/simple_syncroot✅‌pass✅‌pass❌‌FAIL❌‌FAIL✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestAccept/bundle/deploy/jobs/check-metadata✅‌pass✅‌pass✅‌pass✅‌pass❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL
TestAccept/bundle/deployment/bind/dashboard✅‌pass✅‌pass❌‌FAIL❌‌FAIL✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestAccept/bundle/deployment/bind/dashboard/recreation✅‌pass✅‌pass❌‌FAIL❌‌FAIL✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestAccept/bundle/templates/default-python/combinations❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL
TestAccept/bundle/templates/default-python/combinations/D=no/DATABRICKS_CLI_DEPLOYMENT=direct-exp/N=no/P=yes/S=no❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL
TestAccept/bundle/templates/default-python/combinations/D=no/DATABRICKS_CLI_DEPLOYMENT=direct-exp/N=no/P=yes/S=yes🙈‌skip🙈‌skip❌‌FAIL❌‌FAIL🙈‌skip🙈‌skip❌‌FAIL❌‌FAIL🙈‌skip🙈‌skip
TestAccept/bundle/templates/default-python/combinations/D=no/DATABRICKS_CLI_DEPLOYMENT=direct-exp/N=yes/P=yes/S=no❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL
TestAccept/bundle/templates/default-python/combinations/D=no/DATABRICKS_CLI_DEPLOYMENT=direct-exp/N=yes/P=yes/S=yes🙈‌skip🙈‌skip❌‌FAIL❌‌FAIL🙈‌skip🙈‌skip❌‌FAIL❌‌FAIL🙈‌skip🙈‌skip
TestAccept/bundle/templates/default-python/combinations/D=no/DATABRICKS_CLI_DEPLOYMENT=terraform/N=no/P=yes/S=no❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL
TestAccept/bundle/templates/default-python/combinations/D=no/DATABRICKS_CLI_DEPLOYMENT=terraform/N=no/P=yes/S=yes🙈‌skip🙈‌skip❌‌FAIL❌‌FAIL🙈‌skip🙈‌skip❌‌FAIL❌‌FAIL🙈‌skip🙈‌skip
TestAccept/bundle/templates/default-python/combinations/D=no/DATABRICKS_CLI_DEPLOYMENT=terraform/N=yes/P=yes/S=no❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL
TestAccept/bundle/templates/default-python/combinations/D=no/DATABRICKS_CLI_DEPLOYMENT=terraform/N=yes/P=yes/S=yes🙈‌skip🙈‌skip❌‌FAIL❌‌FAIL🙈‌skip🙈‌skip❌‌FAIL❌‌FAIL🙈‌skip🙈‌skip
TestAccept/bundle/templates/default-python/combinations/D=yes/DATABRICKS_CLI_DEPLOYMENT=direct-exp/N=no/P=yes/S=no❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL
TestAccept/bundle/templates/default-python/combinations/D=yes/DATABRICKS_CLI_DEPLOYMENT=direct-exp/N=no/P=yes/S=yes🙈‌skip🙈‌skip❌‌FAIL❌‌FAIL🙈‌skip🙈‌skip❌‌FAIL❌‌FAIL🙈‌skip🙈‌skip
TestAccept/bundle/templates/default-python/combinations/D=yes/DATABRICKS_CLI_DEPLOYMENT=direct-exp/N=yes/P=yes/S=no❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL
TestAccept/bundle/templates/default-python/combinations/D=yes/DATABRICKS_CLI_DEPLOYMENT=direct-exp/N=yes/P=yes/S=yes🙈‌skip🙈‌skip❌‌FAIL❌‌FAIL🙈‌skip🙈‌skip❌‌FAIL❌‌FAIL🙈‌skip🙈‌skip
TestAccept/bundle/templates/default-python/combinations/D=yes/DATABRICKS_CLI_DEPLOYMENT=terraform/N=no/P=yes/S=no❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL
TestAccept/bundle/templates/default-python/combinations/D=yes/DATABRICKS_CLI_DEPLOYMENT=terraform/N=no/P=yes/S=yes🙈‌skip🙈‌skip❌‌FAIL❌‌FAIL🙈‌skip🙈‌skip❌‌FAIL❌‌FAIL🙈‌skip🙈‌skip
TestAccept/bundle/templates/default-python/combinations/D=yes/DATABRICKS_CLI_DEPLOYMENT=terraform/N=yes/P=yes/S=no❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL❌‌FAIL
TestAccept/bundle/templates/default-python/combinations/D=yes/DATABRICKS_CLI_DEPLOYMENT=terraform/N=yes/P=yes/S=yes🙈‌skip🙈‌skip❌‌FAIL❌‌FAIL🙈‌skip🙈‌skip❌‌FAIL❌‌FAIL🙈‌skip🙈‌skip
TestDashboardAssumptions_WorkspaceImport✅‌pass✅‌pass❌‌FAIL❌‌FAIL✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestFetchRepositoryInfoAPI_FromRepo❌‌FAIL❌‌FAIL✅‌pass❌‌FAIL✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestFilerWorkspaceNotebook✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass❌‌FAIL✅‌pass✅‌pass
TestFilerWorkspaceNotebook/scalaNb.scala✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass❌‌FAIL✅‌pass✅‌pass
TestLock❌‌FAIL❌‌FAIL✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestReposCreateWithProvider❌‌FAIL❌‌FAIL✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestReposCreateWithoutProvider❌‌FAIL❌‌FAIL✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestReposDeleteByID❌‌FAIL❌‌FAIL✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestReposDeleteByPath✅‌pass❌‌FAIL✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestReposGet❌‌FAIL❌‌FAIL✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestReposUpdate❌‌FAIL❌‌FAIL✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestSyncEnsureRemotePathIsUsableIfRepoDoesntExist🤯‌MISS🤯‌MISS✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestSyncEnsureRemotePathIsUsableIfRepoExists🤯‌MISS🤯‌MISS✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestSyncEnsureRemotePathIsUsableInWorkspace🤯‌MISS🤯‌MISS✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestSyncIncrementalFileOverwritesFolder🤯‌MISS🤯‌MISS✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestSyncIncrementalFileSync💥‌PANIC✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestSyncIncrementalSyncFileToPythonNotebook🤯‌MISS🤯‌MISS✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestSyncIncrementalSyncPythonNotebookDelete🤯‌MISS🤯‌MISS✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestSyncIncrementalSyncPythonNotebookToFile🤯‌MISS🤯‌MISS✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestSyncNestedFolderDoesntFailOnNonEmptyDirectory🤯‌MISS💥‌PANIC✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestSyncNestedFolderSync🤯‌MISS❌‌FAIL✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass
TestSyncNestedSpacePlusAndHashAreEscapedSync🤯‌MISS🤯‌MISS✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass✅‌pass

@denik
deniktemporarily deployed to test-trigger-is June 30, 2025 10:55 — with GitHub Actions Inactive
Comment threadacceptance/bundle/summary/modified_status/test.toml Outdated
Co-authored-by: shreyas-goenka <88374338+shreyas-goenka@users.noreply.github.com>
@denik
deniktemporarily deployed to test-trigger-is June 30, 2025 11:14 — with GitHub Actions Inactive
@@ -0,0 +1,14 @@
title "Initial view of resources without id and modified_status=created"
trace $CLI bundle summary -o json | jq .resources

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you filter only the fields we are interested in, in jq? That is, modified_status and id. Keeps the test more narrow in scope.

@denikdenikJun 30, 2025

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In general, makes sense, but for "bundle summary" currently we don't have a lot of coverage, so good to record fuller output in some tests.

@denik
denik merged commit 7ca636b into mainJun 30, 2025
@denik
denik deleted the denik/acc-modified_status branch June 30, 2025 11:27
denik added a commit that referenced this pull request May 20, 2026
## Changes
Add an acceptance test covering modified_status lifecycle: "created" ->
"" -> "deleted".
## Why
To support refactoring such as
#3134 and direct deployment method
support in summary.
---------
Co-authored-by: shreyas-goenka <88374338+shreyas-goenka@users.noreply.github.com>
Sign up for freeto 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.

3 participants

@denik@eng-dev-ecosystem-bot@shreyas-goenka