New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ipatests: fix test_replica_promotion.py #4926
Conversation
Adding ipa-4-8 label as the failing tests were also backported to this branch with PR #4917 |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
LGTM |
LGTM as well. Please remove the temp commit. |
Apologies for inadvertently introducing these testing bugs in my PR, and a big thank you to @flo-renaud for fixing it. |
@jsf9k it's more my fault for not catching it in the first place. @flo-renaud reminder to remove the temp commit. |
A new test has been added to TestUnprivilegedUserPermissions that duplicates the steps done in the precedent test. As the tests are usually run sequentially, no need to duplicate. Fixes: https://pagure.io/freeipa/issue/8413
A new test was added to TestReplicaPromotionLevel1 but was run after the replica uninstallation. As the new test checks the content of /etc/sssd/sssd.conf on the replica, merge it with the previous test, when the replica is still installed. Fixes: https://pagure.io/freeipa/issue/8414
Rebased to master and removed the temp commit. |
ipatests: Fix TestReplicaPromotionLevel1
A new test was added to TestReplicaPromotionLevel1 but was run
after the replica uninstallation. As the new test checks
the content of /etc/sssd/sssd.conf on the replica, merge it with the
previous test, when the replica is still installed.
Fixes: https://pagure.io/freeipa/issue/8414
ipatests: fix TestUnprivilegedUserPermissions
A new test has been added to TestUnprivilegedUserPermissions that
duplicates the steps done in the precedent test. As the tests
are usually run sequentially, no need to duplicate.
Fixes: https://pagure.io/freeipa/issue/8413