@webex/plugin-meetings 2.37.0 → 2.37.1

package/test/unit/spec/roap/turnDiscovery.ts

@@ -25,15 +25,15 @@ describe('TurnDiscovery', () => {
     sinon.stub(Metrics, 'sendBehavioralMetric');
 
     mockRoapRequest = {
-      sendRoap: sinon.fake.resolves({mediaConnections: FAKE_MEDIA_CONNECTIONS_FROM_LOCUS})
+      sendRoap: sinon.fake.resolves({mediaConnections: FAKE_MEDIA_CONNECTIONS_FROM_LOCUS}),
     } as unknown as RoapRequest;
 
     testMeeting = {
       id: 'fake meeting id',
       config: {
         experimental: {
-          enableTurnDiscovery: true
-        }
+          enableTurnDiscovery: true,
+        },
       },
       correlationId: 'fake correlation id',
       selfUrl: 'fake self url',
@@ -46,7 +46,7 @@ describe('TurnDiscovery', () => {
         testMeeting.roapSeq = newSeq;
       }),
       updateMediaConnections: sinon.stub(),
-      webex: {meetings: {reachability: {isAnyClusterReachable: () => false}}}
+      webex: {meetings: {reachability: {isAnyClusterReachable: () => false}}},
     };
   });
 
@@ -55,7 +55,11 @@ describe('TurnDiscovery', () => {
     sinon.restore();
   });
 
-  const checkRoapMessageSent = async (messageType, expectedSeq, expectedMediaId = testMeeting.mediaId) => {
+  const checkRoapMessageSent = async (
+    messageType,
+    expectedSeq,
+    expectedMediaId = testMeeting.mediaId
+  ) => {
     await testUtils.flushPromises();
 
     assert.calledOnce(mockRoapRequest.sendRoap);
@@ -70,7 +74,7 @@ describe('TurnDiscovery', () => {
       mediaId: expectedMediaId,
       audioMuted: testMeeting.isAudioMuted(),
       videoMuted: testMeeting.isVideoMuted(),
-      meetingId: testMeeting.id
+      meetingId: testMeeting.id,
     });
 
     if (messageType === 'TURN_DISCOVERY_REQUEST') {
@@ -109,7 +113,7 @@ describe('TurnDiscovery', () => {
           `x-cisco-turn-url=${FAKE_TURN_URL}`,
           `x-cisco-turn-username=${FAKE_TURN_USERNAME}`,
           `x-cisco-turn-password=${FAKE_TURN_PASSWORD}`,
-        ]
+        ],
       });
 
       await testUtils.flushPromises();
@@ -122,7 +126,7 @@ describe('TurnDiscovery', () => {
       assert.deepEqual(turnServerInfo, {
         url: FAKE_TURN_URL,
         username: FAKE_TURN_USERNAME,
-        password: FAKE_TURN_PASSWORD
+        password: FAKE_TURN_PASSWORD,
       });
 
       assert.isUndefined(turnDiscoverySkippedReason);
@@ -145,7 +149,7 @@ describe('TurnDiscovery', () => {
           `x-cisco-turn-url=${FAKE_TURN_URL}`,
           `x-cisco-turn-username=${FAKE_TURN_USERNAME}`,
           `x-cisco-turn-password=${FAKE_TURN_PASSWORD}`,
-        ]
+        ],
       });
 
       await testUtils.flushPromises();
@@ -158,7 +162,7 @@ describe('TurnDiscovery', () => {
       assert.deepEqual(turnServerInfo, {
         url: FAKE_TURN_URL,
         username: FAKE_TURN_USERNAME,
-        password: FAKE_TURN_PASSWORD
+        password: FAKE_TURN_PASSWORD,
       });
       assert.isUndefined(turnDiscoverySkippedReason);
     });
@@ -181,7 +185,7 @@ describe('TurnDiscovery', () => {
           `x-cisco-turn-username=${FAKE_TURN_USERNAME}`,
           `x-cisco-turn-password=${FAKE_TURN_PASSWORD}`,
           'another-header-at-the-end=12345',
-        ]
+        ],
       });
 
       await testUtils.flushPromises();
@@ -193,7 +197,7 @@ describe('TurnDiscovery', () => {
       assert.deepEqual(turnServerInfo, {
         url: FAKE_TURN_URL,
         username: FAKE_TURN_USERNAME,
-        password: FAKE_TURN_PASSWORD
+        password: FAKE_TURN_PASSWORD,
       });
       assert.isUndefined(turnDiscoverySkippedReason);
     });
