@webex/plugin-meetings 3.9.0-next.15 → 3.9.0-next.16

package/dist/types/meeting/util.d.ts

@@ -101,14 +101,14 @@ declare const MeetingUtil: {
      */
     addSequence: (meeting: any, requestBody: any) => void;
     /**
-     * Updates the locus info for the meeting with the delta locus
-     * returned from requests that include the sequence information
+     * Updates the locus info for the meeting with the locus
+     * information returned from API requests made to Locus
      * Returns the original response object
      * @param {Object} meeting The meeting object
      * @param {Object} response The response of the http request
      * @returns {Object}
      */
-    updateLocusWithDelta: (meeting: any, response: any) => any;
+    updateLocusFromApiResponse: (meeting: any, response: any) => any;
     generateBuildLocusDeltaRequestOptions: (originalMeeting: any) => (originalOptions: any) => any;
     generateLocusDeltaRequest: (originalMeeting: any) => (originalOptions: any) => any;
     selfSupportsFeature: (feature: SELF_POLICY, userPolicies: Record<SELF_POLICY, boolean>) => boolean;

package/dist/webinar/index.js

@@ -458,7 +458,7 @@ var Webinar = _webexCore.WebexPlugin.extend({
       }, _callee7);
     }))();
   },
-  version: "3.9.0-next.15"
+  version: "3.9.0-next.16"
 });
 var _default = exports.default = Webinar;
 //# sourceMappingURL=index.js.map

package/package.json

@@ -92,5 +92,5 @@
   "//": [
     "TODO: upgrade jwt-decode when moving to node 18"
   ],
-  "version": "3.9.0-next.15"
+  "version": "3.9.0-next.16"
 }

package/src/locus-info/index.ts

@@ -31,6 +31,51 @@ import LocusDeltaParser from './parser';
 import Metrics from '../metrics';
 import BEHAVIORAL_METRICS from '../metrics/constants';
 
+export type LocusDTO = {
+  controls?: any;
+  fullState?: {
+    active: boolean;
+    count: number;
+    lastActive: string;
+    locked: boolean;
+    sessionId: string;
+    seessionIds: string[];
+    startTime: number;
+    state: string;
+    type: string;
+  };
+  host?: {
+    id: string;
+    incomingCallProtocols: any[];
+    isExternal: boolean;
+    name: string;
+    orgId: string;
+  };
+  info?: any;
+  links?: any;
+  mediaShares?: any[];
+  meetings?: any[];
+  participants: any[];
+  replaces?: any[];
+  self?: any;
+  sequence?: {
+    dirtyParticipants: number;
+    entries: number[];
+    rangeEnd: number;
+    rangeStart: number;
+    sequenceHash: number;
+    sessionToken: string;
+    since: string;
+    totalParticipants: number;
+  };
+  syncUrl?: string;
+  url?: string;
+};
+
+export type LocusApiResponseBody = {
+  locus: LocusDTO; // this LocusDTO here might not be the full one (for example it won't have all the participants, but it should have self)
+};
+
 /**
  * @description LocusInfo extends ChildEmitter to convert locusInfo info a private emitter to parent object
  * @export
@@ -323,6 +368,16 @@ export default class LocusInfo extends EventsScope {
     this.emitChange = true;
   }
 
+  /**
+   * Handles HTTP response from Locus API call.
+   * @param {Meeting} meeting meeting object
+   * @param {LocusApiResponseBody} responseBody body of the http response from Locus API call
+   * @returns {void}
+   */
+  handleLocusAPIResponse(meeting, responseBody: LocusApiResponseBody): void {
+    this.handleLocusDelta(responseBody.locus, meeting);
+  }
+
   /**
    * @param {Meeting} meeting
    * @param {Object} data

package/src/meeting/muteState.ts

@@ -291,18 +291,14 @@ export class MuteState {
     );
 
     return MeetingUtil.remoteUpdateAudioVideo(meeting, audioMuted, videoMuted)
-      .then((locus) => {
+      .then((response) => {
         LoggerProxy.logger.info(
           `Meeting:muteState#sendLocalMuteRequestToServer --> ${this.type}: local mute (audio=${audioMuted}, video=${videoMuted}) applied to server`
         );
 
         this.state.server.localMute = this.type === AUDIO ? audioMuted : videoMuted;
 
-        if (locus) {
-          meeting.locusInfo.handleLocusDelta(locus, meeting);
-        }
-
-        return locus;
+        return MeetingUtil.updateLocusFromApiResponse(meeting, response);
       })
       .catch((remoteUpdateError) => {
         LoggerProxy.logger.warn(

package/src/meeting/util.ts

@@ -59,18 +59,16 @@ const MeetingUtil = {
       );
     }
 
-    return meeting.locusMediaRequest
-      .send({
-        type: 'LocalMute',
-        selfUrl: meeting.selfUrl,
-        mediaId: meeting.mediaId,
-        sequence: meeting.locusInfo.sequence,
-        muteOptions: {
-          audioMuted,
-          videoMuted,
-        },
-      })
-      .then((response) => response?.body?.locus);
+    return meeting.locusMediaRequest.send({
+      type: 'LocalMute',
+      selfUrl: meeting.selfUrl,
+      mediaId: meeting.mediaId,
+      sequence: meeting.locusInfo.sequence,
+      muteOptions: {
+        audioMuted,
+        videoMuted,
+      },
+    });
   },
 
   hasOwner: (info) => info && info.owner,
@@ -695,22 +693,20 @@ const MeetingUtil = {
   },
 
   /**
-   * Updates the locus info for the meeting with the delta locus
-   * returned from requests that include the sequence information
+   * Updates the locus info for the meeting with the locus
+   * information returned from API requests made to Locus
    * Returns the original response object
    * @param {Object} meeting The meeting object
    * @param {Object} response The response of the http request
    * @returns {Object}
    */
