-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add tests for random #872
Add tests for random #872
Conversation
Does the PR have any schema changes?Does the PR have any schema changes?Looking good! No breaking changes found. Maintainer note: consult the runbook for dealing with any breaking changes. |
Is README.md missing any configuration options?No missing config! |
@iwahbe this one is ready, LMK what you think and I'll merge the others as they are ready if this one looks ok. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. LGTM
pulumi/pulumi-terraform-bridge#1849
This PR adds tests for the following resources:
index_randomstring
index_randomshuffle
index_randombytes
index_randomuuid
index_randompassword
index_randomid
index_randominteger
index_randompet
It also adds upgrade tests for the resources.