@@ -242,10 +246,9 @@ describe('TurnDiscovery', () => {
       assert.notCalled(mockRoapRequest.sendRoap);
       assert.notCalled(Metrics.sendBehavioralMetric);
       testMeeting.webex.meetings.reachability.isAnyClusterReachable = prev;
-
     });
 
-    it('resolves with undefined if we don\'t get a response within 10s', async () => {
+    it("resolves with undefined if we don't get a response within 10s", async () => {
       const td = new TurnDiscovery(mockRoapRequest);
 
       const promise = td.doTurnDiscovery(testMeeting, false);
@@ -269,7 +272,7 @@ describe('TurnDiscovery', () => {
         headers: [
           `x-cisco-turn-url=${FAKE_TURN_URL}`,
           `x-cisco-turn-username=${FAKE_TURN_USERNAME}`,
-        ]
+        ],
       });
       await testUtils.flushPromises();
       const {turnServerInfo, turnDiscoverySkippedReason} = await turnDiscoveryPromise;
@@ -302,7 +305,7 @@ describe('TurnDiscovery', () => {
       td.handleTurnDiscoveryResponse({headers: []});
 
       await testUtils.flushPromises();
-      const {turnServerInfo, turnDiscoverySkippedReason}= await turnDiscoveryPromise;
+      const {turnServerInfo, turnDiscoverySkippedReason} = await turnDiscoveryPromise;
 
       assert.isUndefined(turnServerInfo);
       assert.isUndefined(turnDiscoverySkippedReason);
@@ -328,7 +331,7 @@ describe('TurnDiscovery', () => {
           `x-cisco-turn-url=${FAKE_TURN_URL}`,
           `x-cisco-turn-username=${FAKE_TURN_USERNAME}`,
           `x-cisco-turn-password=${FAKE_TURN_PASSWORD}`,
-        ]
+        ],
       });
 
       await testUtils.flushPromises();
@@ -345,7 +348,7 @@ describe('TurnDiscovery', () => {
   });
 
   describe('handleTurnDiscoveryResponse', () => {
-    it('doesn\'t do anything if turn discovery was not started', () => {
+    it("doesn't do anything if turn discovery was not started", () => {
       const td = new TurnDiscovery(mockRoapRequest);
 
       // there is not much we can check, but we mainly want to make
@@ -355,7 +358,7 @@ describe('TurnDiscovery', () => {
           `x-cisco-turn-url=${FAKE_TURN_URL}`,
           `x-cisco-turn-username=${FAKE_TURN_USERNAME}`,
           `x-cisco-turn-password=${FAKE_TURN_PASSWORD}`,
-        ]
+        ],
       });
 
       assert.notCalled(mockRoapRequest.sendRoap);

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

@@ -11,11 +11,11 @@ describe('RoapUtil', () => {
     beforeEach('stub PeerConnectionManager', () => {
       meeting = {
         mediaProperties: {
-          peerConnection: {name: 'peer-connection'}
+          peerConnection: {name: 'peer-connection'},
         },
         roap: {
-          lastRoapOffer: 'lastRoapOffer'
-        }
+          lastRoapOffer: 'lastRoapOffer',
+        },
       };
       session = {OFFER: {sdps: ['sdp1', 'sdp2']}};
 

package/test/unit/spec/stats-analyzer/index.js

@@ -19,31 +19,31 @@ describe('plugin-meetings', () => {
       let sandBoxSpy;
 
       const initialConfig = {
-        videoPacketLossRatioThreshold: 9
+        videoPacketLossRatioThreshold: 9,
       };
 
       const defaultStats = {
         internal: {
           video: {
             send: {
-              totalPacketsLostOnReceiver: 10
-            }
-          }
+              totalPacketsLostOnReceiver: 10,
+            },
+          },
         },
         video: {
           send: {
             packetsSent: 2,
             meanRemoteJitter: [],
-            meanRoundTripTime: []
-          }
-        }
+            meanRoundTripTime: [],
+          },
+        },
       };
 
       const statusResult = {
         type: 'remote-inbound-rtp',
         packetsLost: 11,
         rttThreshold: 501,
-        jitterThreshold: 501
+        jitterThreshold: 501,
       };
 
       const sandbox = sinon.createSandbox();
@@ -53,7 +53,10 @@ describe('plugin-meetings', () => {
 
         statsAnalyzer = new StatsAnalyzer(initialConfig, networkQualityMonitor, defaultStats);
 
-        sandBoxSpy = sandbox.spy(statsAnalyzer.networkQualityMonitor, 'determineUplinkNetworkQuality');
+        sandBoxSpy = sandbox.spy(
+          statsAnalyzer.networkQualityMonitor,
+          'determineUplinkNetworkQuality'
+        );
       });
 
       afterEach(() => {
@@ -64,11 +67,13 @@ describe('plugin-meetings', () => {
         await statsAnalyzer.parseGetStatsResult(statusResult, 'video');
 
         assert.calledOnce(statsAnalyzer.networkQualityMonitor.determineUplinkNetworkQuality);
-        assert(sandBoxSpy.calledWith({
-          mediaType: 'video',
-          remoteRtpResults: statusResult,
-          statsAnalyzerCurrentStats: statsAnalyzer.statsResults
-        }));
+        assert(
+          sandBoxSpy.calledWith({
+            mediaType: 'video',
+            remoteRtpResults: statusResult,
+            statsAnalyzerCurrentStats: statsAnalyzer.statsResults,
+          })
+        );
       });
     });
 
@@ -113,7 +118,7 @@ describe('plugin-meetings', () => {
               type: 'outbound-rtp',
               packetsSent: 0,
               bytesSent: 1,
-            }
+            },
           },
           video: {
             receiver: {
@@ -125,35 +130,35 @@ describe('plugin-meetings', () => {
               type: 'outbound-rtp',
               framesSent: 0,
               bytesSent: 1,
-            }
-          }
+            },
+          },
         };
 
         pc = {
           audioTransceiver: {
             sender: {
-              getStats: sinon.stub().resolves([fakeStats.audio.sender])
+              getStats: sinon.stub().resolves([fakeStats.audio.sender]),
             },
             receiver: {
-              getStats: sinon.stub().resolves([fakeStats.audio.receiver])
+              getStats: sinon.stub().resolves([fakeStats.audio.receiver]),
             },
           },
           videoTransceiver: {
             sender: {
-              getStats: sinon.stub().resolves([fakeStats.video.sender])
+              getStats: sinon.stub().resolves([fakeStats.video.sender]),
             },
             receiver: {
-              getStats: sinon.stub().resolves([fakeStats.video.receiver])
+              getStats: sinon.stub().resolves([fakeStats.video.receiver]),
             },
           },
           shareTransceiver: {
             sender: {
-              getStats: sinon.stub().resolves([])
+              getStats: sinon.stub().resolves([]),
             },
             receiver: {
-              getStats: sinon.stub().resolves([])
+              getStats: sinon.stub().resolves([]),
             },
-          }
+          },
         };
 
         networkQualityMonitor = new NetworkQualityMonitor(initialConfig);

package/test/utils/cmr.js

@@ -1,4 +1,3 @@
-
 import uuid from 'uuid';
 import retry from '@webex/test-helper-retry';
 
@@ -23,16 +22,16 @@ CMR.reserve = (webex, claimed) => {
       authorization: `Bearer ${webex.credentials.supertoken.access_token}`,
       'cisco-no-http-redirect': null,
       'spark-user-agent': null,
-      trackingid: `ITCLIENT_${uuid.v4()}_0_imi:true`
+      trackingid: `ITCLIENT_${uuid.v4()}_0_imi:true`,
     },
     body: {
       resourceType,
       requestMetaData: {
         emailAddress: `test${uuid.v4()}@wx2.example.com`,
-        loginType: 'loginGuest'
+        loginType: 'loginGuest',
       },
-      reservedBy: 'Webex JavaScript SDK Test Suite'
-    }
+      reservedBy: 'Webex JavaScript SDK Test Suite',
+    },
   };
 
   if (claimed) {
@@ -40,7 +39,8 @@ CMR.reserve = (webex, claimed) => {
   }
   console.log('USER ID ', webex.internal.device.userId);
 
-  return webex.request(requestBody)
+  return webex
+    .request(requestBody)
     .then((response) => {
       const cmr = response.body;
 
@@ -56,47 +56,49 @@ CMR.reserve = (webex, claimed) => {
 CMR.release = (webex, reservationUrl) => {
   console.log('releasing cmr');
 
-  return webex.request({
-    method: 'DELETE',
-    uri: reservationUrl,
-    headers: {
-      authorization: `Bearer ${webex.credentials.supertoken.access_token}`,
-      'cisco-no-http-redirect': null,
-      'spark-user-agent': null
-    }
-  })
+  return webex
+    .request({
+      method: 'DELETE',
+      uri: reservationUrl,
+      headers: {
+        authorization: `Bearer ${webex.credentials.supertoken.access_token}`,
+        'cisco-no-http-redirect': null,
+        'spark-user-agent': null,
+      },
+    })
     .then(() => console.log('released cmr'));
 };
 
-CMR.waitForHostToJoin = (webex, resourceUrl) => retry(() => {
-  console.log('checking if the host has joined');
-
-  return webex.request({
-    method: 'GET',
-    uri: resourceUrl,
-    headers: {
-      authorization: `Bearer ${webex.credentials.supertoken.access_token}`,
-      'cisco-no-http-redirect': null,
-      'spark-user-agent': null
-    }
+CMR.waitForHostToJoin = (webex, resourceUrl) =>
+  retry(() => {
+    console.log('checking if the host has joined');
+
+    return webex
+      .request({
+        method: 'GET',
+        uri: resourceUrl,
+        headers: {
+          authorization: `Bearer ${webex.credentials.supertoken.access_token}`,
+          'cisco-no-http-redirect': null,
+          'spark-user-agent': null,
+        },
+      })
+      .then((res) => {
+        if (res.body && res.body.meeting.hostPresent) {
+          console.log('the host has joined');
+
+          return;
+        }
+
+        console.log('the host has not joined');
+        throw new Error('Meeting host has not yet joined');
+      });
   })
-    .then((res) => {
-      if (res.body && res.body.meeting.hostPresent) {
-        console.log('the host has joined');
+    .then(() => true)
+    .catch((reason) => {
+      console.warn(reason);
 
-        return;
-      }
-
-      console.log('the host has not joined');
-      throw new Error('Meeting host has not yet joined');
+      return false;
     });
-})
-  .then(() => true)
-  .catch((reason) => {
-    console.warn(reason);
-
-    return false;
-  });
-
 
 export default CMR;

package/test/utils/testUtils.js

@@ -19,7 +19,7 @@ const waitForSpy = (spy, event) => {
           resolve(spy.args[0][0]);
         }
       }, 1000);
-    })
+    }),
   ]);
 };
 
@@ -40,7 +40,7 @@ const waitForStateChange = (meeting, state) => {
           resolve(meeting.locusInfo.parsedLocus.states);
         }
       }, 1000);
