Skip to content

Commit a2cfcb5

Browse files
authored
Patch revoke session response (#486)
* Update version to 5.31.0 * Remove unused mock session from test
1 parent 15bf6e3 commit a2cfcb5

File tree

2 files changed

+7
-34
lines changed

2 files changed

+7
-34
lines changed

tests/test_user_management_revoke_session.py

Lines changed: 2 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -6,24 +6,6 @@
66
from workos.user_management import AsyncUserManagement, UserManagement
77

88

9-
def _mock_session(id: str):
10-
now = "2025-07-23T14:00:00.000Z"
11-
return {
12-
"object": "session",
13-
"id": id,
14-
"user_id": "user_123",
15-
"organization_id": "org_123",
16-
"status": "revoked",
17-
"auth_method": "password",
18-
"ip_address": "192.168.1.1",
19-
"user_agent": "Mozilla/5.0",
20-
"expires_at": "2025-07-23T15:00:00.000Z",
21-
"ended_at": now,
22-
"created_at": now,
23-
"updated_at": now,
24-
}
25-
26-
279
@pytest.mark.sync_and_async(UserManagement, AsyncUserManagement)
2810
class TestUserManagementRevokeSession:
2911
@pytest.fixture(autouse=True)
@@ -32,10 +14,7 @@ def setup(self, module_instance: Union[UserManagement, AsyncUserManagement]):
3214
self.user_management = module_instance
3315

3416
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-
)
17+
request_kwargs = capture_and_mock_http_client_request(self.http_client, {}, 200)
3918

4019
response = syncify(
4120
self.user_management.revoke_session(session_id="session_abc")
@@ -44,4 +23,4 @@ def test_revoke_session(self, capture_and_mock_http_client_request):
4423
assert request_kwargs["url"].endswith("user_management/sessions/revoke")
4524
assert request_kwargs["method"] == "post"
4625
assert request_kwargs["json"] == {"session_id": "session_abc"}
47-
assert response.id == "session_abc"
26+
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)