You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There was some confusion on how the tests are generated in this PR.
The generation was not including primary_resource_name due to no examples having the field set in the first place.
I'm not sure what's the reasoning for utilizing a name that's set in examples but it seems to make sense to include a field nested in iam_policy that is meant to be used for iam_policy_generated_tests
Thanks for raising the issue! Another improvement would be to have some documentation described on this attribute in the examples spec (in case it will not be moved to the iam_policy).
Uh oh!
There was an error while loading. Please reload this page.
What kind of contribution is this issue about?
MMv1-based resource
Details
There was some confusion on how the tests are generated in this PR.
The generation was not including
primary_resource_name
due to no examples having the field set in the first place.I'm not sure what's the reasoning for utilizing a name that's set in examples but it seems to make sense to include a field nested in
iam_policy
that is meant to be used foriam_policy_generated_tests
The change in the tmpl file would be here:
References
No response
The text was updated successfully, but these errors were encountered: