@webex/plugin-meetings 3.8.0-next.8 → 3.8.0-next.80

package/test/unit/spec/locus-info/index.js

@@ -111,6 +111,8 @@ describe('plugin-meetings', () => {
           },
           webcastControl: {streaming: false},
           practiceSession: {enabled: true},
+          annotationControl: {enabled: true},
+          rdcControl: {enabled: true},
         };
       });
 
@@ -278,6 +280,32 @@ describe('plugin-meetings', () => {
         });
       });
 
+      it('should trigger the CONTROLS_ANNOTATION_CHANGED event when necessary', () => {
+        locusInfo.controls = {};
+        locusInfo.emitScoped = sinon.stub();
+        locusInfo.updateControls(newControls);
+
+        assert.calledWith(
+          locusInfo.emitScoped,
+          {file: 'locus-info', function: 'updateControls'},
+          LOCUSINFO.EVENTS.CONTROLS_ANNOTATION_CHANGED,
+          {state: newControls.annotationControl}
+        );
+      });
+
+      it('should trigger the CONTROLS_REMOTE_DESKTOP_CONTROL_CHANGED event when necessary', () => {
+        locusInfo.controls = {};
+        locusInfo.emitScoped = sinon.stub();
+        locusInfo.updateControls(newControls);
+
+        assert.calledWith(
+          locusInfo.emitScoped,
+          {file: 'locus-info', function: 'updateControls'},
+          LOCUSINFO.EVENTS.CONTROLS_REMOTE_DESKTOP_CONTROL_CHANGED,
+          {state: newControls.rdcControl}
+        );
+      });
+
       it('should keep the recording state to `IDLE`', () => {
         locusInfo.controls = {
           record: {
@@ -797,6 +825,32 @@ describe('plugin-meetings', () => {
 
         assert.isTrue(locusInfo.deltaParticipants.length === 0);
       });
+
+      it('should call with participant display name', () => {
+        const failureParticipant = [
+          {
+            person: {
+              id: 5678,
+              primaryDisplayString: 'Test User',
+            },
+            reason: 'FAILURE',
+          },
+        ];
+
+        locusInfo.emitScoped = sinon.stub();
+        locusInfo.updateParticipants(failureParticipant);
+        assert.calledWith(
+          locusInfo.emitScoped,
+          {
+            file: 'locus-info',
+            function: 'updateParticipants',
+          },
+          LOCUSINFO.EVENTS.PARTICIPANT_REASON_CHANGED,
+          {
+            displayName: 'Test User',
+          }
+        );
+      })
     });
 
     describe('#updateSelf', () => {
@@ -808,7 +862,7 @@ describe('plugin-meetings', () => {
           selfWithBrbChanged.controls.brb = enabled;
 
           locusInfo.emitScoped = sinon.stub();
-          locusInfo.updateSelf(selfWithBrbChanged, []);
+          locusInfo.updateSelf(selfWithBrbChanged);
 
           assert.calledWith(
             locusInfo.emitScoped,
@@ -828,14 +882,15 @@ describe('plugin-meetings', () => {
 
           const selfWithBrbChanged = cloneDeep(self);
           selfWithBrbChanged.controls.brb = value;
-          locusInfo.self = selfWithBrbChanged;
+
+          locusInfo.updateSelf(selfWithBrbChanged);
 
           locusInfo.emitScoped = sinon.stub();
 
           const newSelf = cloneDeep(self);
           newSelf.controls.brb = value;
 
-          locusInfo.updateSelf(newSelf, []);
+          locusInfo.updateSelf(newSelf);
 
           assert.neverCalledWith(
             locusInfo.emitScoped,
@@ -852,14 +907,14 @@ describe('plugin-meetings', () => {
       it('should not trigger SELF_MEETING_BRB_CHANGED when brb state is undefined', () => {
         const selfWithBrbChanged = cloneDeep(self);
         selfWithBrbChanged.controls.brb = false;
-        locusInfo.self = selfWithBrbChanged;
+        locusInfo.updateSelf(selfWithBrbChanged);
 
         locusInfo.emitScoped = sinon.stub();
 
         const newSelf = cloneDeep(self);
         newSelf.controls.brb = undefined;
 
-        locusInfo.updateSelf(newSelf, []);
+        locusInfo.updateSelf(newSelf);
 
         assert.neverCalledWith(
           locusInfo.emitScoped,
@@ -882,7 +937,7 @@ describe('plugin-meetings', () => {
         ];
 
         locusInfo.emitScoped = sinon.stub();
-        locusInfo.updateSelf(selfWithLayoutChanged, []);
+        locusInfo.updateSelf(selfWithLayoutChanged);
 
         assert.calledWith(
           locusInfo.emitScoped,
@@ -908,11 +963,11 @@ describe('plugin-meetings', () => {
         ];
 
         // Set the layout prior to stubbing to validate it does not change.
-        locusInfo.updateSelf(selfWithLayoutChanged, []);
+        locusInfo.updateSelf(selfWithLayoutChanged);
 
         locusInfo.emitScoped = sinon.stub();
 
-        locusInfo.updateSelf(selfWithLayoutChanged, []);
+        locusInfo.updateSelf(selfWithLayoutChanged);
 
         assert.neverCalledWith(
           locusInfo.emitScoped,
@@ -926,11 +981,11 @@ describe('plugin-meetings', () => {
       });
 
       it('should trigger MEDIA_INACTIVITY on server media inactivity', () => {
-        locusInfo.self = self;
-
         locusInfo.webex.internal.device.url = selfWithInactivity.deviceUrl;
+        locusInfo.updateSelf(self);
+
         locusInfo.emitScoped = sinon.stub();
-        locusInfo.updateSelf(selfWithInactivity, []);
+        locusInfo.updateSelf(selfWithInactivity);
 
         assert.calledWith(
           locusInfo.emitScoped,
@@ -952,7 +1007,7 @@ describe('plugin-meetings', () => {
 
         locusInfo.webex.internal.device.url = self.deviceUrl;
         locusInfo.emitScoped = sinon.stub();
-        locusInfo.updateSelf(selfWithMutedByOthers, []);
+        locusInfo.updateSelf(selfWithMutedByOthers);
 
         assert.calledWith(
           locusInfo.emitScoped,
@@ -965,10 +1020,10 @@ describe('plugin-meetings', () => {
         );
 
         // but sometimes "previous self" is defined, but without controls.audio.muted, so we test this here:
-        locusInfo.self = cloneDeep(self);
+        locusInfo.updateSelf(self);
         locusInfo.self.controls.audio = {};
 
-        locusInfo.updateSelf(selfWithMutedByOthers, []);
+        locusInfo.updateSelf(selfWithMutedByOthers);
         assert.calledWith(
           locusInfo.emitScoped,
           {
@@ -988,7 +1043,7 @@ describe('plugin-meetings', () => {
 
         locusInfo.webex.internal.device.url = self.deviceUrl;
         locusInfo.emitScoped = sinon.stub();
-        locusInfo.updateSelf(selfWithMutedByOthersFalse, []);
+        locusInfo.updateSelf(selfWithMutedByOthersFalse);
 
         // we might get some calls to emitScoped, but we need to check that none of them are for SELF_REMOTE_MUTE_STATUS_UPDATED
         locusInfo.emitScoped.getCalls().forEach((x) => {
@@ -997,20 +1052,20 @@ describe('plugin-meetings', () => {
       });
 
       it('should not trigger SELF_REMOTE_MUTE_STATUS_UPDATED when being removed from meeting', () => {
+        locusInfo.webex.internal.device.url = self.deviceUrl;
         const selfWithMutedByOthers = cloneDeep(self);
 
         selfWithMutedByOthers.controls.audio.muted = true;
 
-        locusInfo.self = selfWithMutedByOthers;
+        locusInfo.updateSelf(selfWithMutedByOthers);
 
         // when user gets removed from meeting we receive a Locus DTO without any self.controls
         const selfWithoutControls = cloneDeep(self);
 
         selfWithoutControls.controls = undefined;
 
-        locusInfo.webex.internal.device.url = self.deviceUrl;
         locusInfo.emitScoped = sinon.stub();
-        locusInfo.updateSelf(selfWithoutControls, []);
+        locusInfo.updateSelf(selfWithoutControls);
 
         // we might get some calls to emitScoped, but we need to check that none of them are for SELF_REMOTE_MUTE_STATUS_UPDATED
         locusInfo.emitScoped.getCalls().forEach((x) => {
@@ -1019,14 +1074,14 @@ describe('plugin-meetings', () => {
       });
 
       it('should trigger SELF_REMOTE_MUTE_STATUS_UPDATED on othersMuted', () => {
-        locusInfo.self = self;
+        locusInfo.webex.internal.device.url = self.deviceUrl;
+        locusInfo.updateSelf(self);
         const selfWithMutedByOthers = cloneDeep(self);
 
         selfWithMutedByOthers.controls.audio.muted = true;
 
-        locusInfo.webex.internal.device.url = self.deviceUrl;
         locusInfo.emitScoped = sinon.stub();
-        locusInfo.updateSelf(selfWithMutedByOthers, []);
+        locusInfo.updateSelf(selfWithMutedByOthers);
 
         assert.calledWith(
           locusInfo.emitScoped,
@@ -1050,7 +1105,7 @@ describe('plugin-meetings', () => {
 
           locusInfo.webex.internal.device.url = self.deviceUrl;
           locusInfo.emitScoped = sinon.stub();
-          locusInfo.updateSelf(selfWithMutedByOthers, []);
+          locusInfo.updateSelf(selfWithMutedByOthers);
 
           assert.calledWith(
             locusInfo.emitScoped,
@@ -1063,10 +1118,10 @@ describe('plugin-meetings', () => {
           );
 
           // but sometimes "previous self" is defined, but without controls.audio.muted, so we test this here:
-          locusInfo.self = cloneDeep(self);
+          locusInfo.updateSelf(self);
           locusInfo.self.controls.video = {};
 
-          locusInfo.updateSelf(selfWithMutedByOthers, []);
+          locusInfo.updateSelf(selfWithMutedByOthers);
           assert.calledWith(
             locusInfo.emitScoped,
             {
@@ -1086,7 +1141,7 @@ describe('plugin-meetings', () => {
 
           locusInfo.webex.internal.device.url = self.deviceUrl;
           locusInfo.emitScoped = sinon.stub();
-          locusInfo.updateSelf(selfWithMutedByOthersFalse, []);
+          locusInfo.updateSelf(selfWithMutedByOthersFalse);
 
           // we might get some calls to emitScoped, but we need to check that none of them are for SELF_REMOTE_VIDEO_MUTE_STATUS_UPDATED
           locusInfo.emitScoped.getCalls().forEach((x) => {
@@ -1095,14 +1150,14 @@ describe('plugin-meetings', () => {
         });
 
         it('should emit event when remoteVideoMuted changed', () => {
-          locusInfo.self = self;
+          locusInfo.webex.internal.device.url = self.deviceUrl;
+          locusInfo.updateSelf(self);
           const selfWithMutedByOthers = cloneDeep(self);
 
           selfWithMutedByOthers.controls.video.muted = true;
 
-          locusInfo.webex.internal.device.url = self.deviceUrl;
           locusInfo.emitScoped = sinon.stub();
-          locusInfo.updateSelf(selfWithMutedByOthers, []);
+          locusInfo.updateSelf(selfWithMutedByOthers);
 
           assert.calledWith(
             locusInfo.emitScoped,
@@ -1117,13 +1172,13 @@ describe('plugin-meetings', () => {
       });
 
       it('should trigger SELF_MEETING_BREAKOUTS_CHANGED when breakouts changed', () => {
-        locusInfo.self = self;
+        locusInfo.updateSelf(self);
         const selfWithBreakoutsChanged = cloneDeep(self);
 
         selfWithBreakoutsChanged.controls.breakout.sessions.active[0].name = 'new name';
 
         locusInfo.emitScoped = sinon.stub();
-        locusInfo.updateSelf(selfWithBreakoutsChanged, []);
+        locusInfo.updateSelf(selfWithBreakoutsChanged);
 
         assert.calledWith(
           locusInfo.emitScoped,
@@ -1156,16 +1211,16 @@ describe('plugin-meetings', () => {
       });
 
       it('should trigger SELF_REMOTE_MUTE_STATUS_UPDATED if muted and disallowUnmute changed', () => {
-        locusInfo.self = self;
+        locusInfo.webex.internal.device.url = self.deviceUrl;
+        locusInfo.updateSelf(self);
         const selfWithMutedByOthersAndDissalowUnmute = cloneDeep(self);
 
         // first simulate remote mute
         selfWithMutedByOthersAndDissalowUnmute.controls.audio.muted = true;
         selfWithMutedByOthersAndDissalowUnmute.controls.audio.disallowUnmute = true;
 
-        locusInfo.webex.internal.device.url = self.deviceUrl;
         locusInfo.emitScoped = sinon.stub();
-        locusInfo.updateSelf(selfWithMutedByOthersAndDissalowUnmute, []);
+        locusInfo.updateSelf(selfWithMutedByOthersAndDissalowUnmute);
 
         assert.calledWith(
           locusInfo.emitScoped,
@@ -1183,7 +1238,7 @@ describe('plugin-meetings', () => {
         selfWithMutedByOthers.controls.audio.muted = true;
         selfWithMutedByOthers.controls.audio.disallowUnmute = false;
 
-        locusInfo.updateSelf(selfWithMutedByOthers, []);
+        locusInfo.updateSelf(selfWithMutedByOthers);
 
         assert.calledWith(
           locusInfo.emitScoped,
@@ -1197,15 +1252,15 @@ describe('plugin-meetings', () => {
       });
 
       it('should trigger LOCAL_UNMUTE_REQUIRED on localAudioUnmuteRequired', () => {
-        locusInfo.self = self;
+        locusInfo.webex.internal.device.url = self.deviceUrl;
+        locusInfo.updateSelf(self);
         const selfWithLocalUnmuteRequired = cloneDeep(self);
 
         selfWithLocalUnmuteRequired.controls.audio.muted = false;
         selfWithLocalUnmuteRequired.controls.audio.localAudioUnmuteRequired = true;
 
-        locusInfo.webex.internal.device.url = self.deviceUrl;
         locusInfo.emitScoped = sinon.stub();
-        locusInfo.updateSelf(selfWithLocalUnmuteRequired, []);
+        locusInfo.updateSelf(selfWithLocalUnmuteRequired);
 
         assert.calledWith(
           locusInfo.emitScoped,
@@ -1222,16 +1277,16 @@ describe('plugin-meetings', () => {
       });
 
       it('should trigger LOCAL_UNMUTE_REQUESTED when receiving requestedToUnmute=true', () => {
-        locusInfo.self = self;
+        locusInfo.webex.internal.device.url = self.deviceUrl;
+        locusInfo.updateSelf(self);
         const selfWithRequestedToUnmute = cloneDeep(self);
 
         selfWithRequestedToUnmute.controls.audio.requestedToUnmute = true;
         selfWithRequestedToUnmute.controls.audio.lastModifiedRequestedToUnmute =
           '2023-06-16T19:25:04.369Z';
 
-        locusInfo.webex.internal.device.url = self.deviceUrl;
         locusInfo.emitScoped = sinon.stub();
-        locusInfo.updateSelf(selfWithRequestedToUnmute, []);
+        locusInfo.updateSelf(selfWithRequestedToUnmute);
 
         assert.calledWith(
           locusInfo.emitScoped,
@@ -1249,7 +1304,7 @@ describe('plugin-meetings', () => {
         selfWithoutRequestedToUnmute.controls.audio.requestedToUnmute = false;
 
         locusInfo.emitScoped.resetHistory();
-        locusInfo.updateSelf(selfWithoutRequestedToUnmute, []);
+        locusInfo.updateSelf(selfWithoutRequestedToUnmute);
 
         assert.neverCalledWith(
           locusInfo.emitScoped,
@@ -1263,15 +1318,14 @@ describe('plugin-meetings', () => {
       });
 
       it('should trigger SELF_OBSERVING when moving meeting to DX', () => {
-        locusInfo.self = self;
+        locusInfo.webex.internal.device.url = self.deviceUrl;
+        locusInfo.updateSelf(self);
         const selfInitiatedMove = cloneDeep(self);
 
         // Inital move meeting is iniated
         selfInitiatedMove.devices[0].intent.type = 'MOVE_MEDIA';
 
-        locusInfo.webex.internal.device.url = self.deviceUrl;
-
-        locusInfo.updateSelf(selfInitiatedMove, []);
+        locusInfo.updateSelf(selfInitiatedMove);
 
         locusInfo.emitScoped = sinon.stub();
         // When dx joined the meeting after move
@@ -1279,7 +1333,7 @@ describe('plugin-meetings', () => {
 
         selfAfterDxJoins.devices[0].intent.type = 'OBSERVE';
 
-        locusInfo.updateSelf(selfAfterDxJoins, []);
+        locusInfo.updateSelf(selfAfterDxJoins);
 
         assert.calledWith(
           locusInfo.emitScoped,
@@ -1297,11 +1351,11 @@ describe('plugin-meetings', () => {
         selfClone.canNotViewTheParticipantList = false; // same
 
         // Set the layout prior to stubbing to validate it does not change.
-        locusInfo.updateSelf(self, []);
+        locusInfo.updateSelf(self);
 
         locusInfo.emitScoped = sinon.stub();
 
-        locusInfo.updateSelf(selfClone, []);
+        locusInfo.updateSelf(selfClone);
 
         assert.neverCalledWith(
           locusInfo.emitScoped,
@@ -1320,11 +1374,11 @@ describe('plugin-meetings', () => {
         selfClone.canNotViewTheParticipantList = true; // different
 
         // Set the layout prior to stubbing to validate it does not change.
-        locusInfo.updateSelf(self, []);
+        locusInfo.updateSelf(self);
 
         locusInfo.emitScoped = sinon.stub();
 
-        locusInfo.updateSelf(selfClone, []);
+        locusInfo.updateSelf(selfClone);
 
         assert.calledWith(
           locusInfo.emitScoped,
@@ -1343,11 +1397,11 @@ describe('plugin-meetings', () => {
         selfClone.isSharingBlocked = false; // same
 
         // Set the layout prior to stubbing to validate it does not change.
-        locusInfo.updateSelf(self, []);
+        locusInfo.updateSelf(self);
 
         locusInfo.emitScoped = sinon.stub();
 
-        locusInfo.updateSelf(selfClone, []);
+        locusInfo.updateSelf(selfClone);
 
         assert.neverCalledWith(
           locusInfo.emitScoped,
@@ -1366,11 +1420,11 @@ describe('plugin-meetings', () => {
         selfClone.isSharingBlocked = true; // different
 
         // Set the layout prior to stubbing to validate it does not change.
-        locusInfo.updateSelf(self, []);
+        locusInfo.updateSelf(self);
 
         locusInfo.emitScoped = sinon.stub();
 
-        locusInfo.updateSelf(selfClone, []);
+        locusInfo.updateSelf(selfClone);
 
         assert.calledWith(
           locusInfo.emitScoped,
@@ -1384,12 +1438,12 @@ describe('plugin-meetings', () => {
       });
 
       it('should trigger SELF_ROLES_CHANGED if self roles changed', () => {
-        locusInfo.self = self;
+        locusInfo.updateSelf(self);
         locusInfo.emitScoped = sinon.stub();
         const sampleNewSelf = cloneDeep(self);
         sampleNewSelf.controls.role.roles = [{type: 'COHOST', hasRole: true}];
 
-        locusInfo.updateSelf(sampleNewSelf, []);
+        locusInfo.updateSelf(sampleNewSelf);
 
         assert.calledWith(
           locusInfo.emitScoped,
@@ -1403,12 +1457,12 @@ describe('plugin-meetings', () => {
       });
 
       it('should not trigger SELF_ROLES_CHANGED if self roles not changed', () => {
-        locusInfo.self = self;
+        locusInfo.updateSelf(self);
         locusInfo.emitScoped = sinon.stub();
         const sampleNewSelf = cloneDeep(self);
         sampleNewSelf.controls.role.roles = [{type: 'PRESENTER', hasRole: true}];
 
-        locusInfo.updateSelf(sampleNewSelf, []);
+        locusInfo.updateSelf(sampleNewSelf);
 
         assert.neverCalledWith(
           locusInfo.emitScoped,
@@ -1422,12 +1476,12 @@ describe('plugin-meetings', () => {
       });
 
       it('should trigger SELF_MEETING_INTERPRETATION_CHANGED if self interpretation info changed', () => {
-        locusInfo.self = self;
+        locusInfo.updateSelf(self);
         locusInfo.emitScoped = sinon.stub();
         const sampleNewSelf = cloneDeep(self);
         sampleNewSelf.controls.interpretation.targetLanguage = 'it';
 
-        locusInfo.updateSelf(sampleNewSelf, []);
+        locusInfo.updateSelf(sampleNewSelf);
 
         assert.calledWith(
           locusInfo.emitScoped,
@@ -1444,12 +1498,12 @@ describe('plugin-meetings', () => {
       });
 
       it('should not trigger SELF_MEETING_INTERPRETATION_CHANGED if self interpretation info not changed', () => {
-        locusInfo.self = self;
+        locusInfo.updateSelf(self);
         locusInfo.emitScoped = sinon.stub();
         const sampleNewSelf = cloneDeep(self);
         sampleNewSelf.controls.interpretation.targetLanguage = 'cn'; // same with previous one
 
-        locusInfo.updateSelf(sampleNewSelf, []);
+        locusInfo.updateSelf(sampleNewSelf);
 
         assert.neverCalledWith(
           locusInfo.emitScoped,
@@ -1466,12 +1520,12 @@ describe('plugin-meetings', () => {
       });
 
       it('should not trigger any events if controls is undefined', () => {
-        locusInfo.self = self;
+        locusInfo.updateSelf(self);
         locusInfo.emitScoped = sinon.stub();
         const newSelf = cloneDeep(self);
         newSelf.controls = undefined;
 
-        locusInfo.updateSelf(newSelf, []);
+        locusInfo.updateSelf(newSelf);
 
         const eventsSet = new Set([
           LOCUSINFO.EVENTS.CONTROLS_MEETING_LAYOUT_UPDATED,
@@ -1488,6 +1542,31 @@ describe('plugin-meetings', () => {
           assert.isFalse(eventsSet.has(eventName));
         });
       });
+
+      it('calls getSelves with correct parameters', () => {
+        const getSelvesStub = sinon.stub(SelfUtils, 'getSelves').returns({
+          current: {},
+          previous: {},
+          updates: {},
+        });
+
+        locusInfo.webex.internal.device.url = self.deviceUrl;
+        locusInfo.participants = [{id: '1'}, {id: '2'}];
+        locusInfo.parsedLocus.self = {id: 'fake parsed locus self id'};
+
+        const parsedLocusSelf = locusInfo.parsedLocus.self; // need to store it before it's updated in updateSelf
+        locusInfo.updateSelf(self);
+
+        assert.calledWith(
+          getSelvesStub,
+          parsedLocusSelf,
+          self,
+          locusInfo.webex.internal.device.url,
+          locusInfo.participants
+        );
+
+        getSelvesStub.restore();
+      });
     });
 
     describe('#updateMeetingInfo', () => {
@@ -1754,11 +1833,11 @@ describe('plugin-meetings', () => {
       });
 
       it('should update media shares and emit LOCUS_INFO_UPDATE_MEDIA_SHARES when mediaShares change', () => {
-        const initialMediaShares = { audio: true, video: false };
-        const newMediaShares = { audio: false, video: true };
+        const initialMediaShares = {audio: true, video: false};
+        const newMediaShares = {audio: false, video: true};
 
         locusInfo.mediaShares = initialMediaShares;
-        locusInfo.parsedLocus = { mediaShares: null };
+        locusInfo.parsedLocus = {mediaShares: null};
 
         const parsedMediaShares = {
           current: newMediaShares,
@@ -1795,9 +1874,9 @@ describe('plugin-meetings', () => {
       });
 
       it('should force update media shares and emit LOCUS_INFO_UPDATE_MEDIA_SHARES even if shares are the same', () => {
-        const initialMediaShares = { audio: true, video: false };
+        const initialMediaShares = {audio: true, video: false};
         locusInfo.mediaShares = initialMediaShares;
-        locusInfo.parsedLocus = { mediaShares: null };
+        locusInfo.parsedLocus = {mediaShares: null};
 
         const parsedMediaShares = {
           current: initialMediaShares,
@@ -1829,7 +1908,7 @@ describe('plugin-meetings', () => {
       });
 
       it('should not emit LOCUS_INFO_UPDATE_MEDIA_SHARES if mediaShares do not change and forceUpdate is false', () => {
-        const initialMediaShares = { audio: true, video: false };
+        const initialMediaShares = {audio: true, video: false};
         locusInfo.mediaShares = initialMediaShares;
 
         // Call the function with the same mediaShares and forceUpdate = false
@@ -1843,11 +1922,11 @@ describe('plugin-meetings', () => {
       });
 
       it('should update internal state correctly when mediaShares are updated', () => {
-        const initialMediaShares = { audio: true, video: false };
-        const newMediaShares = { audio: false, video: true };
+        const initialMediaShares = {audio: true, video: false};
+        const newMediaShares = {audio: false, video: true};
 
         locusInfo.mediaShares = initialMediaShares;
-        locusInfo.parsedLocus = { mediaShares: null };
+        locusInfo.parsedLocus = {mediaShares: null};
 
         const parsedMediaShares = {
           current: newMediaShares,
@@ -2386,6 +2465,21 @@ describe('plugin-meetings', () => {
         locusInfo.onDeltaLocus(fakeLocus);
         assert.calledWith(locusInfo.updateParticipants, {}, true);
       });
+
+      it('onDeltaLocus merges delta participants with existing participants', () => {
+        const FAKE_DELTA_PARTICIPANTS = [
+          {id: '1111'}, {id: '2222'}
+        ]
+        fakeLocus.participants = FAKE_DELTA_PARTICIPANTS;
+
+        sinon.spy(locusInfo, 'mergeParticipants');
+        locusInfo.updateParticipants = sinon.stub();
+        const existingParticipants = locusInfo.participants;
+
+        locusInfo.onDeltaLocus(fakeLocus);
+        assert.calledOnceWithExactly(locusInfo.mergeParticipants, existingParticipants, FAKE_DELTA_PARTICIPANTS);
+        assert.calledWith(locusInfo.updateParticipants, FAKE_DELTA_PARTICIPANTS, false);
+      });
     });
 
     describe('#updateLocusCache', () => {