@webex/plugin-meetings 2.60.0-next.1 → 2.60.0-next.2

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

@@ -3,6 +3,10 @@
  */
 import 'jsdom-global/register';
 
+// Polyfill for crypto: https://github.com/jsdom/jsdom/issues/1612#issuecomment-663210638
+import {Crypto} from '@peculiar/webcrypto';
+global.crypto = new Crypto();
+
 import Device from '@webex/internal-plugin-device';
 import Mercury from '@webex/internal-plugin-mercury';
 import {assert} from '@webex/test-helper-chai';
@@ -32,10 +36,10 @@ import {
   EVENT_TRIGGERS,
 } from '../../../../src/constants';
 import CaptchaError from '@webex/plugin-meetings/src/common/errors/captcha-error';
-import { forEach } from 'lodash';
+import {forEach} from 'lodash';
 import PasswordError from '@webex/plugin-meetings/src/common/errors/password-error';
 import PermissionError from '@webex/plugin-meetings/src/common/errors/permission';
-import {NoiseReductionEffect,VirtualBackgroundEffect} from '@webex/media-helpers';
+import {NoiseReductionEffect, VirtualBackgroundEffect} from '@webex/media-helpers';
 import NoMeetingInfoError from '../../../../src/common/errors/no-meeting-info';
 
 describe('plugin-meetings', () => {
@@ -333,10 +337,10 @@ describe('plugin-meetings', () => {
         beforeEach(() => {
           webex.credentials = {
             supertoken: {
-              access_token: "fake_token"
-            }
+              access_token: 'fake_token',
+            },
           };
-        })
+        });
 
         it('creates background effect', async () => {
           const result = await webex.meetings.createVirtualBackgroundEffect();
@@ -350,7 +354,7 @@ describe('plugin-meetings', () => {
             generator: 'worker',
             quality: 'LOW',
             authToken: 'fake_token',
-            mirror: false
+            mirror: false,
           });
           assert.exists(result.enable);
           assert.exists(result.disable);
@@ -359,13 +363,13 @@ describe('plugin-meetings', () => {
 
         it('creates background effect with custom options passed', async () => {
           const effectOptions = {
-            generator: "local",
+            generator: 'local',
             frameRate: 45,
-            mode: "IMAGE",
+            mode: 'IMAGE',
             mirror: false,
-            quality: "HIGH",
-            blurStrength: "STRONG",
-            bgImageUrl: "https://test.webex.com/landscape.5a535788.jpg",
+            quality: 'HIGH',
+            blurStrength: 'STRONG',
+            bgImageUrl: 'https://test.webex.com/landscape.5a535788.jpg',
           };
 
           const result = await webex.meetings.createVirtualBackgroundEffect(effectOptions);
@@ -373,21 +377,21 @@ describe('plugin-meetings', () => {
           assert.exists(result);
           assert.instanceOf(result, VirtualBackgroundEffect);
           assert.containsAllKeys(result, ['loadModel', 'isEnabled', 'options']);
-          assert.deepEqual(result.options, {...effectOptions, authToken: "fake_token"});
+          assert.deepEqual(result.options, {...effectOptions, authToken: 'fake_token'});
           assert.exists(result.enable);
           assert.exists(result.disable);
           assert.exists(result.dispose);
         });
-      })
+      });
 
       describe('noise reduction effect', () => {
         beforeEach(() => {
           webex.credentials = {
             supertoken: {
-              access_token: "fake_token"
-            }
+              access_token: 'fake_token',
+            },
           };
-        })
+        });
 
         it('creates noise reduction effect', async () => {
           const result = await webex.meetings.createNoiseReductionEffect({audioContext: {}});
@@ -397,7 +401,7 @@ describe('plugin-meetings', () => {
           assert.containsAllKeys(result, ['audioContext', 'isEnabled', 'isReady', 'options']);
           assert.deepEqual(result.options, {
             authToken: 'fake_token',
-            audioContext: {}
+            audioContext: {},
           });
           assert.exists(result.enable);
           assert.exists(result.disable);
@@ -407,8 +411,8 @@ describe('plugin-meetings', () => {
         it('creates noise reduction effect with custom options passed', async () => {
           const effectOptions = {
             audioContext: {},
-            mode: "WORKLET",
-            env: "prod"
+            mode: 'WORKLET',
+            env: 'prod',
           };
 
           const result = await webex.meetings.createNoiseReductionEffect(effectOptions);
@@ -416,12 +420,12 @@ describe('plugin-meetings', () => {
           assert.exists(result);
           assert.instanceOf(result, NoiseReductionEffect);
           assert.containsAllKeys(result, ['audioContext', 'isEnabled', 'isReady', 'options']);
-          assert.deepEqual(result.options, {...effectOptions, authToken: "fake_token"});
+          assert.deepEqual(result.options, {...effectOptions, authToken: 'fake_token'});
           assert.exists(result.enable);
           assert.exists(result.disable);
           assert.exists(result.dispose);
         });
-      })
+      });
 
       describe('gets', () => {
         describe('#getReachability', () => {
@@ -431,10 +435,7 @@ describe('plugin-meetings', () => {
           it('gets the reachability data instance from webex.meetings', () => {
             const reachability = webex.meetings.getReachability();
 
-            assert.exists(
-              reachability,
-              'reachability is defined'
-            );
+            assert.exists(reachability, 'reachability is defined');
             assert.instanceOf(reachability, Reachability, 'should be a reachability instance');
           });
         });
@@ -510,7 +511,6 @@ describe('plugin-meetings', () => {
             );
           });
           describe('when meeting is returned', () => {
-
             beforeEach(() => {
               webex.meetings.meetingCollection.getByKey = sinon.stub().returns({
                 locusInfo,
@@ -645,31 +645,56 @@ describe('plugin-meetings', () => {
           await create;
           assert.calledOnce(webex.meetings.createMeeting);
           assert.calledWith(webex.meetings.createMeeting, ...createMeetingParameters);
-        } 
+        };
 
         it('calls createMeeting and returns its promise', async () => {
-          checkCallCreateMeeting([test1, test2, FAKE_USE_RANDOM_DELAY, {}, correlationId, true], [test1, test2, FAKE_USE_RANDOM_DELAY, {}, correlationId, true]);
+          checkCallCreateMeeting(
+            [test1, test2, FAKE_USE_RANDOM_DELAY, {}, correlationId, true],
+            [test1, test2, FAKE_USE_RANDOM_DELAY, {}, correlationId, true]
+          );
         });
 
         it('calls createMeeting when failOnMissingMeetinginfo is undefined and returns its promise', async () => {
-          checkCallCreateMeeting([test1, test2, FAKE_USE_RANDOM_DELAY, {}, correlationId, undefined], [test1, test2, FAKE_USE_RANDOM_DELAY, {}, correlationId, false]);
+          checkCallCreateMeeting(
+            [test1, test2, FAKE_USE_RANDOM_DELAY, {}, correlationId, undefined],
+            [test1, test2, FAKE_USE_RANDOM_DELAY, {}, correlationId, false]
+          );
         });
 
         it('calls createMeeting when failOnMissingMeetinginfo is false and returns its promise', async () => {
-          checkCallCreateMeeting([test1, test2, FAKE_USE_RANDOM_DELAY, {}, correlationId, false], [test1, test2, FAKE_USE_RANDOM_DELAY, {}, correlationId, false]);
+          checkCallCreateMeeting(
+            [test1, test2, FAKE_USE_RANDOM_DELAY, {}, correlationId, false],
+            [test1, test2, FAKE_USE_RANDOM_DELAY, {}, correlationId, false]
+          );
         });
 
         it('calls createMeeting with extra info params and returns its promise', async () => {
           const FAKE_USE_RANDOM_DELAY = false;
           const correlationId = 'my-correlationId';
 
-          const FAKE_INFO_EXTRA_PARAMS = {mtid: 'm9fe0afd8c435e892afcce9ea25b97046', joinTXId: 'TSmrX61wNF'};
-          const create = webex.meetings.create(test1, test2, FAKE_USE_RANDOM_DELAY, FAKE_INFO_EXTRA_PARAMS, correlationId);
+          const FAKE_INFO_EXTRA_PARAMS = {
+            mtid: 'm9fe0afd8c435e892afcce9ea25b97046',
+            joinTXId: 'TSmrX61wNF',
+          };
+          const create = webex.meetings.create(
+            test1,
+            test2,
+            FAKE_USE_RANDOM_DELAY,
+            FAKE_INFO_EXTRA_PARAMS,
+            correlationId
+          );
 
           assert.exists(create.then);
           await create;
           assert.calledOnce(webex.meetings.createMeeting);
-          assert.calledWith(webex.meetings.createMeeting, test1, test2, FAKE_USE_RANDOM_DELAY, FAKE_INFO_EXTRA_PARAMS, correlationId);
+          assert.calledWith(
+            webex.meetings.createMeeting,
+            test1,
+            test2,
+            FAKE_USE_RANDOM_DELAY,
+            FAKE_INFO_EXTRA_PARAMS,
+            correlationId
+          );
         });
 
         it('creates a new meeting when a scheduled meeting exists in the conversation', async () => {
@@ -765,7 +790,6 @@ describe('plugin-meetings', () => {
       });
       describe('#handleLocusEvent', () => {
         describe('there was a meeting', () => {
-
           beforeEach(() => {
             webex.meetings.meetingCollection.getByKey = sinon.stub().returns({
               locusInfo,
@@ -831,7 +855,7 @@ describe('plugin-meetings', () => {
                   },
                 },
                 info: {
-                  webExMeetingId
+                  webExMeetingId,
                 },
               },
               eventType: 'locus.difference',
@@ -839,7 +863,11 @@ describe('plugin-meetings', () => {
             });
             assert.callCount(webex.meetings.meetingCollection.getByKey, 6);
             assert.calledWith(webex.meetings.meetingCollection.getByKey, 'locusUrl', url1);
-            assert.calledWith(webex.meetings.meetingCollection.getByKey, 'meetingNumber', webExMeetingId);
+            assert.calledWith(
+              webex.meetings.meetingCollection.getByKey,
+              'meetingNumber',
+              webExMeetingId
+            );
             assert.calledOnce(initialSetup);
             assert.calledWith(initialSetup, {
               id: uuid1,
@@ -854,7 +882,7 @@ describe('plugin-meetings', () => {
                 },
               },
               info: {
-                webExMeetingId
+                webExMeetingId,
               },
             });
           });
@@ -868,7 +896,7 @@ describe('plugin-meetings', () => {
                   },
                 },
                 info: {
-                  webExMeetingId
+                  webExMeetingId,
                 },
               },
               eventType: 'locus.difference',
@@ -876,7 +904,11 @@ describe('plugin-meetings', () => {
             });
             assert.callCount(webex.meetings.meetingCollection.getByKey, 5);
             assert.calledWith(webex.meetings.meetingCollection.getByKey, 'locusUrl', url1);
-            assert.calledWith(webex.meetings.meetingCollection.getByKey, 'meetingNumber', webExMeetingId);
+            assert.calledWith(
+              webex.meetings.meetingCollection.getByKey,
+              'meetingNumber',
+              webExMeetingId
+            );
             assert.calledOnce(initialSetup);
             assert.calledWith(initialSetup, {
               id: uuid1,
@@ -886,7 +918,7 @@ describe('plugin-meetings', () => {
                 },
               },
               info: {
-                webExMeetingId
+                webExMeetingId,
               },
             });
           });
@@ -933,7 +965,7 @@ describe('plugin-meetings', () => {
                   },
                 },
                 info: {
-                  webExMeetingId
+                  webExMeetingId,
                 },
               },
               eventType: test1,
@@ -941,7 +973,11 @@ describe('plugin-meetings', () => {
             });
             assert.callCount(webex.meetings.meetingCollection.getByKey, 5);
             assert.calledWith(webex.meetings.meetingCollection.getByKey, 'locusUrl', url1);
-            assert.calledWith(webex.meetings.meetingCollection.getByKey, 'meetingNumber', webExMeetingId);
+            assert.calledWith(
+              webex.meetings.meetingCollection.getByKey,
+              'meetingNumber',
+              webExMeetingId
+            );
             assert.calledOnce(initialSetup);
             assert.calledWith(initialSetup, {
               id: uuid1,
@@ -951,7 +987,7 @@ describe('plugin-meetings', () => {
                 },
               },
               info: {
-                webExMeetingId
+                webExMeetingId,
               },
             });
           });
@@ -1023,7 +1059,10 @@ describe('plugin-meetings', () => {
         });
         describe('successful MeetingInfo.#fetchMeetingInfo', () => {
           let clock, setTimeoutSpy, fakeMeetingStartTimeString, FAKE_TIME_TO_START;
-          const FAKE_INFO_EXTRA_PARAMS = {mtid: 'm9fe0afd8c435e892afcce9ea25b97046', joinTXId: 'TSmrX61wNF'};
+          const FAKE_INFO_EXTRA_PARAMS = {
+            mtid: 'm9fe0afd8c435e892afcce9ea25b97046',
+            joinTXId: 'TSmrX61wNF',
+          };
 
           beforeEach(() => {
             clock = sinon.useFakeTimers();
@@ -1055,13 +1094,23 @@ describe('plugin-meetings', () => {
             type,
             extraParams = {},
             expectedMeetingData = {},
-            sendCAevents = false,
+            sendCAevents = false
           ) => {
             assert.calledOnce(webex.meetings.meetingInfo.fetchMeetingInfo);
             assert.calledOnce(MeetingsUtil.getMeetingAddedType);
             assert.notCalled(setTimeoutSpy);
             assert.callCount(TriggerProxy.trigger, 5);
-            assert.calledWith(webex.meetings.meetingInfo.fetchMeetingInfo, destination, type, null, null, undefined, undefined, extraParams, {meetingId: meeting.id, sendCAevents});
+            assert.calledWith(
+              webex.meetings.meetingInfo.fetchMeetingInfo,
+              destination,
+              type,
+              null,
+              null,
+              undefined,
+              undefined,
+              extraParams,
+              {meetingId: meeting.id, sendCAevents}
+            );
             assert.calledWith(MeetingsUtil.getMeetingAddedType, 'test type');
 
             if (expectedMeetingData.permissionToken) {
@@ -1070,7 +1119,7 @@ describe('plugin-meetings', () => {
             if (expectedMeetingData.meetingJoinUrl) {
               assert.equal(meeting.meetingJoinUrl, expectedMeetingData.meetingJoinUrl);
             }
-            if(expectedMeetingData.correlationId) {
+            if (expectedMeetingData.correlationId) {
               assert.equal(meeting.correlationId, expectedMeetingData.correlationId);
             }
             assert.equal(meeting.destination, destination);
@@ -1105,14 +1154,27 @@ describe('plugin-meetings', () => {
               correlationId: meeting.id,
             };
 
-            checkCreateWithoutDelay(meeting, 'test destination', 'test type', {}, expectedMeetingData);
+            checkCreateWithoutDelay(
+              meeting,
+              'test destination',
+              'test type',
+              {},
+              expectedMeetingData
+            );
           });
 
           [undefined, FAKE_INFO_EXTRA_PARAMS].forEach((infoExtraParams) => {
             const infoExtraParamsProvided = infoExtraParams !== undefined;
 
-            it(`creates the meeting from a successful meeting info fetch meeting resolve testing${infoExtraParamsProvided ? ' with infoExtraParams' : ''}`, async () => {
-              const meeting = await webex.meetings.createMeeting('test destination', 'test type', false, infoExtraParams);
+            it(`creates the meeting from a successful meeting info fetch meeting resolve testing${
+              infoExtraParamsProvided ? ' with infoExtraParams' : ''
+            }`, async () => {
+              const meeting = await webex.meetings.createMeeting(
+                'test destination',
+                'test type',
+                false,
+                infoExtraParams
+              );
               const expectedMeetingData = {
                 permissionToken: 'PT',
                 meetingJoinUrl: 'meetingJoinUrl',
@@ -1123,10 +1185,18 @@ describe('plugin-meetings', () => {
                 Meeting,
                 'createMeeting should eventually resolve to a Meeting Object'
               );
-              checkCreateWithoutDelay(meeting, 'test destination', 'test type', infoExtraParamsProvided ? infoExtraParams : {}, expectedMeetingData);
+              checkCreateWithoutDelay(
+                meeting,
+                'test destination',
+                'test type',
+                infoExtraParamsProvided ? infoExtraParams : {},
+                expectedMeetingData
+              );
             });
 
-            it(`creates the meeting from a successful meeting info fetch with random delay${infoExtraParamsProvided ? ' with infoExtraParams' : ''}`, async () => {
+            it(`creates the meeting from a successful meeting info fetch with random delay${
+              infoExtraParamsProvided ? ' with infoExtraParams' : ''
+            }`, async () => {
               const FAKE_LOCUS_MEETING = {
                 conversationUrl: 'locusConvURL',
                 url: 'locusUrl',
@@ -1214,7 +1284,7 @@ describe('plugin-meetings', () => {
                 'meeting:meetingInfoAvailable'
               );
             });
-          })
+          });
 
           it('creates the meeting from a successful meeting info fetch that has no random delay because it is active', async () => {
             const FAKE_LOCUS_MEETING = {
@@ -1315,14 +1385,27 @@ describe('plugin-meetings', () => {
           });
 
           it('creates meeting with the correlationId provided', async () => {
-            const meeting = await webex.meetings.createMeeting('test destination', 'test type', false, {}, 'my-correlationId');
+            const meeting = await webex.meetings.createMeeting(
+              'test destination',
+              'test type',
+              false,
+              {},
+              'my-correlationId'
+            );
 
             const expectedMeetingData = {
               correlationId: 'my-correlationId',
             };
 
-            checkCreateWithoutDelay(meeting, 'test destination', 'test type', {}, expectedMeetingData, true);
-          })
+            checkCreateWithoutDelay(
+              meeting,
+              'test destination',
+              'test type',
+              {},
+              expectedMeetingData,
+              true
+            );
+          });
         });
 
         describe('rejected MeetingInfo.#fetchMeetingInfo', () => {
@@ -1332,16 +1415,21 @@ describe('plugin-meetings', () => {
             webex.meetings.meetingInfo.fetchMeetingInfo = sinon
               .stub()
               .returns(Promise.reject(new Error('test')));
-            webex.meetings.destroy = sinon
-              .stub()
-              .returns(Promise.resolve());
+            webex.meetings.destroy = sinon.stub().returns(Promise.resolve());
             webex.meetings.createMeeting = sinon.spy(webex.meetings.createMeeting);
           });
 
           const checkCreateMeetingWithNoMeetingInfo = async (failOnMissingMeetingInfo, destroy) => {
             try {
-              const meeting = await webex.meetings.createMeeting('test destination', 'test type', undefined, undefined, undefined, failOnMissingMeetingInfo);
-              
+              const meeting = await webex.meetings.createMeeting(
+                'test destination',
+                'test type',
+                undefined,
+                undefined,
+                undefined,
+                failOnMissingMeetingInfo
+              );
+
               assert.instanceOf(
                 meeting,
                 Meeting,
@@ -1355,9 +1443,13 @@ describe('plugin-meetings', () => {
                 'test destination',
                 'test type'
               );
-  
+
               if (destroy) {
-                assert.calledWith(webex.meetings.destroy, sinon.match.instanceOf(Meeting), 'MISSING_MEETING_INFO')
+                assert.calledWith(
+                  webex.meetings.destroy,
+                  sinon.match.instanceOf(Meeting),
+                  'MISSING_MEETING_INFO'
+                );
                 assert.notCalled(MeetingsUtil.getMeetingAddedType);
                 assert.notCalled(TriggerProxy.trigger);
                 assert.throw(webex.meetings.createMeeting, 'meeting information not found');
@@ -1378,10 +1470,10 @@ describe('plugin-meetings', () => {
                   }
                 );
               }
-            } catch (err) { 
+            } catch (err) {
               assert.instanceOf(err, NoMeetingInfoError);
             }
-          }
+          };
 
           it('creates the meeting from a rejected meeting info fetch', async () => {
             checkCreateMeetingWithNoMeetingInfo(false, false);
@@ -1526,7 +1618,6 @@ describe('plugin-meetings', () => {
       });
 
       describe('#fetchUserPreferredWebexSite', () => {
-
         let loggerProxySpy;
 
         it('should call request.getMeetingPreferences to get the preferred webex site ', async () => {
@@ -1544,12 +1635,10 @@ describe('plugin-meetings', () => {
               getMeetingPreferences: sinon.stub().returns(Promise.resolve({})),
             },
             user: {
-              get: sinon.stub().returns(
-                Promise.resolve(user)
-              ),
+              get: sinon.stub().returns(Promise.resolve(user)),
             },
           });
-        }
+        };
 
         it('should not fail if UserPreferred info is not fetched ', async () => {
           setup();
@@ -1586,6 +1675,59 @@ describe('plugin-meetings', () => {
           assert.notCalled(loggerProxySpy);
         });
 
+        forEach(
+          [
+            {user: undefined},
+            {user: {userPreferences: {}}},
+            {user: {userPreferences: {userPreferencesItems: {}}}},
+            {user: {userPreferences: {userPreferencesItems: {preferredWebExSite: undefined}}}},
+          ],
+          ({user}) => {
+            it(`should handle invalid user data ${user}`, async () => {
+              setup({user});
+
+              await webex.meetings.fetchUserPreferredWebexSite();
+
+              assert.equal(webex.meetings.preferredWebexSite, '');
+              assert.calledOnceWithExactly(
+                loggerProxySpy,
+                'Failed to fetch preferred site from user - no site will be set'
+              );
+            });
+          }
+        );
+
+        it('should handle a get user failure', async () => {
+          setup();
+
+          webex.internal.user.get.rejects(new Error());
+
+          await webex.meetings.fetchUserPreferredWebexSite();
+
+          assert.equal(webex.meetings.preferredWebexSite, '');
+          assert.calledOnceWithExactly(
+            loggerProxySpy,
+            'Failed to fetch preferred site from user - no site will be set'
+          );
+        });
+
+        it('should fall back to fetching the site from the user', async () => {
+          setup({
+            user: {
+              userPreferences: {
+                userPreferencesItems: {
+                  preferredWebExSite: 'site.webex.com',
+                },
+              },
+            },
+          });
+
+          await webex.meetings.fetchUserPreferredWebexSite();
+
+          assert.equal(webex.meetings.preferredWebexSite, 'site.webex.com');
+          assert.notCalled(loggerProxySpy);
+        });
+
         forEach([
           {user: undefined},
           {user: {userPreferences: {}}},
@@ -1714,7 +1856,7 @@ describe('plugin-meetings', () => {
         newLocus = {
           controls: {},
           self: {},
-        }
+        };
       });
       afterEach(() => {
         sinon.restore();
@@ -1745,16 +1887,24 @@ describe('plugin-meetings', () => {
       });
 
       it('if newLocus replaceAt time is expired, then return false', () => {
-        sinon.stub(webex.meetings.meetingCollection, 'getActiveBreakoutLocus').returns({joinedWith: {replaces: [{
-              replaceAt: '2023-03-27T02:17:02.506Z',
-            }]}});
+        sinon.stub(webex.meetings.meetingCollection, 'getActiveBreakoutLocus').returns({
+          joinedWith: {
+            replaces: [
+              {
+                replaceAt: '2023-03-27T02:17:02.506Z',
+              },
+            ],
+          },
+        });
         newLocus.self.state = 'JOINED';
         sinon.stub(MeetingsUtil, 'joinedOnThisDevice').returns(true);
         sinon.stub(MeetingsUtil, 'getThisDevice').returns({
-          replaces: [{
-            replaceAt: '2023-03-27T02:17:01.506Z'
-          }]
-        })
+          replaces: [
+            {
+              replaceAt: '2023-03-27T02:17:01.506Z',
+            },
+          ],
+        });
 
         LoggerProxy.logger.log = sinon.stub();
         const result = webex.meetings.isNeedHandleMainLocus(meeting, newLocus);
@@ -1843,7 +1993,7 @@ describe('plugin-meetings', () => {
         newLocus = {
           controls: {},
           self: {},
-        }
+        };
       });
       afterEach(() => {
         sinon.restore();
@@ -1909,13 +2059,13 @@ describe('plugin-meetings', () => {
           self: {
             callbackInfo: {
               callbackAddress: 'address1',
-            }
+            },
           },
           info: {
             webExMeetingId: '123456',
             isUnifiedSpaceMeeting: false,
           },
-          conversationUrl: 'conversationUrl1'
+          conversationUrl: 'conversationUrl1',
         };
 
         sinon.stub(MeetingsUtil, 'checkForCorrelationId').returns('correlationId1');
@@ -1929,9 +2079,17 @@ describe('plugin-meetings', () => {
         assert.isNull(result);
         assert.callCount(webex.meetings.meetingCollection.getByKey, 5);
         assert.calledWith(webex.meetings.meetingCollection.getByKey, 'locusUrl', url1);
-        assert.calledWith(webex.meetings.meetingCollection.getByKey, 'correlationId', 'correlationId1');
+        assert.calledWith(
+          webex.meetings.meetingCollection.getByKey,
+          'correlationId',
+          'correlationId1'
+        );
         assert.calledWith(webex.meetings.meetingCollection.getByKey, 'sipUri', 'address1');
-        assert.calledWith(webex.meetings.meetingCollection.getByKey, 'conversationUrl', 'conversationUrl1');
+        assert.calledWith(
+          webex.meetings.meetingCollection.getByKey,
+          'conversationUrl',
+          'conversationUrl1'
+        );
         assert.calledWith(webex.meetings.meetingCollection.getByKey, 'meetingNumber', '123456');
       });
 
@@ -1941,7 +2099,7 @@ describe('plugin-meetings', () => {
         const result = webex.meetings.getCorrespondingMeetingByLocus({locus, locusUrl: url1});
         assert.isNull(result);
         assert.callCount(webex.meetings.meetingCollection.getByKey, 4);
-      })
+      });
 
       it('check the calls when meeting found by key: locusUrl', () => {
         mockGetByKey('locusUrl');
@@ -1957,7 +2115,11 @@ describe('plugin-meetings', () => {
         assert.deepEqual(result, mockReturnMeeting);
         assert.callCount(webex.meetings.meetingCollection.getByKey, 2);
         assert.calledWith(webex.meetings.meetingCollection.getByKey, 'locusUrl', url1);
-        assert.calledWith(webex.meetings.meetingCollection.getByKey, 'correlationId', 'correlationId1');
+        assert.calledWith(
+          webex.meetings.meetingCollection.getByKey,
+          'correlationId',
+          'correlationId1'
+        );
       });
 
       it('check the calls when meeting found by key: sipUri', () => {
@@ -1966,7 +2128,11 @@ describe('plugin-meetings', () => {
         assert.deepEqual(result, mockReturnMeeting);
         assert.callCount(webex.meetings.meetingCollection.getByKey, 3);
         assert.calledWith(webex.meetings.meetingCollection.getByKey, 'locusUrl', url1);
-        assert.calledWith(webex.meetings.meetingCollection.getByKey, 'correlationId', 'correlationId1');
+        assert.calledWith(
+          webex.meetings.meetingCollection.getByKey,
+          'correlationId',
+          'correlationId1'
+        );
         assert.calledWith(webex.meetings.meetingCollection.getByKey, 'sipUri', 'address1');
       });
 
@@ -1976,9 +2142,17 @@ describe('plugin-meetings', () => {
         assert.deepEqual(result, mockReturnMeeting);
         assert.callCount(webex.meetings.meetingCollection.getByKey, 4);
         assert.calledWith(webex.meetings.meetingCollection.getByKey, 'locusUrl', url1);
-        assert.calledWith(webex.meetings.meetingCollection.getByKey, 'correlationId', 'correlationId1');
+        assert.calledWith(
+          webex.meetings.meetingCollection.getByKey,
+          'correlationId',
+          'correlationId1'
+        );
         assert.calledWith(webex.meetings.meetingCollection.getByKey, 'sipUri', 'address1');
-        assert.calledWith(webex.meetings.meetingCollection.getByKey, 'conversationUrl', 'conversationUrl1');
+        assert.calledWith(
+          webex.meetings.meetingCollection.getByKey,
+          'conversationUrl',
+          'conversationUrl1'
+        );
       });
 
       it('check the calls when meeting found by key: meetingNumber', () => {
@@ -1987,9 +2161,17 @@ describe('plugin-meetings', () => {
         assert.deepEqual(result, mockReturnMeeting);
         assert.callCount(webex.meetings.meetingCollection.getByKey, 5);
         assert.calledWith(webex.meetings.meetingCollection.getByKey, 'locusUrl', url1);
-        assert.calledWith(webex.meetings.meetingCollection.getByKey, 'correlationId', 'correlationId1');
+        assert.calledWith(
+          webex.meetings.meetingCollection.getByKey,
+          'correlationId',
+          'correlationId1'
+        );
         assert.calledWith(webex.meetings.meetingCollection.getByKey, 'sipUri', 'address1');
-        assert.calledWith(webex.meetings.meetingCollection.getByKey, 'conversationUrl', 'conversationUrl1');
+        assert.calledWith(
+          webex.meetings.meetingCollection.getByKey,
+          'conversationUrl',
+          'conversationUrl1'
+        );
         assert.calledWith(webex.meetings.meetingCollection.getByKey, 'meetingNumber', '123456');
       });
     });
@@ -2004,18 +2186,18 @@ describe('plugin-meetings', () => {
           controls: {
             breakout: {
               sessionType: 'MAIN',
-              url: 'breakoutUnifiedUrl1'
-            }
-          }
+              url: 'breakoutUnifiedUrl1',
+            },
+          },
         };
         breakoutLocus = {
           url: 'breakoutUrl1',
           controls: {
             breakout: {
               sessionType: 'BREAKOUT',
-              url: 'breakoutUnifiedUrl1'
-            }
-          }
+              url: 'breakoutUnifiedUrl1',
+            },
+          },
         };
         lociArray = [mainLocus, breakoutLocus];
 
@@ -2058,8 +2240,8 @@ describe('plugin-meetings', () => {
             breakout: {
               sessionType: 'BREAKOUT',
               url: 'breakoutUnifiedUrl1',
-            }
-          }
+            },
+          },
         };
 
         webex.meetings.handleLocusEvent = sinon.stub();
@@ -2089,8 +2271,8 @@ describe('plugin-meetings', () => {
             breakout: {
               sessionType: 'MAIN',
               url: 'breakoutUnifiedUrl2',
-            }
-          }
+            },
+          },
         });
         assert.notCalled(webex.meetings.handleLocusEvent);
       });
@@ -2102,10 +2284,13 @@ describe('plugin-meetings', () => {
             breakout: {
               sessionType: 'MAIN',
               url: 'breakoutUnifiedUrl1',
-            }
-          }
+            },
+          },
+        });
+        assert.calledWith(webex.meetings.handleLocusEvent, {
+          locus: breakoutLocus,
+          locusUrl: breakoutLocus.url,
         });
-        assert.calledWith(webex.meetings.handleLocusEvent, {locus: breakoutLocus, locusUrl: breakoutLocus.url});
       });
     });
 
@@ -2129,17 +2314,16 @@ describe('plugin-meetings', () => {
         meeting.locusId = 'locus id';
         meeting.correlationId = 'correlation id';
         meeting.locusInfo = {
-          fullState: { lastActive: 'last active', sessionId: 'locus session id'},
-          info: { webExMeetingId: 'meeting id'}
-        }
+          fullState: {lastActive: 'last active', sessionId: 'locus session id'},
+          info: {webExMeetingId: 'meeting id'},
+        };
       });
 
       afterEach(() => {
         sinon.restore();
-      })
+      });
 
       it('sends metrics on success', async () => {
-
         await meeting.uploadLogs();
 
         await testUtils.flushPromises();
@@ -2162,16 +2346,20 @@ describe('plugin-meetings', () => {
 
         await testUtils.flushPromises();
 
-        assert.calledOnceWithExactly(metricsSpy, 'js_sdk_upload_logs_failure', sinon.match({
-          callStart: 'last active',
-          correlationId: 'correlation id',
-          feedbackId: 'correlation id',
-          locusId: 'locus id',
-          meetingId: 'meeting id',
-          reason: 'fake error',
-          autoupload: true,
-          locussessionid: 'locus session id',
-        }));
+        assert.calledOnceWithExactly(
+          metricsSpy,
+          'js_sdk_upload_logs_failure',
+          sinon.match({
+            callStart: 'last active',
+            correlationId: 'correlation id',
+            feedbackId: 'correlation id',
+            locusId: 'locus id',
+            meetingId: 'meeting id',
+            reason: 'fake error',
+            autoupload: true,
+            locussessionid: 'locus session id',
+          })
+        );
       });
     });
   });