From 5617e37cd465f120675d9f0fd3e19c2704c9ca47 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Karol=20S=C3=B3jko?= Date: Tue, 8 Aug 2023 15:09:34 +0200 Subject: [PATCH] chore: enable e2e test for already existing members of shared vaults (#2396) --- packages/services/src/Domain/VaultInvite/VaultInviteService.ts | 3 +++ packages/snjs/mocha/vaults/invites.test.js | 2 +- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/packages/services/src/Domain/VaultInvite/VaultInviteService.ts b/packages/services/src/Domain/VaultInvite/VaultInviteService.ts index 32b71dd0f..a48779de4 100644 --- a/packages/services/src/Domain/VaultInvite/VaultInviteService.ts +++ b/packages/services/src/Domain/VaultInvite/VaultInviteService.ts @@ -189,6 +189,9 @@ export class VaultInviteService sharedVaultContacts: contacts, permission, }) + if (result.isFailed()) { + return Result.fail(result.getError()) + } void this.notifyEvent(VaultInviteServiceEvent.InviteSent) diff --git a/packages/snjs/mocha/vaults/invites.test.js b/packages/snjs/mocha/vaults/invites.test.js index f0c54fb24..29db7f51f 100644 --- a/packages/snjs/mocha/vaults/invites.test.js +++ b/packages/snjs/mocha/vaults/invites.test.js @@ -224,7 +224,7 @@ describe('shared vault invites', function () { await deinitContactContext() }) - it.skip('should fail to invite user if already member of shared vault', async () => { + it('should fail to invite user if already member of shared vault', async () => { const { sharedVault, contact, deinitContactContext } = await Collaboration.createSharedVaultWithAcceptedInvite( context, )