Skip to content

fix(tests): Fix Snowflake read test#6373

Merged
kodiakhq[bot] merged 2 commits intomainfrom
snowflake-migrations
Jan 5, 2023
Merged

fix(tests): Fix Snowflake read test#6373
kodiakhq[bot] merged 2 commits intomainfrom
snowflake-migrations

Conversation

@hermanschaaf
Copy link
Copy Markdown
Contributor

This changes the Read query to only select the columns defined in the table schema, so that user-defined columns are ignored. Since Read is only used in tests, this only affects testing right now.

This change should allow the new test in cloudquery/plugin-sdk#574 to pass

@hermanschaaf hermanschaaf requested review from a team and yevgenypats and removed request for a team January 5, 2023 11:21
@hermanschaaf hermanschaaf added the automerge Automatically merge once required checks pass label Jan 5, 2023
@kodiakhq kodiakhq bot merged commit 0c334f5 into main Jan 5, 2023
@kodiakhq kodiakhq bot deleted the snowflake-migrations branch January 5, 2023 14:36
kodiakhq bot pushed a commit that referenced this pull request Jan 10, 2023
🤖 I have created a release *beep* *boop*
---


## [1.1.5](plugins-destination-snowflake-v1.1.4...plugins-destination-snowflake-v1.1.5) (2023-01-10)


### Bug Fixes

* **deps:** Update module github.com/aws/aws-sdk-go-v2/feature/s3/manager to v1.11.47 ([#6395](#6395)) ([71ec9b9](71ec9b9))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.18.0 ([#6339](#6339)) ([158365a](158365a))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.19.0 ([#6363](#6363)) ([ae6967c](ae6967c))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.20.0 ([#6376](#6376)) ([d6187ec](d6187ec))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.21.0 ([#6382](#6382)) ([5baea40](5baea40))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.22.0 ([#6516](#6516)) ([b7e4e73](b7e4e73))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.23.0 ([#6522](#6522)) ([ce24f1d](ce24f1d))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.24.1 ([#6553](#6553)) ([392b848](392b848))
* **deps:** Update plugin-sdk to v1.21.0 for destinations ([#6419](#6419)) ([f3b989f](f3b989f))
* **tests:** Fix Snowflake read test ([#6373](#6373)) ([0c334f5](0c334f5))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants