diff options
| author | Zuul <zuul@review.opendev.org> | 2020-10-14 03:46:00 +0000 |
|---|---|---|
| committer | Gerrit Code Review <review@openstack.org> | 2020-10-14 03:46:00 +0000 |
| commit | f083d42972ec1678ad705b4e1984ba2944a07da2 (patch) | |
| tree | 9937f81e0b187bcc343b8dc987d46d3f56b9e573 /openstackclient/tests/unit | |
| parent | 91027a501f61e4e884b304544bbc2b729f8f8f20 (diff) | |
| parent | 08b0e5855be5dc12a2fd04c5c92027c8a85a0d00 (diff) | |
| download | python-openstackclient-f083d42972ec1678ad705b4e1984ba2944a07da2.tar.gz | |
Merge "Add 'server migration force complete' command"
Diffstat (limited to 'openstackclient/tests/unit')
| -rw-r--r-- | openstackclient/tests/unit/compute/v2/test_server.py | 51 |
1 files changed, 51 insertions, 0 deletions
diff --git a/openstackclient/tests/unit/compute/v2/test_server.py b/openstackclient/tests/unit/compute/v2/test_server.py index 9eb3a20c..100559f3 100644 --- a/openstackclient/tests/unit/compute/v2/test_server.py +++ b/openstackclient/tests/unit/compute/v2/test_server.py @@ -4288,6 +4288,57 @@ class TestServerMigrationAbort(TestServer): str(ex)) +class TestServerMigrationForceComplete(TestServer): + + def setUp(self): + super(TestServerMigrationForceComplete, self).setUp() + + self.server = compute_fakes.FakeServer.create_one_server() + + # Return value for utils.find_resource for server. + self.servers_mock.get.return_value = self.server + + # Get the command object to test + self.cmd = server.ForceCompleteMigration(self.app, None) + + def test_migration_force_complete(self): + self.app.client_manager.compute.api_version = api_versions.APIVersion( + '2.22') + + arglist = [ + self.server.id, + '2', # arbitrary migration ID + ] + verifylist = [] + parsed_args = self.check_parser(self.cmd, arglist, verifylist) + + result = self.cmd.take_action(parsed_args) + + self.servers_mock.get.assert_called_with(self.server.id) + self.server_migrations_mock.live_migrate_force_complete\ + .assert_called_with(self.server.id, '2',) + self.assertIsNone(result) + + def test_migration_force_complete_pre_v222(self): + self.app.client_manager.compute.api_version = api_versions.APIVersion( + '2.21') + + 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.22 or greater is required', + str(ex)) + + class TestServerPause(TestServer): def setUp(self): |
