Merge "Network: Abstract get_body() out to be a private helper."

Author: Jenkins

openstackclient/network/v2/network.py

@@ -47,6 +47,33 @@ def _get_columns(item):
     return tuple(sorted(columns))
 
 
+def _get_attrs(client_manager, parsed_args):
+    attrs = {}
+    if parsed_args.name is not None:
+        attrs['name'] = str(parsed_args.name)
+    if parsed_args.admin_state is not None:
+        attrs['admin_state_up'] = parsed_args.admin_state
+    if parsed_args.shared is not None:
+        attrs['shared'] = parsed_args.shared
+
+    # "network set" command doesn't support setting project.
+    if 'project' in parsed_args and parsed_args.project is not None:
+        identity_client = client_manager.identity
+        project_id = identity_common.find_project(
+            identity_client,
+            parsed_args.project,
+            parsed_args.project_domain,
+        ).id
+        attrs['tenant_id'] = project_id
+
+    # "network set" command doesn't support setting availability zone hints.
+    if 'availability_zone_hints' in parsed_args and \
+       parsed_args.availability_zone_hints is not None:
+        attrs['availability_zone_hints'] = parsed_args.availability_zone_hints
+
+    return attrs
+
+
 class CreateNetwork(command.ShowOne):
     """Create new network"""
 
@@ -105,31 +132,14 @@ def get_parser(self, prog_name):
 
     def take_action(self, parsed_args):
         client = self.app.client_manager.network
-        body = self.get_body(parsed_args)
-        obj = client.create_network(**body)
+
+        attrs = _get_attrs(self.app.client_manager, parsed_args)
+        obj = client.create_network(**attrs)
         columns = _get_columns(obj)
+
         data = utils.get_item_properties(obj, columns, formatters=_formatters)
         return (columns, data)
 
-    def get_body(self, parsed_args):
-        body = {'name': str(parsed_args.name),
-                'admin_state_up': parsed_args.admin_state}
-        if parsed_args.shared is not None:
-            body['shared'] = parsed_args.shared
-        if parsed_args.project is not None:
-            identity_client = self.app.client_manager.identity
-            project_id = identity_common.find_project(
-                identity_client,
-                parsed_args.project,
-                parsed_args.project_domain,
-            ).id
-            body['tenant_id'] = project_id
-        if parsed_args.availability_zone_hints is not None:
-            body['availability_zone_hints'] = \
-                parsed_args.availability_zone_hints
-
-        return body
-
 
 class DeleteNetwork(command.Command):
     """Delete network(s)"""
@@ -271,18 +281,13 @@ def take_action(self, parsed_args):
         client = self.app.client_manager.network
         obj = client.find_network(parsed_args.identifier, ignore_missing=False)
 
-        if parsed_args.name is not None:
-            obj.name = str(parsed_args.name)
-        if parsed_args.admin_state is not None:
-            obj.admin_state_up = parsed_args.admin_state
-        if parsed_args.shared is not None:
-            obj.shared = parsed_args.shared
-
-        if not obj.is_dirty:
+        attrs = _get_attrs(self.app.client_manager, parsed_args)
+        if attrs == {}:
             msg = "Nothing specified to be set"
             raise exceptions.CommandError(msg)
 
-        client.update_network(obj)
+        client.update_network(obj, **attrs)
+        return
 
 
 class ShowNetwork(command.ShowOne):

openstackclient/tests/network/v2/fakes.py

@@ -96,7 +96,6 @@ def create_one_network(attrs={}, methods={}):
             'subnets': ['a', 'b'],
             'provider_network_type': 'vlan',
             'router_external': True,
-            'is_dirty': True,
             'availability_zones': [],
             'availability_zone_hints': [],
         }

openstackclient/tests/network/v2/test_network.py

@@ -440,8 +440,6 @@ def setUp(self):
         self.cmd = network.SetNetwork(self.app, self.namespace)
 
     def test_set_this(self):
-        self._network.is_dirty = True
-
         arglist = [
             self._network.name,
             '--enable',
@@ -458,12 +456,15 @@ def test_set_this(self):
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
         result = self.cmd.take_action(parsed_args)
 
-        self.network.update_network.assert_called_with(self._network)
+        attrs = {
+            'name': 'noob',
+            'admin_state_up': True,
+            'shared': True,
+        }
+        self.network.update_network.assert_called_with(self._network, **attrs)
         self.assertIsNone(result)
 
     def test_set_that(self):
-        self._network.is_dirty = True
-
         arglist = [
             self._network.name,
             '--disable',
@@ -478,12 +479,14 @@ def test_set_that(self):
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
         result = self.cmd.take_action(parsed_args)
 
-        self.network.update_network.assert_called_with(self._network)
+        attrs = {
+            'admin_state_up': False,
+            'shared': False,
+        }
+        self.network.update_network.assert_called_with(self._network, **attrs)
         self.assertIsNone(result)
 
     def test_set_nothing(self):
-        self._network.is_dirty = False
-
         arglist = [self._network.name, ]
         verifylist = [('identifier', self._network.name), ]