Skip to content

Commit debb45a

Browse files
committed
Remove deprecated code
1 parent 2ce66b3 commit debb45a

File tree

3 files changed

+0
-54
lines changed

3 files changed

+0
-54
lines changed

src/posit/connect/_deprecated.py

Lines changed: 0 additions & 23 deletions
This file was deleted.

src/posit/connect/permissions.py

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66

77
from requests.sessions import Session as Session
88

9-
from ._deprecated import warn_deprecated_and_removed_in_future
109
from .resources import Resource, ResourceParameters, Resources
1110

1211
if TYPE_CHECKING:
@@ -15,12 +14,6 @@
1514

1615

1716
class Permission(Resource):
18-
def delete(self) -> None:
19-
"""[Deprecated] Delete the permission."""
20-
warn_deprecated_and_removed_in_future("Please use .destroy() instead of .delete().")
21-
22-
self.destroy()
23-
2417
def destroy(self) -> None:
2518
"""Destroy the permission."""
2619
path = f"v1/content/{self['content_guid']}/permissions/{self['id']}"

tests/posit/connect/test_permissions.py

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66
import responses
77
from responses import matchers
88

9-
from posit.connect._deprecated import PositConnectDeprecationWarning
109
from posit.connect.groups import Group
1110
from posit.connect.permissions import Permission, Permissions
1211
from posit.connect.resources import ResourceParameters
@@ -39,29 +38,6 @@ def test(self):
3938
# assert
4039
assert mock_delete.call_count == 1
4140

42-
@responses.activate
43-
def test_destroy_deprecated(self):
44-
# data
45-
uid = "94"
46-
content_guid = "f2f37341-e21d-3d80-c698-a935ad614066"
47-
48-
# behavior
49-
mock_delete = responses.delete(
50-
f"https://connect.example/__api__/v1/content/{content_guid}/permissions/{uid}",
51-
)
52-
53-
# setup
54-
params = ResourceParameters(requests.Session(), Url("https://connect.example/__api__"))
55-
fake_permission = load_mock_dict(f"v1/content/{content_guid}/permissions/{uid}.json")
56-
permission = Permission(params, **fake_permission)
57-
58-
# invoke
59-
with pytest.warns(PositConnectDeprecationWarning, match="destroy"):
60-
permission.delete()
61-
62-
# assert
63-
assert mock_delete.call_count == 1
64-
6541

6642
class TestPermissionUpdate:
6743
@responses.activate

0 commit comments

Comments
 (0)