diff options
| author | Jan Cholasta <jcholast@redhat.com> | 2016-06-08 16:00:49 +0200 |
|---|---|---|
| committer | Jan Cholasta <jcholast@redhat.com> | 2016-06-20 16:39:12 +0200 |
| commit | f554078291d682d59956998af97f7d3066fbe7e7 (patch) | |
| tree | 536cda31a019bce7886dc4a9f64d23aa24e55f44 /ipatests | |
| parent | db13494045198842a049d40a3481d9081c7ce175 (diff) | |
| download | freeipa-f554078291d682d59956998af97f7d3066fbe7e7.tar.gz freeipa-f554078291d682d59956998af97f7d3066fbe7e7.tar.xz freeipa-f554078291d682d59956998af97f7d3066fbe7e7.zip | |
frontend: don't copy command arguments to output params
Use only object params and params defined in has_output_params as output
params. This removes unnecessary duplication of params defined both in
object plugins and as command arguments.
This requires all command output params to be properly defined in either
the object plugins or the command's has_output_params. Fix the plugins
where this wasn't true.
https://fedorahosted.org/freeipa/ticket/4739
Reviewed-By: David Kupka <dkupka@redhat.com>
Diffstat (limited to 'ipatests')
| -rw-r--r-- | ipatests/test_ipalib/test_frontend.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/ipatests/test_ipalib/test_frontend.py b/ipatests/test_ipalib/test_frontend.py index 518dadcda..93ab547b0 100644 --- a/ipatests/test_ipalib/test_frontend.py +++ b/ipatests/test_ipalib/test_frontend.py @@ -814,16 +814,16 @@ class test_Command(ClassChecker): 'foo', ) takes_options = ( - Str('bar', flags='no_output'), + Str('bar'), 'baz', ) inst = example(api) inst.finalize() assert list(inst.get_output_params()) == [ - 'one', 'two', 'three', inst.params.foo, inst.params.baz + 'one', 'two', 'three' ] - assert list(inst.output_params) == ['one', 'two', 'three', 'foo', 'baz'] + assert list(inst.output_params) == ['one', 'two', 'three'] class test_LocalOrRemote(ClassChecker): |