]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Use base.BaseTestCase in NVP config test
authorAkihiro Motoki <motoki@da.jp.nec.com>
Thu, 19 Dec 2013 09:05:57 +0000 (18:05 +0900)
committerAkihiro Motoki <motoki@da.jp.nec.com>
Mon, 6 Jan 2014 11:08:30 +0000 (20:08 +0900)
BaseTestCase setup stdout/stderr capture and
it is better to be used.

Change-Id: I3537a633125e5e474fda660581f41e5f57af26ca
Closes-Bug: 1262550

neutron/tests/unit/nicira/test_nvpopts.py

index 30afa1f437689f1dfa37110352a04da161346c8a..aa9c108354d1ad822108a56362fa4d5fb42c2d4c 100644 (file)
@@ -16,7 +16,6 @@
 #
 
 import fixtures
-import testtools
 
 import mock
 from oslo.config import cfg
@@ -30,6 +29,7 @@ from neutron.plugins.nicira.common import sync
 from neutron.plugins.nicira.nsxlib import lsn as lsnlib
 from neutron.plugins.nicira import nvp_cluster
 from neutron.plugins.nicira import NvpApiClient as nvp_client
+from neutron.tests import base
 from neutron.tests.unit.nicira import get_fake_conf
 from neutron.tests.unit.nicira import PLUGIN_NAME
 
@@ -40,7 +40,7 @@ NSX_INI_AGENTLESS_PATH = get_fake_conf('nsx.ini.agentless.test')
 NVP_INI_DEPR_PATH = get_fake_conf('nvp.ini.full.test')
 
 
-class NSXClusterTest(testtools.TestCase):
+class NSXClusterTest(base.BaseTestCase):
 
     cluster_opts = {'default_tz_uuid': uuidutils.generate_uuid(),
                     'default_l2_gw_service_uuid': uuidutils.generate_uuid(),
@@ -54,10 +54,6 @@ class NSXClusterTest(testtools.TestCase):
                     'default_interface_name': 'baz',
                     'nsx_controllers': ['1.1.1.1:443']}
 
-    def setUp(self):
-        super(NSXClusterTest, self).setUp()
-        self.addCleanup(cfg.CONF.reset)
-
     def test_create_cluster(self):
         cluster = nvp_cluster.NVPCluster(**self.cluster_opts)
         for (k, v) in self.cluster_opts.iteritems():
@@ -77,11 +73,10 @@ class NSXClusterTest(testtools.TestCase):
                           nvp_cluster.NVPCluster, **opts)
 
 
-class ConfigurationTest(testtools.TestCase):
+class ConfigurationTest(base.BaseTestCase):
 
     def setUp(self):
         super(ConfigurationTest, self).setUp()
-        self.addCleanup(cfg.CONF.reset)
         self.useFixture(fixtures.MonkeyPatch(
                         'neutron.manager.NeutronManager._instance',
                         None))
@@ -206,11 +201,10 @@ class ConfigurationTest(testtools.TestCase):
                       plugin.supported_extension_aliases)
 
 
-class OldNVPConfigurationTest(testtools.TestCase):
+class OldNVPConfigurationTest(base.BaseTestCase):
 
     def setUp(self):
         super(OldNVPConfigurationTest, self).setUp()
-        self.addCleanup(cfg.CONF.reset)
         self.useFixture(fixtures.MonkeyPatch(
                         'neutron.manager.NeutronManager._instance',
                         None))