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

provider/aws: Improved IAM Role description tests & validation #14220

Merged
merged 1 commit into from
May 5, 2017

Conversation

Ninir
Copy link
Contributor

@Ninir Ninir commented May 4, 2017

Description

This augments #14208 to add acceptance tests, adding validation. (I also started the work for the description, but @minamijoyo did it fast 😄 )

Acceptance tests create the role with the description, then update it.
This work also adds more randomization to iam role tests.

Tests

make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSIAMRole_importBasic'
==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
2017/05/04 21:10:03 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSIAMRole_importBasic -timeout 120m
=== RUN   TestAccAWSIAMRole_importBasic
--- PASS: TestAccAWSIAMRole_importBasic (17.14s)
PASS
ok  	github.com/hashicorp/terraform/builtin/providers/aws	17.170s

make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSRole_'
==> Checking that code complies with gofmt requirements...
go generate $(go list ./... | grep -v /terraform/vendor/)
2017/05/04 22:23:31 Generated command/internal_plugin_list.go
TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSRole_ -timeout 120m
=== RUN   TestAccAWSRole_basic
--- PASS: TestAccAWSRole_basic (15.90s)
=== RUN   TestAccAWSRole_basicWithDescription
--- PASS: TestAccAWSRole_basicWithDescription (38.47s)
=== RUN   TestAccAWSRole_namePrefix
--- PASS: TestAccAWSRole_namePrefix (15.29s)
=== RUN   TestAccAWSRole_testNameChange
--- PASS: TestAccAWSRole_testNameChange (35.00s)
=== RUN   TestAccAWSRole_badJSON
--- PASS: TestAccAWSRole_badJSON (0.00s)
PASS
ok  	github.com/hashicorp/terraform/builtin/providers/aws	104.691s

@Ninir Ninir changed the title provider/aws: Improved IAM Role description tests & validation [WIP] provider/aws: Improved IAM Role description tests & validation May 4, 2017
@Ninir Ninir force-pushed the f-aws-iam-role-description branch 2 times, most recently from 437ed94 to 4f55796 Compare May 4, 2017 20:43
@Ninir Ninir force-pushed the f-aws-iam-role-description branch from 4f55796 to d00370c Compare May 4, 2017 21:09
@Ninir Ninir changed the title [WIP] provider/aws: Improved IAM Role description tests & validation provider/aws: Improved IAM Role description tests & validation May 4, 2017
Copy link
Contributor

@stack72 stack72 left a comment

Choose a reason for hiding this comment

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

Nice work as always @Ninir :)

@stack72 stack72 merged commit c8ce988 into hashicorp:master May 5, 2017
@ghost
Copy link

ghost commented Apr 13, 2020

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.

@ghost ghost locked and limited conversation to collaborators Apr 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants