Skip to content
This repository has been archived by the owner on Mar 25, 2022. It is now read-only.

r/security_list: Suppress case diffs for security_list #27

Merged
merged 2 commits into from
Jul 6, 2017

Conversation

grubernaut
Copy link
Contributor

With outbound_cidr_policy and policy attributes, the API returns a fully upper-case value regardless of a lower-case value being supplied. Needed to suppress the diff with the updated value to ignore case.

Fixes: #17

@grubernaut grubernaut added the bug label Jul 6, 2017
With `outbound_cidr_policy` and `policy` attributes, the API returns a fully upper-case value regardless of a lower-case value being supplied. Needed to suppress the diff with the updated value to ignore case.
@grubernaut
Copy link
Contributor Author

$ make testacc TEST=./opc TESTARGS="-run=TestAccOPCSecurityList"
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./opc -v -run=TestAccOPCSecurityList -timeout 120m
=== RUN   TestAccOPCSecurityList_importBasic
--- PASS: TestAccOPCSecurityList_importBasic (12.51s)
=== RUN   TestAccOPCSecurityList_importComplete
--- PASS: TestAccOPCSecurityList_importComplete (12.70s)
=== RUN   TestAccOPCSecurityList_basic
--- PASS: TestAccOPCSecurityList_basic (12.01s)
=== RUN   TestAccOPCSecurityList_complete
--- PASS: TestAccOPCSecurityList_complete (16.06s)
=== RUN   TestAccOPCSecurityList_lowercasePolicies
--- PASS: TestAccOPCSecurityList_lowercasePolicies (13.05s)
PASS
ok  	github.com/terraform-providers/terraform-provider-opc/opc	66.335s

Copy link
Contributor

@mbfrahry mbfrahry left a comment

Choose a reason for hiding this comment

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

LGTM!

@grubernaut grubernaut merged commit 2f54ebc into master Jul 6, 2017
@grubernaut grubernaut deleted the f-security-list-suppress branch July 6, 2017 17:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants