mediasoup-client 3.6.96 → 3.6.98

package/lib/tests/test.js

@@ -212,7 +212,6 @@ test('transport.produce() without "connect" listener rejects', async () => {
         .toThrow(Error);
 }, 500);
 test('transport.produce() succeeds', async () => {
-    var _a, _b;
     const audioTrack = new fake_mediastreamtrack_1.FakeMediaStreamTrack({ kind: 'audio' });
     const videoTrack = new fake_mediastreamtrack_1.FakeMediaStreamTrack({ kind: 'video' });
     let audioProducerId;
@@ -307,12 +306,12 @@ test('transport.produce() succeeds', async () => {
     encodings = audioProducer.rtpParameters.encodings;
     expect(Array.isArray(encodings)).toBe(true);
     expect(encodings.length).toBe(1);
-    expect(typeof (encodings === null || encodings === void 0 ? void 0 : encodings[0])).toBe('object');
+    expect(typeof encodings?.[0]).toBe('object');
     expect(Object.keys(encodings[0])).toEqual(['ssrc', 'dtx']);
-    expect(typeof (encodings === null || encodings === void 0 ? void 0 : encodings[0].ssrc)).toBe('number');
+    expect(typeof encodings?.[0].ssrc).toBe('number');
     rtcp = audioProducer.rtpParameters.rtcp;
     expect(typeof rtcp).toBe('object');
-    expect(typeof (rtcp === null || rtcp === void 0 ? void 0 : rtcp.cname)).toBe('string');
+    expect(typeof rtcp?.cname).toBe('string');
     expect(audioProducer.paused).toBe(true);
     expect(audioProducer.maxSpatialLayer).toBe(undefined);
     expect(audioProducer.appData).toEqual({ foo: 'FOO' });
@@ -401,19 +400,19 @@ test('transport.produce() succeeds', async () => {
     encodings = videoProducer.rtpParameters.encodings;
     expect(Array.isArray(encodings)).toBe(true);
     expect(encodings.length).toBe(2);
-    expect(typeof (encodings === null || encodings === void 0 ? void 0 : encodings[0])).toBe('object');
-    expect(typeof (encodings === null || encodings === void 0 ? void 0 : encodings[0].ssrc)).toBe('number');
-    expect(typeof (encodings === null || encodings === void 0 ? void 0 : encodings[0].rtx)).toBe('object');
+    expect(typeof encodings?.[0]).toBe('object');
+    expect(typeof encodings?.[0].ssrc).toBe('number');
+    expect(typeof encodings?.[0].rtx).toBe('object');
     expect(Object.keys(encodings[0].rtx)).toEqual(['ssrc']);
-    expect(typeof ((_a = encodings === null || encodings === void 0 ? void 0 : encodings[0].rtx) === null || _a === void 0 ? void 0 : _a.ssrc)).toBe('number');
-    expect(typeof (encodings === null || encodings === void 0 ? void 0 : encodings[1])).toBe('object');
-    expect(typeof (encodings === null || encodings === void 0 ? void 0 : encodings[1].ssrc)).toBe('number');
-    expect(typeof (encodings === null || encodings === void 0 ? void 0 : encodings[1].rtx)).toBe('object');
+    expect(typeof encodings?.[0].rtx?.ssrc).toBe('number');
+    expect(typeof encodings?.[1]).toBe('object');
+    expect(typeof encodings?.[1].ssrc).toBe('number');
+    expect(typeof encodings?.[1].rtx).toBe('object');
     expect(Object.keys(encodings[1].rtx)).toEqual(['ssrc']);
-    expect(typeof ((_b = encodings === null || encodings === void 0 ? void 0 : encodings[1].rtx) === null || _b === void 0 ? void 0 : _b.ssrc)).toBe('number');
+    expect(typeof encodings?.[1].rtx?.ssrc).toBe('number');
     rtcp = videoProducer.rtpParameters.rtcp;
     expect(typeof rtcp).toBe('object');
-    expect(typeof (rtcp === null || rtcp === void 0 ? void 0 : rtcp.cname)).toBe('string');
+    expect(typeof rtcp?.cname).toBe('string');
     expect(videoProducer.paused).toBe(false);
     expect(videoProducer.maxSpatialLayer).toBe(undefined);
     expect(videoProducer.appData).toEqual({});
@@ -446,7 +445,6 @@ test('transport.produce() with a non object appData rejects with TypeError', asy
         .toThrow(TypeError);
 }, 500);
 test('transport.consume() succeeds', async () => {
-    var _a;
     const audioConsumerRemoteParameters = fakeParameters.generateConsumerRemoteParameters({ codecMimeType: 'audio/opus' });
     const videoConsumerRemoteParameters = fakeParameters.generateConsumerRemoteParameters({ codecMimeType: 'video/VP8' });
     let connectEventNumTimesCalled = 0;
@@ -517,12 +515,12 @@ test('transport.consume() succeeds', async () => {
     encodings = audioConsumer.rtpParameters.encodings;
     expect(Array.isArray(encodings)).toBe(true);
     expect(encodings.length).toBe(1);
-    expect(typeof (encodings === null || encodings === void 0 ? void 0 : encodings[0])).toBe('object');
+    expect(typeof encodings?.[0]).toBe('object');
     expect(Object.keys(encodings[0])).toEqual(['ssrc', 'dtx']);
     expect(typeof encodings[0].ssrc).toBe('number');
     rtcp = audioProducer.rtpParameters.rtcp;
     expect(typeof rtcp).toBe('object');
-    expect(typeof (rtcp === null || rtcp === void 0 ? void 0 : rtcp.cname)).toBe('string');
+    expect(typeof rtcp?.cname).toBe('string');
     expect(audioConsumer.paused).toBe(false);
     expect(audioConsumer.appData).toEqual({ bar: 'BAR' });
     videoConsumer = await recvTransport.consume({
@@ -602,15 +600,15 @@ test('transport.consume() succeeds', async () => {
     encodings = videoConsumer.rtpParameters.encodings;
     expect(Array.isArray(encodings)).toBe(true);
     expect(encodings.length).toBe(1);
-    expect(typeof (encodings === null || encodings === void 0 ? void 0 : encodings[0])).toBe('object');
+    expect(typeof encodings?.[0]).toBe('object');
     expect(Object.keys(encodings[0])).toEqual(['ssrc', 'rtx', 'dtx']);
-    expect(typeof (encodings === null || encodings === void 0 ? void 0 : encodings[0].ssrc)).toBe('number');
-    expect(typeof (encodings === null || encodings === void 0 ? void 0 : encodings[0].rtx)).toBe('object');
+    expect(typeof encodings?.[0].ssrc).toBe('number');
+    expect(typeof encodings?.[0].rtx).toBe('object');
     expect(Object.keys(encodings[0].rtx)).toEqual(['ssrc']);
-    expect(typeof ((_a = encodings === null || encodings === void 0 ? void 0 : encodings[0].rtx) === null || _a === void 0 ? void 0 : _a.ssrc)).toBe('number');
+    expect(typeof encodings?.[0].rtx?.ssrc).toBe('number');
     rtcp = videoConsumer.rtpParameters.rtcp;
     expect(typeof rtcp).toBe('object');
-    expect(typeof (rtcp === null || rtcp === void 0 ? void 0 : rtcp.cname)).toBe('string');
+    expect(typeof rtcp?.cname).toBe('string');
     expect(videoConsumer.paused).toBe(false);
     expect(videoConsumer.appData).toEqual({});
     recvTransport.removeAllListeners('connect');
@@ -841,6 +839,36 @@ test('transport.updateIceServers() without iceServers rejects with TypeError', a
         .rejects
         .toThrow(TypeError);
 }, 500);
+test('ICE gathering state change fires "icegatheringstatechange" in live Transport', () => {
+    // NOTE: These tests are a bit flaky and we should isolate them. FakeHandler
+    // emits '@connectionstatechange' with value 'connecting' as soon as its
+    // private setupTransport() method is called (which has happens many times in
+    // tests above already). So here we have to reset it manually to test things.
+    // @ts-ignore
+    sendTransport.handler.setIceGatheringState('new');
+    // @ts-ignore
+    sendTransport.handler.setConnectionState('new');
+    let iceGatheringStateChangeEventNumTimesCalled = 0;
+    let connectionStateChangeEventNumTimesCalled = 0;
+    sendTransport.on('icegatheringstatechange', (iceGatheringState) => {
+        iceGatheringStateChangeEventNumTimesCalled++;
+        expect(iceGatheringState).toBe('complete');
+        expect(sendTransport.iceGatheringState).toBe('complete');
+        expect(sendTransport.connectionState).toBe('new');
+    });
+    // eslint-disable-next-line @typescript-eslint/no-unused-vars
+    sendTransport.on('connectionstatechange', (connectionState) => {
+        connectionStateChangeEventNumTimesCalled++;
+    });
+    // @ts-ignore
+    sendTransport.handler.setIceGatheringState('complete');
+    expect(iceGatheringStateChangeEventNumTimesCalled).toBe(1);
+    expect(connectionStateChangeEventNumTimesCalled).toBe(0);
+    expect(sendTransport.iceGatheringState).toBe('complete');
+    expect(sendTransport.connectionState).toBe('new');
+    sendTransport.removeAllListeners('icegatheringstatechange');
+    sendTransport.removeAllListeners('connectionstatechange');
+});
 test('connection state change fires "connectionstatechange" in live Transport', () => {
     let connectionStateChangeEventNumTimesCalled = 0;
     sendTransport.on('connectionstatechange', (connectionState) => {
@@ -854,20 +882,17 @@ test('connection state change fires "connectionstatechange" in live Transport',
     sendTransport.removeAllListeners('connectionstatechange');
 });
 test('producer.pause() succeeds', () => {
-    var _a;
     videoProducer.pause();
     expect(videoProducer.paused).toBe(true);
     // Track will be still enabled due to disableTrackOnPause: false.
-    expect((_a = videoProducer.track) === null || _a === void 0 ? void 0 : _a.enabled).toBe(true);
+    expect(videoProducer.track?.enabled).toBe(true);
 });
 test('producer.resume() succeeds', () => {
-    var _a;
     videoProducer.resume();
     expect(videoProducer.paused).toBe(false);
-    expect((_a = videoProducer.track) === null || _a === void 0 ? void 0 : _a.enabled).toBe(true);
+    expect(videoProducer.track?.enabled).toBe(true);
 });
 test('producer.replaceTrack() with a new track succeeds', async () => {
-    var _a;
     // Have the audio Producer paused.
     audioProducer.pause();
     const audioProducerPreviousTrack = audioProducer.track;
@@ -876,8 +901,8 @@ test('producer.replaceTrack() with a new track succeeds', async () => {
         .resolves
         .toBe(undefined);
     // Previous track must be 'live' due to stopTracks: false.
-    expect(audioProducerPreviousTrack === null || audioProducerPreviousTrack === void 0 ? void 0 : audioProducerPreviousTrack.readyState).toBe('live');
-    expect((_a = audioProducer.track) === null || _a === void 0 ? void 0 : _a.readyState).toBe('live');
+    expect(audioProducerPreviousTrack?.readyState).toBe('live');
+    expect(audioProducer.track?.readyState).toBe('live');
     expect(audioProducer.track).not.toBe(audioProducerPreviousTrack);
     expect(audioProducer.track).toBe(newAudioTrack);
     // Producer was already paused.
@@ -890,7 +915,7 @@ test('producer.replaceTrack() with a new track succeeds', async () => {
         .resolves
         .toBe(undefined);
     // Previous track must be 'ended' due to stopTracks: true.
-    expect(videoProducerPreviousTrack === null || videoProducerPreviousTrack === void 0 ? void 0 : videoProducerPreviousTrack.readyState).toBe('ended');
+    expect(videoProducerPreviousTrack?.readyState).toBe('ended');
     expect(videoProducer.track).not.toBe(videoProducerPreviousTrack);
     expect(videoProducer.track).toBe(newVideoTrack);
     expect(videoProducer.paused).toBe(false);
@@ -903,7 +928,7 @@ test('producer.replaceTrack() with null succeeds', async () => {
         .resolves
         .toBe(undefined);
     // Previous track must be 'live' due to stopTracks: false.
-    expect(audioProducerPreviousTrack === null || audioProducerPreviousTrack === void 0 ? void 0 : audioProducerPreviousTrack.readyState).toBe('live');
+    expect(audioProducerPreviousTrack?.readyState).toBe('live');
     expect(audioProducer.track).toBeNull();
     // Producer was already paused.
     expect(audioProducer.paused).toBe(true);
@@ -919,26 +944,24 @@ test('producer.replaceTrack() with null succeeds', async () => {
     // The given audio track was muted but the Producer was not, so the track
     // must not be muted now.
     expect(audioProducer.paused).toBe(false);
-    expect(audioProducerPreviousTrack === null || audioProducerPreviousTrack === void 0 ? void 0 : audioProducerPreviousTrack.enabled).toBe(true);
+    expect(audioProducerPreviousTrack?.enabled).toBe(true);
     // Reset the audio paused state.
     audioProducer.resume();
 }, 500);
 test('producer.replaceTrack() with an ended track rejects with InvalidStateError', async () => {
-    var _a;
     const track = new fake_mediastreamtrack_1.FakeMediaStreamTrack({ kind: 'audio' });
     track.stop();
     await expect(videoProducer.replaceTrack({ track }))
         .rejects
         .toThrow(errors_1.InvalidStateError);
     expect(track.readyState).toBe('ended');
-    expect((_a = videoProducer.track) === null || _a === void 0 ? void 0 : _a.readyState).toBe('live');
+    expect(videoProducer.track?.readyState).toBe('live');
 }, 500);
 test('producer.replaceTrack() with the same track succeeds', async () => {
-    var _a;
     await expect(audioProducer.replaceTrack({ track: audioProducer.track }))
         .resolves
         .toBe(undefined);
-    expect((_a = audioProducer.track) === null || _a === void 0 ? void 0 : _a.readyState).toBe('live');
+    expect(audioProducer.track?.readyState).toBe('live');
 }, 500);
 test('producer.setMaxSpatialLayer() succeeds', async () => {
     await expect(videoProducer.setMaxSpatialLayer(0))
@@ -987,11 +1010,10 @@ test('consumer.getStats() succeeds', async () => {
     expect(typeof stats).toBe('object');
 }, 500);
 test('producer.close() succeed', () => {
-    var _a;
     audioProducer.close();
     expect(audioProducer.closed).toBe(true);
     // Track will be still 'live' due to stopTracks: false.
-    expect((_a = audioProducer.track) === null || _a === void 0 ? void 0 : _a.readyState).toBe('live');
+    expect(audioProducer.track?.readyState).toBe('live');
 });
 test('producer.replaceTrack() rejects with InvalidStateError if closed', async () => {
     const audioTrack = new fake_mediastreamtrack_1.FakeMediaStreamTrack({ kind: 'audio' });
@@ -1160,7 +1182,6 @@ test('connection state change does not fire "connectionstatechange" in closed Tr
     sendTransport.removeAllListeners('connectionstatechange');
 });
 test('RemoteSdp properly handles multiple streams of the same type in planB', async () => {
-    var _a, _b, _c;
     let sdp = undefined;
     let sdpObject = undefined;
     const remoteSdp = new RemoteSdp_1.RemoteSdp({ planB: true });
@@ -1180,7 +1201,7 @@ test('RemoteSdp properly handles multiple streams of the same type in planB', as
     expect(sdpObject.media[0].rtp[0].codec).toBe('VP8');
     expect(sdpObject.media[0].rtp[1].payload).toBe(102);
     expect(sdpObject.media[0].rtp[1].codec).toBe('rtx');
-    expect((_a = sdpObject.media[0].ssrcs) === null || _a === void 0 ? void 0 : _a.length).toBe(4);
+    expect(sdpObject.media[0].ssrcs?.length).toBe(4);
     await remoteSdp.receive({
         mid: 'video',
         kind: 'video',
@@ -1201,7 +1222,7 @@ test('RemoteSdp properly handles multiple streams of the same type in planB', as
     expect(sdpObject.media[0].rtp[2].codec).toBe('H264');
     expect(sdpObject.media[0].rtp[3].payload).toBe(104);
     expect(sdpObject.media[0].rtp[3].codec).toBe('rtx');
-    expect((_b = sdpObject.media[0].ssrcs) === null || _b === void 0 ? void 0 : _b.length).toBe(8);
+    expect(sdpObject.media[0].ssrcs?.length).toBe(8);
     await remoteSdp.planBStopReceiving({
         mid: 'video',
         offerRtpParameters: fakeParameters.generateConsumerRemoteParameters({ codecMimeType: 'video/H264' }).rtpParameters
@@ -1219,10 +1240,9 @@ test('RemoteSdp properly handles multiple streams of the same type in planB', as
     expect(sdpObject.media[0].rtp[2].codec).toBe('H264');
     expect(sdpObject.media[0].rtp[3].payload).toBe(104);
     expect(sdpObject.media[0].rtp[3].codec).toBe('rtx');
-    expect((_c = sdpObject.media[0].ssrcs) === null || _c === void 0 ? void 0 : _c.length).toBe(4);
+    expect(sdpObject.media[0].ssrcs?.length).toBe(4);
 }, 500);
 test('RemoteSdp does not duplicate codec descriptions', async () => {
-    var _a, _b;
     let sdp = undefined;
     let sdpObject = undefined;
     const remoteSdp = new RemoteSdp_1.RemoteSdp({ planB: true });
@@ -1242,7 +1262,7 @@ test('RemoteSdp does not duplicate codec descriptions', async () => {
     expect(sdpObject.media[0].rtp[0].codec).toBe('VP8');
     expect(sdpObject.media[0].rtp[1].payload).toBe(102);
     expect(sdpObject.media[0].rtp[1].codec).toBe('rtx');
-    expect((_a = sdpObject.media[0].ssrcs) === null || _a === void 0 ? void 0 : _a.length).toBe(4);
+    expect(sdpObject.media[0].ssrcs?.length).toBe(4);
     await remoteSdp.receive({
         mid: 'video',
         kind: 'video',
@@ -1259,7 +1279,7 @@ test('RemoteSdp does not duplicate codec descriptions', async () => {
     expect(sdpObject.media[0].rtp[0].codec).toBe('VP8');
     expect(sdpObject.media[0].rtp[1].payload).toBe(102);
     expect(sdpObject.media[0].rtp[1].codec).toBe('rtx');
-    expect((_b = sdpObject.media[0].ssrcs) === null || _b === void 0 ? void 0 : _b.length).toBe(8);
+    expect(sdpObject.media[0].ssrcs?.length).toBe(8);
 }, 500);
 test('parseScalabilityMode() works', () => {
     expect(parseScalabilityMode('L1T3')).toEqual({ spatialLayers: 1, temporalLayers: 3 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mediasoup-client",
-  "version": "3.6.96",
+  "version": "3.6.98",
   "description": "mediasoup client side JavaScript library",
   "contributors": [
     "Iñaki Baz Castillo <ibc@aliax.net> (https://inakibaz.me)",
@@ -79,7 +79,7 @@
     "@types/ua-parser-js": "^0.7.36",
     "@typescript-eslint/eslint-plugin": "^6.0.0",
     "@typescript-eslint/parser": "^6.0.0",
-    "eslint": "^8.44.0",
+    "eslint": "^8.45.0",
     "eslint-plugin-jest": "^27.2.3",
     "jest": "^29.6.1",
     "open-cli": "^7.2.0",