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

[WIP] Add (currently failing) integration test for ad connection strategy #29

Merged
merged 1 commit into from Sep 24, 2018

Conversation

rdark
Copy link
Contributor

@rdark rdark commented Aug 8, 2018

Do not merge in current state.

Added a quick test that illustrates the current behaviour with a vanilla ad connection strategy.

This results in a fail:

...
    --- FAIL: TestAccAdConnection (1.64s)
        testing.go:518: Step 0 error: After applying this step, the plan was not empty:
                
                DIFF:
                
                UPDATE: auth0_connection.my_ad_connection
                  options.#:                        "1" => "0"
                  options.0.brute_force_protection: "false" => "true"
                
                STATE:
                
                auth0_connection.my_ad_connection:
                  ID = con_XQEUZQX8P2AOfkRe
                  provider = provider.auth0
                  enabled_clients.# = 0
                  name = Acceptance-Test-Ad-Connection
                  options.# = 1
                  options.0.api_enable_users = false
                  options.0.basic_profile = false
                  options.0.brute_force_protection = false
                  options.0.disable_signup = false
                  options.0.enabled_database_customization = false
                  options.0.ext_admin = false
                  options.0.ext_agreed_terms = false
                  options.0.ext_assigned_plans = false
                  options.0.ext_groups = false
                  options.0.ext_is_suspended = false
                  options.0.ext_profile = false
                  options.0.import_mode = false
                  options.0.password_dictionary.% = 0
                  options.0.password_history.% = 0
                  options.0.password_no_personal_info.% = 0
                  options.0.password_policy = 
                  options.0.validation.% = 0
                  realms.# = 1
                  realms.0 = Acceptance-Test-Ad-Connection
                  strategy = ad

@rdark
Copy link
Contributor Author

rdark commented Aug 8, 2018

The brute_force_protection default option addressed in #28

@alexkappa alexkappa merged commit b3c633a into alexkappa:master Sep 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants