@webex/plugin-meetings 2.36.1 → 2.37.1

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

@@ -10,24 +10,27 @@ describe('plugin-meetings', () => {
           sites: [
             {
               siteUrl: 'site1-example.webex.com',
-              default: false
+              default: false,
             },
             {
               siteUrl: 'site2-example.webex.com',
-              default: false
+              default: false,
             },
             {
               siteUrl: 'go.webex.com',
-              default: true
+              default: true,
             },
             {
               siteUrl: 'site3-example.webex.com',
-              default: false
-            }
-          ]
+              default: false,
+            },
+          ],
         };
 
-        assert.equal(MeetingsUtil.parseDefaultSiteFromMeetingPreferences(userPreferences), 'go.webex.com');
+        assert.equal(
+          MeetingsUtil.parseDefaultSiteFromMeetingPreferences(userPreferences),
+          'go.webex.com'
+        );
       });
 
       it('should work fine if no default true site', () => {
@@ -35,16 +38,16 @@ describe('plugin-meetings', () => {
           sites: [
             {
               siteUrl: 'site1-example.webex.com',
-              default: false
+              default: false,
             },
             {
               siteUrl: 'site2-example.webex.com',
-              default: false
+              default: false,
             },
             {
               siteUrl: 'site3-example.webex.com',
-              default: false
-            }
+              default: false,
+            },
           ],
         };
 
@@ -61,4 +64,3 @@ describe('plugin-meetings', () => {
     });
   });
 });
-

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

@@ -4,7 +4,6 @@ import {assert} from '@webex/test-helper-chai';
 import MemberUtil from '@webex/plugin-meetings/src/member/util';
 import Member from '@webex/plugin-meetings/src/member';
 
-
 describe('member', () => {
   afterEach(() => {
     sinon.restore();

package/test/unit/spec/member/util.js

@@ -2,7 +2,6 @@ import {assert} from '@webex/test-helper-chai';
 
 import MemberUtil from '@webex/plugin-meetings/src/member/util';
 
-
 describe('isHandRaised', () => {
   it('throws error when there is no participant', () => {
     assert.throws(() => {
@@ -18,7 +17,7 @@ describe('isHandRaised', () => {
 
   it('returns false when hand is not there in controls', () => {
     const participant = {
-      controls: {}
+      controls: {},
     };
 
     assert.isFalse(MemberUtil.isHandRaised(participant));
@@ -28,9 +27,9 @@ describe('isHandRaised', () => {
     const participant = {
       controls: {
         hand: {
-          raised: true
+          raised: true,
         },
-      }
+      },
     };
 
     assert.isTrue(MemberUtil.isHandRaised(participant));
@@ -40,9 +39,9 @@ describe('isHandRaised', () => {
     const participant = {
       controls: {
         hand: {
-          raised: false
+          raised: false,
         },
-      }
+      },
     };
 
     assert.isFalse(MemberUtil.isHandRaised(participant));

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

@@ -21,7 +21,6 @@ const {assert} = chai;
 chai.use(chaiAsPromised);
 sinon.assert.expose(chai.assert, {prefix: ''});
 
-
 describe('plugin-meetings', () => {
   let webex;
   let url1;
@@ -35,37 +34,33 @@ describe('plugin-meetings', () => {
           id: '6eb08f8b-bf69-3251-a126-b161bead2d21',
           phoneNumber: '+18578675309',
           isExternal: true,
-          primaryDisplayString: '+18578675309'
+          primaryDisplayString: '+18578675309',
         },
         devices: [
           {
             url: 'https://fakeURL.com',
             deviceType: 'SIP',
             state: 'JOINED',
-            intents: [
-              null
-            ],
+            intents: [null],
             correlationId: '1234',
             provisionalUrl: 'dialout:///fake',
-            isSparkPstn: true
+            isSparkPstn: true,
           },
           {
             url: 'dialout:///fakeagain',
             deviceType: 'PROVISIONAL',
             state: 'JOINED',
-            intents: [
-              null
-            ],
+            intents: [null],
             correlationId: '4321',
             isVideoCallback: false,
             clientUrl: 'https://fakeURL',
             provisionalType: 'DIAL_OUT_ONLY',
-            dialingStatus: 'SUCCESS'
-          }
+            dialingStatus: 'SUCCESS',
+          },
         ],
         status: {
           audioStatus: 'SENDRECV',
-          videoStatus: 'INACTIVE'
+          videoStatus: 'INACTIVE',
         },
         id: 'abc-123-abc-123',
         guest: true,
@@ -73,16 +68,15 @@ describe('plugin-meetings', () => {
         moderator: false,
         panelist: false,
         moveToLobbyNotAllowed: true,
-        deviceUrl: 'https://fakeDeviceurl'
+        deviceUrl: 'https://fakeDeviceurl',
       },
       id: 'abc-123-abc-123',
       status: 'IN_MEETING',
       type: 'MEETING',
-      isModerator: false
-    }
+      isModerator: false,
+    },
   };
 
-
   describe('members', () => {
     const sandbox = sinon.createSandbox();
     let createMembers;
@@ -92,21 +86,21 @@ describe('plugin-meetings', () => {
         children: {
           meetings: Meetings,
           credentials: Credentials,
-          support: Support
+          support: Support,
         },
         config: {
           credentials: {
-            client_id: 'mock-client-id'
+            client_id: 'mock-client-id',
           },
           meetings: {
             reconnection: {
-              enabled: false
+              enabled: false,
             },
             mediaSettings: {},
             metrics: {},
-            stats: {}
-          }
-        }
+            stats: {},
+          },
+        },
       });
 
       url1 = `https://example.com/${uuid.v4()}`;
@@ -156,7 +150,13 @@ describe('plugin-meetings', () => {
 
         members.membersCollection.setAll(fakeMembersCollection);
         await members.sendDialPadKey('1', 'test1');
-        assert.calledWith(MembersUtil.genderateSendDTMFOptions, 'https://fakeURL.com', '1', 'test1', url1);
+        assert.calledWith(
+          MembersUtil.genderateSendDTMFOptions,
+          'https://fakeURL.com',
+          '1',
+          'test1',
+          url1
+        );
       });
 
       it('should call the sendDialPadKey method on membersRequest if the member is valid', async () => {
@@ -197,7 +197,10 @@ describe('plugin-meetings', () => {
         const members = createMembers({url: locusUrl});
 
         const spies = {
-          generateRaiseHandMemberOptions: sandbox.spy(MembersUtil, 'generateRaiseHandMemberOptions'),
+          generateRaiseHandMemberOptions: sandbox.spy(
+            MembersUtil,
+            'generateRaiseHandMemberOptions'
+          ),
           raiseOrLowerHandMember: sandbox.spy(members.membersRequest, 'raiseOrLowerHandMember'),
         };
 
@@ -210,10 +213,25 @@ describe('plugin-meetings', () => {
         assert.notCalled(spies.raiseOrLowerHandMember);
       };
 
-      const checkValid = async (resultPromise, spies, expectedMemberId, expectedRaise, expectedLocusUrl) => {
+      const checkValid = async (
+        resultPromise,
+        spies,
+        expectedMemberId,
+        expectedRaise,
+        expectedLocusUrl
+      ) => {
         await assert.isFulfilled(resultPromise);
-        assert.calledOnceWithExactly(spies.generateRaiseHandMemberOptions, expectedMemberId, expectedRaise, expectedLocusUrl);
-        assert.calledOnceWithExactly(spies.raiseOrLowerHandMember, {memberId: expectedMemberId, raised: expectedRaise, locusUrl: expectedLocusUrl});
+        assert.calledOnceWithExactly(
+          spies.generateRaiseHandMemberOptions,
+          expectedMemberId,
+          expectedRaise,
+          expectedLocusUrl
+        );
+        assert.calledOnceWithExactly(spies.raiseOrLowerHandMember, {
+          memberId: expectedMemberId,
+          raised: expectedRaise,
+          locusUrl: expectedLocusUrl,
+        });
         assert.strictEqual(resultPromise, spies.raiseOrLowerHandMember.getCall(0).returnValue);
       };
 
@@ -222,7 +240,11 @@ describe('plugin-meetings', () => {
 
         const resultPromise = members.raiseOrLowerHand();
 
-        await checkInvalid(resultPromise, 'The member id must be defined to raise/lower the hand of the member.', spies);
+        await checkInvalid(
+          resultPromise,
+          'The member id must be defined to raise/lower the hand of the member.',
+          spies
+        );
       });
 
       it('should not make a request if there is no locus url', async () => {
@@ -230,7 +252,11 @@ describe('plugin-meetings', () => {
 
         const resultPromise = members.raiseOrLowerHand(uuid.v4());
 
-        await checkInvalid(resultPromise, 'The associated locus url for this meetings members object must be defined.', spies);
+        await checkInvalid(
+          resultPromise,
+          'The associated locus url for this meetings members object must be defined.',
+          spies
+        );
       });
 
       it('should make the correct request when called with raise as true', async () => {
@@ -266,7 +292,10 @@ describe('plugin-meetings', () => {
         const members = createMembers({url: locusUrl});
 
         const spies = {
-          generateLowerAllHandsMemberOptions: sandbox.spy(MembersUtil, 'generateLowerAllHandsMemberOptions'),
+          generateLowerAllHandsMemberOptions: sandbox.spy(
+            MembersUtil,
+            'generateLowerAllHandsMemberOptions'
+          ),
           lowerAllHandsMember: sandbox.spy(members.membersRequest, 'lowerAllHandsMember'),
         };
 
@@ -279,10 +308,22 @@ describe('plugin-meetings', () => {
         assert.notCalled(spies.lowerAllHandsMember);
       };
 
-      const checkValid = async (resultPromise, spies, expectedRequestingMemberId, expectedLocusUrl) => {
+      const checkValid = async (
+        resultPromise,
+        spies,
+        expectedRequestingMemberId,
+        expectedLocusUrl
+      ) => {
         await assert.isFulfilled(resultPromise);
-        assert.calledOnceWithExactly(spies.generateLowerAllHandsMemberOptions, expectedRequestingMemberId, expectedLocusUrl);
-        assert.calledOnceWithExactly(spies.lowerAllHandsMember, {requestingParticipantId: expectedRequestingMemberId, locusUrl: expectedLocusUrl});
+        assert.calledOnceWithExactly(
+          spies.generateLowerAllHandsMemberOptions,
+          expectedRequestingMemberId,
+          expectedLocusUrl
+        );
+        assert.calledOnceWithExactly(spies.lowerAllHandsMember, {
+          requestingParticipantId: expectedRequestingMemberId,
+          locusUrl: expectedLocusUrl,
+        });
         assert.strictEqual(resultPromise, spies.lowerAllHandsMember.getCall(0).returnValue);
       };
 
@@ -291,7 +332,11 @@ describe('plugin-meetings', () => {
 
         const resultPromise = members.lowerAllHands();
 
-        await checkInvalid(resultPromise, 'The requestingMemberId must be defined to lower all hands in a meeting.', spies);
+        await checkInvalid(
+          resultPromise,
+          'The requestingMemberId must be defined to lower all hands in a meeting.',
+          spies
+        );
       });
 
       it('should not make a request if there is no locus url', async () => {
@@ -299,7 +344,11 @@ describe('plugin-meetings', () => {
 
         const resultPromise = members.lowerAllHands(uuid.v4());
 
-        await checkInvalid(resultPromise, 'The associated locus url for this meetings members object must be defined.', spies);
+        await checkInvalid(
+          resultPromise,
+          'The associated locus url for this meetings members object must be defined.',
+          spies
+        );
       });
 
       it('should make the correct request when called with requestingMemberId', async () => {

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

@@ -22,17 +22,20 @@ describe('plugin-meetings', () => {
   beforeEach(() => {
     const webex = new MockWebex({
       children: {
-        meetings: Meetings
-      }
+        meetings: Meetings,
+      },
     });
 
     sandbox = sinon.createSandbox();
 
     url1 = `https://example.com/${uuid.v4()}`;
 
-    membersRequest = new MembersRequest({}, {
-      parent: webex
-    });
+    membersRequest = new MembersRequest(
+      {},
+      {
+        parent: webex,
+      }
+    );
     membersRequest.request = sinon.mock().returns(Promise.resolve({}));
   });
 
@@ -40,7 +43,6 @@ describe('plugin-meetings', () => {
     sandbox.restore();
   });
 
-
   describe('members request library', () => {
     describe('#sendDialPadKey', () => {
       it('sends a POST to the sendDtmf locus endpoint', async () => {
@@ -53,7 +55,7 @@ describe('plugin-meetings', () => {
           url,
           tones,
           memberId,
-          locusUrl
+          locusUrl,
         });
         const requestParams = membersRequest.request.getCall(0).args[0];
 
@@ -68,9 +70,9 @@ describe('plugin-meetings', () => {
       it('sends a PUT to the locus endpoint', async () => {
         const options = {
           invitee: {
-            phoneNumber: '+18578675309'
+            phoneNumber: '+18578675309',
           },
-          locusUrl: url1
+          locusUrl: url1,
         };
 
         await membersRequest.addMembers(options);
@@ -86,9 +88,9 @@ describe('plugin-meetings', () => {
       it('sends a PUT to the locus endpoint', async () => {
         const options = {
           invitee: {
-            phoneNumber: '+18578675309'
+            phoneNumber: '+18578675309',
           },
-          locusUrl: url1
+          locusUrl: url1,
         };
 
         await membersRequest.cancelPhoneInvite(options);
@@ -109,7 +111,7 @@ describe('plugin-meetings', () => {
         const options = {
           memberId,
           locusUrl,
-          raised: true
+          raised: true,
         };
 
         await membersRequest.raiseOrLowerHandMember(options);
@@ -122,10 +124,15 @@ describe('plugin-meetings', () => {
     });
 
     describe('#lowerAllHands', () => {
-      const parameterErrorMessage = 'requestingParticipantId must be defined, and the associated locus url for this meeting object must be defined.';
+      const parameterErrorMessage =
+        'requestingParticipantId must be defined, and the associated locus url for this meeting object must be defined.';
 
       const checkInvalid = async (functionParams) => {
-        assert.throws(() => membersRequest.lowerAllHandsMember(functionParams), ParameterError, parameterErrorMessage);
+        assert.throws(
+          () => membersRequest.lowerAllHandsMember(functionParams),
+          ParameterError,
+          parameterErrorMessage
+        );
         assert(membersRequest.request.notCalled);
         assert(membersUtil.getLowerAllHandsMemberRequestParams.notCalled);
       };
@@ -151,7 +158,10 @@ describe('plugin-meetings', () => {
           locusUrl,
         };
 
-        assert.strictEqual(membersRequest.lowerAllHandsMember(options), membersRequest.request.getCall(0).returnValue);
+        assert.strictEqual(
+          membersRequest.lowerAllHandsMember(options),
+          membersRequest.request.getCall(0).returnValue
+        );
       });
 
       it('sends a PATCH to the locus endpoint', async () => {
@@ -163,14 +173,13 @@ describe('plugin-meetings', () => {
           locusUrl,
         };
 
-
         const getRequestParamsSpy = sandbox.spy(membersUtil, 'getLowerAllHandsMemberRequestParams');
 
         await membersRequest.lowerAllHandsMember(options);
 
         assert.calledOnceWithExactly(getRequestParamsSpy, {
           requestingParticipantId: memberId,
-          locusUrl: url1
+          locusUrl: url1,
         });
 
         const requestParams = membersRequest.request.getCall(0).args[0];
@@ -180,10 +189,10 @@ describe('plugin-meetings', () => {
           uri: `${locusUrl}/controls`,
           body: {
             hand: {
-              raised: false
+              raised: false,
             },
-            requestingParticipantId: memberId
-          }
+            requestingParticipantId: memberId,
+          },
         });
       });
     });

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

@@ -20,7 +20,7 @@ describe('plugin-meetings', () => {
         assert.deepEqual(MembersUtil.generateRaiseHandMemberOptions(memberId, status, locusUrl), {
           memberId,
           raised: status,
-          locusUrl
+          locusUrl,
         });
       });
     });
@@ -29,10 +29,13 @@ describe('plugin-meetings', () => {
         const requestingParticipantId = 'test';
         const locusUrl = 'urlTest1';
 
-        assert.deepEqual(MembersUtil.generateLowerAllHandsMemberOptions(requestingParticipantId, locusUrl), {
-          requestingParticipantId,
-          locusUrl
-        });
+        assert.deepEqual(
+          MembersUtil.generateLowerAllHandsMemberOptions(requestingParticipantId, locusUrl),
+          {
+            requestingParticipantId,
+            locusUrl,
+          }
+        );
       });
     });
   });

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

@@ -10,7 +10,6 @@ import {assert} from '@webex/test-helper-chai';
 
 import metrics from '@webex/plugin-meetings/src/metrics';
 
-
 /**
  * Meeting can only run in a browser, so we can only send metrics for
  * browser usage.
@@ -23,7 +22,7 @@ browserOnly(describe)('Meeting metrics', () => {
     clientRegion: 'US-WEST',
     countryCode: 'US',
     regionCode: 'US-WEST',
-    timezone: 'America/Los_Angeles'
+    timezone: 'America/Los_Angeles',
   };
 
   beforeEach(() => {
@@ -31,16 +30,16 @@ browserOnly(describe)('Meeting metrics', () => {
     mockSubmitMetric = sandbox.stub();
     webex = new MockWebex({
       children: {
-        metrics: Metrics
-      }
+        metrics: Metrics,
+      },
     });
 
     webex.version = '1.2.3';
     webex.credentials.getOrgId = sinon.fake.returns('7890');
     webex.credentials.config = {
       _values: {
-        clientId: 'mock-client-id'
-      }
+        clientId: 'mock-client-id',
+      },
     };
     webex.config.metrics.type = ['behavioral'];
     webex.internal.metrics.submitClientMetrics = mockSubmitMetric;
@@ -55,7 +54,7 @@ browserOnly(describe)('Meeting metrics', () => {
     it('should create payload with masked IPv4', () => {
       geoHintInfo.clientAddress = '128.0.0.1';
       webex.meetings = {
-        geoHintInfo
+        geoHintInfo,
       };
       metrics.initialSetup({}, webex);
       const payload = metrics.initPayload('myMetric', {}, {clientType: 'TEAMS_CLIENT'});
@@ -71,7 +70,7 @@ browserOnly(describe)('Meeting metrics', () => {
     it('should create payload with masked IPv6', () => {
       geoHintInfo.clientAddress = '2001:0db8:0000:08d3:0000:0000:0070:0000';
       webex.meetings = {
-        geoHintInfo
+        geoHintInfo,
       };
       metrics.initialSetup({}, webex);
       const payload = metrics.initPayload('myIPv6Metric', {}, {clientType: 'TEAMS_CLIENT'});
@@ -98,19 +97,15 @@ browserOnly(describe)('Meeting metrics', () => {
 
       metrics.sendBehavioralMetric('myMetric', fields, metricTags);
 
-      assert.calledWithMatch(
-        mockSubmitMetric,
-        'myMetric',
-        {
-          type: ['behavioral'],
-          fields: {
-            value: 567
-          },
-          tags: {
-            test: true
-          }
-        }
-      );
+      assert.calledWithMatch(mockSubmitMetric, 'myMetric', {
+        type: ['behavioral'],
+        fields: {
+          value: 567,
+        },
+        tags: {
+          test: true,
+        },
+      });
     });
   });
 });

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

@@ -20,7 +20,7 @@ describe('plugin-meetings', () => {
     const initialConfig = {
       videoPacketLossRatioThreshold: 9,
       rttThreshold: 500,
-      jitterThreshold: 500
+      jitterThreshold: 500,
     };
 
     const configObject = {
@@ -39,20 +39,20 @@ describe('plugin-meetings', () => {
         roundTripTime: 0.648,
         fractionLost: 0,
         totalRoundTripTime: 3.554,
-        roundTripTimeMeasurements: 14
+        roundTripTimeMeasurements: 14,
       },
       statsAnalyzerCurrentStats: {
         audio: {
           send: {
-            currentPacketLossRatio: 8
-          }
+            currentPacketLossRatio: 8,
+          },
         },
         video: {
           send: {
-            currentPacketLossRatio: 10
-          }
-        }
-      }
+            currentPacketLossRatio: 10,
+          },
+        },
+      },
     };
 
     const sandbox = sinon.createSandbox();
@@ -75,13 +75,19 @@ describe('plugin-meetings', () => {
 
     it('should emit a network quality judgement event with the proper payload', async () => {
       await networkQualityMonitor.determineUplinkNetworkQuality(configObject);
-      assert(sandBoxEmitSpy.calledWith(sinon.match({
-        file: 'networkQualityMonitor',
-        function: 'emitNetworkQuality'
-      }), sinon.match(EVENT_TRIGGERS.NETWORK_QUALITY), sinon.match({
-        mediaType: 'video',
-        networkQualityScore: 0
-      })));
+      assert(
+        sandBoxEmitSpy.calledWith(
+          sinon.match({
+            file: 'networkQualityMonitor',
+            function: 'emitNetworkQuality',
+          }),
+          sinon.match(EVENT_TRIGGERS.NETWORK_QUALITY),
+          sinon.match({
+            mediaType: 'video',
+            networkQualityScore: 0,
+          })
+        )
+      );
     });
 
     it('should reset to default values after determineUplinkNetworkQuality call stack is complete', async () => {