diff --git a/atst/domain/workspaces/workspaces.py b/atst/domain/workspaces/workspaces.py index 4edecccf..d8960a3d 100644 --- a/atst/domain/workspaces/workspaces.py +++ b/atst/domain/workspaces/workspaces.py @@ -145,7 +145,7 @@ class Workspaces(object): WorkspacesQuery.add_and_commit(workspace) @classmethod - def can_revoke_access(cls, workspace, workspace_role): + def can_revoke_access_for(cls, workspace, workspace_role): return workspace_role.user != workspace.owner @classmethod @@ -159,7 +159,7 @@ class Workspaces(object): ) workspace_role = WorkspaceRoles.get_by_id(workspace_role_id) - if not Workspaces.can_revoke_access(workspace, workspace_role): + if not Workspaces.can_revoke_access_for(workspace, workspace_role): raise WorkspaceError("cannot revoke workspace access for this user") workspace_role.status = WorkspaceRoleStatus.DISABLED diff --git a/tests/domain/test_workspaces.py b/tests/domain/test_workspaces.py index 84adf696..61cb7b52 100644 --- a/tests/domain/test_workspaces.py +++ b/tests/domain/test_workspaces.py @@ -316,8 +316,8 @@ def test_can_revoke_access(): owner_role = workspace.roles[0] workspace_role = WorkspaceRoleFactory.create(workspace=workspace) - assert Workspaces.can_revoke_access(workspace, workspace_role) - assert not Workspaces.can_revoke_access(workspace, owner_role) + assert Workspaces.can_revoke_access_for(workspace, workspace_role) + assert not Workspaces.can_revoke_access_for(workspace, owner_role) def test_cant_revoke_owner_workspace_access():