Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add acc tests for databricks_schemas data source #2074

Merged
merged 3 commits into from
Mar 7, 2023
Merged

Conversation

nkvuong
Copy link
Contributor

@nkvuong nkvuong commented Mar 6, 2023

No description provided.

@nkvuong
Copy link
Contributor Author

nkvuong commented Mar 6, 2023

acceptance test passed

@codecov-commenter
Copy link

Codecov Report

Merging #2074 (f69f54e) into master (bd05be2) will not change coverage.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2074   +/-   ##
=======================================
  Coverage   89.78%   89.78%           
=======================================
  Files         136      136           
  Lines       11077    11077           
=======================================
  Hits         9946     9946           
  Misses        732      732           
  Partials      399      399           
Impacted Files Coverage Δ
catalog/data_schemas.go 100.00% <100.00%> (ø)

@nkvuong nkvuong marked this pull request as ready for review March 6, 2023 19:12
@nkvuong nkvuong requested review from a team March 6, 2023 19:12
@@ -19,7 +19,7 @@ func DataSourceSchemas() *schema.Resource {
return err
}
for _, v := range schemas {
data.Ids = append(data.Ids, v.Name)
data.Ids = append(data.Ids, v.FullName)
Copy link
Contributor

Choose a reason for hiding this comment

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

don't change this

@nfx nfx merged commit 66d718c into master Mar 7, 2023
@nfx nfx deleted the acc/data_schemas branch March 7, 2023 15:58
nfx added a commit that referenced this pull request Mar 9, 2023
* Added databricks_model_serving resource (#2054).
* Added Unity Catalog on GCP support (#2000).
* Deprecate schedule block in databricks_sql_query resource (#2078).
* Fixed InitScripts Type to work with GCS and ABFS in databricks_cluster resource (#2067).
* Added more testing for databricks_tables data source (#2075).
* Added more testing for databricks_schemas data source (#2074).
* Migrate databricks_node_type data source to Go SDK (#2070).
* Migrate databricks_schemas data to Go SDK (#2065).
* Migrate databricks_tables data source to SDK (#2068).
* Migrate databricks_views data source to Go SDK (#2073).
* Migrated databricks_git_credential to Go SDK (#2069).
* Migrated databricks_shares data source to Go SDK (#2072).

Updated dependency versions:

* Bump github.com/databricks/databricks-sdk-go from v0.3.3 to v0.4.0 (#2086).
* Bump golang.org/x/mod from 0.8.0 to 0.9.0 (#2076).
@nfx nfx mentioned this pull request Mar 9, 2023
nfx added a commit that referenced this pull request Mar 9, 2023
* Added databricks_model_serving resource (#2054).
* Added Unity Catalog on GCP support (#2000).
* Deprecate schedule block in databricks_sql_query resource (#2078).
* Fixed InitScripts Type to work with GCS and ABFS in databricks_cluster resource (#2067).
* Added more testing for databricks_tables data source (#2075).
* Added more testing for databricks_schemas data source (#2074).
* Migrate databricks_node_type data source to Go SDK (#2070).
* Migrate databricks_schemas data to Go SDK (#2065).
* Migrate databricks_tables data source to SDK (#2068).
* Migrate databricks_views data source to Go SDK (#2073).
* Migrated databricks_git_credential to Go SDK (#2069).
* Migrated databricks_shares data source to Go SDK (#2072).

Updated dependency versions:

* Bump github.com/databricks/databricks-sdk-go from v0.3.3 to v0.4.0 (#2086).
* Bump golang.org/x/mod from 0.8.0 to 0.9.0 (#2076).
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.

3 participants