openstack
/
heat
Code Issues Proposed changes

Don't mock Target unnecessarily 

Change-Id: I25be61ac5f6e13014529f290cbbe5589cedc6e09
Closes-Bug: #1716080
This commit is contained in:
rabi 2017-09-09 08:13:25 +05:30
parent d2cb548fa4
commit 3089880daa
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)