resource/aws_instance: Added acceptance test when using encrypted volume & IOPS

[trung]

Before PR#1573: failing

After PR#1573:

$ make testacc TESTARGS="-run=TestAccAWSInstance_GP2WithIopsValue"
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test $(go list ./... |grep -v 'vendor') -v -run=TestAccAWSInstance_GP2WithIopsValue -timeout 120m
?   	github.com/terraform-providers/terraform-provider-aws	[no test files]
=== RUN   TestAccAWSInstance_GP2WithIopsValue
--- PASS: TestAccAWSInstance_GP2WithIopsValue (80.04s)
PASS
ok  	github.com/terraform-providers/terraform-provider-aws/aws	80.071s

[Ninir]

Hi @trung

Thanks for the suggestion. Could you provide the failing test?

Ran all the tests when merging the related PR, and except one (that is actually a warning) everything was passing.

Thank you! 👍

[trung]

@Ninir the PR is just a supplement to #1573 as commented (#1573 (comment))

If we didn't have #1573 , the test would fail.

[Ninir]

LGTM, thanks a lot for the work @trung ! 👍

$ make testacc TEST=./aws TESTARGS='-run=TestAccAWSInstance_GP2WithIopsValue'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./aws -v -run=TestAccAWSInstance_GP2WithIopsValue -timeout 120m
=== RUN   TestAccAWSInstance_GP2WithIopsValue
--- PASS: TestAccAWSInstance_GP2WithIopsValue (136.39s)
PASS
ok  	github.com/terraform-providers/terraform-provider-aws/aws	136.426s

[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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!