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

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

@@ -548,34 +548,3 @@ describe('extractMediaStatus', () => {
     assert.deepEqual(mediaStatus, {audio: 'RECVONLY', video: 'SENDRECV'});
   });
 });
-
-describe('extractMediaStatus', () => {
-  it('throws error when there is no participant', () => {
-    assert.throws(() => {
-      MemberUtil.extractMediaStatus()
-    }, 'Media status could not be extracted, participant is undefined.');
-  });
-
-  it('returns undefined media status when participant audio/video status is not present', () => {
-    const participant = {
-      status: {}
-    };
-    
-    const mediaStatus = MemberUtil.extractMediaStatus(participant)
-
-    assert.deepEqual(mediaStatus, {audio: undefined, video: undefined});
-  });
-
-  it('returns correct media status when participant audio/video status is present', () => {
-    const participant = {
-      status: {
-        audioStatus: 'RECVONLY',
-        videoStatus: 'SENDRECV'
-      }
-    };
-    
-    const mediaStatus = MemberUtil.extractMediaStatus(participant)
-
-    assert.deepEqual(mediaStatus, {audio: 'RECVONLY', video: 'SENDRECV'});
-  });
-});

package/test/unit/spec/reconnection-manager/index.js

@@ -3,6 +3,7 @@ import chai from 'chai';
 import chaiAsPromised from 'chai-as-promised';
 import sinon from 'sinon';
 import ReconnectionManager from '@webex/plugin-meetings/src/reconnection-manager';
+import { RECONNECTION } from '../../../../src/constants';
 
 const {assert} = chai;
 
@@ -114,16 +115,6 @@ describe('plugin-meetings', () => {
           meetingId: rm.meeting.id,
         },
       });
