diff options
| author | Petr Viktorin <pviktori@redhat.com> | 2012-04-16 09:29:42 -0400 |
|---|---|---|
| committer | Martin Kosek <mkosek@redhat.com> | 2012-05-07 14:08:50 +0200 |
| commit | c45174d68009076781431ad13a51bd454f6e3ce8 (patch) | |
| tree | a61cd82e5e0a3f1eb12c7e1c2dfb5585d3d23479 /tests | |
| parent | 343aba2486e4b2a406e6f023c2a0a2c24841796f (diff) | |
| download | freeipa.git-c45174d68009076781431ad13a51bd454f6e3ce8.tar.gz freeipa.git-c45174d68009076781431ad13a51bd454f6e3ce8.tar.xz freeipa.git-c45174d68009076781431ad13a51bd454f6e3ce8.zip | |
Do not use extra command options in the automount plugin
Allowing Commands to be called with ignored unknown options opens the
door to problems, for example with misspelled option names.
Before we start rejecting them, we need to make sure IPA itself does
not use them when it calls commands internally.
This patch does that for the automount plugin and its tests.
Part of the work for https://fedorahosted.org/freeipa/ticket/2509
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/test_xmlrpc/test_automount_plugin.py | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/tests/test_xmlrpc/test_automount_plugin.py b/tests/test_xmlrpc/test_automount_plugin.py index dedd2346..0dd65d87 100644 --- a/tests/test_xmlrpc/test_automount_plugin.py +++ b/tests/test_xmlrpc/test_automount_plugin.py @@ -145,7 +145,7 @@ class test_automount(XMLRPC_test): """ Test the `xmlrpc.automountkey_del` method. """ - delkey_kw={'automountkey': self.keyname_rename, 'automountinformation' : self.newinfo, 'raw': True} + delkey_kw={'automountkey': self.keyname_rename, 'automountinformation' : self.newinfo} res = api.Command['automountkey_del'](self.locname, self.mapname, **delkey_kw)['result'] assert res assert_attr_equal(res, 'failed', '') @@ -227,8 +227,7 @@ class test_automount_indirect(XMLRPC_test): mapname = u'auto.home' keyname = u'/home' parentmap = u'auto.master' - info = u'somehost:/homes' - map_kw = {'key': keyname, 'parentmap': parentmap, 'info': info, 'raw': True} + map_kw = {'key': keyname, 'parentmap': parentmap, 'raw': True} key_kw = {'automountkey': keyname, 'automountinformation': mapname} def test_0_automountlocation_add(self): |