Merge "Fix properties format for volume qos in volume v1"

Author: Jenkins

openstackclient/tests/functional/volume/v1/test_qos.py

@@ -94,7 +94,7 @@ def test_volume_qos_set_show_unset(self):
         )
         self.assertEqual(
             "Alpha='c', Beta='b'",
-            cmd_output['specs']
+            cmd_output['properties']
         )
 
         # Test volume qos unset
@@ -115,7 +115,7 @@ def test_volume_qos_set_show_unset(self):
         )
         self.assertEqual(
             "Beta='b'",
-            cmd_output['specs']
+            cmd_output['properties']
         )
 
     # TODO(qiangjiahui): Add tests for associate and disassociate volume type

openstackclient/tests/unit/volume/v1/test_qos_specs.py

@@ -70,23 +70,22 @@ def test_qos_associate(self):
 
 class TestQosCreate(TestQos):
 
-    new_qos_spec = volume_fakes.FakeQos.create_one_qos()
     columns = (
         'consumer',
         'id',
         'name',
-        'specs'
-    )
-    datalist = (
-        new_qos_spec.consumer,
-        new_qos_spec.id,
-        new_qos_spec.name,
-        new_qos_spec.specs
+        'properties'
     )
 
     def setUp(self):
         super(TestQosCreate, self).setUp()
-
+        self.new_qos_spec = volume_fakes.FakeQos.create_one_qos()
+        self.datalist = (
+            self.new_qos_spec.consumer,
+            self.new_qos_spec.id,
+            self.new_qos_spec.name,
+            utils.format_dict(self.new_qos_spec.specs)
+        )
         self.qos_mock.create.return_value = self.new_qos_spec
         # Get the command object to test
         self.cmd = qos_specs.CreateQos(self.app, None)
@@ -336,7 +335,7 @@ def test_qos_list(self):
             'Name',
             'Consumer',
             'Associations',
-            'Specs',
+            'Properties',
         )
         self.assertEqual(collist, columns)
         datalist = ((
@@ -413,7 +412,7 @@ def test_qos_show(self):
             'consumer',
             'id',
             'name',
-            'specs'
+            'properties'
         )
         self.assertEqual(collist, columns)
         datalist = (

openstackclient/volume/v1/qos_specs.py

@@ -94,7 +94,9 @@ def take_action(self, parsed_args):
             specs.update(parsed_args.property)
 
         qos_spec = volume_client.qos_specs.create(parsed_args.name, specs)
-
+        qos_spec._info.update(
+            {'properties': utils.format_dict(qos_spec._info.pop('specs'))}
+        )
         return zip(*sorted(six.iteritems(qos_spec._info)))
 
 
@@ -190,8 +192,10 @@ def take_action(self, parsed_args):
                                 for association in qos_associations]
                 qos._info.update({'associations': associations})
 
+        display_columns = (
+            'ID', 'Name', 'Consumer', 'Associations', 'Properties')
         columns = ('ID', 'Name', 'Consumer', 'Associations', 'Specs')
-        return (columns,
+        return (display_columns,
                 (utils.get_dict_properties(
                     s._info, columns,
                     formatters={
@@ -254,7 +258,8 @@ def take_action(self, parsed_args):
             qos_spec._info.update({
                 'associations': utils.format_list(associations)
             })
-        qos_spec._info.update({'specs': utils.format_dict(qos_spec.specs)})
+        qos_spec._info.update(
+            {'properties': utils.format_dict(qos_spec._info.pop('specs'))})
 
         return zip(*sorted(six.iteritems(qos_spec._info)))