steve/databricks-cli
steve
/
databricks-cli
mirror of https://github.com/databricks/cli.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fixed TestAccBundleDeployUcSchema test (#1997) 

## Changes
It was failing because when schema.yml was removed, `catalog: main`
option left set in the base pipeline definition in databricks.yml which
lead to incorrect config (empty target schema)
https://github.com/databricks/cli/pull/1413

Backend behaviour changed and DLT pipelines stopped to accept empty
targets leading to the error which was ignored before.

## Tests
```
--- PASS: TestAccBundleDeployUcSchema (41.75s)
PASS
coverage: 33.3% of statements in ./...
ok      github.com/databricks/cli/internal/bundle       42.210s coverage: 33.3% of statements in ./...
```
This commit is contained in:
Andrew Nester 2024-12-11 15:00:43 +01:00 committed by GitHub
parent e39e94b12f
commit aa0b6080a4
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
internal/bundle/bundles/uc_schema/template/databricks.yml.tmpl
View File

@ -12,7 +12,6 @@ resources:
- notebook: - notebook:
path: ./nb.sql path: ./nb.sql
development: true development: true
catalog: main
include: include:
- "*.yml" - "*.yml"

1
internal/bundle/bundles/uc_schema/template/schema.yml.tmpl
View File

@ -11,3 +11,4 @@ targets:
pipelines: pipelines:
foo: foo:
target: ${resources.schemas.bar.id} target: ${resources.schemas.bar.id}
catalog: main