@webex/plugin-meetings 2.37.0 → 2.37.1

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

@@ -19,17 +19,17 @@ describe('plugin-meetings', () => {
     meeting = {
       mediaProperties: {
         audioTrack: 'fake audio track',
-        videoTrack: 'fake video track'
+        videoTrack: 'fake video track',
       },
       remoteMuted: false,
       unmuteAllowed: true,
       locusInfo: {
-        onFullLocus: sinon.stub()
+        onFullLocus: sinon.stub(),
       },
       members: {
         selfId: 'fake self id',
-        muteMember: sinon.stub().resolves()
-      }
+        muteMember: sinon.stub().resolves(),
+      },
     };
     audio = createMuteState(AUDIO, meeting, {sendAudio: true});
     video = createMuteState(VIDEO, meeting, {sendVideo: true});
@@ -121,11 +121,14 @@ describe('plugin-meetings', () => {
       // first we need to mute and make that request last forever
       let serverResponseResolve;
 
-      MeetingUtil.remoteUpdateAudioVideo = sinon.stub().returns(new Promise((resolve) => {
-        serverResponseResolve = resolve;
-      }));
+      MeetingUtil.remoteUpdateAudioVideo = sinon.stub().returns(
+        new Promise((resolve) => {
+          serverResponseResolve = resolve;
+        })
+      );
 
-      audio.handleClientRequest(meeting, true)
+      audio
+        .handleClientRequest(meeting, true)
         .then(() => {
           clientPromiseResolved = true;
         })
@@ -229,14 +232,15 @@ describe('plugin-meetings', () => {
 
         let serverResponseResolve;
 
-        MeetingUtil.remoteUpdateAudioVideo = sinon.stub().returns(new Promise((resolve) => {
-          serverResponseResolve = resolve;
-        }));
+        MeetingUtil.remoteUpdateAudioVideo = sinon.stub().returns(
+          new Promise((resolve) => {
+            serverResponseResolve = resolve;
+          })
+        );
 
-        audio.handleClientRequest(meeting, true)
-          .then(() => {
-            clientPromiseResolved = true;
-          });
+        audio.handleClientRequest(meeting, true).then(() => {
+          clientPromiseResolved = true;
+        });
 
         // do a small delay to make sure that the client promise doesn't resolve in that time
         await testUtils.waitUntil(200);
@@ -250,9 +254,11 @@ describe('plugin-meetings', () => {
       });
 
       it('rejects client request promise if server request for local mute fails', async () => {
-        MeetingUtil.remoteUpdateAudioVideo = sinon.stub().returns(new Promise((resolve, reject) => {
-          reject();
-        }));
+        MeetingUtil.remoteUpdateAudioVideo = sinon.stub().returns(
+          new Promise((resolve, reject) => {
+            reject();
+          })
+        );
 
         assert.isRejected(audio.handleClientRequest(meeting, true));
       });
@@ -275,9 +281,11 @@ describe('plugin-meetings', () => {
       it('does not send a server request if client state matches the server', async () => {
         let serverResponseResolve;
 
-        MeetingUtil.remoteUpdateAudioVideo = sinon.stub().returns(new Promise((resolve) => {
-          serverResponseResolve = resolve;
-        }));
+        MeetingUtil.remoteUpdateAudioVideo = sinon.stub().returns(
+          new Promise((resolve) => {
+            serverResponseResolve = resolve;
+          })
+        );
 
         // simulate many client requests, with the last one matching the initial one
         audio.handleClientRequest(meeting, true);
@@ -302,22 +310,22 @@ describe('plugin-meetings', () => {
       it('queues up server requests when multiple client requests are received', async () => {
         let serverResponseResolve;
 
-        MeetingUtil.remoteUpdateAudioVideo = sinon.stub().returns(new Promise((resolve) => {
-          serverResponseResolve = resolve;
-        }));
+        MeetingUtil.remoteUpdateAudioVideo = sinon.stub().returns(
+          new Promise((resolve) => {
+            serverResponseResolve = resolve;
+          })
+        );
 
         let firstClientPromiseResolved = false;
         let secondClientPromiseResolved = false;
 
         // 2 client requests, one after another without waiting for first one to resolve
-        audio.handleClientRequest(meeting, true)
-          .then(() => {
-            firstClientPromiseResolved = true;
-          });
-        audio.handleClientRequest(meeting, false)
-          .then(() => {
-            secondClientPromiseResolved = true;
-          });
+        audio.handleClientRequest(meeting, true).then(() => {
+          firstClientPromiseResolved = true;
+        });
+        audio.handleClientRequest(meeting, false).then(() => {
+          secondClientPromiseResolved = true;
+        });
 
         await testUtils.flushPromises();
 
@@ -344,7 +352,8 @@ describe('plugin-meetings', () => {
       it('rejects client request to unmute if hard mute is used', (done) => {
         audio.handleServerRemoteMuteUpdate(true, false);
 
-        audio.handleClientRequest(meeting, false)
+        audio
+          .handleClientRequest(meeting, false)
           .then(() => {
             done(new Error('expected handleClientRequest to fail, but it did not!'));
           })

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

@@ -10,26 +10,28 @@ describe('plugin-meetings', () => {
   beforeEach(() => {
     const webex = new MockWebex({
       children: {
-        meetings: Meetings
-      }
+        meetings: Meetings,
+      },
     });
 
     webex.meetings.clientRegion = {
       countryCode: 'US',
-      regionCode: 'WEST-COAST'
+      regionCode: 'WEST-COAST',
     };
 
     webex.internal = {
       services: {
         get: sinon.mock().returns('locusUrl'),
-        waitForCatalog: sinon.mock().returns(Promise.resolve({}))
-      }
+        waitForCatalog: sinon.mock().returns(Promise.resolve({})),
+      },
     };
 
-    meetingsRequest = new MeetingRequest({}, {
-      parent: webex
-    });
-
+    meetingsRequest = new MeetingRequest(
+      {},
+      {
+        parent: webex,
+      }
+    );
 
     meetingsRequest.request = sinon.mock().returns(Promise.resolve({}));
   });
@@ -44,7 +46,7 @@ describe('plugin-meetings', () => {
         await meetingsRequest.sendDTMF({
           locusUrl,
           deviceUrl,
-          tones
+          tones,
         });
         const requestParams = meetingsRequest.request.getCall(0).args[0];
 
@@ -66,7 +68,7 @@ describe('plugin-meetings', () => {
           deviceUrl,
           layoutType,
           main: {width: 640, height: 480},
-          content: {width: 1280, height: 720}
+          content: {width: 1280, height: 720},
         });
         const requestParams = meetingsRequest.request.getCall(0).args[0];
 
@@ -74,43 +76,53 @@ describe('plugin-meetings', () => {
         assert.equal(requestParams.uri, `${locusUrl}/controls`);
         assert.equal(requestParams.body.layout.type, layoutType);
         assert.equal(requestParams.body.layout.deviceUrl, deviceUrl);
-        assert.deepEqual(requestParams.body.layout.layoutParams, {renderInfo: {main: {width: 640, height: 480}, content: {width: 1280, height: 720}}});
+        assert.deepEqual(requestParams.body.layout.layoutParams, {
+          renderInfo: {main: {width: 640, height: 480}, content: {width: 1280, height: 720}},
+        });
       });
 
       it('throws if width is missing for main', async () => {
-        await assert.isRejected(meetingsRequest.changeVideoLayout({
-          locusUrl,
-          deviceUrl,
-          layoutType,
-          main: {height: 100}
-        }));
+        await assert.isRejected(
+          meetingsRequest.changeVideoLayout({
+            locusUrl,
+            deviceUrl,
+            layoutType,
+            main: {height: 100},
+          })
+        );
       });
 
       it('throws if height is missing for main', async () => {
-        await assert.isRejected(meetingsRequest.changeVideoLayout({
-          locusUrl,
-          deviceUrl,
-          layoutType,
-          main: {width: 100}
-        }));
+        await assert.isRejected(
+          meetingsRequest.changeVideoLayout({
+            locusUrl,
+            deviceUrl,
+            layoutType,
+            main: {width: 100},
+          })
+        );
       });
 
       it('throws if width is missing for content', async () => {
-        await assert.isRejected(meetingsRequest.changeVideoLayout({
-          locusUrl,
-          deviceUrl,
-          layoutType,
-          content: {height: 100}
-        }));
+        await assert.isRejected(
+          meetingsRequest.changeVideoLayout({
+            locusUrl,
+            deviceUrl,
+            layoutType,
+            content: {height: 100},
+          })
+        );
       });
 
       it('throws if height is missing for content', async () => {
-        await assert.isRejected(meetingsRequest.changeVideoLayout({
-          locusUrl,
-          deviceUrl,
-          layoutType,
-          content: {width: 100}
-        }));
+        await assert.isRejected(
+          meetingsRequest.changeVideoLayout({
+            locusUrl,
+            deviceUrl,
+            layoutType,
+            content: {width: 100},
+          })
+        );
       });
     });
 
@@ -127,8 +139,7 @@ describe('plugin-meetings', () => {
           deviceUrl,
           correlationId,
           roapMessage,
-          permissionToken
-
+          permissionToken,
         });
         const requestParams = meetingsRequest.request.getCall(0).args[0];
 
@@ -150,7 +161,7 @@ describe('plugin-meetings', () => {
           deviceUrl,
           correlationId,
           roapMessage,
-          meetingNumber
+          meetingNumber,
         });
         const requestParams = meetingsRequest.request.getCall(0).args[0];
 
@@ -171,7 +182,7 @@ describe('plugin-meetings', () => {
           correlationId,
           roapMessage,
           meetingNumber,
-          inviteeAddress
+          inviteeAddress,
         });
         const requestParams = meetingsRequest.request.getCall(0).args[0];
 
@@ -192,7 +203,7 @@ describe('plugin-meetings', () => {
           locusUrl,
           clientUrl,
           correlationId,
-          dialInUrl
+          dialInUrl,
         });
         const requestParams = meetingsRequest.request.getCall(0).args[0];
 
@@ -216,7 +227,7 @@ describe('plugin-meetings', () => {
           clientUrl,
           correlationId,
           dialOutUrl,
-          phoneNumber
+          phoneNumber,
         });
         const requestParams = meetingsRequest.request.getCall(0).args[0];
 
@@ -239,7 +250,7 @@ describe('plugin-meetings', () => {
           locusUrl,
           selfId,
           correlationId,
-          phoneUrl
+          phoneUrl,
         });
         const requestParams = meetingsRequest.request.getCall(0).args[0];
 
@@ -286,13 +297,13 @@ describe('plugin-meetings', () => {
           type: 'thumb_down',
           codepoints: '1F44E',
           shortcodes: ':thumbsdown:',
-          tone: {type: 'normal_skin_tone', codepoints: '', shortcodes: ''}
+          tone: {type: 'normal_skin_tone', codepoints: '', shortcodes: ''},
         };
 
         await meetingsRequest.sendReaction({
           reactionChannelUrl,
           reaction,
-          participantId
+          participantId,
         });
         const requestParams = meetingsRequest.request.getCall(0).args[0];
 

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

@@ -2,11 +2,9 @@ import sinon from 'sinon';
 import {assert} from '@webex/test-helper-chai';
 import MeetingUtil from '@webex/plugin-meetings/src/meeting/util';
 import LoggerProxy from '@webex/plugin-meetings/src/common/logs/logger-proxy';
-import LoggerConfig
-  from '@webex/plugin-meetings/src/common/logs/logger-config';
+import LoggerConfig from '@webex/plugin-meetings/src/common/logs/logger-config';
 import Metrics from '@webex/plugin-meetings/src/metrics/index';
 
-
 describe('plugin-meetings', () => {
   describe('Meeting utils function', () => {
     const sandbox = sinon.createSandbox();
@@ -18,23 +16,19 @@ describe('plugin-meetings', () => {
         info: sandbox.stub(),
         log: sandbox.stub(),
         error: sandbox.stub(),
-        warn: sandbox.stub()
+        warn: sandbox.stub(),
       };
 
       LoggerConfig.set({
         verboseEvents: true,
-        enable: true
+        enable: true,
       });
       LoggerProxy.set(logger);
 
-      meeting.closeLocalStream = sinon.stub()
-        .returns(Promise.resolve());
-      meeting.closeLocalShare = sinon.stub()
-        .returns(Promise.resolve());
-      meeting.closeRemoteTracks = sinon.stub()
-        .returns(Promise.resolve());
-      meeting.closePeerConnections = sinon.stub()
-        .returns(Promise.resolve());
+      meeting.closeLocalStream = sinon.stub().returns(Promise.resolve());
+      meeting.closeLocalShare = sinon.stub().returns(Promise.resolve());
+      meeting.closeRemoteTracks = sinon.stub().returns(Promise.resolve());
+      meeting.closePeerConnections = sinon.stub().returns(Promise.resolve());
 
       meeting.unsetLocalVideoTrack = sinon.stub();
       meeting.unsetLocalShareTrack = sinon.stub();
@@ -70,11 +64,11 @@ describe('plugin-meetings', () => {
 
     describe('logging', () => {
       const fakeDevice = sinon.fake.returns({
-        deviceId: 'device-1'
+        deviceId: 'device-1',
       });
 
       const mockTrack = {
-        getSettings: fakeDevice
+        getSettings: fakeDevice,
       };
 
       it('#log - should log [info, warn, error, log] to console', () => {
@@ -124,10 +118,7 @@ describe('plugin-meetings', () => {
         });
 
         it('should log device settings', () => {
-          const mockDevices = [
-            {deviceId: 'device-1'},
-            {deviceId: 'device-2'}
-          ];
+          const mockDevices = [{deviceId: 'device-1'}, {deviceId: 'device-2'}];
 
           assert(MeetingUtil.handleDeviceLogging, 'is defined');
           MeetingUtil.handleDeviceLogging(mockDevices);
@@ -138,7 +129,13 @@ describe('plugin-meetings', () => {
 
     describe('joinMeeting', () => {
       it('#Should call `meetingRequest.joinMeeting', async () => {
-        const meeting = {meetingJoinUrl: 'meetingJoinUrl', locusUrl: 'locusUrl', meetingRequest: {joinMeeting: sinon.stub().returns(Promise.resolve({body: {}, headers: {}}))}};
+        const meeting = {
+          meetingJoinUrl: 'meetingJoinUrl',
+          locusUrl: 'locusUrl',
+          meetingRequest: {
+            joinMeeting: sinon.stub().returns(Promise.resolve({body: {}, headers: {}})),
+          },
+        };
 
         MeetingUtil.parseLocusJoin = sinon.stub();
         await MeetingUtil.joinMeeting(meeting, {});
@@ -150,7 +147,13 @@ describe('plugin-meetings', () => {
       });
 
       it('#Should fallback sipUrl if meetingJoinUrl does not exists', async () => {
-        const meeting = {sipUri: 'sipUri', locusUrl: 'locusUrl', meetingRequest: {joinMeeting: sinon.stub().returns(Promise.resolve({body: {}, headers: {}}))}};
+        const meeting = {
+          sipUri: 'sipUri',
+          locusUrl: 'locusUrl',
+          meetingRequest: {
+            joinMeeting: sinon.stub().returns(Promise.resolve({body: {}, headers: {}})),
+          },
+        };
 
         MeetingUtil.parseLocusJoin = sinon.stub();
         await MeetingUtil.joinMeeting(meeting, {});
@@ -162,7 +165,13 @@ describe('plugin-meetings', () => {
       });
 
       it('#Should fallback to meetingNumber if meetingJoinUrl/sipUrl  does not exists', async () => {
-        const meeting = {meetingNumber: 'meetingNumber', locusUrl: 'locusUrl', meetingRequest: {joinMeeting: sinon.stub().returns(Promise.resolve({body: {}, headers: {}}))}};
+        const meeting = {
+          meetingNumber: 'meetingNumber',
+          locusUrl: 'locusUrl',
+          meetingRequest: {
+            joinMeeting: sinon.stub().returns(Promise.resolve({body: {}, headers: {}})),
+          },
+        };
 
         MeetingUtil.parseLocusJoin = sinon.stub();
         await MeetingUtil.joinMeeting(meeting, {});
@@ -179,15 +188,9 @@ describe('plugin-meetings', () => {
       it('returns display hints', () => {
         assert.deepEqual(MeetingUtil.getUserDisplayHintsFromLocusInfo(), []);
 
-        assert.deepEqual(
-          MeetingUtil.getUserDisplayHintsFromLocusInfo({}),
-          []
-        );
+        assert.deepEqual(MeetingUtil.getUserDisplayHintsFromLocusInfo({}), []);
 
-        assert.deepEqual(
-          MeetingUtil.getUserDisplayHintsFromLocusInfo({parsedLocus: {}}),
-          []
-        );
+        assert.deepEqual(MeetingUtil.getUserDisplayHintsFromLocusInfo({parsedLocus: {}}), []);
 
         assert.deepEqual(
           MeetingUtil.getUserDisplayHintsFromLocusInfo({parsedLocus: {info: {}}}),
@@ -195,7 +198,9 @@ describe('plugin-meetings', () => {
         );
 
         assert.deepEqual(
-          MeetingUtil.getUserDisplayHintsFromLocusInfo({parsedLocus: {info: {userDisplayHints: []}}}),
+          MeetingUtil.getUserDisplayHintsFromLocusInfo({
+            parsedLocus: {info: {userDisplayHints: []}},
+          }),
           []
         );
 
@@ -203,11 +208,9 @@ describe('plugin-meetings', () => {
           MeetingUtil.getUserDisplayHintsFromLocusInfo({
             parsedLocus: {
               info: {
-                userDisplayHints: [
-                  'HINT_1'
-                ]
-              }
-            }
+                userDisplayHints: ['HINT_1'],
+              },
+            },
           }),
           ['HINT_1']
         );
@@ -244,23 +247,38 @@ describe('plugin-meetings', () => {
 
     describe('canUserLowerSomeoneElsesHand', () => {
       it('works as expected', () => {
-        assert.deepEqual(MeetingUtil.canUserLowerSomeoneElsesHand(['LOWER_SOMEONE_ELSES_HAND']), true);
+        assert.deepEqual(
+          MeetingUtil.canUserLowerSomeoneElsesHand(['LOWER_SOMEONE_ELSES_HAND']),
+          true
+        );
         assert.deepEqual(MeetingUtil.canUserLowerSomeoneElsesHand([]), false);
       });
     });
 
     describe('bothLeaveAndEndMeetingAvailable', () => {
       it('works as expected', () => {
-        assert.deepEqual(MeetingUtil.bothLeaveAndEndMeetingAvailable(['LEAVE_TRANSFER_HOST_END_MEETING']), true);
+        assert.deepEqual(
+          MeetingUtil.bothLeaveAndEndMeetingAvailable(['LEAVE_TRANSFER_HOST_END_MEETING']),
+          true
+        );
         assert.deepEqual(MeetingUtil.bothLeaveAndEndMeetingAvailable(['LEAVE_END_MEETING']), true);
-        assert.deepEqual(MeetingUtil.bothLeaveAndEndMeetingAvailable(['LEAVE_TRANSFER_HOST_END_MEETING', 'LEAVE_END_MEETING']), true);
+        assert.deepEqual(
+          MeetingUtil.bothLeaveAndEndMeetingAvailable([
+            'LEAVE_TRANSFER_HOST_END_MEETING',
+            'LEAVE_END_MEETING',
+          ]),
+          true
+        );
         assert.deepEqual(MeetingUtil.bothLeaveAndEndMeetingAvailable([]), false);
       });
     });
 
     describe('canUserLock', () => {
       it('works as expected', () => {
-        assert.deepEqual(MeetingUtil.canUserLock(['LOCK_CONTROL_LOCK', 'LOCK_STATUS_UNLOCKED']), true);
+        assert.deepEqual(
+          MeetingUtil.canUserLock(['LOCK_CONTROL_LOCK', 'LOCK_STATUS_UNLOCKED']),
+          true
+        );
         assert.deepEqual(MeetingUtil.canUserLock(['LOCK_CONTROL_LOCK']), false);
         assert.deepEqual(MeetingUtil.canUserLock(['LOCK_STATUS_UNLOCKED']), false);
         assert.deepEqual(MeetingUtil.canUserLock([]), false);
@@ -269,7 +287,10 @@ describe('plugin-meetings', () => {
 
     describe('canUserUnlock', () => {
       it('works as expected', () => {
-        assert.deepEqual(MeetingUtil.canUserUnlock(['LOCK_CONTROL_UNLOCK', 'LOCK_STATUS_LOCKED']), true);
+        assert.deepEqual(
+          MeetingUtil.canUserUnlock(['LOCK_CONTROL_UNLOCK', 'LOCK_STATUS_LOCKED']),
+          true
+        );
         assert.deepEqual(MeetingUtil.canUserUnlock(['LOCK_CONTROL_UNLOCK']), false);
         assert.deepEqual(MeetingUtil.canUserUnlock(['LOCK_STATUS_LOCKED']), false);
         assert.deepEqual(MeetingUtil.canUserUnlock([]), false);
@@ -297,7 +318,6 @@ describe('plugin-meetings', () => {
       });
     });
 
-
     describe('canUserStop', () => {
       it('works as expected', () => {
         assert.deepEqual(MeetingUtil.canUserStop(['RECORDING_CONTROL_STOP']), true);
@@ -324,7 +344,6 @@ describe('plugin-meetings', () => {
       });
     });
 
-
     describe('recording tests', () => {
       let request;
       let locusInfo;
@@ -334,14 +353,12 @@ describe('plugin-meetings', () => {
         locusInfo = {
           parsedLocus: {
             info: {
-              userDisplayHints: [
-                'RECORDING_CONTROL_START'
-              ]
-            }
-          }
+              userDisplayHints: ['RECORDING_CONTROL_START'],
+            },
+          },
         };
         request = {
-          recordMeeting: sinon.stub().returns(Promise.resolve())
+          recordMeeting: sinon.stub().returns(Promise.resolve()),
         };
       });