ipatests: fix test_authselect

Before the code fix, install/uninstall on a config without
any authselect profile was not able to restore the exact
state but configured sssd profile instead.

Now that the code is doing a pre-install backup, uninstall
restores the exact state and the test needs to be updated
accordingly.

Related: https://pagure.io/freeipa/issue/8189
Reviewed-By: Michal Polovka <mpolovka@redhat.com>
Reviewed-By: Francois Cami <fcami@redhat.com>
This commit is contained in:
Florence Blanc-Renaud 2020-07-29 17:57:53 +02:00
parent aac570bb45
commit 143b23cb75

View File

@ -100,7 +100,9 @@ class TestClientInstallation(IntegrationTest):
['rm', '-f', '/etc/authselect/authselect.conf'])
result = self._install_client()
assert result.returncode == 0
assert self.msg_warn_install in result.stderr_text
# With the fix for 8189, there is no warning any more
# because install is performing a pre-install backup
assert self.msg_warn_install not in result.stderr_text
# Client installation must configure the 'sssd' profile
# with sudo
check_authselect_profile(self.client, default_profile, ('with-sudo',))
@ -109,12 +111,13 @@ class TestClientInstallation(IntegrationTest):
"""
Test client un-installation when there was no authselect profile
"""
# As the client did not have any authselect profile before install,
# uninstall must print a warning about restoring 'sssd' profile
# by default
# The client did not have any authselect profile before install,
# but uninstall must be able to restore the backup
# Check that no profile is configured after uninstall
result = self._uninstall_client()
assert result.returncode == 0
check_authselect_profile(self.client, default_profile)
assert not self.client.transport.file_exists(
'/etc/authselect/authselect.conf')
def test_install_client_preconfigured_profile(self):
"""