fix(group): make getGroupsPublicEncryptionKeys returns unique Keys

requesting keys for the same group multiple times in one call
now returns only unique keys instead of X times the same key

packages/core/src/Groups/Manager.ts

@@ -179,7 +179,8 @@ export default class GroupManager {
   }
 
   async getGroupsPublicEncryptionKeys(groupIds: Array<b64string>): Promise<Array<Uint8Array>> {
-    const result = await this._publicEncryptionKeyLookupCoalescer.run(this._getGroupsPublicEncryptionKeys, groupIds);
+    const uniqueGroupIds = [...new Set(groupIds)];
+    const result = await this._publicEncryptionKeyLookupCoalescer.run(this._getGroupsPublicEncryptionKeys, uniqueGroupIds);
 
     return result.map(record => record.key);
   }

packages/functional-tests/src/encryption.ts

@@ -148,6 +148,11 @@ export const generateEncryptionTests = (args: TestArgs) => {
         await expectDecrypt([aliceLaptop, bobLaptop], clearText, encrypted);
       });
 
+      it('dedupes identities when encrypting/sharing with the same permanent identity twice', async () => {
+        const encrypted = await bobLaptop.encrypt(clearText, { shareWithUsers: [alicePublicIdentity, alicePublicIdentity] });
+        await expectDecrypt([aliceLaptop], clearText, encrypted);
+      });
+
       it('encrypts and shares with a permanent identity and not self', async () => {
         const encrypted = await bobLaptop.encrypt(clearText, { shareWithUsers: [alicePublicIdentity], shareWithSelf: false });
         await expect(bobLaptop.decrypt(encrypted)).to.be.rejectedWith(errors.InvalidArgument);

packages/functional-tests/src/groups.ts

@@ -597,6 +597,15 @@ export const generateGroupsTests = (args: TestArgs) => {
     });
 
     describe('encrypt/share', () => {
+      it('dedupes groupIds when encrypting/sharing with the same group twice', async () => {
+        const alice = await UserSession.create(appHelper);
+        const bob = await UserSession.create(appHelper);
+        const groupId = await alice.session.createGroup([bob.spublicIdentity]);
+        const encrypted = await encrypt(alice.session, { shareWithGroups: [groupId, groupId] });
+
+        await checkDecrypt([bob], [encrypted]);
+      });
+
       it('encrypt for two groups', async () => {
         const alice = await UserSession.create(appHelper);
         const bob = await UserSession.create(appHelper);