openstack
/
heat
Code Issues Proposed changes

Merge "Don't mock Target unnecessarily"

This commit is contained in:
Jenkins 2017-09-11 04:08:28 +00:00 committed by Gerrit Code Review
parent 0de1e372c7 3089880daa
commit 1723ae2542
1 changed files with 1 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
heat/tests/engine/service/test_service_engine.py
View File

@ -385,8 +385,6 @@ class ServiceEngineTest(common.HeatTestCase):
self.eng.reset()
setup_logging_mock.assert_called_once_with(cfg.CONF, 'heat')
@mock.patch('oslo_messaging.Target',
return_value=mock.Mock())
@mock.patch('heat.common.messaging.get_rpc_client',
return_value=mock.Mock())
@mock.patch('heat.common.service_utils.generate_engine_id',
@ -406,8 +404,7 @@ class ServiceEngineTest(common.HeatTestCase):
engine_listener_class,
thread_group_manager_class,
sample_uuid_method,
rpc_client_class,
target_class):
rpc_client_class):
self.eng.start()
self.assertEqual(cfg.CONF.executor_thread_pool_size,
cfg.CONF.database.max_overflow)