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

Add support for allowed_clients #443

Merged
merged 3 commits into from
Nov 9, 2021

Conversation

lpakula
Copy link
Contributor

@lpakula lpakula commented Sep 6, 2021

Proposed Changes

  • add allowed_clients attribute

Fixes #441

Acceptance Test Output

$ make testacc TESTS=TestAccClient
==> Checking that code complies with gofmt requirements...
?   	github.com/alexkappa/terraform-provider-auth0	[no test files]
=== RUN   TestAccClientGrant
--- PASS: TestAccClientGrant (8.52s)
=== RUN   TestAccClient
--- PASS: TestAccClient (1.99s)
=== RUN   TestAccClientZeroValueCheck
--- PASS: TestAccClientZeroValueCheck (4.62s)
=== RUN   TestAccClientRotateSecret
--- PASS: TestAccClientRotateSecret (3.38s)
=== RUN   TestAccClientInitiateLoginUri
--- PASS: TestAccClientInitiateLoginUri (0.06s)
=== RUN   TestAccClientJwtScopes
--- PASS: TestAccClientJwtScopes (3.05s)
=== RUN   TestAccClientMobile
--- PASS: TestAccClientMobile (3.37s)
=== RUN   TestAccClientMobileValidationError
--- PASS: TestAccClientMobileValidationError (0.01s)
PASS
coverage: 20.8% of statements
ok  	github.com/alexkappa/terraform-provider-auth0/auth0	25.019s	coverage: 20.8% 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/random	0.011s	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.001s	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

@alexkappa
Copy link
Owner

@lpakula this looks good, and the tests are passing. I'll merge it in and release it with the next version. Thanks!

@alexkappa alexkappa merged commit 72ef8e5 into alexkappa:master Nov 9, 2021
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.

Add support for allowed_clients
2 participants