feat: Replace external package mock with Python standard library `u…

…nittest.mock` (#697)

Closes: SDK-2005

setup.py

@@ -65,7 +65,6 @@ def main():
     test_requires = [
         'bottle',
         'jsonpatch>1.14',
-        'mock>=2.0.0, <4.0.0',
         'pycodestyle',
         'pylint',
         'sphinx',

test/conftest.py

@@ -3,10 +3,9 @@
 import json
 import logging
 import sys
+from unittest.mock import Mock
 
-from mock import Mock
 import pytest
-import requests
 
 from boxsdk.network.default_network import DefaultNetworkResponse
 
@@ -36,7 +35,7 @@ def _set_content_and_json_from_content(mock_response, content):
 
 @pytest.fixture()
 def generic_successful_request_response():
-    mock_request_response = Mock(requests.Response(), headers={f'header{i}': f'value{i}' for i in range(4)})
+    mock_request_response = Mock(headers={f'header{i}': f'value{i}' for i in range(4)})
     _set_content_and_json_from_json(mock_request_response, json_value={f'key{i}': f'value{i}' for i in range(8)})
     mock_request_response.status_code = 200
     mock_request_response.ok = True
@@ -75,7 +74,7 @@ def successful_token_json_response(access_token, refresh_token):
 @pytest.fixture()
 def successful_token_request_response(successful_token_json_response):
     # pylint:disable=redefined-outer-name
-    successful_token_mock = Mock(requests.Response(), headers={})
+    successful_token_mock = Mock(headers={})
     _set_content_and_json_from_json(successful_token_mock, json_value=successful_token_json_response)
     successful_token_mock.ok = True
     successful_token_mock.status_code = 200
@@ -139,7 +138,7 @@ def retry_after_response(retry_after_response_202, retry_after_response_429, req
 
 
 def _server_error_request_response(status_code):
-    mock_request_response = Mock(requests.Response(), headers={f'header{i}': f'value{i}' for i in range(4)})
+    mock_request_response = Mock(headers={f'header{i}': f'value{i}' for i in range(4)})
     _set_content_and_json_from_json(mock_request_response, json_value={f'key{i}': f'value{i}' for i in range(8)})
     mock_request_response.status_code = status_code
     mock_request_response.ok = False

test/functional/conftest.py

@@ -2,12 +2,12 @@
 from urllib import parse
 from test.functional.mock_box.box import Box
 from test.util.streamable_mock_open import streamable_mock_open
+from unittest.mock import patch
 
 import re
 import pytest
 import requests
 
-from mock import patch
 from boxsdk.auth.oauth2 import OAuth2
 from boxsdk.client import LoggingClient
 from boxsdk.config import API

test/functional/test_delete.py

@@ -1,6 +1,6 @@
 # coding: utf-8
 from test.util.streamable_mock_open import streamable_mock_open
-from mock import patch
+from unittest.mock import patch
 
 import pytest
 

test/functional/test_file_upload_update_download.py

@@ -2,7 +2,7 @@
 from test.util.streamable_mock_open import streamable_mock_open
 
 from io import BytesIO
-from mock import patch
+from unittest.mock import patch
 
 
 def test_upload_then_update(box_client, test_file_path, test_file_content, update_file_content, file_name):

test/functional/test_rate_limits.py

@@ -1,6 +1,6 @@
 # coding: utf-8
 
-from mock import patch
+from unittest.mock import patch
 
 
 def test_too_many_requests_causes_retry(box_client, mock_box, monkeypatch):

test/integration/conftest.py

@@ -1,6 +1,6 @@
 # coding: utf-8
 
-from mock import Mock
+from unittest.mock import Mock
 import pytest
 
 from boxsdk import Client

test/integration/mock_network.py

@@ -1,6 +1,6 @@
 # coding: utf-8
 
-from mock import Mock
+from unittest.mock import Mock
 import requests
 from boxsdk.network.default_network import DefaultNetworkResponse
 from boxsdk.network.network_interface import Network

test/integration/test_as_user.py

@@ -1,6 +1,6 @@
 # coding: utf-8
 
-from mock import call
+from unittest.mock import call
 import pytest
 from boxsdk.config import API, Client
 from boxsdk.object.user import User

test/integration/test_retry_and_refresh.py

@@ -1,6 +1,6 @@
 # coding: utf-8
 
-from mock import ANY, call
+from unittest.mock import ANY, call
 from boxsdk.config import API
 
 

test/integration/test_with_shared_link.py

@@ -1,6 +1,6 @@
 # coding: utf-8
 
-from mock import call
+from unittest.mock import call
 import pytest
 from boxsdk.config import API, Client
 from boxsdk.util.shared_link import get_shared_link_header

test/unit/auth/test_cooperatively_managed_oauth2.py

@@ -1,6 +1,6 @@
 # coding: utf-8
 
-from mock import Mock
+from unittest.mock import Mock
 
 from boxsdk.auth import cooperatively_managed_oauth2
 

test/unit/auth/test_developer_token_auth.py

@@ -1,6 +1,6 @@
 # coding: utf-8
 
-from mock import Mock, patch
+from unittest.mock import Mock, patch
 from boxsdk.auth import developer_token_auth
 
 

test/unit/auth/test_jwt_auth.py

@@ -7,14 +7,13 @@
 import json
 import random
 import string
+from unittest.mock import Mock, mock_open, patch, sentinel, call
 
 from cryptography.hazmat.backends import default_backend
 from cryptography.hazmat.primitives.asymmetric.rsa import RSAPrivateKey, generate_private_key as generate_rsa_private_key
 from cryptography.hazmat.primitives import serialization
-from mock import Mock, mock_open, patch, sentinel, call
 import pytest
 import pytz
-import requests
 
 from boxsdk.auth.jwt_auth import JWTAuth
 from boxsdk.exception import BoxOAuthException
@@ -85,7 +84,7 @@ def test_jwt_auth_init_raises_type_error_unless_exactly_one_of_rsa_private_key_f
         JWTAuth(**kwargs)
 
 
-@pytest.mark.parametrize('key_data', [object(), u'ƒøø'])
+@pytest.mark.parametrize('key_data', [object(), 'ƒøø'])
 @pytest.mark.parametrize('rsa_passphrase', [None])
 def test_jwt_auth_init_raises_type_error_if_rsa_private_key_data_has_unexpected_type(key_data, rsa_private_key_bytes):
     kwargs = dict(
@@ -258,7 +257,7 @@ def test_authenticate_instance_saves_enterprise_id_for_future_calls(jwt_auth_ini
             auth.authenticate_instance('fake_enterprise_id_2')
 
 
-@pytest.yield_fixture
+@pytest.fixture
 def jwt_encode():
     with patch('jwt.encode') as patched_jwt_encode:
         yield patched_jwt_encode
@@ -478,7 +477,7 @@ def box_datetime():
 @pytest.fixture
 def unsuccessful_jwt_response(box_datetime, status_code, error_description, include_date_header, error_code):
     headers = {'Date': box_datetime.strftime('%a, %d %b %Y %H:%M:%S %Z')} if include_date_header else {}
-    unsuccessful_response = Mock(requests.Response(), headers=headers)
+    unsuccessful_response = Mock(headers=headers)
     unsuccessful_response.json.return_value = {'error_description': error_description, 'error': error_code}
     unsuccessful_response.status_code = status_code
     unsuccessful_response.ok = False

test/unit/auth/test_oauth2.py

@@ -3,10 +3,10 @@
 from functools import partial
 import re
 from threading import Thread
+from unittest.mock import Mock, patch
 import uuid
 from urllib import parse
 
-from mock import Mock, patch
 import pytest
 
 from boxsdk.exception import BoxOAuthException

test/unit/auth/test_redis_managed_oauth2.py

@@ -1,9 +1,8 @@
 # coding: utf-8
 
+from unittest.mock import Mock, patch
 import uuid
 
-from mock import Mock, patch
-
 from boxsdk.auth import redis_managed_oauth2
 
 

test/unit/auth/test_remote_managed_oauth2.py

@@ -1,6 +1,6 @@
 # coding: utf-8
 
-from mock import Mock
+from unittest.mock import Mock
 
 from boxsdk.auth import remote_managed_oauth2
 

test/unit/client/test_client.py

@@ -3,8 +3,8 @@
 
 import json
 from io import BytesIO
+from unittest.mock import Mock, ANY
 
-from mock import Mock, ANY
 import pytest
 
 

test/unit/conftest.py

@@ -3,7 +3,7 @@
 import copy
 import json
 
-from mock import Mock, MagicMock
+from unittest.mock import Mock, MagicMock
 import pytest
 
 from boxsdk.config import API, Client, Proxy

test/unit/network/conftest.py

@@ -1,6 +1,6 @@
 # coding: utf-8
 
-from mock import Mock
+from unittest.mock import Mock
 import pytest
 from requests import Session
 

test/unit/network/test_network.py

@@ -5,8 +5,8 @@
 from logging import Logger
 from operator import attrgetter
 from pprint import pformat
+from unittest.mock import DEFAULT, Mock, patch, ANY
 
-from mock import DEFAULT, Mock, patch, ANY
 import pytest
 from requests import Response
 

test/unit/object/conftest.py

@@ -1,7 +1,7 @@
 # coding: utf-8
 
 import os
-from mock import Mock
+from unittest.mock import Mock
 import pytest
 from boxsdk.object.collaboration import Collaboration
 from boxsdk.object.collection import Collection

test/unit/object/test_base_api_json_object.py

@@ -40,7 +40,7 @@ def test_iter(base_api_json_object):
 
 
 def test_meta_registers_new_item_type_in_default_translator(default_translator, original_default_translator):
-    item_type = u'ƒøø'
+    item_type = 'ƒøø'
 
     class Foo(BaseAPIJSONObject):
         _item_type = item_type

test/unit/object/test_chunked_upload.py

@@ -1,11 +1,10 @@
 # coding: utf-8
 # pylint: disable-msg=too-many-locals
-
+from unittest.mock import MagicMock, Mock, call
 import io
 import json
 import pytest
 
-from mock import MagicMock, Mock, call
 from boxsdk.config import API
 from boxsdk.exception import BoxAPIException
 from boxsdk.exception import BoxException

test/unit/object/test_device_pin.py

@@ -1,8 +1,8 @@
 # coding: utf-8
 
+from unittest.mock import Mock
 import pytest
 
-from mock import Mock
 from boxsdk.config import API
 from boxsdk.object.device_pinner import DevicePinner
 from boxsdk.network.default_network import DefaultNetworkResponse

test/unit/object/test_events.py

@@ -4,9 +4,9 @@
 from itertools import chain
 import json
 from typing import Optional, Union
+from unittest.mock import Mock
 from urllib.parse import urlunsplit, urlencode
 
-from mock import Mock
 import pytest
 from requests.exceptions import Timeout
 

test/unit/object/test_file.py

@@ -2,8 +2,8 @@
 
 import json
 from io import BytesIO
+from unittest.mock import mock_open, patch, Mock
 
-from mock import mock_open, patch, Mock
 import pytest
 from boxsdk.config import API
 from boxsdk.exception import BoxAPIException

test/unit/object/test_folder.py

@@ -3,7 +3,7 @@
 import json
 from io import BytesIO
 from os.path import basename
-from mock import mock_open, patch, Mock, MagicMock
+from unittest.mock import mock_open, patch, Mock, MagicMock
 import pytest
 from boxsdk.config import API
 from boxsdk.exception import BoxAPIException

test/unit/object/test_group.py

@@ -3,8 +3,8 @@
 from itertools import chain, islice, repeat, count
 import json
 from operator import sub
+from unittest.mock import Mock
 
-from mock import Mock
 import pytest
 
 from boxsdk.network.default_network import DefaultNetworkResponse

test/unit/object/test_legal_hold_assignment.py

@@ -1,8 +1,7 @@
 # coding: utf-8
-
+from unittest.mock import Mock
 import pytest
 
-from mock import Mock
 from boxsdk.object.legal_hold_policy_assignment import LegalHoldPolicyAssignment
 from boxsdk.config import API
 from boxsdk.network.default_network import DefaultNetworkResponse

test/unit/object/test_legal_hold_policy.py

@@ -1,8 +1,8 @@
 # coding: utf-8
+from unittest.mock import Mock
 import json
 import pytest
 
-from mock import Mock
 from boxsdk.object.legal_hold_policy import LegalHoldPolicy
 from boxsdk.object.legal_hold_policy_assignment import LegalHoldPolicyAssignment
 from boxsdk.config import API

test/unit/object/test_search.py

@@ -1,9 +1,8 @@
 # coding: utf-8
-
+from unittest.mock import ANY
 import json
 import pytest
 
-from mock import ANY
 from boxsdk.config import API
 from boxsdk.object.file import File
 from boxsdk.object.user import User

test/unit/object/test_storage_policy_assignment.py

@@ -1,8 +1,8 @@
 # coding: utf-8
+from unittest.mock import Mock
 import json
 import pytest
 
-from mock import Mock
 from boxsdk.config import API
 from boxsdk.object.storage_policy_assignment import StoragePolicyAssignment
 from boxsdk.network.default_network import DefaultNetworkResponse

test/unit/object/test_task.py

@@ -1,7 +1,7 @@
+from unittest.mock import Mock
 import json
 import pytest
 
-from mock import Mock
 from boxsdk.config import API
 from boxsdk.object.task import Task
 from boxsdk.object.task_assignment import TaskAssignment

test/unit/object/test_task_assignment.py

@@ -1,7 +1,7 @@
+from unittest.mock import Mock
 import json
 import pytest
 
-from mock import Mock
 from boxsdk.config import API
 from boxsdk.object.task_assignment import TaskAssignment, ResolutionState
 from boxsdk.network.default_network import DefaultNetworkResponse