@webex/plugin-meetings 2.59.5 → 2.59.6-next.2

package/src/statsAnalyzer/index.ts

@@ -25,8 +25,6 @@ import {
 
 export const EVENTS = {
   MEDIA_QUALITY: 'MEDIA_QUALITY',
-  NO_FRAMES_SENT: 'NO_FRAMES_SENT',
-  NO_VIDEO_ENCODED: 'NO_VIDEO_ENCODED',
   LOCAL_MEDIA_STARTED: 'LOCAL_MEDIA_STARTED',
   LOCAL_MEDIA_STOPPED: 'LOCAL_MEDIA_STOPPED',
   REMOTE_MEDIA_STARTED: 'REMOTE_MEDIA_STARTED',
@@ -422,6 +420,9 @@ export class StatsAnalyzer extends EventsScope {
       case 'inbound-rtp':
         this.processInboundRTPResult(getStatsResult, type);
         break;
+      case 'track':
+        this.processTrackResult(getStatsResult, type);
+        break;
       case 'remote-inbound-rtp':
       case 'remote-outbound-rtp':
         // @ts-ignore
@@ -519,7 +520,7 @@ export class StatsAnalyzer extends EventsScope {
 
     if (currentValue - previousValue > 0) {
       newEvent = isLocal ? EVENTS.LOCAL_MEDIA_STARTED : EVENTS.REMOTE_MEDIA_STARTED;
-    } else if (currentValue === previousValue && currentValue >= 0) {
+    } else if (currentValue === previousValue && currentValue > 0) {
       newEvent = isLocal ? EVENTS.LOCAL_MEDIA_STOPPED : EVENTS.REMOTE_MEDIA_STOPPED;
     }
 
@@ -634,25 +635,14 @@ export class StatsAnalyzer extends EventsScope {
           LoggerProxy.logger.info(
             `StatsAnalyzer:index#compareLastStatsResult --> No ${mediaType} RTP packets sent`
           );
-        } else if (this.lastEmittedStartStopEvent[mediaType].local !== EVENTS.LOCAL_MEDIA_STOPPED) {
+        } else {
           if (
             currentStats.framesEncoded === previousStats.framesEncoded ||
             currentStats.framesEncoded === 0
           ) {
-            this.lastEmittedStartStopEvent[mediaType].local = EVENTS.NO_VIDEO_ENCODED;
             LoggerProxy.logger.info(
               `StatsAnalyzer:index#compareLastStatsResult --> No ${mediaType} Frames Encoded`
             );
-            this.emit(
-              {
-                file: 'statsAnalyzer',
-                function: 'compareLastStatsResult',
-              },
-              EVENTS.NO_VIDEO_ENCODED,
-              {
-                mediaType,
-              }
-            );
           }
 
           if (
@@ -664,28 +654,8 @@ export class StatsAnalyzer extends EventsScope {
               `StatsAnalyzer:index#compareLastStatsResult --> No ${mediaType} Frames sent`
             );
           }
-
-          // Video is encoded but frames are not sent
-          if (
-            currentStats.framesEncoded !== previousStats.framesEncoded &&
-            (currentStats.framesSent === previousStats.framesSent || currentStats.framesSent === 0)
-          ) {
-            this.lastEmittedStartStopEvent[mediaType].local = EVENTS.NO_FRAMES_SENT;
-            LoggerProxy.logger.info(
-              `StatsAnalyzer:index#compareLastStatsResult --> No ${mediaType} frames sent even though frames are encoded`
-            );
-            this.emit(
-              {
-                file: 'statsAnalyzer',
-                function: 'compareLastStatsResult',
-              },
-              EVENTS.NO_FRAMES_SENT,
-              {
-                mediaType,
-              }
-            );
-          }
         }
+
         this.emitStartStopEvents(
           mediaType,
           previousStats.framesSent,
@@ -761,25 +731,14 @@ export class StatsAnalyzer extends EventsScope {
           LoggerProxy.logger.info(
             `StatsAnalyzer:index#compareLastStatsResult --> No ${mediaType} RTP packets sent`
           );
-        } else if (this.lastEmittedStartStopEvent[mediaType].local !== EVENTS.LOCAL_MEDIA_STOPPED) {
+        } else {
           if (
             currentStats.framesEncoded === previousStats.framesEncoded ||
             currentStats.framesEncoded === 0
           ) {
-            this.lastEmittedStartStopEvent[mediaType].local = EVENTS.NO_VIDEO_ENCODED;
             LoggerProxy.logger.info(
               `StatsAnalyzer:index#compareLastStatsResult --> No ${mediaType} frames getting encoded`
             );
-            this.emit(
-              {
-                file: 'statsAnalyzer',
-                function: 'compareLastStatsResult',
-              },
-              EVENTS.NO_VIDEO_ENCODED,
-              {
-                mediaType,
-              }
-            );
           }
 
           if (
@@ -791,27 +750,6 @@ export class StatsAnalyzer extends EventsScope {
               `StatsAnalyzer:index#compareLastStatsResult --> No ${mediaType} frames sent`
             );
           }
-
-          // Share video is encoded but frames are not sent
-          if (
-            currentStats.framesEncoded !== previousStats.framesEncoded &&
-            (currentStats.framesSent === previousStats.framesSent || currentStats.framesSent === 0)
-          ) {
-            this.lastEmittedStartStopEvent[mediaType].local = EVENTS.NO_FRAMES_SENT;
-            LoggerProxy.logger.info(
-              `StatsAnalyzer:index#compareLastStatsResult --> No ${mediaType} Frames sent even though frames are being encoded`
-            );
-            this.emit(
-              {
-                file: 'statsAnalyzer',
-                function: 'compareLastStatsResult',
-              },
-              EVENTS.NO_FRAMES_SENT,
-              {
-                mediaType,
-              }
-            );
-          }
         }
 
         // TODO:need to check receive share value
@@ -943,7 +881,6 @@ export class StatsAnalyzer extends EventsScope {
     const mediaType = type || STATS.AUDIO_CORRELATE;
     const sendrecvType = STATS.SEND_DIRECTION;
 
-    this.processTrackResult(result, type, sendrecvType);
     if (result.bytesSent) {
       let kilobytes = 0;
 
@@ -967,6 +904,7 @@ export class StatsAnalyzer extends EventsScope {
 
       this.statsResults[mediaType][sendrecvType].availableBandwidth = kilobytes.toFixed(1);
       this.statsResults[mediaType].bytesSent = kilobytes;
+
       this.statsResults[mediaType][sendrecvType].framesEncoded =
         result.framesEncoded - this.statsResults.internal[mediaType][sendrecvType].framesEncoded;
       this.statsResults[mediaType][sendrecvType].keyFramesEncoded =
@@ -1017,7 +955,6 @@ export class StatsAnalyzer extends EventsScope {
     const mediaType = type || STATS.AUDIO_CORRELATE;
     const sendrecvType = STATS.RECEIVE_DIRECTION;
 
-    this.processTrackResult(result, type, sendrecvType);
     if (result.bytesReceived) {
       let kilobytes = 0;
 
@@ -1221,29 +1158,29 @@ export class StatsAnalyzer extends EventsScope {
    * @private
    * @param {*} result
    * @param {*} mediaType
-   * @param {*} sendrecvType
    * @returns {void}
    * @memberof StatsAnalyzer
    */
-  private processTrackResult(result: any, mediaType: any, sendrecvType: any) {
-    if (!result || mediaType === STATS.AUDIO_CORRELATE) {
-      return;
-    }
-    if (result.type !== 'inbound-rtp' && result.type !== 'outbound-rtp') {
+  private processTrackResult(result: any, mediaType: any) {
+    if (!result || result.type !== 'track') {
       return;
     }
+    if (result.type !== 'track') return;
+
+    const sendrecvType =
+      result.remoteSource === true ? STATS.RECEIVE_DIRECTION : STATS.SEND_DIRECTION;
+
     if (result.frameWidth && result.frameHeight) {
       this.statsResults.resolutions[mediaType][sendrecvType].width = result.frameWidth;
       this.statsResults.resolutions[mediaType][sendrecvType].height = result.frameHeight;
+      this.statsResults.resolutions[mediaType][sendrecvType].framesSent = result.framesSent;
+      this.statsResults.resolutions[mediaType][sendrecvType].hugeFramesSent = result.hugeFramesSent;
     }
 
     if (sendrecvType === STATS.RECEIVE_DIRECTION) {
       this.statsResults.resolutions[mediaType][sendrecvType].framesReceived = result.framesReceived;
       this.statsResults.resolutions[mediaType][sendrecvType].framesDecoded = result.framesDecoded;
       this.statsResults.resolutions[mediaType][sendrecvType].framesDropped = result.framesDropped;
-    } else if (sendrecvType === STATS.SEND_DIRECTION) {
-      this.statsResults.resolutions[mediaType][sendrecvType].framesSent = result.framesSent;
-      this.statsResults.resolutions[mediaType][sendrecvType].hugeFramesSent = result.hugeFramesSent;
     }
 
     if (result.trackIdentifier && mediaType !== STATS.AUDIO_CORRELATE) {

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

@@ -770,12 +770,12 @@ describe('plugin-meetings', () => {
           assert.equal(meeting.isTranscriptionSupported(), true);
         });
       });
-      describe('#receiveTranscription', () => {
+      describe('#startTranscription', () => {
         it('should invoke subscribe method to invoke the callback', () => {
           meeting.monitorTranscriptionSocketConnection = sinon.stub();
           meeting.initializeTranscription = sinon.stub();
 
-          meeting.receiveTranscription().then(() => {
+          meeting.startTranscription().then(() => {
             assert.equal(true, false);
             assert.calledOnce(meeting.initializeTranscription);
             assert.calledOnce(meeting.monitorTranscriptionSocketConnection);
@@ -786,7 +786,7 @@ describe('plugin-meetings', () => {
           meeting.request = sinon.stub().returns(Promise.reject());
 
           try {
-            await meeting.receiveTranscription();
+            await meeting.startTranscription();
           } catch (err) {
             assert(err, {});
           }
@@ -840,12 +840,12 @@ describe('plugin-meetings', () => {
             assert.calledOnce(MeetingUtil.joinMeeting);
             assert.calledOnce(meeting.setLocus);
           });
-          it('should invoke `receiveTranscription()` if receiveTranscription is set to true', async () => {
+          it('should invoke `startTranscription()` if receiveTranscription is set to true', async () => {
             meeting.isTranscriptionSupported = sinon.stub().returns(true);
-            meeting.receiveTranscription = sinon.stub().returns(Promise.resolve());
+            meeting.startTranscription = sinon.stub().returns(Promise.resolve());
 
             await meeting.join({receiveTranscription: true});
-            assert.calledOnce(meeting.receiveTranscription);
+            assert.calledOnce(meeting.startTranscription);
           });
 
           it('should not create new correlation ID on join immediately after create', async () => {
@@ -1329,71 +1329,6 @@ describe('plugin-meetings', () => {
               data: {intervalData: fakeData, networkType: 'wifi'},
             });
           });
-          it('NO_FRAMES_SENT triggers "meeting:noFramesSent" event and sends metrics', async () => {
-            meeting.mediaProperties.mediaDirection = {sendVideo: true};
-            statsAnalyzerStub.emit(
-              {file: 'test', function: 'test'},
-              StatsAnalyzerModule.EVENTS.NO_FRAMES_SENT,
-              {mediaType: 'video'}
-            );
-
-            assert.calledWith(
-              TriggerProxy.trigger,
-              sinon.match.instanceOf(Meeting),
-              {
-                file: 'meeting/index',
-                function: 'compareLastStatsResult',
-              },
-              EVENT_TRIGGERS.MEETING_NO_FRAMES_SENT,
-              {
-                mediaType: 'video',
-              }
-            );
-            assert.calledWith(Metrics.sendBehavioralMetric, BEHAVIORAL_METRICS.NO_FRAMES_SENT);
-          });
-          it('NO_FRAMES_SENT triggers "meeting:noFramesSent" event and sends metrics for share', async () => {
-            meeting.mediaProperties.mediaDirection = {sendShare: true};
-            statsAnalyzerStub.emit(
-              {file: 'test', function: 'test'},
-              StatsAnalyzerModule.EVENTS.NO_FRAMES_SENT,
-              {mediaType: 'share'}
-            );
-
-            assert.calledWith(
-              TriggerProxy.trigger,
-              sinon.match.instanceOf(Meeting),
-              {
-                file: 'meeting/index',
-                function: 'compareLastStatsResult',
-              },
-              EVENT_TRIGGERS.MEETING_NO_FRAMES_SENT,
-              {
-                mediaType: 'share',
-              }
-            );
-            assert.calledWith(Metrics.sendBehavioralMetric, BEHAVIORAL_METRICS.NO_FRAMES_SENT);
-          });
-          it('NO_VIDEO_ENCODED triggers "meeting:noVideoEncoded" event and sends metrics', async () => {
-            statsAnalyzerStub.emit(
-              {file: 'test', function: 'test'},
-              StatsAnalyzerModule.EVENTS.NO_VIDEO_ENCODED,
-              {mediaType: 'video'}
-            );
-
-            assert.calledWith(
-              TriggerProxy.trigger,
-              sinon.match.instanceOf(Meeting),
-              {
-                file: 'meeting/index',
-                function: 'compareLastStatsResult',
-              },
-              EVENT_TRIGGERS.MEETING_NO_VIDEO_ENCODED,
-              {
-                mediaType: 'video',
-              }
-            );
-            assert.calledWith(Metrics.sendBehavioralMetric, BEHAVIORAL_METRICS.NO_VIDEO_ENCODED);
-          });
         });
       });
       describe('#acknowledge', () => {
@@ -3643,8 +3578,49 @@ describe('plugin-meetings', () => {
           );
           done();
         });
+        it('transcription should start when configured when guest admitted', (done) => {
+          meeting.isTranscriptionSupported = sinon.stub().returns(true);
+          meeting.receiveTranscription = sinon.stub().returns(true);
+          meeting.startTranscription = sinon.stub();
+
+          meeting.locusInfo.emit({function: 'test', file: 'test'}, 'SELF_ADMITTED_GUEST', test1);
+          assert.calledOnce(meeting.startTranscription);
+          done();
+        });
       });
 
+      describe('#setupLocusControlsListener', () => {
+        it('transcription should start when meeting transcribe state is updated with active transcribing', (done) => {
+          const payload = {caption: true, transcribing: true};
+          meeting.startTranscription = sinon.stub();
+          meeting.config.receiveTranscription = true;
+          meeting.transcription = null;
+
+          meeting.locusInfo.emit({function: 'meeting/index', file: 'setupLocusControlsListener'}, 'CONTROLS_MEETING_TRANSCRIBE_UPDATED', payload);
+          assert.calledOnce(meeting.startTranscription);
+          done();
+        })
+
+        it('transcription should stop when meeting transcribe state is updated with inactive transcribing', (done) => {
+          const payload = {caption: false, transcribing: false};
+          meeting.startTranscription = sinon.stub();
+          meeting.config.receiveTranscription = true;
+          meeting.transcription = {};
+
+          meeting.locusInfo.emit({function: 'meeting/index', file: 'setupLocusControlsListener'}, 'CONTROLS_MEETING_TRANSCRIBE_UPDATED', payload);
+          assert.notCalled(meeting.startTranscription);
+          assert.calledTwice(TriggerProxy.trigger);
+          assert.calledWith(
+            TriggerProxy.trigger,
+            sinon.match.instanceOf(Meeting),
+            {file: 'meeting/index', function: 'setupLocusControlsListener'},
+            'meeting:receiveTranscription:stopped',
+            payload
+          );
+          done();
+        })
+      })
+
       describe('#setUpLocusUrlListener', () => {
         it('listens to the locus url update event', (done) => {
           const newLocusUrl = 'newLocusUrl/12345';
@@ -3670,17 +3646,14 @@ describe('plugin-meetings', () => {
       describe('#setUpLocusServicesListener', () => {
         it('listens to the locus services update event', (done) => {
           const newLocusServices = {
-            services: {
-              record: {
-                url: 'url',
+              services: {
+                record: {
+                  url: 'url',
+                }
               },
-            },
           };
 
-          meeting.recordingController = {
-            setServiceUrl: sinon.stub().returns(undefined),
-            setSessionId: sinon.stub().returns(undefined),
-          };
+          meeting.recordingController = {setServiceUrl: sinon.stub().returns(undefined), setSessionId: sinon.stub().returns(undefined)};
 
           meeting.locusInfo.emit(
             {function: 'test', file: 'test'},
@@ -3688,10 +3661,7 @@ describe('plugin-meetings', () => {
             newLocusServices
           );
 
-          assert.calledWith(
-            meeting.recordingController.setServiceUrl,
-            newLocusServices.services.record.url
-          );
+          assert.calledWith(meeting.recordingController.setServiceUrl, newLocusServices.services.record.url);
           assert.calledOnce(meeting.recordingController.setSessionId);
           done();
         });

package/test/unit/spec/meeting-info/meetinginfov2.js

@@ -354,7 +354,7 @@ describe('plugin-meetings', () => {
 
         assert.calledWith(
           webex.request,
-          conversationUrl,
+          {uri:conversationUrl, qs: {includeParticipants: true}, disableTransform: true}
         )
 
         assert.calledWith(webex.request, {

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

@@ -41,92 +41,88 @@ describe('Roap', () => {
   describe('sendRoapMediaRequest', () => {
     let sendRoapStub;
     let roapHandlerSubmitStub;
-    let meeting;
+
+    const meeting = {
+      id: 'some meeting id',
+      correlationId: 'correlation id',
+      selfUrl: 'self url',
+      mediaId: 'media id',
+      audio:{
+        isLocallyMuted: () => true,
+      },
+      video:{
+        isLocallyMuted: () => false,
+      },
+      setRoapSeq: sinon.stub(),
+      config: {experimental: {enableTurnDiscovery: false}},
+    };
 
     beforeEach(() => {
-      meeting = {
-        id: 'some meeting id',
-        correlationId: 'correlation id',
-        selfUrl: 'self url',
-        mediaId: 'media id',
-        audio: {
-          isLocallyMuted: () => true,
-        },
-        video: {
-          isLocallyMuted: () => false,
-        },
-        setRoapSeq: sinon.stub(),
-        config: {experimental: {enableTurnDiscovery: false}},
-        webex: {meetings: {reachability: {isAnyClusterReachable: () => true}}},
-      };
-
-      beforeEach(() => {
-        sendRoapStub = sinon.stub(RoapRequest.prototype, 'sendRoap').resolves({});
-        roapHandlerSubmitStub = sinon.stub(RoapHandler.prototype, 'submit');
-        meeting.setRoapSeq.resetHistory();
-      });
-
-      afterEach(() => {
-        sinon.restore();
-      });
-
-      [
-        {reconnect: true, turnDiscoverySkipped: false, expectEmptyMediaId: false},
-        {reconnect: true, turnDiscoverySkipped: true, expectEmptyMediaId: true},
-        {reconnect: false, turnDiscoverySkipped: false, expectEmptyMediaId: false},
-        {reconnect: false, turnDiscoverySkipped: true, expectEmptyMediaId: false},
-      ].forEach(({reconnect, turnDiscoverySkipped, expectEmptyMediaId}) =>
-        it(`sends roap OFFER with ${expectEmptyMediaId ? 'empty ' : ''}mediaId when ${
-          reconnect ? '' : 'not '
-        }reconnecting and TURN discovery is ${
-          turnDiscoverySkipped ? 'skipped' : 'not skipped'
-        }`, async () => {
-          const roap = new Roap({}, {parent: 'fake'});
-
-          sinon.stub(roap.turnDiscovery, 'isSkipped').resolves(turnDiscoverySkipped);
-
-          await roap.sendRoapMediaRequest({
-            meeting,
-            sdp: 'sdp',
-            reconnect,
-            roapSeq: 1,
-          });
-
-          const expectedRoapMessage = {
-            messageType: 'OFFER',
-            sdps: ['sdp'],
-            version: '2',
-            seq: 2,
-            tieBreaker: 4294967294,
-          };
-
-          assert.calledOnce(sendRoapStub);
-          assert.calledWith(sendRoapStub, {
-            roapMessage: expectedRoapMessage,
-            correlationId: meeting.correlationId,
-            locusSelfUrl: meeting.selfUrl,
-            mediaId: expectEmptyMediaId ? '' : meeting.mediaId,
-            audioMuted: meeting.audio?.isLocallyMuted(),
-            videoMuted: meeting.video?.isLocallyMuted(),
-            meetingId: meeting.id,
-          });
-
-          assert.calledTwice(roapHandlerSubmitStub);
-          assert.calledWith(roapHandlerSubmitStub, {
-            type: ROAP.SEND_ROAP_MSG,
-            msg: expectedRoapMessage,
-            correlationId: meeting.correlationId,
-          });
-          assert.calledWith(roapHandlerSubmitStub, {
-            type: ROAP.SEND_ROAP_MSG_SUCCESS,
-            seq: 2,
-            correlationId: meeting.correlationId,
-          });
-
-          assert.calledOnce(meeting.setRoapSeq);
-          assert.calledWith(meeting.setRoapSeq, 2);
-        })
-      );
+      sendRoapStub = sinon.stub(RoapRequest.prototype, 'sendRoap').resolves({});
+      roapHandlerSubmitStub = sinon.stub(RoapHandler.prototype, 'submit');
+      meeting.setRoapSeq.resetHistory();
     });
+
+    afterEach(() => {
+      sinon.restore();
+    });
+
+    [
+      {reconnect: true, enableTurnDiscovery: true, expectEmptyMediaId: false},
+      {reconnect: true, enableTurnDiscovery: false, expectEmptyMediaId: true},
+      {reconnect: false, enableTurnDiscovery: true, expectEmptyMediaId: false},
+      {reconnect: false, enableTurnDiscovery: false, expectEmptyMediaId: false},
+    ].forEach(({reconnect, enableTurnDiscovery, expectEmptyMediaId}) =>
+      it(`sends roap OFFER with ${expectEmptyMediaId ? 'empty ' : ''}mediaId when ${
+        reconnect ? '' : 'not '
+      }reconnecting and TURN discovery is ${
+        enableTurnDiscovery ? 'enabled' : 'disabled'
+      }`, async () => {
+        meeting.config.experimental.enableTurnDiscovery = enableTurnDiscovery;
+
+        const roap = new Roap({}, {parent: 'fake'});
+
+        await roap.sendRoapMediaRequest({
+          meeting,
+          sdp: 'sdp',
+          reconnect,
+          roapSeq: 1,
+        });
+
+        const expectedRoapMessage = {
+          messageType: 'OFFER',
+          sdps: ['sdp'],
+          version: '2',
+          seq: 2,
+          tieBreaker: 4294967294,
+        };
+
+        assert.calledOnce(sendRoapStub);
+        assert.calledWith(sendRoapStub, {
+          roapMessage: expectedRoapMessage,
+          correlationId: meeting.correlationId,
+          locusSelfUrl: meeting.selfUrl,
+          mediaId: expectEmptyMediaId ? '' : meeting.mediaId,
+          audioMuted: meeting.audio?.isLocallyMuted(),
+          videoMuted: meeting.video?.isLocallyMuted(),
+          meetingId: meeting.id,
+        });
+
+        assert.calledTwice(roapHandlerSubmitStub);
+        assert.calledWith(roapHandlerSubmitStub, {
+          type: ROAP.SEND_ROAP_MSG,
+          msg: expectedRoapMessage,
+          correlationId: meeting.correlationId,
+        });
+        assert.calledWith(roapHandlerSubmitStub, {
+          type: ROAP.SEND_ROAP_MSG_SUCCESS,
+          seq: 2,
+          correlationId: meeting.correlationId,
+        });
+
+        assert.calledOnce(meeting.setRoapSeq);
+        assert.calledWith(meeting.setRoapSeq, 2);
+      })
+    );
   });
 });

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

@@ -352,27 +352,6 @@ describe('TurnDiscovery', () => {
     });
   });
 
-  describe('isSkipped', () => {
-    [
-      {enabledInConfig: true, isAnyClusterReachable: true, expectedIsSkipped: true},
-      {enabledInConfig: true, isAnyClusterReachable: false, expectedIsSkipped: false},
-      {enabledInConfig: false, isAnyClusterReachable: true, expectedIsSkipped: true},
-      {enabledInConfig: false, isAnyClusterReachable: false, expectedIsSkipped: true},
-    ].forEach(({enabledInConfig, isAnyClusterReachable, expectedIsSkipped}) => {
-      it(`returns ${expectedIsSkipped} when TURN discovery is ${enabledInConfig ? '' : 'not '} enabled in config and isAnyClusterReachable() returns ${isAnyClusterReachable ? 'true' : 'false'}`, async () => {
-        testMeeting.config.experimental.enableTurnDiscovery = enabledInConfig;
-
-        sinon.stub(testMeeting.webex.meetings.reachability, 'isAnyClusterReachable').resolves(isAnyClusterReachable);
-
-        const td = new TurnDiscovery(mockRoapRequest);
-
-        const isSkipped = await td.isSkipped(testMeeting);
-
-        assert.equal(isSkipped, expectedIsSkipped);
-      })
-    })
-  })
-
   describe('handleTurnDiscoveryResponse', () => {
     it("doesn't do anything if turn discovery was not started", () => {
       const td = new TurnDiscovery(mockRoapRequest);