@webex/plugin-meetings 3.6.0 → 3.7.0-next.2

package/src/roap/index.ts

@@ -107,7 +107,7 @@ export default class Roap extends StatelessWebexPlugin {
           roapMessage,
           locusSelfUrl: meeting.selfUrl,
           mediaId: options.mediaId,
-          meetingId: meeting.id,
+          isMultistream: meeting.isMultistream,
           locusMediaRequest: meeting.locusMediaRequest,
         })
         .then(() => {
@@ -141,7 +141,7 @@ export default class Roap extends StatelessWebexPlugin {
       roapMessage,
       locusSelfUrl: meeting.selfUrl,
       mediaId: options.mediaId,
-      meetingId: meeting.id,
+      isMultistream: meeting.isMultistream,
       locusMediaRequest: meeting.locusMediaRequest,
     });
   }
@@ -170,7 +170,7 @@ export default class Roap extends StatelessWebexPlugin {
         roapMessage,
         locusSelfUrl: meeting.selfUrl,
         mediaId: options.mediaId,
-        meetingId: meeting.id,
+        isMultistream: meeting.isMultistream,
         locusMediaRequest: meeting.locusMediaRequest,
       })
       .then(() => {
@@ -207,10 +207,9 @@ export default class Roap extends StatelessWebexPlugin {
         roapMessage,
         locusSelfUrl: meeting.selfUrl,
         mediaId: sendEmptyMediaId ? '' : meeting.mediaId,
-        meetingId: meeting.id,
+        isMultistream: meeting.isMultistream,
         preferTranscoding: !meeting.isMultistream,
         locusMediaRequest: meeting.locusMediaRequest,
-        ipVersion: MeetingUtil.getIpVersion(meeting.webex),
       })
       .then(({locus, mediaConnections}) => {
         if (mediaConnections) {

package/src/roap/request.ts

@@ -4,44 +4,13 @@ import {StatelessWebexPlugin} from '@webex/webex-core';
 import LoggerProxy from '../common/logs/logger-proxy';
 import {IP_VERSION, REACHABILITY} from '../constants';
 import {LocusMediaRequest} from '../meeting/locusMediaRequest';
+import MeetingUtil from '../meeting/util';
+import {ClientMediaPreferences} from '../reachability/reachability.types';
 
 /**
  * @class RoapRequest
  */
 export default class RoapRequest extends StatelessWebexPlugin {
-  /**
-   * Returns reachability data.
-   * @param {Object} localSdp
-   * @returns {Object}
-   */
-  async attachReachabilityData(localSdp) {
-    let joinCookie;
-
-    // @ts-ignore
-    const reachabilityResult = await this.webex.meetings.reachability.getReachabilityResults();
-
-    if (reachabilityResult && Object.keys(reachabilityResult).length) {
-      localSdp.reachability = reachabilityResult;
-    }
-
-    // @ts-ignore
-    const joinCookieRaw = await this.webex.boundedStorage
-      .get(REACHABILITY.namespace, REACHABILITY.localStorageJoinCookie)
-      .catch(() => {});
-
-    if (joinCookieRaw) {
-      try {
-        joinCookie = JSON.parse(joinCookieRaw);
-      } catch (e) {
-        LoggerProxy.logger.error(
-          `MeetingRequest#constructor --> Error in parsing join cookie data: ${e}`
-        );
-      }
-    }
-
-    return {localSdp, joinCookie};
-  }
-
   /**
    * Sends a ROAP message
    * @param {Object} options
@@ -50,18 +19,16 @@ export default class RoapRequest extends StatelessWebexPlugin {
    * @param {String} options.mediaId
    * @param {String} options.correlationId
    * @param {String} options.meetingId
-   * @param {IP_VERSION} options.ipVersion only required for offers
    * @returns {Promise} returns the response/failure of the request
    */
   async sendRoap(options: {
     roapMessage: any;
     locusSelfUrl: string;
     mediaId: string;
-    meetingId: string;
-    ipVersion?: IP_VERSION;
+    isMultistream: boolean;
     locusMediaRequest?: LocusMediaRequest;
   }) {
-    const {roapMessage, locusSelfUrl, mediaId, locusMediaRequest, ipVersion} = options;
+    const {roapMessage, locusSelfUrl, isMultistream, mediaId, locusMediaRequest} = options;
 
     if (!mediaId) {
       LoggerProxy.logger.info('Roap:request#sendRoap --> sending empty mediaID');
@@ -74,23 +41,44 @@ export default class RoapRequest extends StatelessWebexPlugin {
 
       return Promise.reject(new Error('sendRoap called when locusMediaRequest is undefined'));
     }
-    const {localSdp: localSdpWithReachabilityData, joinCookie} = await this.attachReachabilityData({
-      roapMessage,
-    });
+
+    let reachability;
+    let clientMediaPreferences: ClientMediaPreferences = {
+      // bare minimum fallback value that should allow us to join;
+      joinCookie: undefined,
+      ipver: IP_VERSION.unknown,
+      preferTranscoding: !isMultistream,
+    };
+
+    try {
+      clientMediaPreferences =
+        // @ts-ignore
+        await this.webex.meetings.reachability.getClientMediaPreferences(
+          isMultistream,
+          // @ts-ignore
+          MeetingUtil.getIpVersion(this.webex)
+        );
+      reachability =
+        // @ts-ignore
+        await this.webex.meetings.reachability.getReachabilityReportToAttachToRoap();
+    } catch (error) {
+      LoggerProxy.logger.error('Roap:request#sendRoap --> reachability error:', error);
+    }
 
     LoggerProxy.logger.info(
-      `Roap:request#sendRoap --> ${locusSelfUrl} \n ${roapMessage.messageType} \n seq:${roapMessage.seq}`
+      `Roap:request#sendRoap --> ${roapMessage.messageType} seq:${roapMessage.seq} ${
+        clientMediaPreferences?.ipver ? `ipver=${clientMediaPreferences?.ipver} ` : ''
+      } ${locusSelfUrl}`
     );
 
     return locusMediaRequest
       .send({
         type: 'RoapMessage',
         selfUrl: locusSelfUrl,
-        joinCookie,
         mediaId,
         roapMessage,
-        reachability: localSdpWithReachabilityData.reachability,
-        ipVersion,
+        reachability,
+        clientMediaPreferences,
       })
       .then((res) => {
         // always it will be the first mediaConnection Object

package/src/roap/turnDiscovery.ts

@@ -408,10 +408,8 @@ export default class TurnDiscovery {
         locusSelfUrl: meeting.selfUrl,
         // @ts-ignore - Fix missing type
         mediaId: isReconnecting ? '' : meeting.mediaId,
-        meetingId: meeting.id,
+        isMultistream: meeting.isMultistream,
         locusMediaRequest: meeting.locusMediaRequest,
-        // @ts-ignore - because of meeting.webex
-        ipVersion: MeetingUtil.getIpVersion(meeting.webex),
       })
       .then(async (response) => {
         const {mediaConnections} = response;
@@ -451,7 +449,7 @@ export default class TurnDiscovery {
       locusSelfUrl: meeting.selfUrl,
       // @ts-ignore - fix type
       mediaId: meeting.mediaId,
-      meetingId: meeting.id,
+      isMultistream: meeting.isMultistream,
       locusMediaRequest: meeting.locusMediaRequest,
     });
   }

package/src/webinar/index.ts

@@ -2,7 +2,8 @@
  * Copyright (c) 2015-2023 Cisco Systems, Inc. See LICENSE file.
  */
 import {WebexPlugin} from '@webex/webex-core';
-import {MEETINGS} from '../constants';
+import {get} from 'lodash';
+import {MEETINGS, SELF_ROLES} from '../constants';
 
 import WebinarCollection from './collection';
 
@@ -17,14 +18,15 @@ const Webinar = WebexPlugin.extend({
 
   props: {
     locusUrl: 'string', // appears current webinar's locus url
-    webcastUrl: 'string', // current webinar's webcast url
-    webinarAttendeesSearchingUrl: 'string', // current webinarAttendeesSearching url
+    webcastInstanceUrl: 'string', // current webinar's webcast instance url
     canManageWebcast: 'boolean', // appears the ability to manage webcast
+    selfIsPanelist: 'boolean', // self is panelist
+    selfIsAttendee: 'boolean', // self is attendee
   },
 
   /**
    * Update the current locus url of the webinar
-   * @param {string} locusUrl // locus url
+   * @param {string} locusUrl
    * @returns {void}
    */
   locusUrlUpdate(locusUrl) {
@@ -32,21 +34,12 @@ const Webinar = WebexPlugin.extend({
   },
 
   /**
-   * Update the current webcast url of the meeting
-   * @param {string} webcastUrl // webcast url
+   * Update the current webcast instance url of the meeting
+   * @param {object} payload
    * @returns {void}
    */
-  webcastUrlUpdate(webcastUrl) {
-    this.set('webcastUrl', webcastUrl);
-  },
-
-  /**
-   * Update the current webinarAttendeesSearching url of the meeting
-   * @param {string} webinarAttendeesSearchingUrl // webinarAttendeesSearching url
-   * @returns {void}
-   */
-  webinarAttendeesSearchingUrlUpdate(webinarAttendeesSearchingUrl) {
-    this.set('webinarAttendeesSearchingUrl', webinarAttendeesSearchingUrl);
+  updateWebcastUrl(payload) {
+    this.set('webcastInstanceUrl', get(payload, 'resources.webcastInstance.url'));
   },
 
   /**
@@ -57,6 +50,27 @@ const Webinar = WebexPlugin.extend({
   updateCanManageWebcast(canManageWebcast) {
     this.set('canManageWebcast', canManageWebcast);
   },
+
+  /**
+   * Updates user roles and manages associated state transitions
+   * @param {object} payload
+   * @param {string[]} payload.oldRoles - Previous roles of the user
+   * @param {string[]} payload.newRoles - New roles of the user
+   * @returns {{isPromoted: boolean, isDemoted: boolean}} Role transition states
+   */
+  updateRoleChanged(payload) {
+    const isPromoted =
+      get(payload, 'oldRoles', []).includes(SELF_ROLES.ATTENDEE) &&
+      get(payload, 'newRoles', []).includes(SELF_ROLES.PANELIST);
+    const isDemoted =
+      get(payload, 'oldRoles', []).includes(SELF_ROLES.PANELIST) &&
+      get(payload, 'newRoles', []).includes(SELF_ROLES.ATTENDEE);
+    this.set('selfIsPanelist', get(payload, 'newRoles', []).includes(SELF_ROLES.PANELIST));
+    this.set('selfIsAttendee', get(payload, 'newRoles', []).includes(SELF_ROLES.ATTENDEE));
+    this.updateCanManageWebcast(get(payload, 'newRoles', []).includes(SELF_ROLES.MODERATOR));
+
+    return {isPromoted, isDemoted};
+  },
 });
 
 export default Webinar;

package/test/unit/spec/controls-options-manager/index.js

@@ -22,7 +22,7 @@ describe('plugin-meetings', () => {
 
             describe('Mute On Entry', () => {
                 let manager;
-            
+
                 beforeEach(() => {
                   request = {
                     request: sinon.stub().returns(Promise.resolve()),
@@ -37,85 +37,85 @@ describe('plugin-meetings', () => {
                 });
 
                 describe('setMuteOnEntry', () => {
-                    it('rejects when correct display hint is not present enabled=false', () => {  
+                    it('rejects when correct display hint is not present enabled=false', () => {
                       const result = manager.setMuteOnEntry(false);
-              
+
                       assert.notCalled(request.request);
-              
+
                       assert.isRejected(result);
                     });
 
-                    it('rejects when correct display hint is not present enabled=true', () => {  
+                    it('rejects when correct display hint is not present enabled=true', () => {
                       const result = manager.setMuteOnEntry(true);
-              
+
                       assert.notCalled(request.request);
-              
+
                       assert.isRejected(result);
                     });
-      
+
                     it('can set mute on entry when the display hint is available enabled=true', () => {
                       manager.setDisplayHints(['ENABLE_MUTE_ON_ENTRY']);
-              
+
                       const result = manager.setMuteOnEntry(true);
-              
+
                       assert.calledWith(request.request, {  uri: 'test/id/controls',
                       body: { muteOnEntry: { enabled: true } },
                       method: HTTP_VERBS.PATCH});
-              
+
                       assert.deepEqual(result, request.request.firstCall.returnValue);
                     });
 
                     it('can set mute on entry when the display hint is available enabled=false', () => {
                       manager.setDisplayHints(['DISABLE_MUTE_ON_ENTRY']);
-              
+
                       const result = manager.setMuteOnEntry(false);
-              
+
                       assert.calledWith(request.request, {  uri: 'test/id/controls',
                       body: { muteOnEntry: { enabled: false } },
                       method: HTTP_VERBS.PATCH});
-              
+
                       assert.deepEqual(result, request.request.firstCall.returnValue);
                     });
                   });
 
                   describe('setDisallowUnmute', () => {
-                    it('rejects when correct display hint is not present enabled=false', () => {  
+                    it('rejects when correct display hint is not present enabled=false', () => {
                       const result = manager.setDisallowUnmute(false);
-              
+
                       assert.notCalled(request.request);
-              
+
                       assert.isRejected(result);
                     });
 
-                    it('rejects when correct display hint is not present enabled=true', () => {  
+                    it('rejects when correct display hint is not present enabled=true', () => {
                       const result = manager.setDisallowUnmute(true);
-              
+
                       assert.notCalled(request.request);
-              
+
                       assert.isRejected(result);
                     });
-      
-                    it('can set mute on entry when the display hint is available enabled=true', () => {
+
+                    it('can set disallow unmute when ENABLE_HARD_MUTE display hint is available', () => {
                       manager.setDisplayHints(['ENABLE_HARD_MUTE']);
-              
+
                       const result = manager.setDisallowUnmute(true);
-              
+
                       assert.calledWith(request.request, {  uri: 'test/id/controls',
                       body: { disallowUnmute: { enabled: true } },
                       method: HTTP_VERBS.PATCH});
-              
+
                       assert.deepEqual(result, request.request.firstCall.returnValue);
                     });
 
-                    it('can set mute on entry when the display hint is available enabled=false', () => {
+                    it('can set allow unmute when DISABLE_HARD_MUTE display hint is available', () => {
                       manager.setDisplayHints(['DISABLE_HARD_MUTE']);
-              
+
                       const result = manager.setDisallowUnmute(false);
-              
+
                       assert.calledWith(request.request, {  uri: 'test/id/controls',
                       body: { disallowUnmute: { enabled: false } },
                       method: HTTP_VERBS.PATCH});
-              
+
                       assert.deepEqual(result, request.request.firstCall.returnValue);
                     });
                   });
@@ -218,7 +218,7 @@ describe('plugin-meetings', () => {
                 })
               });
 
-              it('rejects when correct display hint is not present mutedEnabled=false', () => {  
+              it('rejects when correct display hint is not present mutedEnabled=false', () => {
                 const result = manager.setMuteAll(false, false, false);
 
                 assert.notCalled(request.request);
@@ -226,7 +226,7 @@ describe('plugin-meetings', () => {
                 assert.isRejected(result);
               });
 
-              it('rejects when correct display hint is not present mutedEnabled=true', () => {  
+              it('rejects when correct display hint is not present mutedEnabled=true', () => {
                 const result = manager.setMuteAll(true, false, false);
 
                 assert.notCalled(request.request);
@@ -281,7 +281,31 @@ describe('plugin-meetings', () => {
 
                 assert.deepEqual(result, request.request.firstCall.returnValue);
               });
+
+              it('can set mute all panelists when the display hint is available mutedEnabled=true', () => {
+                manager.setDisplayHints(['MUTE_ALL', 'DISABLE_HARD_MUTE', 'DISABLE_MUTE_ON_ENTRY']);
+
+                const result = manager.setMuteAll(true, true, true, ['panelist']);
+
+                assert.calledWith(request.request, {  uri: 'test/id/controls',
+                  body: { audio: { muted: true, disallowUnmute: true, muteOnEntry: true, roles: ['panelist'] } },
+                  method: HTTP_VERBS.PATCH});
+
+                assert.deepEqual(result, request.request.firstCall.returnValue);
+              });
+
+              it('can set mute all attendees when the display hint is available mutedEnabled=true', () => {
+                manager.setDisplayHints(['MUTE_ALL', 'DISABLE_HARD_MUTE', 'DISABLE_MUTE_ON_ENTRY']);
+
+                const result = manager.setMuteAll(true, true, true, ['attendee']);
+
+                assert.calledWith(request.request, {  uri: 'test/id/controls',
+                  body: { audio: { muted: true, disallowUnmute: true, muteOnEntry: true, roles: ['attendee'] } },
+                  method: HTTP_VERBS.PATCH});
+
+                assert.deepEqual(result, request.request.firstCall.returnValue);
+              });
             });
           });
     });
-});
+});

package/test/unit/spec/controls-options-manager/util.js

@@ -348,6 +348,50 @@ describe('plugin-meetings', () => {
           });
         });
 
+        it('should call hasHints() with proper hints when `panelistEnabled` is true, attendeeCount is false', () => {
+          ControlsOptionsUtil.canUpdateViewTheParticipantsList({properties: {enabled: true, panelistEnabled: true, attendeeCount: false}}, []);
+
+          assert.calledWith(ControlsOptionsUtil.hasHints, {
+            requiredHints: [DISPLAY_HINTS.ENABLE_VIEW_THE_PARTICIPANT_LIST,
+              DISPLAY_HINTS.ENABLE_VIEW_THE_PARTICIPANT_LIST_PANELIST,
+              DISPLAY_HINTS.DISABLE_SHOW_ATTENDEE_COUNT],
+            displayHints: [],
+          });
+        });
+
+        it('should call hasHints() with proper hints when `panelistEnabled` is true, attendeeCount is true', () => {
+          ControlsOptionsUtil.canUpdateViewTheParticipantsList({properties: {enabled: true, panelistEnabled: true, attendeeCount: true}}, []);
+
+          assert.calledWith(ControlsOptionsUtil.hasHints, {
+            requiredHints: [DISPLAY_HINTS.ENABLE_VIEW_THE_PARTICIPANT_LIST,
+            DISPLAY_HINTS.ENABLE_VIEW_THE_PARTICIPANT_LIST_PANELIST,
+            DISPLAY_HINTS.ENABLE_SHOW_ATTENDEE_COUNT],
+            displayHints: [],
+          });
+        });
+
+        it('should call hasHints() with proper hints when `panelistEnabled` is false, attendeeCount is false', () => {
+          ControlsOptionsUtil.canUpdateViewTheParticipantsList({properties: {enabled: true, panelistEnabled: false, attendeeCount: false}}, []);
+
+          assert.calledWith(ControlsOptionsUtil.hasHints, {
+            requiredHints: [DISPLAY_HINTS.ENABLE_VIEW_THE_PARTICIPANT_LIST,
+              DISPLAY_HINTS.DISABLE_VIEW_THE_PARTICIPANT_LIST_PANELIST,
+              DISPLAY_HINTS.DISABLE_SHOW_ATTENDEE_COUNT],
+            displayHints: [],
+          });
+        });
+
+        it('should call hasHints() with proper hints when `panelistEnabled` is false, attendeeCount is true', () => {
+          ControlsOptionsUtil.canUpdateViewTheParticipantsList({properties: {enabled: true, panelistEnabled: false, attendeeCount: true}}, []);
+
+          assert.calledWith(ControlsOptionsUtil.hasHints, {
+            requiredHints: [DISPLAY_HINTS.ENABLE_VIEW_THE_PARTICIPANT_LIST,
+              DISPLAY_HINTS.DISABLE_VIEW_THE_PARTICIPANT_LIST_PANELIST,
+              DISPLAY_HINTS.ENABLE_SHOW_ATTENDEE_COUNT],
+            displayHints: [],
+          });
+        });
+
         it('should return the resolution of hasHints()', () => {
           const expected = 'example-return-value';
           ControlsOptionsUtil.hasHints.returns(expected);

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

@@ -82,11 +82,13 @@ describe('plugin-meetings', () => {
       });
 
       it('should parse the viewTheParticipantList control', () => {
-        const newControls = {viewTheParticipantList: {enabled: true}};
+        const newControls = {viewTheParticipantList: {enabled: true, panelistEnabled: true, attendeeCount: false}};
 
         const parsedControls = ControlsUtils.parse(newControls);
 
         assert.equal(parsedControls.viewTheParticipantList.enabled, newControls.viewTheParticipantList.enabled);
+        assert.equal(parsedControls.viewTheParticipantList.panelistEnabled, newControls.viewTheParticipantList.panelistEnabled);
+        assert.equal(parsedControls.viewTheParticipantList.attendeeCount, newControls.viewTheParticipantList.attendeeCount);
       });
 
       it('should parse the raiseHand control', () => {
@@ -105,6 +107,42 @@ describe('plugin-meetings', () => {
         assert.equal(parsedControls.video.enabled, newControls.video.enabled);
       });
 
+      it('should parse the webcast control', () => {
+        const newControls = {webcastControl: {streaming: true}};
+
+        const parsedControls = ControlsUtils.parse(newControls);
+
+        assert.equal(parsedControls.webcastControl.streaming, newControls.webcastControl.streaming);
+      });
+
+      it('should parse the meeting full control', () => {
+        const newControls = {meetingFull: {meetingFull: true, meetingPanelistFull: false}};
+
+        const parsedControls = ControlsUtils.parse(newControls);
+
+        assert.equal(parsedControls.meetingFull.meetingFull, newControls.meetingFull.meetingFull);
+        assert.equal(parsedControls.meetingFull.meetingPanelistFull, newControls.meetingFull.meetingPanelistFull);
+      });
+
+      it('should parse the practiceSession control', () => {
+        const newControls = {practiceSession: {enabled: true}};
+
+        const parsedControls = ControlsUtils.parse(newControls);
+
+        assert.equal(parsedControls.practiceSession.enabled, newControls.practiceSession.enabled);
+      });
+
+      it('should parse the videoLayout control', () => {
+        const newControls = {videoLayout: {overrideDefault: true, lockAttendeeViewOnStageOnly:false, stageParameters: {}}};
+
+        const parsedControls = ControlsUtils.parse(newControls);
+
+        assert.equal(parsedControls.videoLayout.overrideDefault, newControls.videoLayout.overrideDefault);
+        assert.equal(parsedControls.videoLayout.lockAttendeeViewOnStageOnly, newControls.videoLayout.lockAttendeeViewOnStageOnly);
+        assert.equal(parsedControls.videoLayout.stageParameters, newControls.videoLayout.stageParameters);
+
+      });
+
       describe('videoEnabled', () => {
         it('returns expected', () => {
           const result = ControlsUtils.parse({video: {enabled: true}});
@@ -170,11 +208,21 @@ describe('plugin-meetings', () => {
       });
 
       it('returns hasViewTheParticipantListChanged = true when changed', () => {
-        const newControls = {viewTheParticipantList: {enabled: true}};
+        const oldControls = {viewTheParticipantList: {enabled: true, panelistEnabled: true, attendeeCount: false}};
 
-        const {updates} = ControlsUtils.getControls(defaultControls, newControls);
+        let result = ControlsUtils.getControls(oldControls, {viewTheParticipantList: {enabled: false, panelistEnabled: true, attendeeCount: false}});
+
+        assert.equal(result.updates.hasViewTheParticipantListChanged, true);
+
+        result = ControlsUtils.getControls(oldControls, {viewTheParticipantList: {enabled: true, panelistEnabled: false, attendeeCount: false}});
 
-        assert.equal(updates.hasViewTheParticipantListChanged, true);
+        assert.equal(result.updates.hasViewTheParticipantListChanged, true);
+        result = ControlsUtils.getControls(oldControls, {viewTheParticipantList: {enabled: true, panelistEnabled: true, attendeeCount: true}});
+
+        assert.equal(result.updates.hasViewTheParticipantListChanged, true);
+        result = ControlsUtils.getControls(oldControls, {viewTheParticipantList: {enabled: true, panelistEnabled: true, attendeeCount: false}});
+
+        assert.equal(result.updates.hasViewTheParticipantListChanged, false);
       });
 
       it('returns hasRaiseHandChanged = true when changed', () => {
@@ -193,6 +241,34 @@ describe('plugin-meetings', () => {
         assert.equal(updates.hasVideoChanged, true);
       });
 
+      it('returns hasWebcastChanged = true when changed', () => {
+        const newControls = {webcastControl: {streaming: true}};
+
+        const {updates} = ControlsUtils.getControls(defaultControls, newControls);
+
+        assert.equal(updates.hasWebcastChanged, true);
+      });
+
+      it('returns hasMeetingFullChanged = true when changed', () => {
+        const newControls = {meetingFull: {meetingFull: true, meetingPanelistFull: false}};
+
+        let result = ControlsUtils.getControls(defaultControls, newControls);
+
+        assert.equal(result.updates.hasMeetingFullChanged, true);
+
+        result = ControlsUtils.getControls(newControls, {meetingFull: {meetingFull: true, meetingPanelistFull: true}});
+
+        assert.equal(result.updates.hasMeetingFullChanged, true);
+      });
+
+      it('returns hasPracticeSessionEnabledChanged = true when changed', () => {
+        const newControls = {practiceSession: {enabled: true}};
+
+        const {updates} = ControlsUtils.getControls(defaultControls, newControls);
+
+        assert.equal(updates.hasPracticeSessionEnabledChanged, true);
+      });
+
       it('returns hasEntryExitToneChanged = true when mode changed', () => {
         const newControls = {
           entryExitTone: {