-  updateLocusWithDelta: (meeting, response) => {
+  updateLocusFromApiResponse: (meeting, response) => {
     if (!meeting) {
       return response;
     }
 
-    const locus = response?.body?.locus;
-
-    if (locus) {
-      meeting.locusInfo.handleLocusDelta(locus, meeting);
+    if (response?.body?.locus) {
+      meeting.locusInfo.handleLocusAPIResponse(meeting, response.body);
     }
 
     return response;
@@ -757,7 +753,7 @@ const MeetingUtil = {
 
       return meeting
         .request(options)
-        .then((response) => MeetingUtil.updateLocusWithDelta(meeting, response));
+        .then((response) => MeetingUtil.updateLocusFromApiResponse(meeting, response));
     };
 
     return locusDeltaRequest;

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

@@ -2040,6 +2040,18 @@ describe('plugin-meetings', () => {
       });
     });
 
+    describe('#handleLocusAPIResponse', () => {
+      it('calls handleLocusDelta', () => {
+        const fakeLocus = {eventType: LOCUSEVENT.DIFFERENCE};
+
+        sinon.stub(locusInfo, 'handleLocusDelta');
+
+        locusInfo.handleLocusAPIResponse(mockMeeting, {locus: fakeLocus});
+
+        assert.calledWith(locusInfo.handleLocusDelta, fakeLocus, mockMeeting);
+      });
+    });
+
     describe('#LocusDeltaEvents', () => {
       const fakeMeeting = 'fakeMeeting';
       let sandbox = null;

package/test/unit/spec/meeting/muteState.js

@@ -12,7 +12,7 @@ describe('plugin-meetings', () => {
   let video;
   let originalRemoteUpdateAudioVideo;
 
-  const fakeLocus = {info: 'this is a fake locus'};
+  const fakeLocusResponse = {body: {locus: {info: 'this is a fake locus'}}};
 
   const createFakeLocalStream = (id, userMuted, systemMuted) => {
     return {
@@ -38,9 +38,6 @@ describe('plugin-meetings', () => {
       unmuteAllowed: true,
       remoteVideoMuted: false,
       unmuteVideoAllowed: true,
-      locusInfo: {
-        handleLocusDelta: sinon.stub(),
-      },
       members: {
         selfId: 'fake self id',
         muteMember: sinon.stub().resolves(),
@@ -49,7 +46,8 @@ describe('plugin-meetings', () => {
 
     originalRemoteUpdateAudioVideo = MeetingUtil.remoteUpdateAudioVideo;
 
-    MeetingUtil.remoteUpdateAudioVideo = sinon.stub().resolves(fakeLocus);
+    MeetingUtil.remoteUpdateAudioVideo = sinon.stub().resolves(fakeLocusResponse);
+    MeetingUtil.updateLocusFromApiResponse = sinon.stub();
 
     audio = createMuteState(AUDIO, meeting, true);
     video = createMuteState(VIDEO, meeting, true);
@@ -141,6 +139,7 @@ describe('plugin-meetings', () => {
       // and local unmute was sent to server
       assert.calledOnce(MeetingUtil.remoteUpdateAudioVideo);
       assert.calledWith(MeetingUtil.remoteUpdateAudioVideo, meeting, false, undefined);
+      assert.calledWith(MeetingUtil.updateLocusFromApiResponse, meeting, fakeLocusResponse);
 
       assert.isFalse(audio.isMuted());
     });
@@ -173,6 +172,7 @@ describe('plugin-meetings', () => {
 
       // system was muted so local unmute was not sent to server
       assert.notCalled(MeetingUtil.remoteUpdateAudioVideo);
+      assert.notCalled(MeetingUtil.updateLocusFromApiResponse);
 
       assert.isTrue(audio.isMuted());
     });
@@ -207,6 +207,7 @@ describe('plugin-meetings', () => {
       // and local unmute was sent to server
       assert.calledOnce(MeetingUtil.remoteUpdateAudioVideo);
       assert.calledWith(MeetingUtil.remoteUpdateAudioVideo, meeting, undefined, false);
+      assert.calledWith(MeetingUtil.updateLocusFromApiResponse, meeting, fakeLocusResponse);
 
       assert.isFalse(video.isMuted());
     });
@@ -219,7 +220,9 @@ describe('plugin-meetings', () => {
 
       assert.isTrue(video.isMuted());
 
+      await testUtils.flushPromises();
       MeetingUtil.remoteUpdateAudioVideo.resetHistory();
+      MeetingUtil.updateLocusFromApiResponse.resetHistory();
 
       // now simulate server requiring us to locally unmute
       // assuming setServerMuted succeeds at updating userMuted
@@ -239,6 +242,7 @@ describe('plugin-meetings', () => {
 
       // system was muted so local unmute was not sent to server
       assert.notCalled(MeetingUtil.remoteUpdateAudioVideo);
+      assert.notCalled(MeetingUtil.updateLocusFromApiResponse);
 
       assert.isTrue(video.isMuted());
     });
@@ -443,6 +447,7 @@ describe('plugin-meetings', () => {
 
         assert.calledOnce(MeetingUtil.remoteUpdateAudioVideo);
         assert.calledWith(MeetingUtil.remoteUpdateAudioVideo, meeting, true, undefined);
+        assert.calledWith(MeetingUtil.updateLocusFromApiResponse, meeting, fakeLocusResponse);
 
         // now allow the first request to complete
         serverResponseResolve();
@@ -559,6 +564,7 @@ describe('plugin-meetings', () => {
         await testUtils.flushPromises();
 
         MeetingUtil.remoteUpdateAudioVideo.resetHistory();
+        MeetingUtil.updateLocusFromApiResponse.resetHistory();
       };
 
       const setupSpies = (mediaType) => {
@@ -605,13 +611,15 @@ describe('plugin-meetings', () => {
         {mediaType: VIDEO, title: 'video'},
       ];
 
+      const fakeLocusResponse = {body: {locus: {info: 'fake locus'}}};
+
       tests.forEach(({mediaType, title}) =>
         describe(title, () => {
           let originalRemoteUpdateAudioVideo;
 
           beforeEach(() => {
             originalRemoteUpdateAudioVideo = MeetingUtil.remoteUpdateAudioVideo;
-            MeetingUtil.remoteUpdateAudioVideo = sinon.stub().resolves({info: 'fake locus'});
+            MeetingUtil.remoteUpdateAudioVideo = sinon.stub().resolves(fakeLocusResponse);
           });
 
           afterEach(() => {
@@ -660,6 +668,7 @@ describe('plugin-meetings', () => {
               assert.calledWith(setUnmuteAllowedSpy, muteState.state.server.unmuteAllowed);
               assert.notCalled(setServerMutedSpy);
               assert.notCalled(MeetingUtil.remoteUpdateAudioVideo);
+              assert.notCalled(MeetingUtil.updateLocusFromApiResponse);
               assert.isTrue(muteState.state.client.localMute);
             });
 
@@ -672,6 +681,7 @@ describe('plugin-meetings', () => {
               assert.calledWith(setUnmuteAllowedSpy, muteState.state.server.unmuteAllowed);
               assert.notCalled(setServerMutedSpy);
               assert.notCalled(MeetingUtil.remoteUpdateAudioVideo);
+              assert.notCalled(MeetingUtil.updateLocusFromApiResponse);
               assert.isTrue(muteState.state.client.localMute);
             });
 
@@ -681,9 +691,12 @@ describe('plugin-meetings', () => {
 
               muteState.init(meeting);
 
+              await testUtils.flushPromises();
+
               assert.calledWith(setUnmuteAllowedSpy, muteState.state.server.unmuteAllowed);
               assert.notCalled(setServerMutedSpy);
               assert.calledOnce(MeetingUtil.remoteUpdateAudioVideo);
+              assert.calledOnceWithExactly(MeetingUtil.updateLocusFromApiResponse, meeting, fakeLocusResponse);
               assert.isFalse(muteState.state.client.localMute);
             });
 
@@ -707,6 +720,7 @@ describe('plugin-meetings', () => {
               simulateUserMute(mediaType, true);
               muteState.handleLocalStreamMuteStateChange(meeting);
               assert.notCalled(MeetingUtil.remoteUpdateAudioVideo);
+              assert.notCalled(MeetingUtil.updateLocusFromApiResponse);
 
               assert.isFalse(muteState.state.client.localMute);
             });
@@ -716,8 +730,11 @@ describe('plugin-meetings', () => {
 
               simulateUserMute(mediaType, false);
               muteState.handleLocalStreamMuteStateChange(meeting);
+              await testUtils.flushPromises();
+
               assert.equal(muteState.state.client.localMute, false);
               assert.called(MeetingUtil.remoteUpdateAudioVideo);
+              assert.calledOnceWithExactly(MeetingUtil.updateLocusFromApiResponse, meeting, fakeLocusResponse);
             });
 
             it('tests localMute - user mute from false to true', async () => {
@@ -725,8 +742,11 @@ describe('plugin-meetings', () => {
 
               simulateUserMute(mediaType, true);
               muteState.handleLocalStreamMuteStateChange(meeting);
+              await testUtils.flushPromises();
+
               assert.equal(muteState.state.client.localMute, true);
               assert.called(MeetingUtil.remoteUpdateAudioVideo);
+              assert.calledOnceWithExactly(MeetingUtil.updateLocusFromApiResponse, meeting, fakeLocusResponse);
             });
 
             it('tests localMute - system mute from true to false', async () => {
@@ -734,8 +754,11 @@ describe('plugin-meetings', () => {
 
               simulateSystemMute(mediaType, false);
               muteState.handleLocalStreamMuteStateChange(meeting);
+              await testUtils.flushPromises();
+
               assert.equal(muteState.state.client.localMute, false);
               assert.called(MeetingUtil.remoteUpdateAudioVideo);
+              assert.calledOnceWithExactly(MeetingUtil.updateLocusFromApiResponse, meeting, fakeLocusResponse);
             });
 
             it('tests localMute - system mute from false to true', async () => {
@@ -743,8 +766,11 @@ describe('plugin-meetings', () => {
 
               simulateSystemMute(mediaType, true);
               muteState.handleLocalStreamMuteStateChange(meeting);
+              await testUtils.flushPromises();
+
               assert.equal(muteState.state.client.localMute, true);
               assert.called(MeetingUtil.remoteUpdateAudioVideo);
+              assert.calledOnceWithExactly(MeetingUtil.updateLocusFromApiResponse, meeting, fakeLocusResponse);
             });
           });
 

package/test/unit/spec/meeting/utils.js

@@ -228,11 +228,11 @@ describe('plugin-meetings', () => {
       });
     });
 
-    describe('updateLocusWithDelta', () => {
-      it('should call handleLocusDelta with the new delta locus', () => {
+    describe('updateLocusFromApiResponse', () => {
+      it('should call handleLocusAPIResponse with the response body', () => {
         const meeting = {
           locusInfo: {
-            handleLocusDelta: sinon.stub(),
+            handleLocusAPIResponse: sinon.stub(),
           },
         };
 
@@ -242,16 +242,16 @@ describe('plugin-meetings', () => {
           },
         };
 
-        const response = MeetingUtil.updateLocusWithDelta(meeting, originalResponse);
+        const response = MeetingUtil.updateLocusFromApiResponse(meeting, originalResponse);
 
         assert.deepEqual(response, originalResponse);
-        assert.calledOnceWithExactly(meeting.locusInfo.handleLocusDelta, 'locus', meeting);
+        assert.calledOnceWithExactly(meeting.locusInfo.handleLocusAPIResponse, meeting, originalResponse.body);
       });
 
       it('should handle locus being missing from the response', () => {
         const meeting = {
           locusInfo: {
-            handleLocusDelta: sinon.stub(),
+            handleLocusAPIResponse: sinon.stub(),
           },
         };
 
@@ -259,10 +259,10 @@ describe('plugin-meetings', () => {
           body: {},
         };
 
-        const response = MeetingUtil.updateLocusWithDelta(meeting, originalResponse);
+        const response = MeetingUtil.updateLocusFromApiResponse(meeting, originalResponse);
 
         assert.deepEqual(response, originalResponse);
-        assert.notCalled(meeting.locusInfo.handleLocusDelta);
+        assert.notCalled(meeting.locusInfo.handleLocusAPIResponse);
       });
 
       it('should work with an undefined meeting', () => {
@@ -272,14 +272,14 @@ describe('plugin-meetings', () => {
           },
         };
 
-        const response = MeetingUtil.updateLocusWithDelta(undefined, originalResponse);
+        const response = MeetingUtil.updateLocusFromApiResponse(undefined, originalResponse);
         assert.deepEqual(response, originalResponse);
       });
     });
 
     describe('generateLocusDeltaRequest', () => {
       it('generates the correct wrapper function', async () => {
-        const updateLocusWithDeltaSpy = sinon.spy(MeetingUtil, 'updateLocusWithDelta');
+        const updateLocusFromApiResponseSpy = sinon.spy(MeetingUtil, 'updateLocusFromApiResponse');
         const addSequenceSpy = sinon.spy(MeetingUtil, 'addSequence');
 
         const meeting = {
@@ -296,16 +296,16 @@ describe('plugin-meetings', () => {
         let result = await locusDeltaRequest(options);
 
         assert.equal(result, 'result');
-        assert.calledOnceWithExactly(updateLocusWithDeltaSpy, meeting, 'result');
+        assert.calledOnceWithExactly(updateLocusFromApiResponseSpy, meeting, 'result');
         assert.calledOnceWithExactly(addSequenceSpy, meeting, options.body);
 
-        updateLocusWithDeltaSpy.resetHistory();
+        updateLocusFromApiResponseSpy.resetHistory();
         addSequenceSpy.resetHistory();
 
         // body missing from options
         result = await locusDeltaRequest({});
         assert.equal(result, 'result');
-        assert.calledOnceWithExactly(updateLocusWithDeltaSpy, meeting, 'result');
+        assert.calledOnceWithExactly(updateLocusFromApiResponseSpy, meeting, 'result');
         assert.calledOnceWithExactly(addSequenceSpy, meeting, options.body);
 
         // meeting disappears so the WeakRef returns undefined
@@ -359,7 +359,11 @@ describe('plugin-meetings', () => {
     });
 
     describe('remoteUpdateAudioVideo', () => {
-      it('#Should call meetingRequest.locusMediaRequest with correct parameters', async () => {
+      it('#Should call meetingRequest.locusMediaRequest with correct parameters and return the full response', async () => {
+        const fakeResponse = {
+          body: { locus: { url: 'locusUrl'}},
+          headers: { },
+        };
         const meeting = {
           id: 'meeting-id',
           mediaId: '12345',
@@ -368,13 +372,14 @@ describe('plugin-meetings', () => {
             sequence: {},
           },
           locusMediaRequest: {
-            send: sinon.stub().resolves({body: {}, headers: {}}),
+            send: sinon.stub().resolves(fakeResponse),
           },
           getWebexObject: sinon.stub().returns(webex),
         };
 
-        await MeetingUtil.remoteUpdateAudioVideo(meeting, true, false);
+        const result = await MeetingUtil.remoteUpdateAudioVideo(meeting, true, false);
 
+        assert.deepEqual(result, fakeResponse);
         assert.calledOnceWithExactly(meeting.locusMediaRequest.send, {
           mediaId: '12345',
           muteOptions: {