From 2caf7b19e207f232530534ad2d7407af5e3355d9 Mon Sep 17 00:00:00 2001 From: Tang Chen Date: Sat, 5 Dec 2015 10:40:09 +0800 Subject: Trivial: Do not use plural format in command parameter in "network delete" Since "network delete" could delete more than one network, add a (s) to the doc. And also, rename the parameter "networks" to "network". The naming style is not using plural format in parameter. Change-Id: Id434ea905af34457f84ea6bcb18addef5800429a --- openstackclient/network/v2/network.py | 6 +++--- openstackclient/tests/network/v2/test_network.py | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) (limited to 'openstackclient') diff --git a/openstackclient/network/v2/network.py b/openstackclient/network/v2/network.py index 64b98f5d..519356b4 100644 --- a/openstackclient/network/v2/network.py +++ b/openstackclient/network/v2/network.py @@ -119,17 +119,17 @@ class DeleteNetwork(command.Command): def get_parser(self, prog_name): parser = super(DeleteNetwork, self).get_parser(prog_name) parser.add_argument( - 'networks', + 'network', metavar="", nargs="+", - help=("Network to delete (name or ID)") + help=("Network(s) to delete (name or ID)") ) return parser def take_action(self, parsed_args): self.log.debug('take_action(%s)' % parsed_args) client = self.app.client_manager.network - for network in parsed_args.networks: + for network in parsed_args.network: obj = client.find_network(network) client.delete_network(obj) return diff --git a/openstackclient/tests/network/v2/test_network.py b/openstackclient/tests/network/v2/test_network.py index 4c897bcb..e8651492 100644 --- a/openstackclient/tests/network/v2/test_network.py +++ b/openstackclient/tests/network/v2/test_network.py @@ -284,7 +284,7 @@ class TestDeleteNetwork(TestNetwork): self._network.name, ] verifylist = [ - ('networks', [self._network.name]), + ('network', [self._network.name]), ] parsed_args = self.check_parser(self.cmd, arglist, verifylist) -- cgit v1.2.1