Skip to content

Commit 0a3ddef

Browse files
committed
Update return for revoke session
2 parents 619d452 + 15bf6e3 commit 0a3ddef

File tree

2 files changed

+7
-16
lines changed

2 files changed

+7
-16
lines changed

tests/test_user_management_revoke_session.py

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,7 @@ def setup(self, module_instance: Union[UserManagement, AsyncUserManagement]):
3232
self.user_management = module_instance
3333

3434
def test_revoke_session(self, capture_and_mock_http_client_request):
35-
mock = _mock_session("session_abc")
36-
request_kwargs = capture_and_mock_http_client_request(
37-
self.http_client, mock, 200
38-
)
35+
request_kwargs = capture_and_mock_http_client_request(self.http_client, {}, 200)
3936

4037
response = syncify(
4138
self.user_management.revoke_session(session_id="session_abc")
@@ -44,4 +41,4 @@ def test_revoke_session(self, capture_and_mock_http_client_request):
4441
assert request_kwargs["url"].endswith("user_management/sessions/revoke")
4542
assert request_kwargs["method"] == "post"
4643
assert request_kwargs["json"] == {"session_id": "session_abc"}
47-
assert response.id == "session_abc"
44+
assert response is None

workos/user_management.py

Lines changed: 5 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -739,9 +739,7 @@ def list_sessions(
739739
order: Optional[PaginationOrder] = "desc",
740740
) -> SyncOrAsync["SessionsListResource"]: ...
741741

742-
def revoke_session(
743-
self, *, session_id: str
744-
) -> SyncOrAsync[UserManagementSession]: ...
742+
def revoke_session(self, *, session_id: str) -> SyncOrAsync[None]: ...
745743

746744
def get_magic_auth(self, magic_auth_id: str) -> SyncOrAsync[MagicAuth]:
747745
"""Get the details of a Magic Auth object.
@@ -1439,15 +1437,13 @@ def list_sessions(
14391437
**ListPage[UserManagementSession](**response).model_dump(),
14401438
)
14411439

1442-
def revoke_session(self, *, session_id: str) -> UserManagementSession:
1440+
def revoke_session(self, *, session_id: str) -> None:
14431441
json = {"session_id": session_id}
14441442

1445-
response = self._http_client.request(
1443+
self._http_client.request(
14461444
SESSIONS_REVOKE_PATH, method=REQUEST_METHOD_POST, json=json
14471445
)
14481446

1449-
return UserManagementSession.model_validate(response)
1450-
14511447
def enroll_auth_factor(
14521448
self,
14531449
*,
@@ -2143,15 +2139,13 @@ async def list_sessions(
21432139
**ListPage[UserManagementSession](**response).model_dump(),
21442140
)
21452141

2146-
async def revoke_session(self, *, session_id: str) -> UserManagementSession:
2142+
async def revoke_session(self, *, session_id: str) -> None:
21472143
json = {"session_id": session_id}
21482144

2149-
response = await self._http_client.request(
2145+
await self._http_client.request(
21502146
SESSIONS_REVOKE_PATH, method=REQUEST_METHOD_POST, json=json
21512147
)
21522148

2153-
return UserManagementSession.model_validate(response)
2154-
21552149
async def enroll_auth_factor(
21562150
self,
21572151
*,

0 commit comments

Comments
 (0)