From: Oleg Bondarev <obondarev@mirantis.com>
Date: Wed, 18 Dec 2013 08:14:30 +0000 (+0400)
Subject: Mock looping_call in metadata agent tests
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=4ded12bef837e707c76af87d71155f04534dcb0d;p=openstack-build%2Fneutron-build.git

Mock looping_call in metadata agent tests

Change-Id: Icb5146084730384bac1fb7d3176ce0cfcd565cc5
Closes-Bug: #1262089
---

diff --git a/neutron/tests/unit/test_metadata_agent.py b/neutron/tests/unit/test_metadata_agent.py
index 8eb378449..d903a52c9 100644
--- a/neutron/tests/unit/test_metadata_agent.py
+++ b/neutron/tests/unit/test_metadata_agent.py
@@ -308,7 +308,10 @@ class TestUnixDomainMetadataProxy(base.BaseTestCase):
         super(TestUnixDomainMetadataProxy, self).setUp()
         self.cfg_p = mock.patch.object(agent, 'cfg')
         self.cfg = self.cfg_p.start()
-        self.addCleanup(self.cfg_p.stop)
+        looping_call_p = mock.patch(
+            'neutron.openstack.common.loopingcall.FixedIntervalLoopingCall')
+        self.looping_mock = looping_call_p.start()
+        self.addCleanup(mock.patch.stopall)
         self.cfg.CONF.metadata_proxy_socket = '/the/path'
 
     def test_init_doesnot_exists(self):
@@ -393,13 +396,11 @@ class TestUnixDomainMetadataProxy(base.BaseTestCase):
                             )
 
     def test_init_state_reporting(self):
-        with mock.patch('neutron.openstack.common.loopingcall.'
-                        'FixedIntervalLoopingCall') as loop_call:
-            with mock.patch('os.makedirs'):
-                proxy = agent.UnixDomainMetadataProxy(mock.Mock())
-                loop_call.assert_called_once_with(proxy._report_state)
-                loop_call.return_value.start.assert_called_once_with(
-                    interval=mock.ANY)
+        with mock.patch('os.makedirs'):
+            proxy = agent.UnixDomainMetadataProxy(mock.Mock())
+            self.looping_mock.assert_called_once_with(proxy._report_state)
+            self.looping_mock.return_value.start.assert_called_once_with(
+                interval=mock.ANY)
 
     def test_report_state(self):
         with mock.patch('neutron.agent.rpc.PluginReportStateAPI') as state_api: