diff options
author | V02460 <V02460@gmail.com> | 2021-12-20 16:34:46 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-12-20 10:34:46 -0500 |
commit | 7a7ca8f2263f8750b8ff2c18b9aefaf4a3626235 (patch) | |
tree | 1cfb2b70b55e5b8497fd35efdebb956211d5949a | |
parent | Add opentracing types (#11603) (diff) | |
download | synapse-7a7ca8f2263f8750b8ff2c18b9aefaf4a3626235.tar.xz |
Use mock from standard library (#11588)
Instead of the backported version.
-rw-r--r-- | changelog.d/11588.removal | 1 | ||||
-rwxr-xr-x | setup.py | 4 | ||||
-rw-r--r-- | tests/storage/test_background_update.py | 17 |
3 files changed, 10 insertions, 12 deletions
diff --git a/changelog.d/11588.removal b/changelog.d/11588.removal new file mode 100644 index 0000000000..f781021e11 --- /dev/null +++ b/changelog.d/11588.removal @@ -0,0 +1 @@ +Replace `mock` package by its standard library version. diff --git a/setup.py b/setup.py index 812459074a..e113da6782 100755 --- a/setup.py +++ b/setup.py @@ -120,9 +120,7 @@ CONDITIONAL_REQUIREMENTS["mypy"] = [ # Tests assume that all optional dependencies are installed. # # parameterized_class decorator was introduced in parameterized 0.7.0 -# -# We use `mock` library as that backports `AsyncMock` to Python 3.6 -CONDITIONAL_REQUIREMENTS["test"] = ["parameterized>=0.7.0", "mock>=4.0.0"] +CONDITIONAL_REQUIREMENTS["test"] = ["parameterized>=0.7.0"] CONDITIONAL_REQUIREMENTS["dev"] = ( CONDITIONAL_REQUIREMENTS["lint"] diff --git a/tests/storage/test_background_update.py b/tests/storage/test_background_update.py index d77c001506..6156dfac4e 100644 --- a/tests/storage/test_background_update.py +++ b/tests/storage/test_background_update.py @@ -12,15 +12,14 @@ # See the License for the specific language governing permissions and # limitations under the License. -# Use backported mock for AsyncMock support on Python 3.6. -from mock import Mock +from unittest.mock import Mock from twisted.internet.defer import Deferred, ensureDeferred from synapse.storage.background_updates import BackgroundUpdater from tests import unittest -from tests.test_utils import make_awaitable +from tests.test_utils import make_awaitable, simple_async_mock class BackgroundUpdateTestCase(unittest.HomeserverTestCase): @@ -116,14 +115,14 @@ class BackgroundUpdateControllerTestCase(unittest.HomeserverTestCase): ) # Mock out the AsyncContextManager - self._update_ctx_manager = Mock(spec=["__aenter__", "__aexit__"]) - self._update_ctx_manager.__aenter__ = Mock( - return_value=make_awaitable(None), - ) - self._update_ctx_manager.__aexit__ = Mock(return_value=make_awaitable(None)) + class MockCM: + __aenter__ = simple_async_mock(return_value=None) + __aexit__ = simple_async_mock(return_value=None) + + self._update_ctx_manager = MockCM # Mock out the `update_handler` callback - self._on_update = Mock(return_value=self._update_ctx_manager) + self._on_update = Mock(return_value=self._update_ctx_manager()) # Define a default batch size value that's not the same as the internal default # value (100). |