Don't explicitly call .stop() on mock.patch objects

BaseTestCase already cleans up all mocks via .stopall(), so there is no need to do it in each test case. Change-Id: I7165ea9682a3dce3a76e90e309b763c939729d84
openstack · Jun 17, 2014 · 9e6b0ad · 9e6b0ad
1 parent 469a1ca
commit 9e6b0ad
Showing 1 changed file with 0 additions and 2 deletions.
diff --git a/neutron/tests/unit/openvswitch/test_agent_scheduler.py b/neutron/tests/unit/openvswitch/test_agent_scheduler.py
@@ -1040,7 +1040,6 @@ def setUp(self):
         # the global attribute map
         attributes.RESOURCE_ATTRIBUTE_MAP.update(
             agent.RESOURCE_ATTRIBUTE_MAP)
-        self.addCleanup(self.dhcp_notifier_cls_p.stop)
         self.addCleanup(self.restore_attribute_map)
 
     def restore_attribute_map(self):
@@ -1190,7 +1189,6 @@ def setUp(self):
         # the global attribute map
         attributes.RESOURCE_ATTRIBUTE_MAP.update(
             agent.RESOURCE_ATTRIBUTE_MAP)
-        self.addCleanup(self.dhcp_notifier_cls_p.stop)
         self.addCleanup(self.restore_attribute_map)
 
     def restore_attribute_map(self):