diff options
| author | Tang Chen <tangchen@cn.fujitsu.com> | 2015-12-05 10:40:09 +0800 |
|---|---|---|
| committer | Tang Chen <tangchen@cn.fujitsu.com> | 2015-12-11 19:39:36 +0800 |
| commit | 2caf7b19e207f232530534ad2d7407af5e3355d9 (patch) | |
| tree | f1f82ebac7255490f52cce6f2d4579c167a6e35c /openstackclient | |
| parent | f65f82e4350757fcae93891c42ed8b01bb8577f2 (diff) | |
| download | python-openstackclient-2caf7b19e207f232530534ad2d7407af5e3355d9.tar.gz | |
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
Diffstat (limited to 'openstackclient')
| -rw-r--r-- | openstackclient/network/v2/network.py | 6 | ||||
| -rw-r--r-- | openstackclient/tests/network/v2/test_network.py | 2 |
2 files changed, 4 insertions, 4 deletions
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="<network>", 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) |
