summaryrefslogtreecommitdiff
path: root/openstackclient
diff options
context:
space:
mode:
authorZuul <zuul@review.opendev.org>2021-01-12 13:54:11 +0000
committerGerrit Code Review <review@openstack.org>2021-01-12 13:54:11 +0000
commitd6646d714b959d350b976defef00547c4da8d320 (patch)
tree27795baa2d6291356aa0ad6fe488843cac30d2d2 /openstackclient
parentd828b04c9f69075c1e38ff2c9cb3f26a356f11cc (diff)
parentdd89efd5acbabcd4de0f071a2f7fbb93d067a950 (diff)
downloadpython-openstackclient-d6646d714b959d350b976defef00547c4da8d320.tar.gz
Merge "network: Address nits for I3c313fc9329837dde67815901528a34dca98ebcc"
Diffstat (limited to 'openstackclient')
-rw-r--r--openstackclient/network/v2/address_group.py6
-rw-r--r--openstackclient/tests/unit/network/v2/test_address_group.py7
2 files changed, 4 insertions, 9 deletions
diff --git a/openstackclient/network/v2/address_group.py b/openstackclient/network/v2/address_group.py
index fe1e14a3..c5b2f126 100644
--- a/openstackclient/network/v2/address_group.py
+++ b/openstackclient/network/v2/address_group.py
@@ -36,10 +36,7 @@ def _get_columns(item):
def _format_addresses(addresses):
- ret = []
- for addr in addresses:
- ret.append(str(netaddr.IPNetwork(addr)))
- return ret
+ return [str(netaddr.IPNetwork(addr)) for addr in addresses]
def _get_attrs(client_manager, parsed_args):
@@ -185,7 +182,6 @@ class ListAddressGroup(command.Lister):
parsed_args.project,
parsed_args.project_domain,
).id
- attrs['tenant_id'] = project_id
attrs['project_id'] = project_id
data = client.address_groups(**attrs)
diff --git a/openstackclient/tests/unit/network/v2/test_address_group.py b/openstackclient/tests/unit/network/v2/test_address_group.py
index 3b2b1ab6..e4fa8ab3 100644
--- a/openstackclient/tests/unit/network/v2/test_address_group.py
+++ b/openstackclient/tests/unit/network/v2/test_address_group.py
@@ -282,7 +282,7 @@ class TestListAddressGroup(TestAddressGroup):
columns, data = self.cmd.take_action(parsed_args)
self.network.address_groups.assert_called_once_with(
- **{'tenant_id': project.id, 'project_id': project.id})
+ project_id=project.id)
self.assertEqual(self.columns, columns)
self.assertItemsEqual(self.data, list(data))
@@ -297,11 +297,10 @@ class TestListAddressGroup(TestAddressGroup):
('project', project.id),
]
parsed_args = self.check_parser(self.cmd, arglist, verifylist)
-
columns, data = self.cmd.take_action(parsed_args)
- filters = {'tenant_id': project.id, 'project_id': project.id}
- self.network.address_groups.assert_called_once_with(**filters)
+ self.network.address_groups.assert_called_once_with(
+ project_id=project.id)
self.assertEqual(self.columns, columns)
self.assertItemsEqual(self.data, list(data))