smartsheet-python-sdk 3.5.2__tar.gz → 3.5.4__tar.gz

{smartsheet_python_sdk-3.5.2 → smartsheet_python_sdk-3.5.4}/CHANGELOG.md

@@ -5,6 +5,22 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## [3.5.4] - 2025-10-31
+
+### Fixed
+
+- [Issue-92](https://github.com/smartsheet/smartsheet-python-sdk/issues/92): Only include the `smartsheet` package in the build.
+
+## [3.5.3] - 2025-10-31
+
+### Fixed
+
+- The new sharing endpoints (/2.0/shares) return string for userId and groupId, adjust the SDK to that.
+
+### Added
+
+- `AssetShare` model to fully comply with the new sharing endpoints (/2.0/shares) DTO.
+
 ## [3.5.2] - 2025-10-29
 
 ### Fixed

{smartsheet_python_sdk-3.5.2 → smartsheet_python_sdk-3.5.4}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.1
+Metadata-Version: 2.4
 Name: smartsheet_python_sdk
-Version: 3.5.2
+Version: 3.5.4
 Summary: Library that uses Python to connect to Smartsheet services (using API 2.0).
 Home-page: http://smartsheet-platform.github.io/api-docs/
 Author: Smartsheet
@@ -20,10 +20,34 @@ Classifier: Programming Language :: Python :: 3.10
 Classifier: Topic :: Software Development :: Libraries :: Python Modules
 Classifier: Topic :: Office/Business :: Financial :: Spreadsheet
 Description-Content-Type: text/markdown
-Provides-Extra: test
-Provides-Extra: develop
 License-File: LICENSE.md
 License-File: NOTICE
+Requires-Dist: requests
+Requires-Dist: requests-toolbelt
+Requires-Dist: six>=1.9
+Requires-Dist: certifi
+Requires-Dist: python-dateutil
+Provides-Extra: test
+Requires-Dist: coverage; extra == "test"
+Requires-Dist: coveralls; extra == "test"
+Requires-Dist: pytest; extra == "test"
+Provides-Extra: develop
+Requires-Dist: coverage; extra == "develop"
+Requires-Dist: coveralls[yaml]; extra == "develop"
+Requires-Dist: pytest; extra == "develop"
+Requires-Dist: pytest-instafail; extra == "develop"
+Dynamic: author
+Dynamic: author-email
+Dynamic: classifier
+Dynamic: description
+Dynamic: description-content-type
+Dynamic: home-page
+Dynamic: keywords
+Dynamic: license
+Dynamic: license-file
+Dynamic: provides-extra
+Dynamic: requires-dist
+Dynamic: summary
 
 # Smartsheet Python SDK
 

{smartsheet_python_sdk-3.5.2 → smartsheet_python_sdk-3.5.4}/setup.py

@@ -65,7 +65,7 @@ setup(
     },
     setup_requires=['setuptools_scm==7.0.1'],
     install_requires=REQUIRES,
-    packages=find_packages(),
+    packages=find_packages(include=['smartsheet', 'smartsheet.*']),
     include_package_data=True,
     long_description=open('README.md').read(),
     long_description_content_type='text/markdown',

{smartsheet_python_sdk-3.5.2 → smartsheet_python_sdk-3.5.4}/smartsheet/models/__init__.py

@@ -123,3 +123,4 @@ from .widget_content import WidgetContent
 from .workspace import Workspace
 from .user_plan import UserPlan
 from .asset_shares_paginated_result import AssetSharesPaginatedResult
+from .asset_share import AssetShare

smartsheet_python_sdk-3.5.4/smartsheet/models/asset_share.py

@@ -0,0 +1,138 @@
+# pylint: disable=C0111,R0902,R0904,R0912,R0913,R0915,E1101
+# Smartsheet Python SDK.
+#
+# Copyright 2018 Smartsheet.com, Inc.
+#
+# Licensed under the Apache License, Version 2.0 (the "License"): you may
+# not use this file except in compliance with the License. You may obtain
+# a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
+# License for the specific language governing permissions and limitations
+# under the License.
+
+from __future__ import absolute_import
+
+from ..types import EnumeratedValue, String, json
+from ..util import deserialize, serialize
+from .enums import AccessLevel, ShareScope, ShareType
+
+
+class AssetShare:
+
+    """Smartsheet AssetShare data model."""
+
+    def __init__(self, props=None, base_obj=None):
+        """Initialize the AssetShare model."""
+        self._base = None
+        if base_obj is not None:
+            self._base = base_obj
+
+        self._access_level = EnumeratedValue(AccessLevel)
+        self._email = String()
+        self._group_id = String()
+        self._id_ = String()
+        self._name = String()
+        self._scope = EnumeratedValue(ShareScope)
+        self._type_ = EnumeratedValue(ShareType)
+        self._user_id = String()
+
+        if props:
+            deserialize(self, props)
+
+        # requests package Response object
+        self.request_response = None
+        self.__initialized = True
+
+    def __getattr__(self, key):
+        if key == "id":
+            return self.id_
+        elif key == "type":
+            return self.type_
+        else:
+            raise AttributeError(key)
+
+    def __setattr__(self, key, value):
+        if key == "id":
+            self.id_ = value
+        elif key == "type":
+            self.type_ = value
+        else:
+            super().__setattr__(key, value)
+
+    @property
+    def access_level(self):
+        return self._access_level
+
+    @access_level.setter
+    def access_level(self, value):
+        self._access_level.set(value)
+
+    @property
+    def email(self):
+        return self._email.value
+
+    @email.setter
+    def email(self, value):
+        self._email.value = value
+
+    @property
+    def group_id(self):
+        return self._group_id.value
+
+    @group_id.setter
+    def group_id(self, value):
+        self._group_id.value = value
+
+    @property
+    def id_(self):
+        return self._id_.value
+
+    @id_.setter
+    def id_(self, value):
+        self._id_.value = value
+
+    @property
+    def name(self):
+        return self._name.value
+
+    @name.setter
+    def name(self, value):
+        self._name.value = value
+
+    @property
+    def scope(self):
+        return self._scope
+
+    @scope.setter
+    def scope(self, value):
+        self._scope.set(value)
+
+    @property
+    def type_(self):
+        return self._type_
+
+    @type_.setter
+    def type_(self, value):
+        self._type_.set(value)
+
+    @property
+    def user_id(self):
+        return self._user_id.value
+
+    @user_id.setter
+    def user_id(self, value):
+        self._user_id.value = value
+
+    def to_dict(self):
+        return serialize(self)
+
+    def to_json(self):
+        return json.dumps(self.to_dict())
+
+    def __str__(self):
+        return self.to_json()

{smartsheet_python_sdk-3.5.2 → smartsheet_python_sdk-3.5.4}/smartsheet/models/asset_shares_paginated_result.py

@@ -20,7 +20,7 @@ from __future__ import absolute_import
 from typing import TypeVar, List
 from ..types import String, json, TypedList, importlib
 from ..util import deserialize, serialize
-from . import Share
+from .asset_share import AssetShare
 
 T = TypeVar('T')
 
@@ -52,7 +52,7 @@ class AssetSharesPaginatedResult:
         self.__initialized = True
 
     @property
-    def items(self) -> List[Share]:
+    def items(self) -> List[AssetShare]:
         return self._items
 
     @items.setter

{smartsheet_python_sdk-3.5.2 → smartsheet_python_sdk-3.5.4}/smartsheet/sharing.py

@@ -54,7 +54,7 @@ class Sharing:
         _op['query_params']['lastKey'] = last_key
         _op['query_params']['sharingInclude'] = sharing_include.name if sharing_include else None
 
-        expected = ['AssetSharesPaginatedResult', 'Share']
+        expected = ['AssetSharesPaginatedResult', 'AssetShare']
 
         prepped_request = self._base.prepare_request(_op)
         response = self._base.request(prepped_request, expected, _op)
@@ -70,7 +70,7 @@ class Sharing:
             share_id (str): Share ID
 
         Returns:
-            Share
+            AssetShare
         """
         _op = fresh_operation('get_asset_share')
         _op['method'] = 'GET'
@@ -78,7 +78,7 @@ class Sharing:
         _op['query_params']['assetType'] = asset_type
         _op['query_params']['assetId'] = asset_id
 
-        expected = 'Share'
+        expected = 'AssetShare'
         prepped_request = self._base.prepare_request(_op)
         response = self._base.request(prepped_request, expected, _op)
 
@@ -106,7 +106,7 @@ class Sharing:
         _op['query_params']['sendEmail'] = send_email
         _op['json'] = share_obj
 
-        expected = ['Result', 'Share']
+        expected = ['Result', 'AssetShare']
 
         prepped_request = self._base.prepare_request(_op)
         response = self._base.request(prepped_request, expected, _op)
@@ -132,7 +132,7 @@ class Sharing:
         _op['query_params']['assetId'] = asset_id
         _op['json'] = share_obj
 
-        expected = 'Share'
+        expected = 'AssetShare'
 
         prepped_request = self._base.prepare_request(_op)
         response = self._base.request(prepped_request, expected, _op)

{smartsheet_python_sdk-3.5.2 → smartsheet_python_sdk-3.5.4}/smartsheet/version.py

@@ -1,4 +1,4 @@
 # file generated by setuptools_scm
 # don't change, don't track in version control
-__version__ = version = '3.5.2'
-__version_tuple__ = version_tuple = (3, 5, 2)
+__version__ = version = '3.5.4'
+__version_tuple__ = version_tuple = (3, 5, 4)

{smartsheet_python_sdk-3.5.2 → smartsheet_python_sdk-3.5.4}/smartsheet_python_sdk.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.1
-Name: smartsheet-python-sdk
-Version: 3.5.2
+Metadata-Version: 2.4
+Name: smartsheet_python_sdk
+Version: 3.5.4
 Summary: Library that uses Python to connect to Smartsheet services (using API 2.0).
 Home-page: http://smartsheet-platform.github.io/api-docs/
 Author: Smartsheet
@@ -20,10 +20,34 @@ Classifier: Programming Language :: Python :: 3.10
 Classifier: Topic :: Software Development :: Libraries :: Python Modules
 Classifier: Topic :: Office/Business :: Financial :: Spreadsheet
 Description-Content-Type: text/markdown
-Provides-Extra: test
-Provides-Extra: develop
 License-File: LICENSE.md
 License-File: NOTICE
+Requires-Dist: requests
+Requires-Dist: requests-toolbelt
+Requires-Dist: six>=1.9
+Requires-Dist: certifi
+Requires-Dist: python-dateutil
+Provides-Extra: test
+Requires-Dist: coverage; extra == "test"
+Requires-Dist: coveralls; extra == "test"
+Requires-Dist: pytest; extra == "test"
+Provides-Extra: develop
+Requires-Dist: coverage; extra == "develop"
+Requires-Dist: coveralls[yaml]; extra == "develop"
+Requires-Dist: pytest; extra == "develop"
+Requires-Dist: pytest-instafail; extra == "develop"
+Dynamic: author
+Dynamic: author-email
+Dynamic: classifier
+Dynamic: description
+Dynamic: description-content-type
+Dynamic: home-page
+Dynamic: keywords
+Dynamic: license
+Dynamic: license-file
+Dynamic: provides-extra
+Dynamic: requires-dist
+Dynamic: summary
 
 # Smartsheet Python SDK
 

{smartsheet_python_sdk-3.5.2 → smartsheet_python_sdk-3.5.4}/smartsheet_python_sdk.egg-info/SOURCES.txt

@@ -55,6 +55,7 @@ smartsheet/models/__init__.py
 smartsheet/models/access_token.py
 smartsheet/models/account.py
 smartsheet/models/alternate_email.py
+smartsheet/models/asset_share.py
 smartsheet/models/asset_shares_paginated_result.py
 smartsheet/models/attachment.py
 smartsheet/models/auto_number_format.py
@@ -261,4 +262,8 @@ tests/mock_api/test_mock_api_sheets.py
 tests/mock_api/test_mock_api_sights.py
 tests/mock_api/test_mock_api_workspaces.py
 tests/mock_api/test_mock_change_agent.py
-tests/mock_api/test_mock_serialization.py
+tests/mock_api/test_mock_serialization.py
+tests/mock_api/users/common_test_constants.py
+tests/mock_api/users/test_list_user_plans.py
+tests/mock_api/users/test_list_users.py
+tests/mock_api/users/test_user_upgrade_downgrade.py

{smartsheet_python_sdk-3.5.2 → smartsheet_python_sdk-3.5.4}/smartsheet_python_sdk.egg-info/top_level.txt

@@ -1,2 +1 @@
 smartsheet
-tests

{smartsheet_python_sdk-3.5.2 → smartsheet_python_sdk-3.5.4}/tests/mock_api/test_mock_api_sharing.py

@@ -4,7 +4,7 @@ from urllib.parse import urlparse, parse_qs
 
 import pytest
 from smartsheet.models.enums import AssetType, AccessLevel, ShareScope, ShareType
-from smartsheet.models import Share, AssetSharesPaginatedResult, Error
+from smartsheet.models import AssetShare, AssetSharesPaginatedResult, Error
 from tests.mock_api.mock_api_test_helper import (
     get_mock_api_client,
     get_wiremock_request,
@@ -16,8 +16,8 @@ TEST_ASSET_ID = "AAAMCmYGFOeE"
 TEST_SHARE_ID = "AAABbbbCccDdd"
 TEST_ASSET_TYPE = AssetType.SHEET
 TEST_EMAIL = "test.email@smartsheet.com"
-TEST_USER_ID = 9876543210
-TEST_GROUP_ID = 1234567890
+TEST_USER_ID = "9876543210"
+TEST_GROUP_ID = "1234567890"
 TEST_NAME = "Example Name"
 TEST_SHARE_TYPE = ShareType.USER
 TEST_ACCESS_LEVEL = AccessLevel.ADMIN
@@ -25,11 +25,14 @@ TEST_SCOPE = ShareScope.ITEM
 TEST_MAX_ITEMS = 100
 TEST_LAST_KEY = "test_last_key"
 TEST_LAST_KEY_RESPONSE = "abcDefGhIjKlMnOpQrStUvWxYz"
+TEST_SUCCESS_MESSAGE = "SUCCESS"
+TEST_SUCCESS_RESULT_CODE = 0
+TEST_SEND_EMAIL = False
 
 
 def assert_share_properties(share, include_name=True):
     """Helper function to assert common share properties."""
-    assert isinstance(share, Share)
+    assert isinstance(share, AssetShare)
     assert share.id == TEST_ASSET_ID
     assert share.email == TEST_EMAIL
     assert share.user_id == TEST_USER_ID
@@ -43,10 +46,10 @@ def assert_share_properties(share, include_name=True):
     assert share.scope == TEST_SCOPE
 
 
-@pytest.fixture
-def test_share():
-    """Pytest fixture to create a test Share object."""
-    return Share({"email": TEST_EMAIL, "access_level": AccessLevel.VIEWER})
+@pytest.fixture(name="test_share")
+def test_share_fixture():
+    """Pytest fixture to create a test AssetShare object."""
+    return AssetShare({"email": TEST_EMAIL, "access_level": TEST_ACCESS_LEVEL})
 
 
 def test_list_asset_shares_generated_url_is_correct():
@@ -56,22 +59,22 @@ def test_list_asset_shares_generated_url_is_correct():
     )
 
     client.Sharing.list_asset_shares(
-        asset_type=AssetType.SHEET,
-        asset_id="AAAMCmYGFOeE",
-        max_items=100,
-        last_key="test_last_key",
-        sharing_include=ShareScope.ITEM,
+        asset_type=TEST_ASSET_TYPE,
+        asset_id=TEST_ASSET_ID,
+        max_items=TEST_MAX_ITEMS,
+        last_key=TEST_LAST_KEY,
+        sharing_include=TEST_SCOPE,
     )
 
     wiremock_request = get_wiremock_request(request_id)
     url = urlparse(wiremock_request["absoluteUrl"])
     query = parse_qs(url.query)
     assert query == {
-        "assetType": ["sheet"],
-        "assetId": ["AAAMCmYGFOeE"],
-        "maxItems": ["100"],
-        "lastKey": ["test_last_key"],
-        "sharingInclude": ["ITEM"],
+        "assetType": [TEST_ASSET_TYPE.value],
+        "assetId": [TEST_ASSET_ID],
+        "maxItems": [str(TEST_MAX_ITEMS)],
+        "lastKey": [TEST_LAST_KEY],
+        "sharingInclude": [TEST_SCOPE.name],
     }
 
 
@@ -82,25 +85,25 @@ def test_list_asset_shares_all_response_properties():
     )
 
     response = client.Sharing.list_asset_shares(
-        asset_type=AssetType.SHEET, asset_id="AAAMCmYGFOeE"
+        asset_type=TEST_ASSET_TYPE, asset_id=TEST_ASSET_ID
     )
 
     assert isinstance(response, AssetSharesPaginatedResult)
 
-    assert response.last_key == "abcDefGhIjKlMnOpQrStUvWxYz"
+    assert response.last_key == TEST_LAST_KEY_RESPONSE
     assert len(response.items) == 1
 
     share = response.items[0]
-    assert isinstance(share, Share)
+    assert isinstance(share, AssetShare)
 
-    assert share.id == "AAAMCmYGFOeE"
-    assert share.email == "test.email@smartsheet.com"
-    assert share.user_id == 9876543210
-    assert share.group_id == 1234567890
-    assert share.name == "Example Name"
-    assert share.type == ShareType.USER
-    assert share.access_level == AccessLevel.ADMIN
-    assert share.scope == ShareScope.ITEM
+    assert share.id == TEST_ASSET_ID
+    assert share.email == TEST_EMAIL
+    assert share.user_id == TEST_USER_ID
+    assert share.group_id == TEST_GROUP_ID
+    assert share.name == TEST_NAME
+    assert share.type == TEST_SHARE_TYPE
+    assert share.access_level == TEST_ACCESS_LEVEL
+    assert share.scope == TEST_SCOPE
 
 
 def test_list_asset_shares_required_response_properties():
@@ -110,7 +113,7 @@ def test_list_asset_shares_required_response_properties():
     )
 
     response = client.Sharing.list_asset_shares(
-        asset_type=AssetType.SHEET, asset_id="AAAMCmYGFOeE"
+        asset_type=TEST_ASSET_TYPE, asset_id=TEST_ASSET_ID
     )
 
     assert isinstance(response, AssetSharesPaginatedResult)
@@ -119,16 +122,16 @@ def test_list_asset_shares_required_response_properties():
     assert len(response.items) == 1
 
     share = response.items[0]
-    assert isinstance(share, Share)
+    assert isinstance(share, AssetShare)
 
-    assert share.id == "AAAMCmYGFOeE"
-    assert share.email == "test.email@smartsheet.com"
-    assert share.user_id == 9876543210
-    assert share.group_id == 1234567890
+    assert share.id == TEST_ASSET_ID
+    assert share.email == TEST_EMAIL
+    assert share.user_id == TEST_USER_ID
+    assert share.group_id == TEST_GROUP_ID
     assert share.name is None
-    assert share.type == ShareType.USER
-    assert share.access_level == AccessLevel.ADMIN
-    assert share.scope == ShareScope.ITEM
+    assert share.type == TEST_SHARE_TYPE
+    assert share.access_level == TEST_ACCESS_LEVEL
+    assert share.scope == TEST_SCOPE
 
 def test_list_asset_shares_error_4xx():
     request_id = uuid.uuid4().hex
@@ -137,11 +140,11 @@ def test_list_asset_shares_error_4xx():
     )
 
     response = client.Sharing.list_asset_shares(
-        asset_type=AssetType.SHEET,
-        asset_id="AAAMCmYGFOeE",
-        max_items=100,
-        last_key="test_last_key",
-        sharing_include=ShareScope.ITEM,
+        asset_type=TEST_ASSET_TYPE,
+        asset_id=TEST_ASSET_ID,
+        max_items=TEST_MAX_ITEMS,
+        last_key=TEST_LAST_KEY,
+        sharing_include=TEST_SCOPE,
     )
 
     assert isinstance(response, Error)
@@ -153,11 +156,11 @@ def test_list_asset_shares_error_5xx():
     )
 
     response = client.Sharing.list_asset_shares(
-        asset_type=AssetType.SHEET,
-        asset_id="AAAMCmYGFOeE",
-        max_items=100,
-        last_key="test_last_key",
-        sharing_include=ShareScope.ITEM,
+        asset_type=TEST_ASSET_TYPE,
+        asset_id=TEST_ASSET_ID,
+        max_items=TEST_MAX_ITEMS,
+        last_key=TEST_LAST_KEY,
+        sharing_include=TEST_SCOPE,
     )
 
     assert isinstance(response, Error)
@@ -177,7 +180,7 @@ def test_get_asset_share_generated_url_is_correct():
     assert url.path == f'/2.0/shares/{TEST_SHARE_ID}'
     query = parse_qs(url.query)
     assert query == {
-        "assetType": ["sheet"],
+        "assetType": [TEST_ASSET_TYPE.value],
         "assetId": [TEST_ASSET_ID],
     }
 
@@ -248,9 +251,9 @@ def test_share_asset_generated_url_is_correct(test_share):
     assert url.path == '/2.0/shares'
     query = parse_qs(url.query)
     assert query == {
-        "assetType": ["sheet"],
+        "assetType": [TEST_ASSET_TYPE.value],
         "assetId": [TEST_ASSET_ID],
-        "sendEmail": ["False"]
+        "sendEmail": [str(TEST_SEND_EMAIL)]
     }
 
 def test_share_asset_all_response_properties(test_share):
@@ -275,7 +278,7 @@ def test_share_asset_all_response_properties(test_share):
 
     wiremock_request = get_wiremock_request(request_id)
     body = json.loads(wiremock_request["body"])
-    assert body == {"accessLevel": "VIEWER", "email": TEST_EMAIL}
+    assert body == {"accessLevel": TEST_ACCESS_LEVEL.name, "email": TEST_EMAIL}
 
 
 def test_share_multiple_assets_all_response_properties(test_share):
@@ -302,7 +305,7 @@ def test_share_multiple_assets_all_response_properties(test_share):
 
     wiremock_request = get_wiremock_request(request_id)
     body = json.loads(wiremock_request["body"])
-    assert body == [{"accessLevel": "VIEWER", "email": TEST_EMAIL}]
+    assert body == [{"accessLevel": TEST_ACCESS_LEVEL.name, "email": TEST_EMAIL}]
 
 
 
@@ -362,7 +365,7 @@ def test_update_asset_share_generated_url_is_correct():
         "/sharing/update-asset-share/all-response-body-properties", request_id
     )
 
-    share = Share({"access_level": AccessLevel.EDITOR})
+    share = AssetShare({"access_level": TEST_ACCESS_LEVEL})
 
     client.Sharing.update_asset_share(
         share_obj=share,
@@ -376,7 +379,7 @@ def test_update_asset_share_generated_url_is_correct():
     assert url.path == f'/2.0/shares/{TEST_SHARE_ID}'
     query = parse_qs(url.query)
     assert query == {
-        "assetType": ["sheet"],
+        "assetType": [TEST_ASSET_TYPE.value],
         "assetId": [TEST_ASSET_ID],
     }
 
@@ -386,7 +389,7 @@ def test_update_asset_share_all_response_properties():
         "/sharing/update-asset-share/all-response-body-properties", request_id
     )
 
-    share = Share({"access_level": AccessLevel.EDITOR})
+    share = AssetShare({"access_level": TEST_ACCESS_LEVEL})
 
     response = client.Sharing.update_asset_share(
         share_obj=share,
@@ -395,7 +398,7 @@ def test_update_asset_share_all_response_properties():
         share_id=TEST_SHARE_ID,
     )
 
-    # Update returns the Share object directly, not wrapped in result
+    # Update returns the AssetShare object directly, not wrapped in result
     assert_share_properties(response, include_name=True)
 
 
@@ -405,7 +408,7 @@ def test_update_asset_share_required_response_properties():
         "/sharing/update-asset-share/required-response-body-properties", request_id
     )
 
-    share = Share({"access_level": TEST_ACCESS_LEVEL})
+    share = AssetShare({"access_level": TEST_ACCESS_LEVEL})
 
     response = client.Sharing.update_asset_share(
         share_obj=share,
@@ -414,7 +417,7 @@ def test_update_asset_share_required_response_properties():
         share_id=TEST_SHARE_ID,
     )
 
-    # Update returns the Share object directly, not wrapped in result
+    # Update returns the AssetShare object directly, not wrapped in result
     assert_share_properties(response, include_name=False)
 
 def test_update_asset_share_error_4xx():
@@ -423,7 +426,7 @@ def test_update_asset_share_error_4xx():
         "/errors/400-response", request_id
     )
 
-    share = Share({"access_level": AccessLevel.EDITOR})
+    share = AssetShare({"access_level": TEST_ACCESS_LEVEL})
 
     response = client.Sharing.update_asset_share(
         share_obj=share,
@@ -440,7 +443,7 @@ def test_update_asset_share_error_5xx():
         "/errors/500-response", request_id
     )
 
-    share = Share({"access_level": AccessLevel.EDITOR})
+    share = AssetShare({"access_level": TEST_ACCESS_LEVEL})
 
     response = client.Sharing.update_asset_share(
         share_obj=share,
@@ -466,7 +469,7 @@ def test_delete_asset_share_generated_url_is_correct():
     assert url.path == f'/2.0/shares/{TEST_SHARE_ID}'
     query = parse_qs(url.query)
     assert query == {
-        "assetType": ["sheet"],
+        "assetType": [TEST_ASSET_TYPE.value],
         "assetId": [TEST_ASSET_ID],
     }
 
@@ -480,8 +483,8 @@ def test_delete_asset_share_all_response_properties():
         asset_type=TEST_ASSET_TYPE, asset_id=TEST_ASSET_ID, share_id=TEST_SHARE_ID
     )
 
-    assert response.message == "SUCCESS"
-    assert response.result_code == 0
+    assert response.message == TEST_SUCCESS_MESSAGE
+    assert response.result_code == TEST_SUCCESS_RESULT_CODE
 
 def test_delete_asset_share_error_4xx():
     request_id = uuid.uuid4().hex

smartsheet_python_sdk-3.5.4/tests/mock_api/users/common_test_constants.py

@@ -0,0 +1,4 @@
+TEST_USER_ID = 1234567890
+TEST_PLAN_ID = 1234567890123456
+TEST_SUCCESS_MESSAGE = "SUCCESS"
+TEST_RESULT_CODE = 0