diff options
author | Ben Nemec <bnemec@redhat.com> | 2019-05-28 17:04:19 +0000 |
---|---|---|
committer | Ben Nemec <bnemec@redhat.com> | 2019-05-28 17:04:19 +0000 |
commit | f8f6b14088637fd87de7cf385b840d6b5e5f9b22 (patch) | |
tree | 1f62b00692b7f2235ee8f200a94093dcc862300b | |
parent | b40f1fc3d4d16806f04c46297c202ef0c08cd0e3 (diff) | |
download | oslotest-f8f6b14088637fd87de7cf385b840d6b5e5f9b22.tar.gz |
Stop testing mock functionality
There's no particular reason for us to be verifying mock's behavior.
They have their own unit tests for doing that.
This is specifically a problem right now because mock changed its
behavior in 3.0.0 and broke this test: "Issue #36366: Calling stop()
on an unstarted or stopped unittest.mock.patch() object will now
return None instead of raising RuntimeError, making the method
idempotent."
Change-Id: I57db9bad15cab343e152615e115e2ba3ac4d87fe
Closes-Bug: 1830429
-rw-r--r-- | oslotest/tests/unit/test_base.py | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/oslotest/tests/unit/test_base.py b/oslotest/tests/unit/test_base.py index 93c3c52..b98c672 100644 --- a/oslotest/tests/unit/test_base.py +++ b/oslotest/tests/unit/test_base.py @@ -126,14 +126,6 @@ class TestManualMock(base.BaseTestCase): self.addCleanup(patcher.stop) super(TestManualMock, self).setUp() self.useFixture(fixtures.MockPatch('fixtures.Timeout')) - self.unstopped = mock.patch('os.environ.put') - - def tearDown(self): - super(TestManualMock, self).tearDown() - self.assertRaises( - RuntimeError, - self.unstopped.stop, - ) def test_mock_patch_manually(self): # Verify that if a test instance creates its own mock and |