Use unittest.mock instead of mock

The mock third party library was needed for mock support in py2
runtimes. Since we now only support py36 and later, we can use the
standard lib unittest.mock module instead.

Note that https://github.com/openstack/charms.openstack is used during tests
and he need `mock`, unfortunatelly it doesn't declare `mock` in its
requirements so it retrieve mock from other charm project (cross dependency).
So we depend on charms.openstack first and when
Ib1ed5b598a52375e29e247db9ab4786df5b6d142 will be merged then CI
will pass without errors.

Depends-On: Ib1ed5b598a52375e29e247db9ab4786df5b6d142
Change-Id: Ibefa1987730d29c04beb41f6e129e47f5524d49a

.zuul.yaml

@@ -1,4 +1,4 @@
 - project:
     templates:
-      - openstack-python3-charm-jobs
+      - openstack-python3-ussuri-jobs
       - openstack-cover-jobs

test-requirements.txt

@@ -13,11 +13,6 @@ setuptools<50.0.0  # https://github.com/pypa/setuptools/commit/04e3df22df840c6bb
 
 requests>=2.18.4
 
-# Newer mock seems to have some syntax which is newer than python3.5 (e.g.
-# f'{something}'
-mock>=1.2,<4.0.0; python_version < '3.6'
-mock>=1.2; python_version >= '3.6'
-
 stestr>=2.2.0
 
 # Dependency of stestr. Workaround for

unit_tests/__init__.py

@@ -13,7 +13,7 @@
 # limitations under the License.
 
 import sys
-from mock import MagicMock
+from unittest.mock import MagicMock
 
 # python-apt is not installed as part of test-requirements but is imported by
 # some charmhelpers modules so create a fake import.

unit_tests/test_actions_hugepagereport.py

@@ -12,7 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-import mock
+from unittest import mock
 import os
 import shutil
 from tempfile import mkdtemp

unit_tests/test_actions_openstack_upgrade.py

@@ -12,7 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-from mock import patch
+from unittest.mock import patch
 import os
 
 os.environ['JUJU_UNIT_NAME'] = 'nova_compute'

unit_tests/test_actions_pause_resume.py

@@ -12,8 +12,8 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-import mock
-from mock import patch
+from unittest import mock
+from unittest.mock import patch
 
 from test_utils import CharmTestCase
 

unit_tests/test_actions_virsh_audit.py

@@ -12,7 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-import mock
+import unittest.mock as mock
 from test_utils import CharmTestCase
 import virshaudit as actions
 

unit_tests/test_nova_compute_contexts.py

@@ -14,7 +14,7 @@
 
 import platform
 
-from mock import patch
+from unittest.mock import patch
 from test_utils import CharmTestCase
 
 import nova_compute_context as context

unit_tests/test_nova_compute_hooks.py

@@ -15,7 +15,7 @@
 import importlib
 import json
 
-from mock import (
+from unittest.mock import (
     ANY,
     call,
     patch,

unit_tests/test_nova_compute_utils.py

@@ -20,7 +20,7 @@ import charmhelpers.contrib.openstack.utils as os_utils
 import nova_compute_context as compute_context
 import nova_compute_utils as utils
 
-from mock import (
+from unittest.mock import (
     patch,
     MagicMock,
     call

unit_tests/test_utils.py

@@ -19,7 +19,7 @@ import os
 import yaml
 
 from contextlib import contextmanager
-from mock import patch, MagicMock
+from unittest.mock import patch, MagicMock
 
 
 patch('charmhelpers.contrib.openstack.utils.set_os_workload_status').start()