diff options
author | Petr Viktorin <pviktori@redhat.com> | 2012-04-17 12:48:33 -0400 |
---|---|---|
committer | Martin Kosek <mkosek@redhat.com> | 2012-05-14 10:38:07 +0200 |
commit | c5689e7fafc688165945e83dd4bf63dd568b3650 (patch) | |
tree | d1bd88acdf92fabde68c0fbfd3d25f089113547d /tests | |
parent | 95bb8d0f45c3061035633c7b9c3b1609b11a158d (diff) | |
download | freeipa-c5689e7fafc688165945e83dd4bf63dd568b3650.tar.gz freeipa-c5689e7fafc688165945e83dd4bf63dd568b3650.tar.xz freeipa-c5689e7fafc688165945e83dd4bf63dd568b3650.zip |
Do not use extra command options in ACI, permission, selfservice
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 ACI-related plugins.
Part of the work for https://fedorahosted.org/freeipa/ticket/2509
Diffstat (limited to 'tests')
-rw-r--r-- | tests/test_xmlrpc/test_selfservice_plugin.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tests/test_xmlrpc/test_selfservice_plugin.py b/tests/test_xmlrpc/test_selfservice_plugin.py index e60eb5d52..fa67cbc2d 100644 --- a/tests/test_xmlrpc/test_selfservice_plugin.py +++ b/tests/test_xmlrpc/test_selfservice_plugin.py @@ -46,7 +46,8 @@ class test_selfservice(Declarative): dict( desc='Try to update non-existent %r' % selfservice1, - command=('selfservice_mod', [selfservice1], dict(description=u'Foo')), + command=('selfservice_mod', [selfservice1], + dict(permissions=u'write')), expected=errors.NotFound( reason=u'ACI with name "%s" not found' % selfservice1), ), |