]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Remove failing SafeFixture tests
authorAssaf Muller <amuller@redhat.com>
Mon, 29 Jun 2015 15:38:51 +0000 (11:38 -0400)
committerAssaf Muller <amuller@redhat.com>
Mon, 29 Jun 2015 15:43:49 +0000 (11:43 -0400)
The fixtures 1.3 release attempted to fix the fixtures resource
leak issue, but failed to do so completely. Our own SafeFixture
is still needed: The 1.3 release broke our SafeFixture tests,
but not the usage of SafeFixture itself. This patch removes
those failing tests for now to unbreak the gate. Jakub reported
a bug on fixtures 1.3:
https://bugs.launchpad.net/python-fixtures/+bug/1469759

We will continue to use SafeFixture until that bug is fixed
in fixtures, at which point we will be able to require
fixtures > 1.3.

Change-Id: I59457c3bb198ff86d5ad55a1e623d008f0034b8f
Closes-Bug: #1469734

neutron/tests/functional/test_tools.py [deleted file]

diff --git a/neutron/tests/functional/test_tools.py b/neutron/tests/functional/test_tools.py
deleted file mode 100644 (file)
index 9a709e5..0000000
+++ /dev/null
@@ -1,81 +0,0 @@
-# Copyright (c) 2015 Thales Services SAS
-# All Rights Reserved.
-#
-#    Licensed under the Apache License, Version 2.0 (the "License"); you may
-#    not use this file except in compliance with the License. You may obtain
-#    a copy of the License at
-#
-#         http://www.apache.org/licenses/LICENSE-2.0
-#
-#    Unless required by applicable law or agreed to in writing, software
-#    distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
-#    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
-#    License for the specific language governing permissions and limitations
-#    under the License.
-
-import fixtures
-import testscenarios
-
-from neutron.tests import base
-from neutron.tests import tools
-
-
-class NoErrorFixture(tools.SafeFixture):
-
-    def __init__(self):
-        super(NoErrorFixture, self).__init__()
-        self.cleaned = False
-        self.called = False
-
-    def setUp(self):
-        super(NoErrorFixture, self).setUp()
-        self.called = True
-
-    def cleanUp(self):
-        self.cleaned = True
-        super(NoErrorFixture, self).cleanUp()
-
-
-class ErrorAfterFixtureSetup(NoErrorFixture):
-
-    def setUp(self):
-        super(tools.SafeFixture, self).setUp()
-        raise ValueError
-
-
-class ErrorBeforeFixtureSetup(NoErrorFixture):
-
-        def setUp(self):
-            raise ValueError
-
-
-class TestSafeFixture(testscenarios.WithScenarios, base.BaseTestCase):
-    scenarios = [
-        ('testtools useFixture', dict(fixtures=False)),
-        ('fixtures useFixture', dict(fixtures=True)),
-    ]
-
-    def setUp(self):
-        super(TestSafeFixture, self).setUp()
-        if self.fixtures:
-            self.parent = self.useFixture(fixtures.Fixture())
-        else:
-            self.parent = self
-
-    def test_no_error(self):
-        fixture = NoErrorFixture()
-        self.parent.useFixture(fixture)
-        self.assertTrue(fixture.called)
-        self.assertFalse(fixture.cleaned)
-
-    def test_error_after_root_setup(self):
-        fixture = ErrorAfterFixtureSetup()
-        self.assertRaises(ValueError, self.parent.useFixture, fixture)
-        self.assertTrue(fixture.cleaned)
-
-    def test_error_before_root_setup(self):
-        fixture = ErrorBeforeFixtureSetup()
-        # NOTE(cbrandily); testtools.useFixture crashs badly if Fixture.setUp
-        # is not called or fails.
-        self.assertRaises(AttributeError, self.parent.useFixture, fixture)
-        self.assertFalse(fixture.cleaned)