trivial: Make better use of argparse

Change-Id: Ib76c0d18bf5e44bfb4dcd9d729d9a1c5635cdba7
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>

Author: stephenfin

openstackclient/tests/unit/volume/v2/test_volume_type.py

@@ -173,6 +173,7 @@ def test_public_type_create_with_project_public(self):
             self.new_volume_type.name,
         ]
         verifylist = [
+            ('is_public', None),
             ('project', self.project.id),
             ('name', self.new_volume_type.name),
         ]

openstackclient/volume/v2/volume_type.py

@@ -250,11 +250,14 @@ def take_action(self, parsed_args):
             raise exceptions.CommandError(msg)
 
         kwargs = {}
+
         if parsed_args.is_public is not None:
             kwargs['is_public'] = parsed_args.is_public
 
         volume_type = volume_client.volume_types.create(
-            parsed_args.name, description=parsed_args.description, **kwargs
+            parsed_args.name,
+            description=parsed_args.description,
+            **kwargs,
         )
         volume_type._info.pop('extra_specs')