Skip to content

fix(tests): Postgresql: Read only columns defined in the schema#6371

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

fix(tests): Postgresql: Read only columns defined in the schema#6371
kodiakhq[bot] merged 2 commits intomainfrom
postgresql-migrations

Conversation

@hermanschaaf
Copy link
Copy Markdown
Contributor

The PostgreSQL read test should only read and return the columns defined in our schema. This will allow https://github.com/cloudquery/plugin-sdk/pull/574/files to pass.

@hermanschaaf hermanschaaf requested review from a team and bbernays and removed request for a team January 5, 2023 11:08
@hermanschaaf hermanschaaf changed the title Postgresql: Read only columns defined in the schema fix(tests): Postgresql: Read only columns defined in the schema Jan 5, 2023
@hermanschaaf hermanschaaf added the automerge Automatically merge once required checks pass label Jan 5, 2023
@kodiakhq kodiakhq bot merged commit 491941a into main Jan 5, 2023
@kodiakhq kodiakhq bot deleted the postgresql-migrations branch January 5, 2023 14:28
kodiakhq bot pushed a commit that referenced this pull request Jan 6, 2023
🤖 I have created a release *beep* *boop*
---


## [2.0.2](plugins-destination-postgresql-v2.0.1...plugins-destination-postgresql-v2.0.2) (2023-01-06)


### Bug Fixes

* **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.20.0 ([#6376](#6376)) ([d6187ec](d6187ec))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.21.0 ([#6382](#6382)) ([5baea40](5baea40))
* **deps:** Update plugin-sdk to v1.21.0 for destinations ([#6419](#6419)) ([f3b989f](f3b989f))
* Error if after the migration there are `not null` columns that are not part of the new schema ([#6282](#6282)) ([c5a4bf5](c5a4bf5))
* **pg:** Return more detailed pg errors ([#6421](#6421)) ([acb3e21](acb3e21))
* **postgresql:** Revert [#6282](#6282) ([#6434](#6434)) ([2cecacd](2cecacd))
* **tests:** Postgresql: Read only columns defined in the schema ([#6371](#6371)) ([491941a](491941a))

---
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