-
-      assert.calledWith(fakeMeeting.webex.internal.newMetrics.submitClientEvent, {
-        name: 'client.media.recovered',
-        payload: {
-          recoveredBy: 'new',
-        },
-        options: {
-          meetingId: rm.meeting.id,
-        },
-      });
     });
 
     it('does not clear previous requests and re-request media for non-multistream meetings', async () => {
@@ -187,9 +178,10 @@ describe('plugin-meetings', () => {
    */
   describe('ReconnectionManager', () => {
     let reconnectionManager;
+    let fakeMeeting;
 
     beforeEach(() => {
-      reconnectionManager = new ReconnectionManager({
+      fakeMeeting = {
         config: {
           reconnection: {
             enabled: true,
@@ -204,7 +196,9 @@ describe('plugin-meetings', () => {
             },
           },
         },
-      });
+      };
+
+      reconnectionManager = new ReconnectionManager(fakeMeeting);
     });
 
     describe('iceReconnected()', () => {
@@ -310,5 +304,41 @@ describe('plugin-meetings', () => {
         });
       });
     });
+
+    describe('setStatus()', () => {
+      beforeEach(() => {
+        reconnectionManager.status = RECONNECTION.STATE.DEFAULT_STATUS;
+      });
+
+      it('should correctly change status to in progress', () => {
+        reconnectionManager.setStatus(RECONNECTION.STATE.IN_PROGRESS);
+
+        assert.equal(reconnectionManager.status, RECONNECTION.STATE.IN_PROGRESS);
+      });
+
+      it('should correctly change status to complete', () => {
+        reconnectionManager.setStatus(RECONNECTION.STATE.COMPLETE);
+
+        assert.equal(reconnectionManager.status, RECONNECTION.STATE.COMPLETE);
+      });
+
+      it('should correctly change status to failure', () => {
+        reconnectionManager.setStatus(RECONNECTION.STATE.FAILURE);
+
+        assert.equal(reconnectionManager.status, RECONNECTION.STATE.FAILURE);
+      });
+    });
+
+    describe('cleanUp()', () => {
+      it('should call reset and keep reference to meeting object', () => {
+        const resetSpy = sinon.spy(reconnectionManager, 'reset');
+        assert.equal(reconnectionManager.meeting, fakeMeeting);
+
+        reconnectionManager.cleanUp();
+
+        assert.equal(reconnectionManager.meeting, fakeMeeting);
+        assert.calledOnce(reconnectionManager.reset);
+      });
+    });
   });
 });

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

@@ -6,15 +6,26 @@ import RTC_METRICS from '../../../../src/rtcMetrics/constants';
 
 const FAKE_METRICS_ITEM = {payload: ['fake-metrics']};
 
+const STATS_WITH_IP = '{\"id\":\"RTCIceCandidate_/kQs0ZNU\",\"type\":\"remote-candidate\",\"transportId\":\"RTCTransport_0_1\",\"isRemote\":true,\"ip\":\"11.22.111.255\",\"address\":\"11.22.111.255\",\"port\":5004,\"protocol\":\"udp\",\"candidateType\":\"host\",\"priority\":2130706431}';
+const STATS_WITH_IP_RESULT = '{\"id\":\"RTCIceCandidate_/kQs0ZNU\",\"type\":\"remote-candidate\",\"transportId\":\"RTCTransport_0_1\",\"isRemote\":true,\"ip\":\"11.22.111.240\",\"address\":\"11.22.111.240\",\"port\":5004,\"protocol\":\"udp\",\"candidateType\":\"host\",\"priority\":2130706431}';
+
 describe('RtcMetrics', () => {
   let metrics: RtcMetrics;
   let webex: MockWebex;
   let clock;
+  let anonymizeIpSpy;
+
+  const sandbox = sinon.createSandbox();
 
   beforeEach(() => {
     clock = sinon.useFakeTimers();
     webex = new MockWebex();
     metrics = new RtcMetrics(webex, 'mock-meeting-id', 'mock-correlation-id');
+    anonymizeIpSpy = sandbox.spy(metrics, 'anonymizeIp');
+  });
+
+  afterEach(() => {
+    sandbox.restore();
   });
 
   it('sendMetrics should send a webex request', () => {
@@ -70,4 +81,13 @@ describe('RtcMetrics', () => {
 
     assert.callCount(webex.request, 1);
   });
+
+  it('should anonymize IP addresses', () => {
+    assert.strictEqual(metrics.anonymizeIp(STATS_WITH_IP), STATS_WITH_IP_RESULT);
+  });
+
+  it('should call anonymizeIp', () => {
+    metrics.addMetrics({ name: 'stats-report', payload: [STATS_WITH_IP] });
+    assert.calledOnce(anonymizeIpSpy);
+  })
 });

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

@@ -179,7 +179,7 @@ describe('plugin-meetings', () => {
                 report: [
                   {
                     type: 'outbound-rtp',
-                    framesSent: 0,
+                    framesSent: 1500,
                     bytesSent: 1,
                   },
                   {
@@ -209,7 +209,7 @@ describe('plugin-meetings', () => {
                     bytesReceived: 1,
                     frameHeight: 720,
                     frameWidth: 1280,
-                    framesReceived: 1,
+                    framesReceived: 1500,
                   },
                   {
                     type: 'candidate-pair',
@@ -437,6 +437,16 @@ describe('plugin-meetings', () => {
         assert.strictEqual(mqeData.intervalMetadata.peripherals.find((val) => val.name === MEDIA_DEVICES.MICROPHONE).information, _UNKNOWN_);
         assert.strictEqual(mqeData.intervalMetadata.peripherals.find((val) => val.name === MEDIA_DEVICES.CAMERA).information, _UNKNOWN_);
       });
+
+      it('emits the correct frameRate', async () => {
+        await startStatsAnalyzer({expected: {receiveVideo: true}});
+
+        await progressTime();
+        assert.strictEqual(mqeData.videoReceive[0].streams[0].common.receivedFrameRate, 25);
+        fakeStats.video.receivers[0].framesReceived = 3000;
+        await progressTime();
+        assert.strictEqual(mqeData.videoReceive[0].streams[0].common.receivedFrameRate, 25);
+      });
     });
   });
 });