@webex/plugin-meetings 3.12.0-next.73 → 3.12.0-next.74

package/src/locus-info/infoUtils.ts

@@ -1,5 +1,15 @@
 import {SELF_ROLES, DISPLAY_HINTS, INTERSTITIAL_DISPLAY_HINTS} from '../constants';
 
+// these values have to match what Locus sends us
+export enum DisplayHintSection {
+  JOINED = 'joined',
+  MODERATOR = 'moderator',
+  COHOST = 'coHost',
+  PRESENTER = 'presenter',
+  PANELIST = 'panelist',
+  ATTENDEE = 'attendee',
+}
+
 const InfoUtils: any = {};
 
 InfoUtils.parse = (info, roles, isJoined = true) => {
@@ -7,6 +17,9 @@ InfoUtils.parse = (info, roles, isJoined = true) => {
     policy: InfoUtils.parsePolicy(info),
     moderator: InfoUtils.parseModerator(info),
     coHost: InfoUtils.parseCoHost(info),
+    presenter: InfoUtils.parsePresenter(info),
+    panelist: InfoUtils.parsePanelist(info),
+    attendee: InfoUtils.parseAttendee(info),
   };
 
   let userDisplayHints = isJoined
@@ -27,6 +40,18 @@ InfoUtils.parse = (info, roles, isJoined = true) => {
     userDisplayHints = {...userDisplayHints, ...parsed.moderator};
   }
 
+  if (roles.includes(SELF_ROLES.PRESENTER)) {
+    userDisplayHints = {...userDisplayHints, ...parsed.presenter};
+  }
+
+  if (roles.includes(SELF_ROLES.PANELIST)) {
+    userDisplayHints = {...userDisplayHints, ...parsed.panelist};
+  }
+
+  if (roles.includes(SELF_ROLES.ATTENDEE)) {
+    userDisplayHints = {...userDisplayHints, ...parsed.attendee};
+  }
+
   parsed.userDisplayHints = Object.keys(userDisplayHints);
 
   if (info.sipUri) {
@@ -44,8 +69,8 @@ InfoUtils.parse = (info, roles, isJoined = true) => {
   return parsed;
 };
 
-InfoUtils.parseDisplayHintSection = (info, displayHintKey) => {
-  const displayHints = {};
+InfoUtils.parseDisplayHintSection = (info: any, displayHintKey: DisplayHintSection) => {
+  const displayHints: Record<string, boolean> = {};
 
   if (
     info &&
@@ -53,7 +78,7 @@ InfoUtils.parseDisplayHintSection = (info, displayHintKey) => {
     info.displayHints[displayHintKey] &&
     info.displayHints[displayHintKey].length > 0
   ) {
-    info.displayHints[displayHintKey].forEach((key) => {
+    info.displayHints[displayHintKey].forEach((key: any) => {
       displayHints[key] = true;
     });
   }
@@ -61,20 +86,30 @@ InfoUtils.parseDisplayHintSection = (info, displayHintKey) => {
   return displayHints;
 };
 
-InfoUtils.parsePolicy = (info) => InfoUtils.parseDisplayHintSection(info, 'joined');
+InfoUtils.parsePolicy = (info) =>
+  InfoUtils.parseDisplayHintSection(info, DisplayHintSection.JOINED);
 
 InfoUtils.parseModerator = (info) => {
-  const displayHints = InfoUtils.parseDisplayHintSection(info, 'moderator');
+  const displayHints = InfoUtils.parseDisplayHintSection(info, DisplayHintSection.MODERATOR);
 
   return {...displayHints, [DISPLAY_HINTS.LOWER_SOMEONE_ELSES_HAND]: true};
 };
 
 InfoUtils.parseCoHost = (info) => {
-  const displayHints = InfoUtils.parseDisplayHintSection(info, 'coHost');
+  const displayHints = InfoUtils.parseDisplayHintSection(info, DisplayHintSection.COHOST);
 
   return {...displayHints, [DISPLAY_HINTS.LOWER_SOMEONE_ELSES_HAND]: true};
 };
 
+InfoUtils.parsePresenter = (info) =>
+  InfoUtils.parseDisplayHintSection(info, DisplayHintSection.PRESENTER);
+
+InfoUtils.parsePanelist = (info) =>
+  InfoUtils.parseDisplayHintSection(info, DisplayHintSection.PANELIST);
+
+InfoUtils.parseAttendee = (info) =>
+  InfoUtils.parseDisplayHintSection(info, DisplayHintSection.ATTENDEE);
+
 InfoUtils.isLocked = (policy) => policy.LOCK_STATUS_LOCKED || false;
 
 InfoUtils.isUnlocked = (policy) => policy.LOCK_STATUS_UNLOCKED || false;

package/src/meeting/in-meeting-actions.ts

@@ -121,6 +121,7 @@ interface IInMeetingActions {
   canAttendeeRequestAiAssistantEnabled?: boolean;
   isAttendeeRequestAiAssistantDeclinedAll?: boolean;
   isAnonymizeDisplayNamesEnabled?: boolean;
+  canViewTheParticipantList?: boolean;
 }
 
 /**
@@ -349,6 +350,8 @@ export default class InMeetingActions implements IInMeetingActions {
 
   isAnonymizeDisplayNamesEnabled = null;
 
+  canViewTheParticipantList = null;
+
   /**
    * Returns all meeting action options
    * @returns {Object}
@@ -464,6 +467,7 @@ export default class InMeetingActions implements IInMeetingActions {
     canAttendeeRequestAiAssistantEnabled: this.canAttendeeRequestAiAssistantEnabled,
     isAttendeeRequestAiAssistantDeclinedAll: this.isAttendeeRequestAiAssistantDeclinedAll,
     isAnonymizeDisplayNamesEnabled: this.isAnonymizeDisplayNamesEnabled,
+    canViewTheParticipantList: this.canViewTheParticipantList,
   });
 
   /**

package/src/meeting/index.ts

@@ -665,6 +665,7 @@ export default class Meeting extends StatelessWebexPlugin {
   joinedWith?: any;
   selfId?: string;
   roles: any[];
+  canNotViewTheParticipantList?: boolean;
   // ... there is more ... see SelfUtils.parse()
   // end of the group
   locusMediaRequest?: LocusMediaRequest;
@@ -3859,6 +3860,9 @@ export default class Meeting extends StatelessWebexPlugin {
     });
 
     this.locusInfo.on(LOCUSINFO.EVENTS.SELF_CANNOT_VIEW_PARTICIPANT_LIST_CHANGE, (payload) => {
+      // canViewTheParticipantList meeting action depends on this value, so we need to update meeting actions
+      this.updateMeetingActions();
+
       Trigger.trigger(
         this,
         {
@@ -4660,6 +4664,10 @@ export default class Meeting extends StatelessWebexPlugin {
           isAnonymizeDisplayNamesEnabled: MeetingUtil.isAnonymizeDisplayNamesEnabled(
             this.userDisplayHints
           ),
+          canViewTheParticipantList: MeetingUtil.canViewTheParticipantList(
+            this.userDisplayHints,
+            this.canNotViewTheParticipantList ?? false
+          ),
         }) || changed;
     }
     if (changed) {

package/src/meeting/util.ts

@@ -949,6 +949,18 @@ const MeetingUtil = {
   isAnonymizeDisplayNamesEnabled: (displayHints) =>
     displayHints.includes(DISPLAY_HINTS.ANONYMOUS_DISPLAY_NAMES_ENABLED),
 
+  canViewTheParticipantList: (displayHints, canNotViewTheParticipantList: boolean) => {
+    if (!displayHints.includes(DISPLAY_HINTS.VIEW_THE_PARTICIPANT_LIST)) {
+      return false;
+    }
+
+    if (canNotViewTheParticipantList) {
+      return false;
+    }
+
+    return displayHints.includes(DISPLAY_HINTS.CAN_VIEW_THE_PARTICIPANT_LIST);
+  },
+
   selfSupportsFeature: (feature: SELF_POLICY, userPolicies: Record<SELF_POLICY, boolean>) => {
     if (!userPolicies) {
       return true;

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

@@ -2856,6 +2856,7 @@ describe('plugin-meetings', () => {
 
         // set the info initially as locusInfo.info starts as undefined
         expectedMeeting = {
+          attendee: {},
           coHost: {
             LOWER_SOMEONE_ELSES_HAND: true,
           },
@@ -2864,10 +2865,12 @@ describe('plugin-meetings', () => {
           moderator: {
             LOWER_SOMEONE_ELSES_HAND: true,
           },
+          panelist: {},
           policy: {
             LOCK_STATUS_UNLOCKED: true,
             ROSTER_IN_MEETING: true,
           },
+          presenter: {},
           userDisplayHints: ['ROSTER_IN_MEETING', 'LOCK_STATUS_UNLOCKED'],
         };
         locusInfo.updateMeetingInfo(initialInfo, self);
@@ -2882,6 +2885,7 @@ describe('plugin-meetings', () => {
 
         // Updating with different info should trigger the event
         expectedMeeting = {
+          attendee: {},
           coHost: {
             LOWER_SOMEONE_ELSES_HAND: true,
             LOCK_CONTROL_LOCK: true,
@@ -2891,10 +2895,12 @@ describe('plugin-meetings', () => {
           moderator: {
             LOWER_SOMEONE_ELSES_HAND: true,
           },
+          panelist: {},
           policy: {
             LOCK_STATUS_UNLOCKED: true,
             ROSTER_IN_MEETING: true,
           },
+          presenter: {},
           userDisplayHints: ['ROSTER_IN_MEETING', 'LOCK_STATUS_UNLOCKED'],
         };
         locusInfo.updateMeetingInfo(newInfo, self);
@@ -2903,6 +2909,7 @@ describe('plugin-meetings', () => {
 
         // update it with the same info
         expectedMeeting = {
+          attendee: {},
           coHost: {
             LOWER_SOMEONE_ELSES_HAND: true,
             LOCK_CONTROL_LOCK: true,
@@ -2912,10 +2919,12 @@ describe('plugin-meetings', () => {
           moderator: {
             LOWER_SOMEONE_ELSES_HAND: true,
           },
+          panelist: {},
           policy: {
             LOCK_STATUS_UNLOCKED: true,
             ROSTER_IN_MEETING: true,
           },
+          presenter: {},
           userDisplayHints: ['ROSTER_IN_MEETING', 'LOCK_STATUS_UNLOCKED'],
         };
         locusInfo.updateMeetingInfo(newInfo, self);
@@ -2930,6 +2939,7 @@ describe('plugin-meetings', () => {
           hasRole: true,
         });
         expectedMeeting = {
+          attendee: {},
           coHost: {
             LOWER_SOMEONE_ELSES_HAND: true,
             LOCK_CONTROL_LOCK: true,
@@ -2939,10 +2949,12 @@ describe('plugin-meetings', () => {
           moderator: {
             LOWER_SOMEONE_ELSES_HAND: true,
           },
+          panelist: {},
           policy: {
             LOCK_STATUS_UNLOCKED: true,
             ROSTER_IN_MEETING: true,
           },
+          presenter: {},
           userDisplayHints: [
             'ROSTER_IN_MEETING',
             'LOCK_STATUS_UNLOCKED',

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

@@ -9,6 +9,9 @@ describe('plugin-meetings', () => {
         moderator: ['HINT_1', 'HINT_2'],
         joined: ['HINT_3', 'VOIP_IS_ENABLED'],
         coHost: ['HINT_4'],
+        presenter: ['HINT_5'],
+        panelist: ['HINT_6'],
+        attendee: ['HINT_7'],
       },
     };
 
@@ -30,6 +33,9 @@ describe('plugin-meetings', () => {
           policy: {HINT_3: true, VOIP_IS_ENABLED: true},
           moderator: {HINT_1: true, HINT_2: true, LOWER_SOMEONE_ELSES_HAND: true},
           coHost: {HINT_4: true, LOWER_SOMEONE_ELSES_HAND: true},
+          presenter: {HINT_5: true},
+          panelist: {HINT_6: true},
+          attendee: {HINT_7: true},
           userDisplayHints: [
             'HINT_3',
             'VOIP_IS_ENABLED',
@@ -43,6 +49,9 @@ describe('plugin-meetings', () => {
           policy: {HINT_3: true, VOIP_IS_ENABLED: true},
           moderator: {HINT_1: true, HINT_2: true, LOWER_SOMEONE_ELSES_HAND: true},
           coHost: {HINT_4: true, LOWER_SOMEONE_ELSES_HAND: true},
+          presenter: {HINT_5: true},
+          panelist: {HINT_6: true},
+          attendee: {HINT_7: true},
           userDisplayHints: [
             'HINT_3',
             'VOIP_IS_ENABLED',
@@ -57,6 +66,9 @@ describe('plugin-meetings', () => {
           policy: {HINT_3: true, VOIP_IS_ENABLED: true},
           moderator: {HINT_1: true, HINT_2: true, LOWER_SOMEONE_ELSES_HAND: true},
           coHost: {HINT_4: true, LOWER_SOMEONE_ELSES_HAND: true},
+          presenter: {HINT_5: true},
+          panelist: {HINT_6: true},
+          attendee: {HINT_7: true},
           userDisplayHints: ['HINT_3', 'VOIP_IS_ENABLED', 'HINT_4', 'LOWER_SOMEONE_ELSES_HAND'],
         });
 
@@ -64,6 +76,9 @@ describe('plugin-meetings', () => {
           policy: {HINT_3: true, VOIP_IS_ENABLED: true},
           moderator: {HINT_1: true, HINT_2: true, LOWER_SOMEONE_ELSES_HAND: true},
           coHost: {HINT_4: true, LOWER_SOMEONE_ELSES_HAND: true},
+          presenter: {HINT_5: true},
+          panelist: {HINT_6: true},
+          attendee: {HINT_7: true},
           userDisplayHints: ['HINT_3', 'VOIP_IS_ENABLED'],
         });
       });
@@ -73,6 +88,9 @@ describe('plugin-meetings', () => {
           policy: {HINT_3: true, VOIP_IS_ENABLED: true},
           moderator: {HINT_1: true, HINT_2: true, LOWER_SOMEONE_ELSES_HAND: true},
           coHost: {HINT_4: true, LOWER_SOMEONE_ELSES_HAND: true},
+          presenter: {HINT_5: true},
+          panelist: {HINT_6: true},
+          attendee: {HINT_7: true},
           userDisplayHints: ['VOIP_IS_ENABLED', 'HINT_1', 'HINT_2', 'LOWER_SOMEONE_ELSES_HAND'],
         });
 
@@ -80,6 +98,9 @@ describe('plugin-meetings', () => {
           policy: {HINT_3: true, VOIP_IS_ENABLED: true},
           moderator: {HINT_1: true, HINT_2: true, LOWER_SOMEONE_ELSES_HAND: true},
           coHost: {HINT_4: true, LOWER_SOMEONE_ELSES_HAND: true},
+          presenter: {HINT_5: true},
+          panelist: {HINT_6: true},
+          attendee: {HINT_7: true},
           userDisplayHints: [
             'HINT_3',
             'VOIP_IS_ENABLED',
@@ -90,10 +111,49 @@ describe('plugin-meetings', () => {
         });
       });
 
+      it('merges presenter hints when user has PRESENTER role', () => {
+        assert.deepEqual(InfoUtils.parse(info, ['PRESENTER']), {
+          policy: {HINT_3: true, VOIP_IS_ENABLED: true},
+          moderator: {HINT_1: true, HINT_2: true, LOWER_SOMEONE_ELSES_HAND: true},
+          coHost: {HINT_4: true, LOWER_SOMEONE_ELSES_HAND: true},
+          presenter: {HINT_5: true},
+          panelist: {HINT_6: true},
+          attendee: {HINT_7: true},
+          userDisplayHints: ['HINT_3', 'VOIP_IS_ENABLED', 'HINT_5'],
+        });
+      });
+
+      it('merges panelist hints when user has PANELIST role', () => {
+        assert.deepEqual(InfoUtils.parse(info, ['PANELIST']), {
+          policy: {HINT_3: true, VOIP_IS_ENABLED: true},
+          moderator: {HINT_1: true, HINT_2: true, LOWER_SOMEONE_ELSES_HAND: true},
+          coHost: {HINT_4: true, LOWER_SOMEONE_ELSES_HAND: true},
+          presenter: {HINT_5: true},
+          panelist: {HINT_6: true},
+          attendee: {HINT_7: true},
+          userDisplayHints: ['HINT_3', 'VOIP_IS_ENABLED', 'HINT_6'],
+        });
+      });
+
+      it('merges attendee hints when user has ATTENDEE role', () => {
+        assert.deepEqual(InfoUtils.parse(info, ['ATTENDEE']), {
+          policy: {HINT_3: true, VOIP_IS_ENABLED: true},
+          moderator: {HINT_1: true, HINT_2: true, LOWER_SOMEONE_ELSES_HAND: true},
+          coHost: {HINT_4: true, LOWER_SOMEONE_ELSES_HAND: true},
+          presenter: {HINT_5: true},
+          panelist: {HINT_6: true},
+          attendee: {HINT_7: true},
+          userDisplayHints: ['HINT_3', 'VOIP_IS_ENABLED', 'HINT_7'],
+        });
+      });
+
       it('only adds datachannel url when present', () => {
         assert.deepEqual(InfoUtils.parse({datachannelUrl: 'some url'}, []), {
           coHost: {LOWER_SOMEONE_ELSES_HAND: true},
           moderator: {LOWER_SOMEONE_ELSES_HAND: true},
+          presenter: {},
+          panelist: {},
+          attendee: {},
           datachannelUrl: 'some url',
           policy: {},
           userDisplayHints: [],
@@ -102,6 +162,9 @@ describe('plugin-meetings', () => {
         assert.deepEqual(InfoUtils.parse({}, []), {
           coHost: {LOWER_SOMEONE_ELSES_HAND: true},
           moderator: {LOWER_SOMEONE_ELSES_HAND: true},
+          presenter: {},
+          panelist: {},
+          attendee: {},
           policy: {},
           userDisplayHints: [],
         });
@@ -171,6 +234,30 @@ describe('plugin-meetings', () => {
           LOWER_SOMEONE_ELSES_HAND: true,
         });
       });
+
+      it('parsePresenter calls parseDisplayHintSection correctly and returns the result', () => {
+        const result = InfoUtils.parsePresenter(info);
+
+        assert.calledWith(parseDisplayHintSectionSpy, info, 'presenter');
+
+        assert.deepEqual(result, parseDisplayHintSectionSpy.firstCall.returnValue);
+      });
+
+      it('parsePanelist calls parseDisplayHintSection correctly and returns the result', () => {
+        const result = InfoUtils.parsePanelist(info);
+
+        assert.calledWith(parseDisplayHintSectionSpy, info, 'panelist');
+
+        assert.deepEqual(result, parseDisplayHintSectionSpy.firstCall.returnValue);
+      });
+
+      it('parseAttendee calls parseDisplayHintSection correctly and returns the result', () => {
+        const result = InfoUtils.parseAttendee(info);
+
+        assert.calledWith(parseDisplayHintSectionSpy, info, 'attendee');
+
+        assert.deepEqual(result, parseDisplayHintSectionSpy.firstCall.returnValue);
+      });
     });
   });
 });

package/test/unit/spec/meeting/in-meeting-actions.ts

@@ -115,6 +115,7 @@ describe('plugin-meetings', () => {
         canAttendeeRequestAiAssistantEnabled: null,
         isAttendeeRequestAiAssistantDeclinedAll: null,
         isAnonymizeDisplayNamesEnabled: null,
+        canViewTheParticipantList: null,
 
         ...expected,
       };
@@ -236,6 +237,7 @@ describe('plugin-meetings', () => {
       'canAttendeeRequestAiAssistantEnabled',
       'isAttendeeRequestAiAssistantDeclinedAll',
       'isAnonymizeDisplayNamesEnabled',
+      'canViewTheParticipantList',
     ].forEach((key) => {
       it(`get and set for ${key} work as expected`, () => {
         const inMeetingActions = new InMeetingActions();

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

@@ -967,6 +967,55 @@ describe('plugin-meetings', () => {
       });
     });
 
+    describe('canViewTheParticipantList', () => {
+      it('returns true when both VIEW_THE_PARTICIPANT_LIST and CAN_VIEW_THE_PARTICIPANT_LIST hints are present and canNotViewTheParticipantList is false', () => {
+        assert.isTrue(
+          MeetingUtil.canViewTheParticipantList(
+            ['VIEW_THE_PARTICIPANT_LIST', 'CAN_VIEW_THE_PARTICIPANT_LIST'],
+            false
+          )
+        );
+      });
+
+      it('returns false when VIEW_THE_PARTICIPANT_LIST hint is missing', () => {
+        assert.isFalse(
+          MeetingUtil.canViewTheParticipantList(['CAN_VIEW_THE_PARTICIPANT_LIST'], false)
+        );
+      });
+
+      it('returns false when CAN_VIEW_THE_PARTICIPANT_LIST hint is missing', () => {
+        assert.isFalse(
+          MeetingUtil.canViewTheParticipantList(['VIEW_THE_PARTICIPANT_LIST'], false)
+        );
+      });
+
+      it('returns false when canNotViewTheParticipantList is true', () => {
+        assert.isFalse(
+          MeetingUtil.canViewTheParticipantList(
+            ['VIEW_THE_PARTICIPANT_LIST', 'CAN_VIEW_THE_PARTICIPANT_LIST'],
+            true
+          )
+        );
+      });
+
+      it('returns false when display hints array is empty', () => {
+        assert.isFalse(MeetingUtil.canViewTheParticipantList([], false));
+      });
+
+      it('returns false when both conditions are violated', () => {
+        assert.isFalse(MeetingUtil.canViewTheParticipantList([], true));
+      });
+
+      it('returns true when canNotViewTheParticipantList is undefined (not yet set on meeting)', () => {
+        assert.isTrue(
+          MeetingUtil.canViewTheParticipantList(
+            ['VIEW_THE_PARTICIPANT_LIST', 'CAN_VIEW_THE_PARTICIPANT_LIST'],
+            undefined
+          )
+        );
+      });
+    });
+
     describe('canAttendeeRequestAiAssistantEnabled', () => {
       it('returns false when user is a cohost', () => {
         assert.deepEqual(

package/test/unit/spec/meetings/index.js

@@ -3593,10 +3593,13 @@ describe('plugin-meetings', () => {
       };
 
       it('triggers correct event when SELF_CANNOT_VIEW_PARTICIPANT_LIST_CHANGE emitted', async () => {
+        sinon.stub(meeting, 'updateMeetingActions');
         checkSelfTrigger(
           LOCUSINFO.EVENTS.SELF_CANNOT_VIEW_PARTICIPANT_LIST_CHANGE,
           EVENT_TRIGGERS.MEETING_SELF_CANNOT_VIEW_PARTICIPANT_LIST
         );
+        assert.calledOnce(meeting.updateMeetingActions);
+        meeting.updateMeetingActions.restore();
       });
 
       it('triggers correct event when SELF_IS_SHARING_BLOCKED_CHANGE emitted', async () => {