From 5648ad6e9d13fb96e9d870fc66c2d6bd545809bb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Rados=C5=82aw=20Piliszek?= Date: Thu, 2 Apr 2020 18:12:37 +0200 Subject: [PATCH] Use unittest.mock instead of PyPI mock Now that py2 is gone, oslotest dropped dependency on mock and will soon affect Ussuri CI [1], let's use unittest.mock built in py3. This also fixes py38 jobs and proactively prevents py36 and py37 failing due to [1]. This is because we never included mock in test-requirements (but in lower-constraints where it does not really belong at all) and instead relied on oslotest to bring it in. [1] https://review.opendev.org/716322 Change-Id: I30e82e2d87418272a71c7ee089a8acdaf8872158 --- kolla_ansible/tests/unit/test_filters.py | 2 +- lower-constraints.txt | 1 - tests/test_kolla_docker.py | 5 +---- tests/test_merge_config.py | 2 +- 4 files changed, 3 insertions(+), 7 deletions(-) diff --git a/kolla_ansible/tests/unit/test_filters.py b/kolla_ansible/tests/unit/test_filters.py index 8eef4f0e09..2ae334ce37 100644 --- a/kolla_ansible/tests/unit/test_filters.py +++ b/kolla_ansible/tests/unit/test_filters.py @@ -13,9 +13,9 @@ # under the License. import unittest +from unittest import mock import jinja2 -import mock from kolla_ansible import exception from kolla_ansible import filters diff --git a/lower-constraints.txt b/lower-constraints.txt index b25acf67d3..a714d26beb 100644 --- a/lower-constraints.txt +++ b/lower-constraints.txt @@ -39,7 +39,6 @@ keystoneauth1==3.4.0 linecache2==1.0.0 MarkupSafe==1.0 mccabe==0.2.1 -mock==2.0.0 monotonic==1.4 mox3==0.25.0 msgpack==0.5.6 diff --git a/tests/test_kolla_docker.py b/tests/test_kolla_docker.py index 7fb9b4f0bc..af00bc89f5 100644 --- a/tests/test_kolla_docker.py +++ b/tests/test_kolla_docker.py @@ -19,11 +19,8 @@ import copy import imp import os import sys +from unittest import mock -try: - from unittest import mock -except ImportError: - import mock from docker import errors as docker_error from docker.types import Ulimit from oslotest import base diff --git a/tests/test_merge_config.py b/tests/test_merge_config.py index 8091025d18..ccc7029479 100644 --- a/tests/test_merge_config.py +++ b/tests/test_merge_config.py @@ -16,9 +16,9 @@ import imp import os import sys +from unittest import mock from io import StringIO -import mock from oslotest import base