-
Notifications
You must be signed in to change notification settings - Fork 632
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
CI fixes #928
CI fixes #928
Conversation
@@ -10,6 +11,58 @@ import ( | |||
"github.com/hashicorp/terraform-plugin-sdk/terraform" | |||
) | |||
|
|||
func init() { |
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.
@@ -597,7 +597,7 @@ func TransformAccessGroupForSchema(accessGroup []interface{}) []map[string]inter | |||
gsuiteCfg := groupValue.(map[string]interface{}) | |||
gsuiteID = gsuiteCfg["identity_provider_id"].(string) | |||
gsuiteEmails = append(gsuiteEmails, gsuiteCfg["name"].(string)) | |||
case "github": | |||
case "github-organization": |
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.
@@ -235,7 +235,7 @@ func testAccessPolicyGroupConfig(resourceID, zone, accountID string) string { | |||
name = "%[1]s" | |||
|
|||
include { | |||
ip = ["127.0.0.1"] | |||
ip = ["127.0.0.1/32"] |
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.
@@ -10,6 +11,42 @@ import ( | |||
"github.com/hashicorp/terraform-plugin-sdk/terraform" | |||
) | |||
|
|||
func init() { |
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.
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.
woah nice 🎉
Gets the CI runs for acceptance tests closer to a fully green run.
github
=>github-organization
(follow up from Correctly handle Access Group schema transformations #917/resource/access_group: pass group conditions viaTransformAccessGroupForSchema
#918)cloudflare_authenticated_origin_pulls_certificate
to make sure we are only testing the managed resources