diff --git a/tests/test_user_management_revoke_session.py b/tests/test_user_management_revoke_session.py index 7efedc65..6f6247ca 100644 --- a/tests/test_user_management_revoke_session.py +++ b/tests/test_user_management_revoke_session.py @@ -6,24 +6,6 @@ from workos.user_management import AsyncUserManagement, UserManagement -def _mock_session(id: str): - now = "2025-07-23T14:00:00.000Z" - return { - "object": "session", - "id": id, - "user_id": "user_123", - "organization_id": "org_123", - "status": "revoked", - "auth_method": "password", - "ip_address": "192.168.1.1", - "user_agent": "Mozilla/5.0", - "expires_at": "2025-07-23T15:00:00.000Z", - "ended_at": now, - "created_at": now, - "updated_at": now, - } - - @pytest.mark.sync_and_async(UserManagement, AsyncUserManagement) class TestUserManagementRevokeSession: @pytest.fixture(autouse=True) @@ -32,10 +14,7 @@ def setup(self, module_instance: Union[UserManagement, AsyncUserManagement]): self.user_management = module_instance def test_revoke_session(self, capture_and_mock_http_client_request): - mock = _mock_session("session_abc") - request_kwargs = capture_and_mock_http_client_request( - self.http_client, mock, 200 - ) + request_kwargs = capture_and_mock_http_client_request(self.http_client, {}, 200) response = syncify( self.user_management.revoke_session(session_id="session_abc") @@ -44,4 +23,4 @@ def test_revoke_session(self, capture_and_mock_http_client_request): assert request_kwargs["url"].endswith("user_management/sessions/revoke") assert request_kwargs["method"] == "post" assert request_kwargs["json"] == {"session_id": "session_abc"} - assert response.id == "session_abc" + assert response is None diff --git a/workos/user_management.py b/workos/user_management.py index 9e72bb14..d06b90c7 100644 --- a/workos/user_management.py +++ b/workos/user_management.py @@ -739,9 +739,7 @@ def list_sessions( order: Optional[PaginationOrder] = "desc", ) -> SyncOrAsync["SessionsListResource"]: ... - def revoke_session( - self, *, session_id: str - ) -> SyncOrAsync[UserManagementSession]: ... + def revoke_session(self, *, session_id: str) -> SyncOrAsync[None]: ... def get_magic_auth(self, magic_auth_id: str) -> SyncOrAsync[MagicAuth]: """Get the details of a Magic Auth object. @@ -1439,15 +1437,13 @@ def list_sessions( **ListPage[UserManagementSession](**response).model_dump(), ) - def revoke_session(self, *, session_id: str) -> UserManagementSession: + def revoke_session(self, *, session_id: str) -> None: json = {"session_id": session_id} - response = self._http_client.request( + self._http_client.request( SESSIONS_REVOKE_PATH, method=REQUEST_METHOD_POST, json=json ) - return UserManagementSession.model_validate(response) - def enroll_auth_factor( self, *, @@ -2143,15 +2139,13 @@ async def list_sessions( **ListPage[UserManagementSession](**response).model_dump(), ) - async def revoke_session(self, *, session_id: str) -> UserManagementSession: + async def revoke_session(self, *, session_id: str) -> None: json = {"session_id": session_id} - response = await self._http_client.request( + await self._http_client.request( SESSIONS_REVOKE_PATH, method=REQUEST_METHOD_POST, json=json ) - return UserManagementSession.model_validate(response) - async def enroll_auth_factor( self, *,