]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Remove double callback manager mocks
authorKevin Benton <blak111@gmail.com>
Fri, 26 Jun 2015 00:55:16 +0000 (17:55 -0700)
committerKevin Benton <blak111@gmail.com>
Fri, 26 Jun 2015 01:38:09 +0000 (18:38 -0700)
setup_test_registry_instance() in the base test case class gives
each test its own registry by mocking out the get_callback_manager.
The L3 agent test cases were duplicating this.

Partial-Bug: #1468998
Change-Id: I7356daa846524611e9f92365939e8ad15d1e1cd8

neutron/tests/unit/agent/l3/test_agent.py
neutron/tests/unit/agent/l3/test_dvr_local_router.py

index d99b1c42daa1a08e4d8367b67dbc0e870740207f..ba74a975ff4acd7951f4fabb22490e6c512b28f5 100644 (file)
@@ -40,8 +40,6 @@ from neutron.agent.linux import interface
 from neutron.agent.linux import ra
 from neutron.agent.metadata import driver as metadata_driver
 from neutron.agent import rpc as agent_rpc
-from neutron.callbacks import manager
-from neutron.callbacks import registry
 from neutron.common import config as base_config
 from neutron.common import constants as l3_constants
 from neutron.common import exceptions as n_exc
@@ -163,10 +161,6 @@ class BasicRouterOperationsFramework(base.BaseTestCase):
         self.ri_kwargs = {'agent_conf': self.conf,
                           'interface_driver': self.mock_driver}
 
-        self._callback_manager = manager.CallbacksManager()
-        mock.patch.object(registry, '_get_callback_manager',
-                          return_value=self._callback_manager).start()
-
     def _process_router_instance_for_agent(self, agent, ri, router):
         ri.router = router
         if not ri.radvd:
index 1e81c8d48546d1bbdbb1f6c4baae86d95ba4c62c..fe41f2fe87bc008c7d7d0ae99cd6d38c38793ed6 100644 (file)
@@ -27,8 +27,6 @@ from neutron.agent.l3 import router_info
 from neutron.agent.linux import external_process
 from neutron.agent.linux import interface
 from neutron.agent.linux import ip_lib
-from neutron.callbacks import manager
-from neutron.callbacks import registry
 from neutron.common import config as base_config
 from neutron.common import constants as l3_constants
 from neutron.common import utils as common_utils
@@ -147,10 +145,6 @@ class TestDvrRouterOperations(base.BaseTestCase):
         self.ri_kwargs = {'agent_conf': self.conf,
                           'interface_driver': self.mock_driver}
 
-        self._callback_manager = manager.CallbacksManager()
-        mock.patch.object(registry, '_get_callback_manager',
-                          return_value=self._callback_manager).start()
-
     def _create_router(self, router=None, **kwargs):
         agent_conf = mock.Mock()
         self.router_id = _uuid()