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

Flatten/expand ADFS #467

Merged
merged 1 commit into from
Nov 9, 2021
Merged

Flatten/expand ADFS #467

merged 1 commit into from
Nov 9, 2021

Conversation

alexkappa
Copy link
Owner

Proposed Changes

  • Fix auth0_connection where strategy is adfs.

The provider didn't support ADFS as there was no expand/flatten functionality for connection options.

Fixes #457

Acceptance Test Output

$ make testacc TESTS=TestAccConnection                                                                                                                                                                                                                 [9:49:23]
==> Checking that code complies with gofmt requirements...
?       github.com/alexkappa/terraform-provider-auth0   [no test files]
=== RUN   TestAccConnection
--- PASS: TestAccConnection (2.90s)
=== RUN   TestAccConnectionAD
--- PASS: TestAccConnectionAD (1.14s)
=== RUN   TestAccConnectionAzureAD
--- PASS: TestAccConnectionAzureAD (1.66s)
=== RUN   TestAccConnectionOIDC
--- PASS: TestAccConnectionOIDC (2.26s)
=== RUN   TestAccConnectionOAuth2
--- PASS: TestAccConnectionOAuth2 (2.21s)
=== RUN   TestAccConnectionWithEnbledClients
--- PASS: TestAccConnectionWithEnbledClients (2.19s)
=== RUN   TestAccConnectionSMS
--- PASS: TestAccConnectionSMS (1.25s)
=== RUN   TestAccConnectionEmail
--- PASS: TestAccConnectionEmail (1.92s)
=== RUN   TestAccConnectionSalesforce
--- PASS: TestAccConnectionSalesforce (1.18s)
=== RUN   TestAccConnectionGoogleOAuth2
--- PASS: TestAccConnectionGoogleOAuth2 (1.21s)
=== RUN   TestAccConnectionFacebook
--- PASS: TestAccConnectionFacebook (1.97s)
=== RUN   TestAccConnectionApple
--- PASS: TestAccConnectionApple (2.08s)
=== RUN   TestAccConnectionLinkedin
--- PASS: TestAccConnectionLinkedin (2.08s)
=== RUN   TestAccConnectionGitHub
--- PASS: TestAccConnectionGitHub (1.26s)
=== RUN   TestAccConnectionWindowslive
--- PASS: TestAccConnectionWindowslive (2.10s)
=== RUN   TestAccConnectionConfiguration
--- PASS: TestAccConnectionConfiguration (2.06s)
=== RUN   TestAccConnectionSAML
--- PASS: TestAccConnectionSAML (2.04s)
PASS
coverage: 21.0% of statements
ok      github.com/alexkappa/terraform-provider-auth0/auth0     31.949s coverage: 21.0% of statements
?       github.com/alexkappa/terraform-provider-auth0/auth0/internal/debug      [no test files]
testing: warning: no tests to run
PASS
coverage: 0.0% of statements
ok      github.com/alexkappa/terraform-provider-auth0/auth0/internal/hash       0.238s  coverage: 0.0% of statements [no tests to run]
testing: warning: no tests to run
PASS
coverage: 0.0% of statements
ok      github.com/alexkappa/terraform-provider-auth0/auth0/internal/random     0.421s  coverage: 0.0% of statements [no tests to run]
testing: warning: no tests to run
PASS
coverage: 0.0% of statements
ok      github.com/alexkappa/terraform-provider-auth0/auth0/internal/validation 0.268s  coverage: 0.0% of statements [no tests to run]
?       github.com/alexkappa/terraform-provider-auth0/version   [no test files]

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't create a ADFS connection
1 participant