diff --git a/gbpclient/gbp/v2_0/groupbasedpolicy.py b/gbpclient/gbp/v2_0/groupbasedpolicy.py index 46e851c..cfddec2 100644 --- a/gbpclient/gbp/v2_0/groupbasedpolicy.py +++ b/gbpclient/gbp/v2_0/groupbasedpolicy.py @@ -136,11 +136,11 @@ class CreatePolicyTargetGroup(neutronV20.CreateCommand): help=_('L2 policy uuid')) parser.add_argument( '--provided-policy-rule-sets', type=utils.str2dict, - default={}, + # default={}, help=_('Dictionary of provided policy rule set uuids')) parser.add_argument( '--consumed-policy-rule-sets', type=utils.str2dict, - default={}, + # default={}, help=_('Dictionary of consumed policy rule set uuids')) parser.add_argument( '--network-service-policy', metavar='NETWORK_SERVICE_POLICY', diff --git a/gbpclient/tests/unit/test_cli20_policytargetgroup.py b/gbpclient/tests/unit/test_cli20_policytargetgroup.py new file mode 100644 index 0000000..aea8603 --- /dev/null +++ b/gbpclient/tests/unit/test_cli20_policytargetgroup.py @@ -0,0 +1,71 @@ +# Licensed under the Apache License, Version 2.0 (the "License"); you may +# not use this file except in compliance with the License. You may obtain +# a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT +# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the +# License for the specific language governing permissions and limitations +# under the License. +# + +import logging +import sys + +from gbpclient.gbp.v2_0 import groupbasedpolicy as gbp +from gbpclient.tests.unit import test_cli20 + + +class CLITestV20PolicyTargetGroupJSON(test_cli20.CLITestV20Base): + + LOG = logging.getLogger(__name__) + + def setUp(self): + super(CLITestV20PolicyTargetGroupJSON, self).setUp() + + def test_create_policy_target_group_with_mandatory_params(self): + """policy-target-group-create with all mandatory params.""" + resource = 'policy_target_group' + cmd = gbp.CreatePolicyTargetGroup(test_cli20.MyApp(sys.stdout), None) + name = 'my-name' + tenant_id = 'my-tenant' + my_id = 'my-id' + args = ['--tenant-id', tenant_id, + name] + position_names = ['name', ] + position_values = [name, ] + self._test_create_resource(resource, cmd, name, my_id, args, + position_names, position_values, + tenant_id=tenant_id) + + def test_list_policy_target_groups(self): + """policy-target-group-list.""" + resource = 'policy_target_groups' + cmd = gbp.ListPolicyTargetGroup(test_cli20.MyApp(sys.stdout), None) + self._test_list_resources(resource, cmd, True) + + def test_show_policy_target_group_name(self): + """policy-target-group-show.""" + resource = 'policy_target_group' + cmd = gbp.ShowPolicyTargetGroup(test_cli20.MyApp(sys.stdout), None) + args = ['--fields', 'id', self.test_id] + self._test_show_resource(resource, cmd, self.test_id, args, ['id']) + + def test_update_policy_target_group_action(self): + """policy-policy-action-update.""" + resource = 'policy_target_group' + cmd = gbp.UpdatePolicyTargetGroup(test_cli20.MyApp(sys.stdout), None) + self._test_update_resource(resource, cmd, 'myid', + ['myid', '--name', 'myname', + '--tags', 'a', 'b'], + {'name': 'myname', 'tags': ['a', 'b'], }) + + def test_delete_policy_target_group_name(self): + """policy-target-group-delete.""" + resource = 'policy_target_group' + cmd = gbp.DeletePolicyTargetGroup(test_cli20.MyApp(sys.stdout), None) + my_id = 'my-id' + args = [my_id] + self._test_delete_resource(resource, cmd, my_id, args)