x/vmware-nsx
Code Issues Proposed changes

Merge "Fix TypeError: <MagicMock name='LinuxBridgeManager().local_ip'"

Browse Source
This commit is contained in:
Jenkins 2013-10-25 02:42:08 +00:00 committed by Gerrit Code Review
commit eb155d5d26
1 changed files with 21 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
neutron/tests/unit/linuxbridge/test_lb_neutron_agent.py
View File

@ -97,11 +97,6 @@ class TestLinuxBridgeAgent(base.BaseTestCase):
super(TestLinuxBridgeAgent, self).setUp() super(TestLinuxBridgeAgent, self).setUp()
cfg.CONF.set_override('rpc_backend', cfg.CONF.set_override('rpc_backend',
'neutron.openstack.common.rpc.impl_fake') 'neutron.openstack.common.rpc.impl_fake')
self.lbmgr_patcher = mock.patch('neutron.plugins.linuxbridge.agent.'
'linuxbridge_neutron_agent.'
'LinuxBridgeManager')
self.lbmgr_mock = self.lbmgr_patcher.start()
self.addCleanup(self.lbmgr_patcher.stop)
self.execute_p = mock.patch.object(ip_lib.IPWrapper, '_execute') self.execute_p = mock.patch.object(ip_lib.IPWrapper, '_execute')
self.execute = self.execute_p.start() self.execute = self.execute_p.start()
self.addCleanup(self.execute_p.stop) self.addCleanup(self.execute_p.stop)
@ -113,8 +108,6 @@ class TestLinuxBridgeAgent(base.BaseTestCase):
self.get_mac.return_value = '00:00:00:00:00:01' self.get_mac.return_value = '00:00:00:00:00:01'
def test_update_devices_failed(self): def test_update_devices_failed(self):
lbmgr_instance = self.lbmgr_mock.return_value
lbmgr_instance.update_devices.side_effect = RuntimeError
agent = linuxbridge_neutron_agent.LinuxBridgeNeutronAgentRPC({}, agent = linuxbridge_neutron_agent.LinuxBridgeNeutronAgentRPC({},
0, 0,
None) None)
@ -125,8 +118,11 @@ class TestLinuxBridgeAgent(base.BaseTestCase):
raise_exception[0] += 1 raise_exception[0] += 1
else: else:
raise RuntimeError() raise RuntimeError()
with mock.patch.object(agent.br_mgr,
with mock.patch.object(linuxbridge_neutron_agent.LOG, 'info') as log: "update_devices") as update_devices:
update_devices.side_effect = RuntimeError
with mock.patch.object(linuxbridge_neutron_agent.LOG,
'info') as log:
log.side_effect = info_mock log.side_effect = info_mock
with testtools.ExpectedException(RuntimeError): with testtools.ExpectedException(RuntimeError):
agent.daemon_loop() agent.daemon_loop()
@ -134,8 +130,6 @@ class TestLinuxBridgeAgent(base.BaseTestCase):
def test_process_network_devices_failed(self): def test_process_network_devices_failed(self):
device_info = {'current': [1, 2, 3]} device_info = {'current': [1, 2, 3]}
lbmgr_instance = self.lbmgr_mock.return_value
lbmgr_instance.update_devices.return_value = device_info
agent = linuxbridge_neutron_agent.LinuxBridgeNeutronAgentRPC({}, agent = linuxbridge_neutron_agent.LinuxBridgeNeutronAgentRPC({},
0, 0,
None) None)
@ -147,6 +141,9 @@ class TestLinuxBridgeAgent(base.BaseTestCase):
else: else:
raise RuntimeError() raise RuntimeError()
with mock.patch.object(agent.br_mgr,
"update_devices") as update_devices:
update_devices.side_effect = device_info
with contextlib.nested( with contextlib.nested(
mock.patch.object(linuxbridge_neutron_agent.LOG, 'info'), mock.patch.object(linuxbridge_neutron_agent.LOG, 'info'),
mock.patch.object(agent, 'process_network_devices') mock.patch.object(agent, 'process_network_devices')