URL: https://github.com/SSSD/sssd/pull/5639 Author: sgoveas Title: #5639: TEST: Fixes after running new tests downstream Action: opened
PR body: """ tests have been synced downstream. Some test were failing or needed docstring updates for new polarion format """ To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5639/head:pr5639 git checkout pr5639
From efbe18e2352b710ef838042ee7c214273c05eaa8 Mon Sep 17 00:00:00 2001 From: Steeve Goveas <[email protected]> Date: Tue, 18 May 2021 16:14:20 +0530 Subject: [PATCH] TEST: Fixes after running new tests downstream tests have been synced downstream. Some test were failing or needed docstring updates for new polarion format --- src/tests/multihost/ad/test_sudo.py | 1 - .../multihost/alltests/test_automount.py | 21 +++++++++---------- src/tests/multihost/alltests/test_sudo.py | 4 ++-- src/tests/multihost/ipa/test_misc.py | 3 ++- 4 files changed, 14 insertions(+), 15 deletions(-) diff --git a/src/tests/multihost/ad/test_sudo.py b/src/tests/multihost/ad/test_sudo.py index f61bd4a675..26125dbc77 100644 --- a/src/tests/multihost/ad/test_sudo.py +++ b/src/tests/multihost/ad/test_sudo.py @@ -89,7 +89,6 @@ def test_002_bz1372440(self, multihost): 2. Add users to the group. 3. Verify sudo_idmuser1 can fetch the sudo rule 4. Run the required command as sudo - :expectedresults: 1. Should succeed 2. Should succeed diff --git a/src/tests/multihost/alltests/test_automount.py b/src/tests/multihost/alltests/test_automount.py index df6b75b89a..4607f14863 100644 --- a/src/tests/multihost/alltests/test_automount.py +++ b/src/tests/multihost/alltests/test_automount.py @@ -455,6 +455,7 @@ def test_008_wildcardsearch(self, multihost, indirect_nismaps, del_pcap = 'rm -f %s' % auto_pcapfile multihost.client[0].run_command(del_pcap) + @pytest.mark.parametrize('add_nisobject', ['/export'], indirect=True) @pytest.mark.tier2 def test_009_maps_after_coming_online(self, multihost, add_nisobject): """ @@ -462,19 +463,15 @@ def test_009_maps_after_coming_online(self, multihost, add_nisobject): :id: b9da6e0e-3d8b-4465-b435-338708d0d51e :bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1113639 :customerscenario: True - - :setup: + :steps: 1. edit sssd.conf and specify autofs_provider = ad 2. restart autofs - - :steps: - 1. firewalld block 389 and 636 - 2. stop sssd, autofs. - 3. remove sssd cache - 4. Start sssd - 5. remove firewall rule - 6. start autofs - + 3. firewalld block 389 and 636 + 4. stop sssd, autofs. + 5. remove sssd cache + 6. Start sssd + 7. remove firewall rule + 8. start autofs :expectedresults: 1. Should succeed 2. Should succeed @@ -482,6 +479,8 @@ def test_009_maps_after_coming_online(self, multihost, add_nisobject): 4. Should succeed 5. Should succeed 6. Should succeed + 7. Should succeed + 8. Should succeed """ multihost.master[0].run_command(['touch', '/export/nfs-test']) client = sssdTools(multihost.client[0]) diff --git a/src/tests/multihost/alltests/test_sudo.py b/src/tests/multihost/alltests/test_sudo.py index aa1d1790f7..ac7400b505 100644 --- a/src/tests/multihost/alltests/test_sudo.py +++ b/src/tests/multihost/alltests/test_sudo.py @@ -33,7 +33,7 @@ def test_bz1294670(self, multihost, backupsssdconf, localusers): multihost.client[0].service_sssd('stop') tools = sssdTools(multihost.client[0]) # remove sssd cache - tools.remove_sss_cache('/var/lib/sss/db') + tools.remove_sss_cache('/var/lib/sss/db/') tools = sssdTools(multihost.client[0]) ldap_uri = 'ldap://%s' % multihost.master[0].sys_hostname sssd_params = {'services': 'nss, pam, sudo'} @@ -87,7 +87,7 @@ def test_timed_sudoers_entry(self, # pylint: disable=unused-argument tools = sssdTools(multihost.client[0]) multihost.client[0].service_sssd('stop') - tools.remove_sss_cache('/var/lib/sss/db') + tools.remove_sss_cache('/var/lib/sss/db/') sudo_base = 'ou=sudoers,dc=example,dc=test' sudo_uri = "ldap://%s"; % multihost.master[0].sys_hostname params = {'ldap_sudo_search_base': sudo_base, diff --git a/src/tests/multihost/ipa/test_misc.py b/src/tests/multihost/ipa/test_misc.py index cd22f55d6d..a15ac5f08a 100644 --- a/src/tests/multihost/ipa/test_misc.py +++ b/src/tests/multihost/ipa/test_misc.py @@ -85,8 +85,9 @@ def test_filter_groups(self, multihost, default_ipa_groups, add_group_member, backupsssdconf): """ :title: filter_groups option partially filters the group from id - output of the user because gidNumber still appears in id output + output of the user because gidNumber still appears in id output :id: 8babb6ee-7141-4723-a79d-c5cf7879a9b4 + :customerscenario: True :description: filter_groups option partially filters the group from 'id' output of the user because gidNumber still appears in 'id' output
_______________________________________________ sssd-devel mailing list -- [email protected] To unsubscribe send an email to [email protected] Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/[email protected] Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