-    })
+    }),
   ]);
 };
 
@@ -59,44 +59,46 @@ const waitForCallEnded = (user, email) => {
           clearTimeout(timer);
           clearInterval(interval);
           resolve();
-        }
-        else {
+        } else {
           console.log('MEETING STILL EXISTS!', user.webex.meetings.getAllMeetings());
         }
       }, 3000);
-    })
+    }),
   ]);
 };
 
-const syncAndEndMeeting = (user) => user.webex.meetings.syncMeetings()
-  .then(() => {
-    const promise = [];
-    const meetings = user.webex.meetings.getAllMeetings();
+const syncAndEndMeeting = (user) =>
+  user.webex.meetings
+    .syncMeetings()
+    .then(() => {
+      const promise = [];
+      const meetings = user.webex.meetings.getAllMeetings();
 
-    if (Object.keys(meetings).length === 0) {
-      return Promise.resolve();
-    }
-    Object.keys(meetings)
-      .forEach((key) => {
+      if (Object.keys(meetings).length === 0) {
+        return Promise.resolve();
+      }
+      Object.keys(meetings).forEach((key) => {
         promise.push(meetings[key].leave());
       });
 
-    return Promise.all(promise);
-  })
-  .then(() => new Promise((resolve) => {
-    const interval = setInterval(() => {
-      if (Object.keys(user.webex.meetings.getAllMeetings()).length === 0) {
-        clearInterval(interval);
-        resolve();
-      }
-      else {
-        console.log('End Meetings before test failed');
-      }
-    }, 3000);
-  }))
-  .catch((e) => {
-    console.log('ERROR on syncMeeting', e);
-  });
+      return Promise.all(promise);
+    })
+    .then(
+      () =>
+        new Promise((resolve) => {
+          const interval = setInterval(() => {
+            if (Object.keys(user.webex.meetings.getAllMeetings()).length === 0) {
+              clearInterval(interval);
+              resolve();
+            } else {
+              console.log('End Meetings before test failed');
+            }
+          }, 3000);
+        })
+    )
+    .catch((e) => {
+      console.log('ERROR on syncMeeting', e);
+    });
 
 //
 const waitForEvents = (scopeEvents, timeout = max) => {
@@ -125,11 +127,12 @@ const waitForEvents = (scopeEvents, timeout = max) => {
               obj.user.memberId = value.meeting.locusInfo.parsedLocus.selfId;
             }
             if (obj.event === 'meeting:removed') {
-              console.log(`MEETING:REMOVED ${obj.user.name} ID ${obj.user.meeting.id} correlationID ${obj.user.meeting.correlationId}`);
+              console.log(
+                `MEETING:REMOVED ${obj.user.name} ID ${obj.user.meeting.id} correlationID ${obj.user.meeting.correlationId}`
+              );
               if (obj.user.meeting.id === value.meetingId) {
                 obj.user.meeting = null;
-              }
-              else {
+              } else {
                 console.log('MEETING EXISTING ', obj.user.webex.meetings.getAllMeetings());
                 reject(new Error(`Different Meeting Object ${value}`));
               }
@@ -142,8 +145,7 @@ const waitForEvents = (scopeEvents, timeout = max) => {
 
                 resolve(result);
               }
-            }
-            else if (result.length === scopeEvents.length) {
+            } else if (result.length === scopeEvents.length) {
               resolve(result);
               obj.scope.off(obj.event, handler);
               clearTimeout(timer);
@@ -152,12 +154,11 @@ const waitForEvents = (scopeEvents, timeout = max) => {
 
           obj.scope.on(obj.event, handler);
         });
-      }
-      catch (e) {
+      } catch (e) {
         console.error('waitForEvents', e);
         reject(e);
       }
-    })
+    }),
   ]);
 };
 
