provider/aws: Add JSON validation to the aws_glacier_vault resource.

[kwilczynski]

This commit adds support for new helper function which is used to
normalise and validate JSON string.

Signed-off-by: Krzysztof Wilczynski [email protected]

[kwilczynski]

Part of work which began in #8028.

[kwilczynski]

Acceptance test is passing:

$ make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSGlacierVault_'
==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
2016/09/14 17:50:12 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSGlacierVault_ -timeout 120m
=== RUN   TestAccAWSGlacierVault_importBasic
--- PASS: TestAccAWSGlacierVault_importBasic (26.93s)
=== RUN   TestAccAWSGlacierVault_basic
--- PASS: TestAccAWSGlacierVault_basic (19.52s)
=== RUN   TestAccAWSGlacierVault_full
--- PASS: TestAccAWSGlacierVault_full (25.49s)
=== RUN   TestAccAWSGlacierVault_RemoveNotifications
--- PASS: TestAccAWSGlacierVault_RemoveNotifications (39.44s)
PASS
ok      github.com/hashicorp/terraform/builtin/providers/aws    111.399s

[radeksimko]

We should replace this with the diff policy helper we use elsewhere, but it's unrelated to the original purpose of this PR, so I can submit a separate PR for it.

[kwilczynski]

Makes sense. Where do we have the function? I am happy to send a PR.

[kwilczynski]

Actually, as per our conversation I will hold for the time being.

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.