From 6cc916b210cb6af8ecf34154caa2e89bf80dcf70 Mon Sep 17 00:00:00 2001 From: Mike Date: Wed, 23 Apr 2025 01:28:15 +0200 Subject: [PATCH] Fix error message (#4559) --- packages/server/modules/core/repositories/streams.ts | 7 +++---- packages/server/modules/core/tests/streams.spec.ts | 2 +- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/packages/server/modules/core/repositories/streams.ts b/packages/server/modules/core/repositories/streams.ts index c510bf337..426e55c13 100644 --- a/packages/server/modules/core/repositories/streams.ts +++ b/packages/server/modules/core/repositories/streams.ts @@ -1167,10 +1167,9 @@ export const revokeStreamPermissionsFactory = .count<{ count: string }[]>() if (parseInt(streamAclEntriesCount.count) === 1) - throw new StreamAccessUpdateError( - 'Stream has only one ownership link left - cannot revoke permissions.', - { info: { streamId, userId } } - ) + throw new StreamAccessUpdateError('A project needs at least one project owner', { + info: { streamId, userId } + }) const aclEntry = existingPermission if (aclEntry?.role === Roles.Stream.Owner) { diff --git a/packages/server/modules/core/tests/streams.spec.ts b/packages/server/modules/core/tests/streams.spec.ts index fc83f3965..2faa2bd59 100644 --- a/packages/server/modules/core/tests/streams.spec.ts +++ b/packages/server/modules/core/tests/streams.spec.ts @@ -348,7 +348,7 @@ describe('Streams @core-streams', () => { throw new Error('This should have thrown') }) .catch((err) => { - expect(err.message).to.include('cannot revoke permissions.') + expect(err.message).to.include('A project needs at least one project owner') }) })