diff options
| author | Thrivikram Mudunuri <mthrivikram+opendev@gmail.com> | 2021-11-30 00:56:39 -0500 |
|---|---|---|
| committer | Stephen Finucane <sfinucan@redhat.com> | 2022-12-19 16:56:20 +0000 |
| commit | 9241514137cd8d45fa6f36808536c5d0f218cb97 (patch) | |
| tree | c434cc6820755831b830347269c51577f3608ed9 /openstackclient/tests | |
| parent | ce8171bad90f15ac8a753f50bb3792862b66cc7c (diff) | |
| download | python-openstackclient-9241514137cd8d45fa6f36808536c5d0f218cb97.tar.gz | |
Switch server start, server stop to SDK
Switch the server start and server stop commands from novaclient to SDK.
Change-Id: I5ebfa6b2468d5f20b99ea0eab1aea9377be09b8c
Diffstat (limited to 'openstackclient/tests')
| -rw-r--r-- | openstackclient/tests/unit/compute/v2/test_server.py | 50 |
1 files changed, 19 insertions, 31 deletions
diff --git a/openstackclient/tests/unit/compute/v2/test_server.py b/openstackclient/tests/unit/compute/v2/test_server.py index 01c71b6c..8cb8ceb5 100644 --- a/openstackclient/tests/unit/compute/v2/test_server.py +++ b/openstackclient/tests/unit/compute/v2/test_server.py @@ -8217,28 +8217,19 @@ class TestServerSsh(TestServer): class TestServerStart(TestServer): def setUp(self): - super(TestServerStart, self).setUp() + super().setUp() # Get the command object to test self.cmd = server.StartServer(self.app, None) - # Set methods to be tested. - self.methods = { - 'start': None, - } - def test_server_start_one_server(self): - self.run_method_with_servers('start', 1) + self.run_method_with_sdk_servers('start_server', 1) def test_server_start_multi_servers(self): - self.run_method_with_servers('start', 3) + self.run_method_with_sdk_servers('start_server', 3) - @mock.patch.object(common_utils, 'find_resource') - def test_server_start_with_all_projects(self, mock_find_resource): - servers = self.setup_servers_mock(count=1) - mock_find_resource.side_effect = compute_fakes.FakeServer.get_servers( - servers, 0, - ) + def test_server_start_with_all_projects(self): + servers = self.setup_sdk_servers_mock(count=1) arglist = [ servers[0].id, @@ -8251,36 +8242,30 @@ class TestServerStart(TestServer): self.cmd.take_action(parsed_args) - mock_find_resource.assert_called_once_with( - mock.ANY, servers[0].id, all_tenants=True, + self.sdk_client.find_server.assert_called_once_with( + name=servers[0].id, + details=False, + all_projects=True, + ignore_missing=False, ) class TestServerStop(TestServer): def setUp(self): - super(TestServerStop, self).setUp() + super().setUp() # Get the command object to test self.cmd = server.StopServer(self.app, None) - # Set methods to be tested. - self.methods = { - 'stop': None, - } - def test_server_stop_one_server(self): - self.run_method_with_servers('stop', 1) + self.run_method_with_sdk_servers('stop_server', 1) def test_server_stop_multi_servers(self): - self.run_method_with_servers('stop', 3) + self.run_method_with_sdk_servers('stop_server', 3) - @mock.patch.object(common_utils, 'find_resource') - def test_server_start_with_all_projects(self, mock_find_resource): + def test_server_start_with_all_projects(self): servers = self.setup_servers_mock(count=1) - mock_find_resource.side_effect = compute_fakes.FakeServer.get_servers( - servers, 0, - ) arglist = [ servers[0].id, @@ -8293,8 +8278,11 @@ class TestServerStop(TestServer): self.cmd.take_action(parsed_args) - mock_find_resource.assert_called_once_with( - mock.ANY, servers[0].id, all_tenants=True, + self.sdk_client.find_server.assert_called_once_with( + name=servers[0].id, + details=False, + all_projects=True, + ignore_missing=False, ) |