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

[SDK-v2] Fix TypeSet tests and Re-record cassettes #983

Merged
merged 8 commits into from
Mar 15, 2021

Conversation

skarimo
Copy link
Member

@skarimo skarimo commented Mar 12, 2021

This PR does several things:

  1. Rebases on latest master
  2. Fixes TypeSet tests based on the helpers introduced here: Port TypeSet helpers from AWS hashicorp/terraform-plugin-sdk#614
  3. Re records cassettes due to merge conflicts + original cassettes not being recorded properly in the original sdkv2 pr

rymir and others added 8 commits March 9, 2021 18:51
* Fix typo from cases to case

* Do not persist state changes when modifying bad rule case

* Update default rule case notifications

* align default rule case notification recipients when rule case tracked in terraform configuration
* clear default rule case notification recipients when rule case removed from terraform configuration

* Remove broken tests

It is not possible to use both ImportState: True and Check

* Revert "Remove broken tests"

This reverts commit fed89bf.

* Reapply test changes, re-record

* Lint

Co-authored-by: Alexandre Trufanow <[email protected]>
Co-authored-by: Thomas Hervé <[email protected]>
Don't use removed metric
* add dashboard_json resource

* refactor and use client exported request methods

* lint and move helper functions to util

* lint

* handle diffs on computed fields

* normalize json string

* add tests and refactor

* update go client

* lint

* update tests and lint

* apply code review requested changes

* apply code review suggestions and re-record cassettes

* generate resource docs

* fmt
@skarimo skarimo requested review from a team as code owners March 12, 2021 22:35
@skarimo skarimo changed the base branch from sdk-v2 to master March 12, 2021 22:35
@skarimo skarimo requested review from a team as code owners March 12, 2021 22:35
@skarimo skarimo changed the base branch from master to sdk-v2 March 12, 2021 22:36
@skarimo skarimo changed the title [SDK-v2] Fix TypeSet's and Re-record Import tests [SDK-v2] Fix TypeSet's and Re-record cassettes Mar 12, 2021
@skarimo skarimo changed the title [SDK-v2] Fix TypeSet's and Re-record cassettes [SDK-v2] Fix TypeSet tests and Re-record cassettes Mar 12, 2021
@skarimo skarimo merged commit 5d1bcfc into sdk-v2 Mar 15, 2021
@skarimo skarimo deleted the sherz/fix-sdkv2-tests branch March 15, 2021 12:30
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