-
Notifications
You must be signed in to change notification settings - Fork 9.7k
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
IAM Acceptance Tests #1591
IAM Acceptance Tests #1591
Conversation
Sets of strings are pretty common. Let's not duplicate the function necessary to create a set of strings in so many places.
Implements support for IAM users, user policies, and access keys. This is only a subset of what IAM can do (notably missing: roles and instance profiles and associated policies), but it's a start. Makes a dent in #28.
@johnrengelman Thanks for doing this. I've been meaning to figure out how to write acceptance tests but never got around to it. I pulled these commits into #939. I think that's better since if I rebase or extend my changes I can take your tests with it. I'm working on adding managed policies now and I should be done next week. I might also want to clean up some of the duplicated code...the tests will make that much easier. |
Closed in favor of original PR. |
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. |
Adding acceptance tasks on top of #939 so that we can get the IAM support merged in.
I don't have a clean account that I can use to run the acceptance test suite, so I haven't run these to verify they actually work...but they do build :) Hopefully any potential issues are minor/quick fixes.