@webex/plugin-meetings 2.59.3-next.1 → 2.59.4-next.1

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

@@ -1,30 +1,30 @@
-import 'jsdom-global/register';
-import {assert} from '@webex/test-helper-chai';
-import RoapUtil from '@webex/plugin-meetings/src/roap/util';
-import PeerConnectionManager from '@webex/plugin-meetings/src/peer-connection-manager/index';
-import sinon from 'sinon';
-
-describe('RoapUtil', () => {
-  describe('updatePeerConnection', () => {
-    let meeting, session;
-
-    beforeEach('stub PeerConnectionManager', () => {
-      meeting = {
-        mediaProperties: {
-          peerConnection: {name: 'peer-connection'},
-        },
-        roap: {
-          lastRoapOffer: 'lastRoapOffer',
-        },
-      };
-      session = {OFFER: {sdps: ['sdp1', 'sdp2']}};
-
-      PeerConnectionManager.updatePeerConnection = sinon.stub().returns(Promise.resolve());
-    });
-
-    it('should just call PeerConnection.updatePeerConnection', async () => {
-      await RoapUtil.updatePeerConnection(meeting, session);
-      assert.calledOnce(PeerConnectionManager.updatePeerConnection);
-    });
-  });
-});
+import 'jsdom-global/register';
+import {assert} from '@webex/test-helper-chai';
+import RoapUtil from '@webex/plugin-meetings/src/roap/util';
+import PeerConnectionManager from '@webex/plugin-meetings/src/peer-connection-manager/index';
+import sinon from 'sinon';
+
+describe('RoapUtil', () => {
+  describe('updatePeerConnection', () => {
+    let meeting, session;
+
+    beforeEach('stub PeerConnectionManager', () => {
+      meeting = {
+        mediaProperties: {
+          peerConnection: {name: 'peer-connection'},
+        },
+        roap: {
+          lastRoapOffer: 'lastRoapOffer',
+        },
+      };
+      session = {OFFER: {sdps: ['sdp1', 'sdp2']}};
+
+      PeerConnectionManager.updatePeerConnection = sinon.stub().returns(Promise.resolve());
+    });
+
+    it('should just call PeerConnection.updatePeerConnection', async () => {
+      await RoapUtil.updatePeerConnection(meeting, session);
+      assert.calledOnce(PeerConnectionManager.updatePeerConnection);
+    });
+  });
+});

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

