From 2d4a7371601290abca1878f07b952dc8d9deff02 Mon Sep 17 00:00:00 2001
From: TerryHowe <terrylhowe@gmail.com>
Date: Wed, 8 Jul 2015 11:21:41 -0600
Subject: Remove the --dhcp option to network list

The --dhcp option lists agents, not networks.  This does not make
a lot of sense.  Another command should be created to list agents.

BackwardsIncompatibleImpact
Closes-Bug: #1472613

Change-Id: I5ecfe3fc046a07eb64a4dabd41dbd99de7c2215f
---
 openstackclient/network/v2/network.py            | 64 ++++++++++--------------
 openstackclient/tests/network/v2/test_network.py | 48 ------------------
 2 files changed, 27 insertions(+), 85 deletions(-)

(limited to 'openstackclient')

diff --git a/openstackclient/network/v2/network.py b/openstackclient/network/v2/network.py
index 1fa05462..336b3086 100644
--- a/openstackclient/network/v2/network.py
+++ b/openstackclient/network/v2/network.py
@@ -156,10 +156,6 @@ class ListNetwork(lister.Lister):
             default=False,
             help='List external networks',
         )
-        parser.add_argument(
-            '--dhcp',
-            metavar='<dhcp-id>',
-            help='DHCP agent ID')
         parser.add_argument(
             '--long',
             action='store_true',
@@ -172,40 +168,34 @@ class ListNetwork(lister.Lister):
         self.log.debug('take_action(%s)' % parsed_args)
         client = self.app.client_manager.network
 
-        if parsed_args.dhcp:
-            data = client.api.dhcp_agent_list(dhcp_id=parsed_args.dhcp)
-
-            columns = ('ID',)
-            column_headers = columns
+        data = client.api.network_list(external=parsed_args.external)
+
+        if parsed_args.long:
+            columns = (
+                'ID',
+                'Name',
+                'Status',
+                'project_id',
+                'state',
+                'Shared',
+                'Subnets',
+                'provider:network_type',
+                'router_type',
+            )
+            column_headers = (
+                'ID',
+                'Name',
+                'Status',
+                'Project',
+                'State',
+                'Shared',
+                'Subnets',
+                'Network Type',
+                'Router Type',
+            )
         else:
-            data = client.api.network_list(external=parsed_args.external)
-
-            if parsed_args.long:
-                columns = (
-                    'ID',
-                    'Name',
-                    'Status',
-                    'project_id',
-                    'state',
-                    'Shared',
-                    'Subnets',
-                    'provider:network_type',
-                    'router_type',
-                )
-                column_headers = (
-                    'ID',
-                    'Name',
-                    'Status',
-                    'Project',
-                    'State',
-                    'Shared',
-                    'Subnets',
-                    'Network Type',
-                    'Router Type',
-                )
-            else:
-                columns = ('ID', 'Name', 'Subnets')
-                column_headers = columns
+            columns = ('ID', 'Name', 'Subnets')
+            column_headers = columns
 
         for d in data:
             d = _prep_network_detail(d)
diff --git a/openstackclient/tests/network/v2/test_network.py b/openstackclient/tests/network/v2/test_network.py
index 36133a3b..61541823 100644
--- a/openstackclient/tests/network/v2/test_network.py
+++ b/openstackclient/tests/network/v2/test_network.py
@@ -278,7 +278,6 @@ class TestListNetwork(common.TestNetworkBase):
         arglist = []
         verifylist = [
             ('external', False),
-            ('dhcp', None),
             ('long', False),
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
@@ -306,7 +305,6 @@ class TestListNetwork(common.TestNetworkBase):
         ]
         verifylist = [
             ('external', True),
-            ('dhcp', None),
             ('long', False),
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
@@ -334,7 +332,6 @@ class TestListNetwork(common.TestNetworkBase):
         ]
         verifylist = [
             ('long', True),
-            ('dhcp', None),
             ('external', False),
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
@@ -377,51 +374,6 @@ class TestListNetwork(common.TestNetworkBase):
         self.assertEqual(list(data), datalist)
 
 
-@mock.patch(
-    'openstackclient.api.network_v2.APIv2.dhcp_agent_list'
-)
-class TestListDhcpAgent(common.TestNetworkBase):
-
-    def setUp(self):
-        super(TestListDhcpAgent, self).setUp()
-
-        # Get the command object to test
-        self.cmd = network.ListNetwork(self.app, self.namespace)
-
-        self.DHCP_LIST = [
-            {'id': '1'},
-            {'id': '2'},
-        ]
-
-    def test_list_dhcp(self, n_mock):
-        n_mock.return_value = self.DHCP_LIST
-
-        arglist = [
-            '--dhcp', 'dhcpid',
-        ]
-        verifylist = [
-            ('external', False),
-            ('dhcp', 'dhcpid'),
-            ('long', False),
-        ]
-        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
-
-        # DisplayCommandBase.take_action() returns two tuples
-        columns, data = self.cmd.take_action(parsed_args)
-
-        # Set expected values
-        n_mock.assert_called_with(
-            dhcp_id='dhcpid',
-        )
-
-        self.assertEqual(('ID',), columns)
-        datalist = [
-            ('1',),
-            ('2',),
-        ]
-        self.assertEqual(datalist, list(data))
-
-
 class TestSetNetwork(common.TestNetworkBase):
     def test_set_this(self):
         arglist = [
-- 
cgit v1.2.1