compute: Add 'server migration show' command

This replaces the 'server-migration-show' command provided by
novaclient.

Change-Id: I413310b481cc13b70853eb579417f6e6fad10d98
Signed-off-by: Stephen Finucane <sfinucan@redhat.com>

2 files changed, 171 insertions, 0 deletions

diff --git a/openstackclient/tests/unit/compute/v2/fakes.py b/openstackclient/tests/unit/compute/v2/fakes.py
index 7f3dcae4..4a2a44de 100644
--- a/openstackclient/tests/unit/compute/v2/fakes.py
+++ b/openstackclient/tests/unit/compute/v2/fakes.py
@@ -1633,6 +1633,59 @@ class FakeMigration(object):
         return migrations
 
 
+class FakeServerMigration(object):
+    """Fake one or more server migrations."""
+
+    @staticmethod
+    def create_one_server_migration(attrs=None, methods=None):
+        """Create a fake server migration.
+
+        :param Dictionary attrs:
+            A dictionary with all attributes
+        :param Dictionary methods:
+            A dictionary with all methods
+        :return:
+            A FakeResource object, with id, type, and so on
+        """
+        attrs = attrs or {}
+        methods = methods or {}
+
+        # Set default attributes.
+
+        migration_info = {
+            "created_at": "2016-01-29T13:42:02.000000",
+            "dest_compute": "compute2",
+            "dest_host": "1.2.3.4",
+            "dest_node": "node2",
+            "id": random.randint(1, 999),
+            "server_uuid": uuid.uuid4().hex,
+            "source_compute": "compute1",
+            "source_node": "node1",
+            "status": "running",
+            "memory_total_bytes": random.randint(1, 99999),
+            "memory_processed_bytes": random.randint(1, 99999),
+            "memory_remaining_bytes": random.randint(1, 99999),
+            "disk_total_bytes": random.randint(1, 99999),
+            "disk_processed_bytes": random.randint(1, 99999),
+            "disk_remaining_bytes": random.randint(1, 99999),
+            "updated_at": "2016-01-29T13:42:02.000000",
+            # added in 2.59
+            "uuid": uuid.uuid4().hex,
+            # added in 2.80
+            "user_id": uuid.uuid4().hex,
+            "project_id": uuid.uuid4().hex,
+        }
+
+        # Overwrite default attributes.
+        migration_info.update(attrs)
+
+        migration = fakes.FakeResource(
+            info=copy.deepcopy(migration_info),
+            methods=methods,
+            loaded=True)
+        return migration
+
+
 class FakeVolumeAttachment(object):
     """Fake one or more volume attachments (BDMs)."""
 
diff --git a/openstackclient/tests/unit/compute/v2/test_server.py b/openstackclient/tests/unit/compute/v2/test_server.py
index 5025f020..8d040472 100644
--- a/openstackclient/tests/unit/compute/v2/test_server.py
+++ b/openstackclient/tests/unit/compute/v2/test_server.py
@@ -4888,6 +4888,124 @@ class TestListMigrationV280(TestListMigration):
             str(ex))
 
 
+class TestServerMigrationShow(TestServer):
+
+    def setUp(self):
+        super().setUp()
+
+        self.server = compute_fakes.FakeServer.create_one_server()
+        self.servers_mock.get.return_value = self.server
+
+        self.server_migration = compute_fakes.FakeServerMigration\
+            .create_one_server_migration()
+        self.server_migrations_mock.get.return_value = self.server_migration
+
+        self.columns = (
+            'ID',
+            'Server UUID',
+            'Status',
+            'Source Compute',
+            'Source Node',
+            'Dest Compute',
+            'Dest Host',
+            'Dest Node',
+            'Memory Total Bytes',
+            'Memory Processed Bytes',
+            'Memory Remaining Bytes',
+            'Disk Total Bytes',
+            'Disk Processed Bytes',
+            'Disk Remaining Bytes',
+            'Created At',
+            'Updated At',
+        )
+
+        self.data = (
+            self.server_migration.id,
+            self.server_migration.server_uuid,
+            self.server_migration.status,
+            self.server_migration.source_compute,
+            self.server_migration.source_node,
+            self.server_migration.dest_compute,
+            self.server_migration.dest_host,
+            self.server_migration.dest_node,
+            self.server_migration.memory_total_bytes,
+            self.server_migration.memory_processed_bytes,
+            self.server_migration.memory_remaining_bytes,
+            self.server_migration.disk_total_bytes,
+            self.server_migration.disk_processed_bytes,
+            self.server_migration.disk_remaining_bytes,
+            self.server_migration.created_at,
+            self.server_migration.updated_at,
+        )
+
+        # Get the command object to test
+        self.cmd = server.ShowMigration(self.app, None)
+
+    def _test_server_migration_show(self):
+        arglist = [
+            self.server.id,
+            '2',  # arbitrary migration ID
+        ]
+        verifylist = []
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        columns, data = self.cmd.take_action(parsed_args)
+
+        self.assertEqual(self.columns, columns)
+        self.assertEqual(self.data, data)
+
+        self.servers_mock.get.assert_called_with(self.server.id)
+        self.server_migrations_mock.get.assert_called_with(
+            self.server.id, '2',)
+
+    def test_server_migration_show(self):
+        self.app.client_manager.compute.api_version = api_versions.APIVersion(
+            '2.24')
+
+        self._test_server_migration_show()
+
+    def test_server_migration_show_v259(self):
+        self.app.client_manager.compute.api_version = api_versions.APIVersion(
+            '2.59')
+
+        self.columns += ('UUID',)
+        self.data += (self.server_migration.uuid,)
+
+        self._test_server_migration_show()
+
+    def test_server_migration_show_v280(self):
+        self.app.client_manager.compute.api_version = api_versions.APIVersion(
+            '2.80')
+
+        self.columns += ('UUID', 'User ID', 'Project ID')
+        self.data += (
+            self.server_migration.uuid,
+            self.server_migration.user_id,
+            self.server_migration.project_id,
+        )
+
+        self._test_server_migration_show()
+
+    def test_server_migration_show_pre_v224(self):
+        self.app.client_manager.compute.api_version = api_versions.APIVersion(
+            '2.23')
+
+        arglist = [
+            self.server.id,
+            '2',  # arbitrary migration ID
+        ]
+        verifylist = []
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        ex = self.assertRaises(
+            exceptions.CommandError,
+            self.cmd.take_action,
+            parsed_args)
+        self.assertIn(
+            '--os-compute-api-version 2.24 or greater is required',
+            str(ex))
+
+
 class TestServerMigrationAbort(TestServer):
 
     def setUp(self):