@@ -1,287 +1,287 @@
-import 'jsdom-global/register';
-import chai from 'chai';
-import chaiAsPromised from 'chai-as-promised';
-import sinon from 'sinon';
-
-import {StatsAnalyzer, EVENTS} from '../../../../src/statsAnalyzer';
-import NetworkQualityMonitor from '../../../../src/networkQualityMonitor';
-import testUtils from '../../../utils/testUtils';
-
-const {assert} = chai;
-
-chai.use(chaiAsPromised);
-sinon.assert.expose(chai.assert, {prefix: ''});
-
-describe('plugin-meetings', () => {
-  describe('StatsAnalyzer', () => {
-    describe('compareSentAndReceived()', () => {
-      let statsAnalyzer;
-      let sandBoxSpy;
-
-      const initialConfig = {
-        videoPacketLossRatioThreshold: 9,
-      };
-
-      const defaultStats = {
-        internal: {
-          video: {
-            send: {
-              totalPacketsLostOnReceiver: 10,
-            },
-          },
-        },
-        video: {
-          send: {
-            packetsSent: 2,
-            meanRemoteJitter: [],
-            meanRoundTripTime: [],
-          },
-        },
-      };
-
-      const statusResult = {
-        type: 'remote-inbound-rtp',
-        packetsLost: 11,
-        rttThreshold: 501,
-        jitterThreshold: 501,
-      };
-
-      const sandbox = sinon.createSandbox();
-
-      beforeEach(() => {
-        const networkQualityMonitor = new NetworkQualityMonitor(initialConfig);
-
-        statsAnalyzer = new StatsAnalyzer(initialConfig, networkQualityMonitor, defaultStats);
-
-        sandBoxSpy = sandbox.spy(
-          statsAnalyzer.networkQualityMonitor,
-          'determineUplinkNetworkQuality'
-        );
-      });
-
-      afterEach(() => {
-        sandbox.restore();
-      });
-
-      it('should trigger determineUplinkNetworkQuality with specific arguments', async () => {
-        await statsAnalyzer.parseGetStatsResult(statusResult, 'video');
-
-        assert.calledOnce(statsAnalyzer.networkQualityMonitor.determineUplinkNetworkQuality);
-        assert(
-          sandBoxSpy.calledWith({
-            mediaType: 'video',
-            remoteRtpResults: statusResult,
-            statsAnalyzerCurrentStats: statsAnalyzer.statsResults,
-          })
-        );
-      });
-    });
-
-    describe('startAnalyzer', () => {
-      let clock;
-      let pc;
-      let networkQualityMonitor;
-      let statsAnalyzer;
-
-      let receivedEventsData = {
-        local: {},
-        remote: {},
-      };
-
-      const initialConfig = {
-        analyzerInterval: 1000,
-      };
-
-      let fakeStats;
-
-      const resetReceivedEvents = () => {
-        receivedEventsData = {
-          local: {},
-          remote: {},
-        };
-      };
-
-      beforeEach(() => {
-        clock = sinon.useFakeTimers();
-
-        resetReceivedEvents();
-
-        // bytesReceived and bytesSent need to be non-zero in order for StatsAnalyzer to parse any other values
-        fakeStats = {
-          audio: {
-            receiver: {
-              type: 'inbound-rtp',
-              packetsReceived: 0,
-              bytesReceived: 1,
-            },
-            sender: {
-              type: 'outbound-rtp',
-              packetsSent: 0,
-              bytesSent: 1,
-            },
-          },
-          video: {
-            receiver: {
-              type: 'inbound-rtp',
-              framesDecoded: 0,
-              bytesReceived: 1,
-            },
-            sender: {
-              type: 'outbound-rtp',
-              framesSent: 0,
-              bytesSent: 1,
-            },
-          },
-        };
-
-        pc = {
-          audioTransceiver: {
-            sender: {
-              getStats: sinon.stub().resolves([fakeStats.audio.sender]),
-            },
-            receiver: {
-              getStats: sinon.stub().resolves([fakeStats.audio.receiver]),
-            },
-          },
-          videoTransceiver: {
-            sender: {
-              getStats: sinon.stub().resolves([fakeStats.video.sender]),
-            },
-            receiver: {
-              getStats: sinon.stub().resolves([fakeStats.video.receiver]),
-            },
-          },
-          shareTransceiver: {
-            sender: {
-              getStats: sinon.stub().resolves([]),
-            },
-            receiver: {
-              getStats: sinon.stub().resolves([]),
-            },
-          },
-        };
-
-        networkQualityMonitor = new NetworkQualityMonitor(initialConfig);
-
-        statsAnalyzer = new StatsAnalyzer(initialConfig, networkQualityMonitor);
-
-        statsAnalyzer.on(EVENTS.LOCAL_MEDIA_STARTED, (data) => {
-          receivedEventsData.local.started = data;
-        });
-        statsAnalyzer.on(EVENTS.LOCAL_MEDIA_STOPPED, (data) => {
-          receivedEventsData.local.stopped = data;
-        });
-        statsAnalyzer.on(EVENTS.REMOTE_MEDIA_STARTED, (data) => {
-          receivedEventsData.remote.started = data;
-        });
-        statsAnalyzer.on(EVENTS.REMOTE_MEDIA_STOPPED, (data) => {
-          receivedEventsData.remote.stopped = data;
-        });
-      });
-
-      afterEach(() => {
-        clock.restore();
-      });
-
-      const startStatsAnalyzer = async (mediaStatus) => {
-        statsAnalyzer.updateMediaStatus(mediaStatus);
-        statsAnalyzer.startAnalyzer(pc);
-
-        await testUtils.flushPromises();
-      };
-
-      const progressTime = async () => {
-        await clock.tickAsync(initialConfig.analyzerInterval);
-        await testUtils.flushPromises();
-      };
-
-      const checkReceivedEvent = ({expected}) => {
-        // check that we got the REMOTE_MEDIA_STARTED event for audio
-        assert.deepEqual(receivedEventsData.local.started, expected.local?.started);
-        assert.deepEqual(receivedEventsData.local.stopped, expected.local?.stopped);
-        assert.deepEqual(receivedEventsData.remote.started, expected.remote?.started);
-        assert.deepEqual(receivedEventsData.remote.stopped, expected.remote?.stopped);
-      };
-
-      it('emits LOCAL_MEDIA_STARTED and LOCAL_MEDIA_STOPPED events for audio', async () => {
-        await startStatsAnalyzer({expected: {sendAudio: true}});
-
-        // check that we haven't received any events yet
-        checkReceivedEvent({expected: {}});
-
-        // setup a mock to return some values higher the previous ones
-        fakeStats.audio.sender.packetsSent += 10;
-
-        await progressTime();
-
-        // check that we got the LOCAL_MEDIA_STARTED event for audio
-        checkReceivedEvent({expected: {local: {started: {type: 'audio'}}}});
-
-        // now advance the clock and the mock still returns same values, so only "stopped" event should be triggered
-        resetReceivedEvents();
-        await progressTime();
-        checkReceivedEvent({expected: {local: {stopped: {type: 'audio'}}}});
-      });
-
-      it('emits LOCAL_MEDIA_STARTED and LOCAL_MEDIA_STOPPED events for video', async () => {
-        await startStatsAnalyzer({expected: {sendVideo: true}});
-
-        // check that we haven't received any events yet
-        checkReceivedEvent({expected: {}});
-
-        // setup a mock to return some values higher the previous ones
-        fakeStats.video.sender.framesSent += 1;
-
-        await progressTime();
-
-        // check that we got the LOCAL_MEDIA_STARTED event for audio
-        checkReceivedEvent({expected: {local: {started: {type: 'video'}}}});
-
-        // now advance the clock and the mock still returns same values, so only "stopped" event should be triggered
-        resetReceivedEvents();
-        await progressTime();
-        checkReceivedEvent({expected: {local: {stopped: {type: 'video'}}}});
-      });
-
-      it('emits REMOTE_MEDIA_STARTED and REMOTE_MEDIA_STOPPED events for audio', async () => {
-        await startStatsAnalyzer({expected: {receiveAudio: true}});
-
-        // check that we haven't received any events yet
-        checkReceivedEvent({expected: {}});
-
-        // setup a mock to return some values higher the previous ones
-        fakeStats.audio.receiver.packetsReceived += 5;
-
-        await progressTime();
-        // check that we got the REMOTE_MEDIA_STARTED event for audio
-        checkReceivedEvent({expected: {remote: {started: {type: 'audio'}}}});
-
-        // now advance the clock and the mock still returns same values, so only "stopped" event should be triggered
-        resetReceivedEvents();
-        await progressTime();
-
-        checkReceivedEvent({expected: {remote: {stopped: {type: 'audio'}}}});
-      });
-
-      it('emits REMOTE_MEDIA_STARTED and REMOTE_MEDIA_STOPPED events for video', async () => {
-        await startStatsAnalyzer({expected: {receiveVideo: true}});
-
-        // check that we haven't received any events yet
-        checkReceivedEvent({expected: {}});
-
-        // setup a mock to return some values higher the previous ones
-        fakeStats.video.receiver.framesDecoded += 1;
-
-        await progressTime();
-        // check that we got the REMOTE_MEDIA_STARTED event for video
-        checkReceivedEvent({expected: {remote: {started: {type: 'video'}}}});
-
-        // now advance the clock and the mock still returns same values, so only "stopped" event should be triggered
-        resetReceivedEvents();
-        await progressTime();
-
-        checkReceivedEvent({expected: {remote: {stopped: {type: 'video'}}}});
-      });
-    });
-  });
-});
+import 'jsdom-global/register';
+import chai from 'chai';
+import chaiAsPromised from 'chai-as-promised';
+import sinon from 'sinon';
+
+import {StatsAnalyzer, EVENTS} from '../../../../src/statsAnalyzer';
+import NetworkQualityMonitor from '../../../../src/networkQualityMonitor';
+import testUtils from '../../../utils/testUtils';
+
+const {assert} = chai;
+
+chai.use(chaiAsPromised);
+sinon.assert.expose(chai.assert, {prefix: ''});
+
+describe('plugin-meetings', () => {
+  describe('StatsAnalyzer', () => {
+    describe('compareSentAndReceived()', () => {
+      let statsAnalyzer;
+      let sandBoxSpy;
+
+      const initialConfig = {
+        videoPacketLossRatioThreshold: 9,
+      };
+
+      const defaultStats = {
+        internal: {
+          video: {
+            send: {
+              totalPacketsLostOnReceiver: 10,
+            },
+          },
+        },
+        video: {
+          send: {
+            packetsSent: 2,
+            meanRemoteJitter: [],
+            meanRoundTripTime: [],
+          },
+        },
+      };
+
+      const statusResult = {
+        type: 'remote-inbound-rtp',
+        packetsLost: 11,
+        rttThreshold: 501,
+        jitterThreshold: 501,
+      };
+
+      const sandbox = sinon.createSandbox();
+
+      beforeEach(() => {
+        const networkQualityMonitor = new NetworkQualityMonitor(initialConfig);
+
+        statsAnalyzer = new StatsAnalyzer(initialConfig, networkQualityMonitor, defaultStats);
+
+        sandBoxSpy = sandbox.spy(
+          statsAnalyzer.networkQualityMonitor,
+          'determineUplinkNetworkQuality'
+        );
+      });
+
+      afterEach(() => {
+        sandbox.restore();
+      });
+
+      it('should trigger determineUplinkNetworkQuality with specific arguments', async () => {
+        await statsAnalyzer.parseGetStatsResult(statusResult, 'video');
+
+        assert.calledOnce(statsAnalyzer.networkQualityMonitor.determineUplinkNetworkQuality);
+        assert(
+          sandBoxSpy.calledWith({
+            mediaType: 'video',
+            remoteRtpResults: statusResult,
+            statsAnalyzerCurrentStats: statsAnalyzer.statsResults,
+          })
+        );
+      });
+    });
+
+    describe('startAnalyzer', () => {
+      let clock;
+      let pc;
+      let networkQualityMonitor;
+      let statsAnalyzer;
+
+      let receivedEventsData = {
+        local: {},
+        remote: {},
+      };
+
+      const initialConfig = {
+        analyzerInterval: 1000,
+      };
+
+      let fakeStats;
+
+      const resetReceivedEvents = () => {
+        receivedEventsData = {
+          local: {},
+          remote: {},
+        };
+      };
+
+      beforeEach(() => {
+        clock = sinon.useFakeTimers();
+
+        resetReceivedEvents();
+
+        // bytesReceived and bytesSent need to be non-zero in order for StatsAnalyzer to parse any other values
+        fakeStats = {
+          audio: {
+            receiver: {
+              type: 'inbound-rtp',
+              packetsReceived: 0,
+              bytesReceived: 1,
+            },
+            sender: {
+              type: 'outbound-rtp',
+              packetsSent: 0,
+              bytesSent: 1,
+            },
+          },
+          video: {
+            receiver: {
+              type: 'inbound-rtp',
+              framesDecoded: 0,
+              bytesReceived: 1,
+            },
+            sender: {
+              type: 'outbound-rtp',
+              framesSent: 0,
+              bytesSent: 1,
+            },
+          },
+        };
+
+        pc = {
+          audioTransceiver: {
+            sender: {
+              getStats: sinon.stub().resolves([fakeStats.audio.sender]),
+            },
+            receiver: {
+              getStats: sinon.stub().resolves([fakeStats.audio.receiver]),
+            },
+          },
+          videoTransceiver: {
+            sender: {
+              getStats: sinon.stub().resolves([fakeStats.video.sender]),
+            },
+            receiver: {
+              getStats: sinon.stub().resolves([fakeStats.video.receiver]),
+            },
+          },
+          shareTransceiver: {
+            sender: {
+              getStats: sinon.stub().resolves([]),
+            },
+            receiver: {
+              getStats: sinon.stub().resolves([]),
+            },
+          },
+        };
+
+        networkQualityMonitor = new NetworkQualityMonitor(initialConfig);
+
+        statsAnalyzer = new StatsAnalyzer(initialConfig, networkQualityMonitor);
+
+        statsAnalyzer.on(EVENTS.LOCAL_MEDIA_STARTED, (data) => {
+          receivedEventsData.local.started = data;
+        });
+        statsAnalyzer.on(EVENTS.LOCAL_MEDIA_STOPPED, (data) => {
+          receivedEventsData.local.stopped = data;
+        });
+        statsAnalyzer.on(EVENTS.REMOTE_MEDIA_STARTED, (data) => {
+          receivedEventsData.remote.started = data;
+        });
+        statsAnalyzer.on(EVENTS.REMOTE_MEDIA_STOPPED, (data) => {
+          receivedEventsData.remote.stopped = data;
+        });
+      });
+
+      afterEach(() => {
+        clock.restore();
+      });
+
+      const startStatsAnalyzer = async (mediaStatus) => {
+        statsAnalyzer.updateMediaStatus(mediaStatus);
+        statsAnalyzer.startAnalyzer(pc);
+
+        await testUtils.flushPromises();
+      };
+
+      const progressTime = async () => {
+        await clock.tickAsync(initialConfig.analyzerInterval);
+        await testUtils.flushPromises();
+      };
+
+      const checkReceivedEvent = ({expected}) => {
+        // check that we got the REMOTE_MEDIA_STARTED event for audio
+        assert.deepEqual(receivedEventsData.local.started, expected.local?.started);
+        assert.deepEqual(receivedEventsData.local.stopped, expected.local?.stopped);
+        assert.deepEqual(receivedEventsData.remote.started, expected.remote?.started);
+        assert.deepEqual(receivedEventsData.remote.stopped, expected.remote?.stopped);
+      };
+
+      it('emits LOCAL_MEDIA_STARTED and LOCAL_MEDIA_STOPPED events for audio', async () => {
+        await startStatsAnalyzer({expected: {sendAudio: true}});
+
+        // check that we haven't received any events yet
+        checkReceivedEvent({expected: {}});
+
+        // setup a mock to return some values higher the previous ones
+        fakeStats.audio.sender.packetsSent += 10;
+
+        await progressTime();
+
+        // check that we got the LOCAL_MEDIA_STARTED event for audio
+        checkReceivedEvent({expected: {local: {started: {type: 'audio'}}}});
+
+        // now advance the clock and the mock still returns same values, so only "stopped" event should be triggered
+        resetReceivedEvents();
+        await progressTime();
+        checkReceivedEvent({expected: {local: {stopped: {type: 'audio'}}}});
+      });
+
+      it('emits LOCAL_MEDIA_STARTED and LOCAL_MEDIA_STOPPED events for video', async () => {
+        await startStatsAnalyzer({expected: {sendVideo: true}});
+
+        // check that we haven't received any events yet
+        checkReceivedEvent({expected: {}});
+
+        // setup a mock to return some values higher the previous ones
+        fakeStats.video.sender.framesSent += 1;
+
+        await progressTime();
+
+        // check that we got the LOCAL_MEDIA_STARTED event for audio
+        checkReceivedEvent({expected: {local: {started: {type: 'video'}}}});
+
+        // now advance the clock and the mock still returns same values, so only "stopped" event should be triggered
+        resetReceivedEvents();
+        await progressTime();
+        checkReceivedEvent({expected: {local: {stopped: {type: 'video'}}}});
+      });
+
+      it('emits REMOTE_MEDIA_STARTED and REMOTE_MEDIA_STOPPED events for audio', async () => {
+        await startStatsAnalyzer({expected: {receiveAudio: true}});
+
+        // check that we haven't received any events yet
+        checkReceivedEvent({expected: {}});
+
+        // setup a mock to return some values higher the previous ones
+        fakeStats.audio.receiver.packetsReceived += 5;
+
+        await progressTime();
+        // check that we got the REMOTE_MEDIA_STARTED event for audio
+        checkReceivedEvent({expected: {remote: {started: {type: 'audio'}}}});
+
+        // now advance the clock and the mock still returns same values, so only "stopped" event should be triggered
+        resetReceivedEvents();
+        await progressTime();
+
+        checkReceivedEvent({expected: {remote: {stopped: {type: 'audio'}}}});
+      });
+
+      it('emits REMOTE_MEDIA_STARTED and REMOTE_MEDIA_STOPPED events for video', async () => {
+        await startStatsAnalyzer({expected: {receiveVideo: true}});
+
+        // check that we haven't received any events yet
+        checkReceivedEvent({expected: {}});
+
+        // setup a mock to return some values higher the previous ones
+        fakeStats.video.receiver.framesDecoded += 1;
+
+        await progressTime();
+        // check that we got the REMOTE_MEDIA_STARTED event for video
+        checkReceivedEvent({expected: {remote: {started: {type: 'video'}}}});
+
+        // now advance the clock and the mock still returns same values, so only "stopped" event should be triggered
+        resetReceivedEvents();
+        await progressTime();
+
+        checkReceivedEvent({expected: {remote: {stopped: {type: 'video'}}}});
+      });
+    });
+  });
+});