@@ -172,25 +173,27 @@ const delayedPromise = (promise) => {
     }),
     new Promise((resolve, reject) => {
       setTimeout(() => {
-        promise.then((res) => {
-          resolve(res);
-          clearTimeout(timer);
-        })
+        promise
+          .then((res) => {
+            resolve(res);
+            clearTimeout(timer);
+          })
           .catch((e) => {
             console.error('delayedPromise', e);
             reject(e);
           });
       }, 2000);
-    })
+    }),
   ]);
 };
 
-const delayedTest = (callback, timeout) => new Promise((resolve) => {
-  setTimeout(() => {
-    callback();
-    resolve();
-  }, timeout);
-});
+const delayedTest = (callback, timeout) =>
+  new Promise((resolve) => {
+    setTimeout(() => {
+      callback();
+      resolve();
+    }, timeout);
+  });
 
 const addMedia = (user) => {
   const mediaReadyPromises = {
@@ -209,23 +212,26 @@ const addMedia = (user) => {
 
   user.meeting.on('media:ready', mediaReady);
 
-  return user.meeting.getMediaStreams({
-    sendAudio: true,
-    sendVideo: true,
-    sendShare: false
-  })
-    .then(([localStream, localShare]) => user.meeting.addMedia({
-      mediaSettings: {
-        sendAudio: true,
-        sendVideo: true,
-        sendShare: false,
-        receiveShare: true,
-        receiveAudio: true,
-        receiveVideo: true
-      },
-      localShare,
-      localStream
-    }))
+  return user.meeting
+    .getMediaStreams({
+      sendAudio: true,
+      sendVideo: true,
+      sendShare: false,
+    })
+    .then(([localStream, localShare]) =>
+      user.meeting.addMedia({
+        mediaSettings: {
+          sendAudio: true,
+          sendVideo: true,
+          sendShare: false,
+          receiveShare: true,
+          receiveAudio: true,
+          receiveVideo: true,
+        },
+        localShare,
+        localStream,
+      })
+    )
     .then(() => Promise.all(Object.values(mediaReadyPromises).map((defer) => defer.promise)))
     .then(() => {
       assert.exists(user.meeting.mediaProperties.audioTrack, 'audioTrack not present');
@@ -233,11 +239,12 @@ const addMedia = (user) => {
     });
 };
 
-const waitUntil = (waitTime) => new Promise((resolve) => {
-  setTimeout(() => {
-    resolve();
-  }, waitTime);
-});
+const waitUntil = (waitTime) =>
+  new Promise((resolve) => {
+    setTimeout(() => {
+      resolve();
+    }, waitTime);
+  });
 
 const flushPromises = () => new Promise(setImmediate);
 
@@ -259,7 +266,10 @@ const getCircularReplacer = () => {
 
 // this function is meant to be used as the "match" callback with waitForEvents()
 // when you want to wait for a particular users's status to reach a certain value
-const checkParticipantUpdatedStatus = (user, expectedStatus) => (event) => !!event.delta.updated.find((member) => user.meeting.members.selfId === member.id && member.status === expectedStatus);
+const checkParticipantUpdatedStatus = (user, expectedStatus) => (event) =>
+  !!event.delta.updated.find(
+    (member) => user.meeting.members.selfId === member.id && member.status === expectedStatus
+  );
 
 export default {
   waitForSpy,
@@ -273,6 +283,5 @@ export default {
   waitUntil,
   delayedTest,
   flushPromises,
-  getCircularReplacer
+  getCircularReplacer,
 };
-