Add datastore-version-show to OSC

This change adds database support to the python-openstackclient
project for the datastore-version-show command.

The trove command datastore-version-show is now:
    openstack datastore version show

Change-Id: I14844d887565bc58d44181db7ed3da454dc574d0
Partially-Implements: trove-support-in-python-openstackclient

2 files changed, 77 insertions, 0 deletions

diff --git a/troveclient/osc/v1/datastores.py b/troveclient/osc/v1/datastores.py
index d442a6c..85983f0 100644
--- a/troveclient/osc/v1/datastores.py
+++ b/troveclient/osc/v1/datastores.py
@@ -16,7 +16,9 @@ from osc_lib.command import command
 from osc_lib import utils
 import six
 
+from troveclient import exceptions
 from troveclient.i18n import _
+from troveclient import utils as tc_utils
 
 
 def set_attributes_for_print_detail(datastore):
@@ -86,3 +88,40 @@ class ListDatastoreVersions(command.Lister):
         versions = datastore_version_client.list(parsed_args.datastore)
         ds = [utils.get_item_properties(d, self.columns) for d in versions]
         return self.columns, ds
+
+
+class ShowDatastoreVersion(command.ShowOne):
+    _description = _("Shows details of a datastore version.")
+
+    def get_parser(self, prog_name):
+        parser = super(ShowDatastoreVersion, self).get_parser(prog_name)
+        parser.add_argument(
+            'datastore_version',
+            metavar='<datastore_version>',
+            help=_('ID or name of the datastore version.'),
+        )
+        parser.add_argument(
+            '--datastore',
+            metavar='<datastore>',
+            default=None,
+            help=_('ID or name of the datastore. Optional if the ID of'
+                   'the datastore_version is provided.'),
+        )
+        return parser
+
+    def take_action(self, parsed_args):
+        datastore_version_client =\
+            self.app.client_manager.database.datastore_versions
+        if parsed_args.datastore:
+            datastore_version = datastore_version_client.\
+                get(parsed_args.datastore, parsed_args.datastore_version)
+        elif tc_utils.is_uuid_like(parsed_args.datastore_version):
+            datastore_version = datastore_version_client.\
+                get_by_uuid(parsed_args.datastore_version)
+        else:
+            raise exceptions.NoUniqueMatch(_('The datastore name or id is'
+                                             ' required to retrieve a'
+                                             ' datastore version by name.'))
+        if datastore_version._info.get('links'):
+            del(datastore_version._info['links'])
+        return zip(*sorted(six.iteritems(datastore_version._info)))
diff --git a/troveclient/tests/osc/v1/test_datastores.py b/troveclient/tests/osc/v1/test_datastores.py
index 4a58667..c9e5806 100644
--- a/troveclient/tests/osc/v1/test_datastores.py
+++ b/troveclient/tests/osc/v1/test_datastores.py
@@ -11,6 +11,7 @@
 #   under the License.
 
 from troveclient import common
+from troveclient import exceptions
 from troveclient.osc.v1 import datastores
 from troveclient.tests.osc.v1 import fakes
 
@@ -85,3 +86,40 @@ class TestDatastoreVersionList(TestDatastores):
         self.datastore_version_client.list.assert_called_once_with(args[0])
         self.assertEqual(self.columns, columns)
         self.assertEqual([self.values], data)
+
+
+class TestDatastoreVersionShow(TestDatastores):
+    values = ('v-56', '5.6')
+
+    def setUp(self):
+        super(TestDatastoreVersionShow, self).setUp()
+        self.cmd = datastores.ShowDatastoreVersion(self.app, None)
+        self.data = self.fake_datastores.get_datastores_d_123_versions()
+        self.datastore_version_client.get.return_value = self.data
+        self.columns = (
+            'id',
+            'name',
+        )
+
+    def test_datastore_version_show_defaults(self):
+        args = ['5.6', '--datastore', 'mysql']
+        verifylist = [
+            ('datastore_version', '5.6'),
+            ('datastore', 'mysql'),
+        ]
+        parsed_args = self.check_parser(self.cmd, args, verifylist)
+        columns, data = self.cmd.take_action(parsed_args)
+        self.assertEqual(self.columns, columns)
+        self.assertEqual(self.values, data)
+
+    def test_datastore_version_show_with_version_id_exception(self):
+        args = [
+            'v-56',
+        ]
+        verifylist = [
+            ('datastore_version', 'v-56'),
+        ]
+        parsed_args = self.check_parser(self.cmd, args, verifylist)
+        self.assertRaises(exceptions.NoUniqueMatch,
+                          self.cmd.take_action,
+                          parsed_args)