diff options
author | Stephen Gallagher <sgallagh@redhat.com> | 2009-12-07 14:50:48 -0500 |
---|---|---|
committer | Stephen Gallagher <sgallagh@redhat.com> | 2009-12-08 23:35:29 -0500 |
commit | afa260fcc56bde83e4b77282c7b3b22cfb4eca47 (patch) | |
tree | 48fb986e0e32629b198919e19c4fc26cb985ac70 /server/config/SSSDConfigTest.py | |
parent | 62bbadfe0aaa9348e3a05b5ce960e8c4e5a8d44b (diff) | |
download | sssd-afa260fcc56bde83e4b77282c7b3b22cfb4eca47.tar.gz sssd-afa260fcc56bde83e4b77282c7b3b22cfb4eca47.tar.bz2 sssd-afa260fcc56bde83e4b77282c7b3b22cfb4eca47.zip |
SSSDDomain.remove_provider() requires only the provider type
There was no valid reason to require the backend type when
specifying a provider to remove.
Diffstat (limited to 'server/config/SSSDConfigTest.py')
-rw-r--r-- | server/config/SSSDConfigTest.py | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/server/config/SSSDConfigTest.py b/server/config/SSSDConfigTest.py index ac37aec9..fa111819 100644 --- a/server/config/SSSDConfigTest.py +++ b/server/config/SSSDConfigTest.py @@ -142,7 +142,7 @@ class SSSDConfigTestValid(unittest.TestCase): ldap_domain = sssdconfig.get_domain('LDAP') ldap_domain.set_option('debug_level', 3) - ldap_domain.remove_provider('ldap', 'auth') + ldap_domain.remove_provider('auth') ldap_domain.add_provider('krb5', 'auth') ldap_domain.set_active(True) sssdconfig.save_domain(ldap_domain) @@ -455,7 +455,7 @@ class SSSDConfigTestSSSDDomain(unittest.TestCase): # Remove the auth domain and verify that the options # revert to the backup_list - domain.remove_provider('krb5', 'auth') + domain.remove_provider('auth') options = domain.list_options() self.assertTrue(type(options) == dict, @@ -666,7 +666,7 @@ class SSSDConfigTestSSSDDomain(unittest.TestCase): # Remove the auth domain and verify that the options # revert to the backup_list - domain.remove_provider('krb5', 'auth') + domain.remove_provider('auth') options = domain.list_options() self.assertTrue(type(options) == dict, @@ -685,15 +685,15 @@ class SSSDConfigTestSSSDDomain(unittest.TestCase): option) # Test removing nonexistent provider - Real - domain.remove_provider('ldap', 'id') + domain.remove_provider('id') # Test removing nonexistent provider - Bad backend type # Should pass without complaint - domain.remove_provider('nosuchbackend', 'id') + domain.remove_provider('id') # Test removing nonexistent provider - Bad provider type # Should pass without complaint - domain.remove_provider('ldap', 'nosuchprovider') + domain.remove_provider('nosuchprovider') def testGetOption(self): domain = SSSDConfig.SSSDDomain('sssd', self.schema) |