Merge "Remove None valued network quota entries"

Author: Zuul

openstackclient/common/quota.py

@@ -161,6 +161,13 @@ def get_volume_quota(self, client, parsed_args):
                 raise
         return quota._info
 
+    def _network_quota_to_dict(self, network_quota):
+        if type(network_quota) is not dict:
+            dict_quota = network_quota.to_dict()
+        else:
+            dict_quota = network_quota
+        return {k: v for k, v in dict_quota.items() if v is not None}
+
     def get_network_quota(self, parsed_args):
         quota_class = (
             parsed_args.quota_class if 'quota_class' in parsed_args else False)
@@ -174,13 +181,11 @@ def get_network_quota(self, parsed_args):
             client = self.app.client_manager.network
             if default:
                 network_quota = client.get_quota_default(project)
-                if type(network_quota) is not dict:
-                    network_quota = network_quota.to_dict()
+                network_quota = self._network_quota_to_dict(network_quota)
             else:
                 network_quota = client.get_quota(project,
                                                  details=detail)
-                if type(network_quota) is not dict:
-                    network_quota = network_quota.to_dict()
+                network_quota = self._network_quota_to_dict(network_quota)
                 if detail:
                     # NOTE(slaweq): Neutron returns values with key "used" but
                     # Nova for example returns same data with key "in_use"

openstackclient/tests/unit/common/test_quota.py

@@ -1087,3 +1087,26 @@ def test_quota_show_no_project(self):
             identity_fakes.project_id, details=False
         )
         self.assertNotCalled(self.network.get_quota_default)
+
+    def test_network_quota_show_remove_empty(self):
+        arglist = [
+            self.projects[0].name,
+        ]
+        verifylist = [
+            ('project', self.projects[0].name),
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        # First check that all regular values are returned
+        result = self.cmd.get_network_quota(parsed_args)
+        self.assertEqual(len(network_fakes.QUOTA), len(result))
+
+        # set 1 of the values to None, and verify it is not returned
+        orig_get_quota = self.network.get_quota
+        network_quotas = copy.copy(network_fakes.QUOTA)
+        network_quotas['healthmonitor'] = None
+        self.network.get_quota = mock.Mock(return_value=network_quotas)
+        result = self.cmd.get_network_quota(parsed_args)
+        self.assertEqual(len(network_fakes.QUOTA) - 1, len(result))
+        # Go back to default mock
+        self.network.get_quota = orig_get_quota

releasenotes/notes/remove-nlbaas-quota-8b38e0c91ab113cb.yaml

@@ -0,0 +1,4 @@
+---
+other:
+  - |
+    Remove deprecated neutron-lbaas results from ``quota show`` command.