feat!: Remove deprecated API: CollaborationWhitelist (#651)

Removed deprecated classes: `CollaborationWhitelist`,  `CollaborationWhitelistEntry` and `CollaborationWhitelistExemptTarget`. Replaced with `CollaborationAllowlist`,  `CollaborationAllowlistEntry` and `CollaborationAllowlistExemptTarget.`
Closes: SDK-1887

Author: lukaszsocha2

boxsdk/client/client.py

@@ -8,7 +8,6 @@
 from ..util.api_call_decorator import api_call
 from ..object.search import Search
 from ..object.events import Events
-from ..object.collaboration_whitelist import CollaborationWhitelist
 from ..object.collaboration_allowlist import CollaborationAllowlist
 from ..object.trash import Trash
 from ..pagination.limit_offset_based_object_collection import LimitOffsetBasedObjectCollection
@@ -228,53 +227,6 @@ def collaboration(self, collab_id):
         """
         return self.translator.get('collaboration')(session=self._session, object_id=collab_id)
 
-    @deprecated('Use collaboration_allowlist instead')
-    def collaboration_whitelist(self):
-        """
-        Initilializes a :class:`CollaborationWhitelist` object.
-
-        :return:
-            A :class:`CollaborationWhitelist` object.
-        :rype:
-            :class:`CollaborationWhitelist`.
-        """
-        return CollaborationWhitelist(self._session)
-
-    @deprecated('Use collaboration_allowlist_entry instead')
-    def collaboration_whitelist_entry(self, entry_id):
-        """
-        Initialize a :class:`CollaborationWhitelistEntry` object, whose box id is entry_id.
-
-        :param entry_id:
-            The box id of the :class:`CollaborationWhitelistEntry` object.
-        :type entry_id:
-            `unicode`
-        :return:
-            A :class:`CollaborationWhitelistEntry` object with the given entry id.
-        :rtype:
-            :class:`CollaborationWhitelistEntry`
-        """
-        return self.translator.get('collaboration_whitelist_entry')(session=self._session, object_id=entry_id)
-
-    @deprecated('Use collaboration_allowlist_exempt_target instead')
-    def collaboration_whitelist_exempt_target(self, exemption_id):
-        """
-        Initialize a :class:`CollaborationWhitelistExemptTarget` object, whose box id is target_id.
-
-        :param exemption_id:
-            The box id of the :class:`CollaborationWhitelistExemptTarget` object.
-        :type exemption_id:
-            `unicode`
-        :return:
-            A :class:`CollaborationWhitelistExemptTarget` object with the given target id.
-        :rtype:
-            :class:`CollaborationWhitelistExemptTarget`
-        """
-        return self.translator.get('collaboration_whitelist_exempt_target')(
-            session=self._session,
-            object_id=exemption_id
-        )
-
     def collaboration_allowlist(self):
         """
         Initilializes a :class:`CollaborationAllowlist` object.

boxsdk/object/__init__.py

@@ -2,9 +2,6 @@
 
 __all__ = list(map(str, [
     'collaboration',
-    'collaboration_whitelist',
-    'collaboration_whitelist_entry',
-    'collaboration_whitelist_exempt_target',
     'collaboration_allowlist',
     'collaboration_allowlist_entry',
     'collaboration_allowlist_exempt_target',

boxsdk/object/collaboration_allowlist_entry.py

@@ -1,8 +1,8 @@
 # coding: utf-8
-from .collaboration_whitelist_entry import CollaborationWhitelistEntry
+from .base_object import BaseObject
 
 
-class CollaborationAllowlistEntry(CollaborationWhitelistEntry):
+class CollaborationAllowlistEntry(BaseObject):
     """Represents a allowlisted email domain for enterprise collaboration."""
 
     _item_type = 'collaboration_whitelist_entry'

boxsdk/object/collaboration_allowlist_exempt_target.py

@@ -1,8 +1,8 @@
 # coding: utf-8
-from boxsdk.object.collaboration_whitelist_exempt_target import CollaborationWhitelistExemptTarget
+from .base_object import BaseObject
 
 
-class CollaborationAllowlistExemptTarget(CollaborationWhitelistExemptTarget):
+class CollaborationAllowlistExemptTarget(BaseObject):
     """Represents a user who is exempted from the collaboration allowlist."""
 
     _item_type = 'collaboration_whitelist_exempt_target'

boxsdk/object/collaboration_whitelist.py

@@ -84,30 +84,6 @@ boxsdk.object.collaboration\_allowlist\_exempt\_target module
    :undoc-members:
    :show-inheritance:
 
-boxsdk.object.collaboration\_whitelist module
----------------------------------------------
-
-.. automodule:: boxsdk.object.collaboration_whitelist
-   :members:
-   :undoc-members:
-   :show-inheritance:
-
-boxsdk.object.collaboration\_whitelist\_entry module
-----------------------------------------------------
-
-.. automodule:: boxsdk.object.collaboration_whitelist_entry
-   :members:
-   :undoc-members:
-   :show-inheritance:
-
-boxsdk.object.collaboration\_whitelist\_exempt\_target module
--------------------------------------------------------------
-
-.. automodule:: boxsdk.object.collaboration_whitelist_exempt_target
-   :members:
-   :undoc-members:
-   :show-inheritance:
-
 boxsdk.object.collection module
 -------------------------------
 

boxsdk/object/collaboration_whitelist_entry.py

@@ -13,7 +13,6 @@
 from boxsdk.config import API
 from boxsdk.network.default_network import DefaultNetworkResponse
 from boxsdk.object.collaboration import Collaboration
-from boxsdk.object.collaboration_whitelist import CollaborationWhitelist
 from boxsdk.object.collaboration_allowlist import CollaborationAllowlist
 from boxsdk.object.email_alias import EmailAlias
 from boxsdk.object.collection import Collection
@@ -555,11 +554,6 @@ def test_events_returns_event_object(mock_client):
     assert isinstance(mock_client.events(), Events)
 
 
-def test_collaboration_whitelist_initializer(mock_client):
-    collaboration_whitelist = mock_client.collaboration_whitelist()
-    assert isinstance(collaboration_whitelist, CollaborationWhitelist)
-
-
 def test_collaboration_allowlist_initializer(mock_client):
     collaboration_allowlist = mock_client.collaboration_allowlist()
     assert isinstance(collaboration_allowlist, CollaborationAllowlist)

boxsdk/object/collaboration_whitelist_exempt_target.py

@@ -27,9 +27,6 @@
 from boxsdk.object.storage_policy_assignment import StoragePolicyAssignment
 from boxsdk.object.terms_of_service import TermsOfService
 from boxsdk.object.terms_of_service_user_status import TermsOfServiceUserStatus
-from boxsdk.object.collaboration_whitelist import CollaborationWhitelist
-from boxsdk.object.collaboration_whitelist_entry import CollaborationWhitelistEntry
-from boxsdk.object.collaboration_whitelist_exempt_target import CollaborationWhitelistExemptTarget
 from boxsdk.object.collaboration_allowlist import CollaborationAllowlist
 from boxsdk.object.collaboration_allowlist_entry import CollaborationAllowlistEntry
 from boxsdk.object.collaboration_allowlist_exempt_target import CollaborationAllowlistExemptTarget
@@ -109,21 +106,6 @@ def test_comment(mock_box_session, mock_object_id):
     return Comment(mock_box_session, mock_object_id)
 
 
-@pytest.fixture()
-def test_collaboration_whitelist(mock_box_session):
-    return CollaborationWhitelist(mock_box_session)
-
-
-@pytest.fixture()
-def test_collaboration_whitelist_entry(mock_box_session, mock_object_id):
-    return CollaborationWhitelistEntry(mock_box_session, mock_object_id)
-
-
-@pytest.fixture()
-def test_collaboration_whitelist_exemption(mock_box_session, mock_object_id):
-    return CollaborationWhitelistExemptTarget(mock_box_session, mock_object_id)
-
-
 @pytest.fixture()
 def test_collaboration_allowlist(mock_box_session):
     return CollaborationAllowlist(mock_box_session)