Skip to content
This repository has been archived by the owner on Mar 8, 2022. It is now read-only.

Connection v4 #179

Merged
merged 9 commits into from
Mar 21, 2020
Merged

Connection v4 #179

merged 9 commits into from
Mar 21, 2020

Conversation

alexkappa
Copy link
Owner

@alexkappa alexkappa commented Mar 20, 2020

Community Note

  • Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
  • Please do not leave "+1" comments, they generate extra noise for pull request followers and do not help prioritize the request

Fixes #163 #160 #118

Changes proposed in this pull request:

  • Use in-progress go-auth0/auth0 for polymorphic connections.
  • Improvements to this providers internal List and Set data access functions which help nest *schema.ResourceData allowing for better control over nesting.

Output from acceptance testing:

==> Checking that code complies with gofmt requirements...
?       github.com/terraform-providers/terraform-provider-auth0 [no test files]
=== RUN   TestProvider
--- PASS: TestProvider (0.00s)
=== RUN   TestProvider_debugDefaults
--- PASS: TestProvider_debugDefaults (0.00s)
=== RUN   TestAccClientGrant
--- PASS: TestAccClientGrant (2.81s)
=== RUN   TestAccClient
--- PASS: TestAccClient (0.86s)
=== RUN   TestAccClientZeroValueCheck
--- PASS: TestAccClientZeroValueCheck (2.00s)
=== RUN   TestAccClientRotateSecret
--- PASS: TestAccClientRotateSecret (1.41s)
=== RUN   TestAccClientInitiateLoginUri
--- PASS: TestAccClientInitiateLoginUri (0.04s)
=== RUN   TestAccConnection
--- PASS: TestAccConnection (1.38s)
=== RUN   TestAccConnectionAD
--- PASS: TestAccConnectionAD (0.77s)
=== RUN   TestAccConnectionAzureAD
--- PASS: TestAccConnectionAzureAD (1.07s)
=== RUN   TestAccConnectionWithEnbledClients
--- PASS: TestAccConnectionWithEnbledClients (1.39s)
=== RUN   TestAccConnectionSMS
--- PASS: TestAccConnectionSMS (0.77s)
=== RUN   TestAccConnectionEmail
--- PASS: TestAccConnectionEmail (0.76s)
=== RUN   TestAccConnectionSalesforce
--- PASS: TestAccConnectionSalesforce (0.75s)
=== RUN   TestAccConnectionGoogleOAuth2
--- PASS: TestAccConnectionGoogleOAuth2 (0.79s)
=== RUN   TestAccCustomDomain
--- PASS: TestAccCustomDomain (0.78s)
=== RUN   TestAccEmailTemplate
--- PASS: TestAccEmailTemplate (0.81s)
=== RUN   TestAccEmail
--- PASS: TestAccEmail (1.69s)
=== RUN   TestAccGlobalClient
--- PASS: TestAccGlobalClient (1.82s)
=== RUN   TestAccHook
--- PASS: TestAccHook (0.78s)
=== RUN   TestHookNameRegexp
--- PASS: TestHookNameRegexp (0.00s)
=== RUN   TestAccResourceServer
--- PASS: TestAccResourceServer (1.19s)
=== RUN   TestAccRole
--- PASS: TestAccRole (1.81s)
=== RUN   TestAccRuleConfig
--- PASS: TestAccRuleConfig (0.58s)
=== RUN   TestAccRule
--- PASS: TestAccRule (0.58s)
=== RUN   TestRuleNameRegexp
--- PASS: TestRuleNameRegexp (0.00s)
=== RUN   TestAccTenant
--- PASS: TestAccTenant (1.20s)
=== RUN   TestAccUserMissingRequiredParams
--- PASS: TestAccUserMissingRequiredParams (0.02s)
=== RUN   TestAccUser
--- PASS: TestAccUser (2.76s)
=== RUN   TestMapData
--- PASS: TestMapData (0.00s)
=== RUN   TestJSON
--- PASS: TestJSON (0.00s)
PASS
coverage: 75.8% of statements
ok      github.com/terraform-providers/terraform-provider-auth0/auth0   28.902s coverage: 75.8% of statements
?       github.com/terraform-providers/terraform-provider-auth0/auth0/internal/debug    [no test files]
=== RUN   TestString
--- PASS: TestString (0.00s)
=== RUN   TestTemplate
--- PASS: TestTemplate (0.00s)
PASS
coverage: 75.0% of statements
ok      github.com/terraform-providers/terraform-provider-auth0/auth0/internal/random   0.047s  coverage: 75.0% of statements
=== RUN   TestIsURLWithNoFragment
--- PASS: TestIsURLWithNoFragment (0.00s)
PASS
coverage: 52.9% of statements
ok      github.com/terraform-providers/terraform-provider-auth0/auth0/internal/validation       0.012s  coverage: 52.9% of statements

@alexkappa alexkappa merged commit c8f3b76 into master Mar 21, 2020
@yvovandoorn yvovandoorn deleted the connection-v4 branch March 24, 2021 14:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.