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

[RFR] Fix test_login_local_group tests #9672

Merged
merged 1 commit into from
Nov 26, 2019

Conversation

john-dupuy
Copy link
Contributor

@john-dupuy john-dupuy commented Nov 25, 2019

These tests were facing an error because the auth setting change to NOT retrieve groups from the LDAP server takes some time to register. Therefore, when the LDAP user was logging into the appliance, MIQ pulled the LDAP groups rather than the local groups.

{{ pytest: cfme/tests/integration/test_cfme_auth.py::test_login_local_group --long-running --use-provider complete -k "ad_dev_2012-upn-ldapuser10" }}

@john-dupuy john-dupuy force-pushed the fix-test-login-local-group branch from 897d2ef to 5ce6cc2 Compare November 25, 2019 20:40
@john-dupuy john-dupuy changed the title [WIP] Fix test_login_local_group tests [WIPTEST] Fix test_login_local_group tests Nov 25, 2019
@dajoRH dajoRH added WIP-testing and removed WIP labels Nov 25, 2019
@dajoRH
Copy link
Contributor

dajoRH commented Nov 25, 2019

I detected some fixture changes in commit 5ce6cc2

The local fixture do_not_fetch_remote_groups is used in the following files:

  • cfme/tests/integration/test_cfme_auth.py

Please, consider creating a PRT run to make sure your fixture changes do not break existing usage 😃

@john-dupuy john-dupuy changed the title [WIPTEST] Fix test_login_local_group tests [RFR] Fix test_login_local_group tests Nov 26, 2019
@mshriver mshriver merged commit 2759117 into ManageIQ:master Nov 26, 2019
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.

3 participants