summaryrefslogtreecommitdiffstats
path: root/nova/tests
diff options
context:
space:
mode:
Diffstat (limited to 'nova/tests')
-rw-r--r--nova/tests/cert/test_rpcapi.py7
-rw-r--r--nova/tests/compute/test_compute.py31
-rw-r--r--nova/tests/compute/test_compute_cells.py3
-rw-r--r--nova/tests/compute/test_rpcapi.py3
-rw-r--r--nova/tests/conductor/test_conductor.py24
-rw-r--r--nova/tests/console/test_console.py10
-rw-r--r--nova/tests/console/test_rpcapi.py7
-rw-r--r--nova/tests/consoleauth/test_consoleauth.py5
-rw-r--r--nova/tests/consoleauth/test_rpcapi.py7
-rw-r--r--nova/tests/fake_policy.py1
-rw-r--r--nova/tests/network/test_api.py12
-rw-r--r--nova/tests/network/test_manager.py13
-rw-r--r--nova/tests/network/test_rpcapi.py6
-rw-r--r--nova/tests/scheduler/test_rpcapi.py7
-rw-r--r--nova/tests/test_baserpc.py5
-rw-r--r--nova/tests/test_service.py2
16 files changed, 7 insertions, 136 deletions
diff --git a/nova/tests/cert/test_rpcapi.py b/nova/tests/cert/test_rpcapi.py
index b743ca1e5..bacbfc194 100644
--- a/nova/tests/cert/test_rpcapi.py
+++ b/nova/tests/cert/test_rpcapi.py
@@ -37,9 +37,6 @@ class CertRpcAPITestCase(test.TestCase):
expected_msg = rpcapi.make_msg(method, **kwargs)
expected_msg['version'] = expected_version
- if method == 'get_backdoor_port':
- del expected_msg['args']['host']
-
self.call_ctxt = None
self.call_topic = None
self.call_msg = None
@@ -88,7 +85,3 @@ class CertRpcAPITestCase(test.TestCase):
def test_decrypt_text(self):
self._test_cert_api('decrypt_text',
project_id='fake_project_id', text='blah')
-
- def test_get_backdoor_port(self):
- self._test_cert_api('get_backdoor_port', host='fake_host',
- version='1.1')
diff --git a/nova/tests/compute/test_compute.py b/nova/tests/compute/test_compute.py
index dbd72a797..c9de65ed7 100644
--- a/nova/tests/compute/test_compute.py
+++ b/nova/tests/compute/test_compute.py
@@ -6822,24 +6822,6 @@ class ComputeAPITestCase(BaseTestCase):
db.instance_destroy(self.context, instance['uuid'])
- def test_get_backdoor_port(self):
- # Test api call to get backdoor_port.
- fake_backdoor_port = 59697
-
- self.mox.StubOutWithMock(rpc, 'call')
-
- rpc_msg = {'method': 'get_backdoor_port',
- 'namespace': None,
- 'args': {},
- 'version': compute_rpcapi.ComputeAPI.BASE_RPC_API_VERSION}
- rpc.call(self.context, 'compute.fake_host', rpc_msg,
- None).AndReturn(fake_backdoor_port)
-
- self.mox.ReplayAll()
-
- port = self.compute_api.get_backdoor_port(self.context, 'fake_host')
- self.assertEqual(port, fake_backdoor_port)
-
def test_console_output(self):
fake_instance = {'uuid': 'fake_uuid',
'host': 'fake_compute_host'}
@@ -7529,19 +7511,6 @@ class ComputeAPIAggrTestCase(BaseTestCase):
self.context, aggr['id'], 'invalid_host')
-class ComputeBackdoorPortTestCase(BaseTestCase):
- """This is for unit test coverage of backdoor port rpc."""
-
- def setUp(self):
- super(ComputeBackdoorPortTestCase, self).setUp()
- self.context = context.get_admin_context()
- self.compute.backdoor_port = 59697
-
- def test_get_backdoor_port(self):
- port = self.compute.get_backdoor_port(self.context)
- self.assertEqual(port, self.compute.backdoor_port)
-
-
class ComputeAggrTestCase(BaseTestCase):
"""This is for unit coverage of aggregate-related methods
defined in nova.compute.manager."""
diff --git a/nova/tests/compute/test_compute_cells.py b/nova/tests/compute/test_compute_cells.py
index 952a557fb..40ae4e3de 100644
--- a/nova/tests/compute/test_compute_cells.py
+++ b/nova/tests/compute/test_compute_cells.py
@@ -148,9 +148,6 @@ class CellsComputeAPITestCase(test_compute.ComputeAPITestCase):
def test_live_migrate(self):
self.skipTest("Test is incompatible with cells.")
- def test_get_backdoor_port(self):
- self.skipTest("Test is incompatible with cells.")
-
def test_snapshot_given_image_uuid(self):
self.skipTest("Test doesn't apply to API cell.")
diff --git a/nova/tests/compute/test_rpcapi.py b/nova/tests/compute/test_rpcapi.py
index 2edf2e20f..2c04f8060 100644
--- a/nova/tests/compute/test_rpcapi.py
+++ b/nova/tests/compute/test_rpcapi.py
@@ -189,9 +189,6 @@ class ComputeRpcAPITestCase(test.TestCase):
self._test_compute_api('host_power_action', 'call', action='action',
host='host')
- def test_get_backdoor_port(self):
- self._test_compute_api('get_backdoor_port', 'call', host='host')
-
def test_inject_file(self):
self._test_compute_api('inject_file', 'cast',
instance=self.fake_instance, path='path', file_contents='fc')
diff --git a/nova/tests/conductor/test_conductor.py b/nova/tests/conductor/test_conductor.py
index 73bdf5d71..9731a76d7 100644
--- a/nova/tests/conductor/test_conductor.py
+++ b/nova/tests/conductor/test_conductor.py
@@ -266,30 +266,6 @@ class _BaseTestCase(object):
result = self.conductor.bw_usage_update(*update_args)
self.assertEqual(result, 'foo')
- def test_get_backdoor_port(self):
- backdoor_port = 59697
-
- def fake_get_backdoor_port(self, context):
- return backdoor_port
-
- if isinstance(self.conductor, conductor_api.API):
- self.stubs.Set(conductor_manager.ConductorManager,
- 'get_backdoor_port', fake_get_backdoor_port)
- port = self.conductor.get_backdoor_port(self.context, 'fake_host')
- elif isinstance(self.conductor, conductor_api.LocalAPI):
- try:
- self.conductor.get_backdoor_port(self.context, 'fake_host')
- except exc.InvalidRequest:
- port = backdoor_port
- else:
- if isinstance(self.conductor, conductor_rpcapi.ConductorAPI):
- self.stubs.Set(conductor_manager.ConductorManager,
- 'get_backdoor_port', fake_get_backdoor_port)
- self.conductor.backdoor_port = backdoor_port
- port = self.conductor.get_backdoor_port(self.context)
-
- self.assertEqual(port, backdoor_port)
-
def test_security_group_get_by_instance(self):
fake_instance = {'id': 'fake-instance'}
self.mox.StubOutWithMock(db, 'security_group_get_by_instance')
diff --git a/nova/tests/console/test_console.py b/nova/tests/console/test_console.py
index 54bcb32ca..e4211f258 100644
--- a/nova/tests/console/test_console.py
+++ b/nova/tests/console/test_console.py
@@ -186,13 +186,3 @@ class ConsoleAPITestCase(test.TestCase):
self.mox.ReplayAll()
self.console_api.create_console(self.context, self.fake_uuid)
-
- def test_get_backdoor_port(self):
- self.mox.StubOutWithMock(console_rpcapi.ConsoleAPI,
- 'get_backdoor_port')
-
- console_rpcapi.ConsoleAPI.get_backdoor_port(self.context, 'fake_host')
-
- self.mox.ReplayAll()
-
- self.console_api.get_backdoor_port(self.context, 'fake_host')
diff --git a/nova/tests/console/test_rpcapi.py b/nova/tests/console/test_rpcapi.py
index 298c59c4e..691e0ef8d 100644
--- a/nova/tests/console/test_rpcapi.py
+++ b/nova/tests/console/test_rpcapi.py
@@ -37,9 +37,6 @@ class ConsoleRpcAPITestCase(test.TestCase):
expected_msg = rpcapi.make_msg(method, **kwargs)
expected_msg['version'] = expected_version
- if method == 'get_backdoor_port':
- del expected_msg['args']['host']
-
self.fake_args = None
self.fake_kwargs = None
@@ -65,7 +62,3 @@ class ConsoleRpcAPITestCase(test.TestCase):
def test_remove_console(self):
self._test_console_api('remove_console', console_id='i',
rpc_method='cast')
-
- def test_get_backdoor_port(self):
- self._test_console_api('get_backdoor_port', host='fake_host',
- rpc_method='call', version='1.1')
diff --git a/nova/tests/consoleauth/test_consoleauth.py b/nova/tests/consoleauth/test_consoleauth.py
index 41bec05e1..459da2fd8 100644
--- a/nova/tests/consoleauth/test_consoleauth.py
+++ b/nova/tests/consoleauth/test_consoleauth.py
@@ -102,11 +102,6 @@ class ConsoleauthTestCase(test.TestCase):
instance_uuid=None)
self.assertFalse(self.manager.check_token(self.context, "token"))
- def test_get_backdoor_port(self):
- self.manager.backdoor_port = 59697
- port = self.manager.get_backdoor_port(self.context)
- self.assertEqual(port, self.manager.backdoor_port)
-
class CellsConsoleauthTestCase(ConsoleauthTestCase):
"""Test Case for consoleauth w/ cells enabled."""
diff --git a/nova/tests/consoleauth/test_rpcapi.py b/nova/tests/consoleauth/test_rpcapi.py
index 308c0d812..605322e75 100644
--- a/nova/tests/consoleauth/test_rpcapi.py
+++ b/nova/tests/consoleauth/test_rpcapi.py
@@ -38,9 +38,6 @@ class ConsoleAuthRpcAPITestCase(test.TestCase):
expected_msg = rpcapi.make_msg(method, **kwargs)
expected_msg['version'] = expected_version
- if method == 'get_backdoor_port':
- del expected_msg['args']['host']
-
self.call_ctxt = None
self.call_topic = None
self.call_msg = None
@@ -81,7 +78,3 @@ class ConsoleAuthRpcAPITestCase(test.TestCase):
_do_cast=True,
instance_uuid="instance",
version='1.2')
-
- def test_get_backdoor_port(self):
- self._test_consoleauth_api('get_backdoor_port', host='fake_host',
- version='1.1')
diff --git a/nova/tests/fake_policy.py b/nova/tests/fake_policy.py
index b87360d0e..3f2d423f4 100644
--- a/nova/tests/fake_policy.py
+++ b/nova/tests/fake_policy.py
@@ -228,7 +228,6 @@ policy_data = """
"network:get_instance_uuids_by_ip_filter": "",
"network:get_instance_id_by_floating_address": "",
"network:setup_networks_on_host": "",
- "network:get_backdoor_port": "",
"network:get_floating_ip": "",
"network:get_floating_ip_pools": "",
diff --git a/nova/tests/network/test_api.py b/nova/tests/network/test_api.py
index 7a53dc8d4..304229b20 100644
--- a/nova/tests/network/test_api.py
+++ b/nova/tests/network/test_api.py
@@ -255,15 +255,3 @@ class ApiTestCase(test.TestCase):
instance = {'uuid': FAKE_UUID}
result = self.network_api._is_multi_host(self.context, instance)
self.assertEqual(is_multi_host, result)
-
- def test_get_backdoor_port(self):
- backdoor_port = 59697
-
- def fake_get_backdoor_port(ctxt, host):
- return backdoor_port
-
- self.stubs.Set(self.network_api.network_rpcapi, 'get_backdoor_port',
- fake_get_backdoor_port)
-
- port = self.network_api.get_backdoor_port(self.context, 'fake_host')
- self.assertEqual(port, backdoor_port)
diff --git a/nova/tests/network/test_manager.py b/nova/tests/network/test_manager.py
index f0d8e58a9..fd70e1584 100644
--- a/nova/tests/network/test_manager.py
+++ b/nova/tests/network/test_manager.py
@@ -1870,19 +1870,6 @@ class RPCAllocateTestCase(test.TestCase):
self.assertEqual(rval, address)
-class BackdoorPortTestCase(test.TestCase):
- """Tests nova.network.manager.get_backdoor_port."""
- def setUp(self):
- super(BackdoorPortTestCase, self).setUp()
- self.manager = network_manager.NetworkManager()
- self.manager.backdoor_port = 59697
- self.context = context.RequestContext('fake', 'fake')
-
- def test_backdoor_port(self):
- port = self.manager.get_backdoor_port(self.context)
- self.assertEqual(port, self.manager.backdoor_port)
-
-
class TestFloatingIPManager(floating_ips.FloatingIP,
network_manager.NetworkManager):
"""Dummy manager that implements FloatingIP."""
diff --git a/nova/tests/network/test_rpcapi.py b/nova/tests/network/test_rpcapi.py
index 55d502915..fbe0e7a8b 100644
--- a/nova/tests/network/test_rpcapi.py
+++ b/nova/tests/network/test_rpcapi.py
@@ -50,7 +50,7 @@ class NetworkRpcAPITestCase(test.TestCase):
'_rpc_allocate_fixed_ip', 'deallocate_fixed_ip', 'update_dns',
'_associate_floating_ip', '_disassociate_floating_ip',
'lease_fixed_ip', 'release_fixed_ip', 'migrate_instance_start',
- 'migrate_instance_finish', 'get_backdoor_port',
+ 'migrate_instance_finish',
'allocate_for_instance', 'deallocate_for_instance',
]
if method in targeted_methods and 'host' in kwargs:
@@ -138,10 +138,6 @@ class NetworkRpcAPITestCase(test.TestCase):
self._test_network_api('get_instance_id_by_floating_address',
rpc_method='call', address='w.x.y.z')
- def test_get_backdoor_port(self):
- self._test_network_api('get_backdoor_port', rpc_method='call',
- host='fake_host', version='1.4')
-
def test_get_vifs_by_instance(self):
self._test_network_api('get_vifs_by_instance',
rpc_method='call', instance_id='fake_id')
diff --git a/nova/tests/scheduler/test_rpcapi.py b/nova/tests/scheduler/test_rpcapi.py
index 62038c722..9a7615c86 100644
--- a/nova/tests/scheduler/test_rpcapi.py
+++ b/nova/tests/scheduler/test_rpcapi.py
@@ -37,9 +37,6 @@ class SchedulerRpcAPITestCase(test.TestCase):
expected_msg = rpcapi.make_msg(method, **kwargs)
expected_msg['version'] = expected_version
- if method == 'get_backdoor_port':
- del expected_msg['args']['host']
-
self.fake_args = None
self.fake_kwargs = None
@@ -88,10 +85,6 @@ class SchedulerRpcAPITestCase(test.TestCase):
host='fake_host', capabilities='fake_capabilities',
version='2.4')
- def test_get_backdoor_port(self):
- self._test_scheduler_api('get_backdoor_port', rpc_method='call',
- host='fake_host', version='2.5')
-
def test_select_hosts(self):
self._test_scheduler_api('select_hosts', rpc_method='call',
request_spec='fake_request_spec',
diff --git a/nova/tests/test_baserpc.py b/nova/tests/test_baserpc.py
index 1e33fba67..d9013fb99 100644
--- a/nova/tests/test_baserpc.py
+++ b/nova/tests/test_baserpc.py
@@ -43,3 +43,8 @@ class BaseAPITestCase(test.TestCase):
def test_ping(self):
res = self.base_rpcapi.ping(self.context, 'foo')
self.assertEqual(res, {'service': 'compute', 'arg': 'foo'})
+
+ def test_get_backdoor_port(self):
+ res = self.base_rpcapi.get_backdoor_port(self.context,
+ self.compute.host)
+ self.assertEqual(res, self.compute.backdoor_port)
diff --git a/nova/tests/test_service.py b/nova/tests/test_service.py
index efe84fbb1..2cbc82fda 100644
--- a/nova/tests/test_service.py
+++ b/nova/tests/test_service.py
@@ -159,7 +159,7 @@ class ServiceTestCase(test.TestCase):
# pre_start_hook is called after service record is created,
# but before RPC consumer is created
self.manager_mock.pre_start_hook(rpc_connection=mox.IgnoreArg())
- self.manager_mock.create_rpc_dispatcher()
+ self.manager_mock.create_rpc_dispatcher(None)
# post_start_hook is called after RPC consumer is created.
self.manager_mock.post_start_hook()