@webex/plugin-meetings 3.0.0-beta.9 → 3.0.0-bnr.0

package/test/unit/spec/multistream/mediaRequestManager.ts

@@ -2,21 +2,29 @@ import {MediaRequestManager} from '@webex/plugin-meetings/src/multistream/mediaR
 import {ReceiveSlot} from '@webex/plugin-meetings/src/multistream/receiveSlot';
 import sinon from 'sinon';
 import {assert} from '@webex/test-helper-chai';
+import {getMaxFs} from '@webex/plugin-meetings/src/multistream/remoteMedia';
+import FakeTimers from '@sinonjs/fake-timers';
 
 type ExpectedActiveSpeaker = {
   policy: 'active-speaker';
   priority: number;
   receiveSlots: Array<ReceiveSlot>;
+  maxFs: number;
 };
 type ExpectedReceiverSelected = {
   policy: 'receiver-selected';
   csi: number;
   receiveSlot: ReceiveSlot;
+  maxFs: number;
 };
 type ExpectedRequest = ExpectedActiveSpeaker | ExpectedReceiverSelected;
 
 const maxPayloadBitsPerSecond = 10 * 1000 * 1000; // for now we always send this fixed constant
 
+const degradationPreferences = {
+  maxMacroblocksLimit: Infinity, // no limit
+};
+
 describe('MediaRequestManager', () => {
   const CROSS_PRIORITY_DUPLICATION = true;
   const CROSS_POLICY_DUPLICATION = true;
@@ -33,7 +41,10 @@ describe('MediaRequestManager', () => {
 
   beforeEach(() => {
     sendMediaRequestsCallback = sinon.stub();
-    mediaRequestManager = new MediaRequestManager(sendMediaRequestsCallback);
+    mediaRequestManager = new MediaRequestManager(
+      degradationPreferences,
+      sendMediaRequestsCallback
+    );
 
     // create some fake receive slots used by the tests
     fakeWcmeSlots = Array(NUM_SLOTS)
@@ -48,14 +59,16 @@ describe('MediaRequestManager', () => {
         (_, index) =>
           ({
             id: `fake receive slot ${index}`,
+            on: sinon.stub(),
+            off: sinon.stub(),
+            sourceState: 'live',
             wcmeReceiveSlot: fakeWcmeSlots[index],
-            resetSourceState: sinon.stub(),
           } as unknown as ReceiveSlot)
       );
   });
 
   // helper function for adding an active speaker request
-  const addActiveSpeakerRequest = (priority, receiveSlots, commit = false) =>
+  const addActiveSpeakerRequest = (priority, receiveSlots, maxFs, commit = false) =>
     mediaRequestManager.addRequest(
       {
         policyInfo: {
@@ -68,14 +81,14 @@ describe('MediaRequestManager', () => {
         receiveSlots,
         codecInfo: {
           codec: 'h264',
-          maxFs: ACTIVE_SPEAKER_MAX_FS,
+          maxFs: maxFs,
         },
       },
       commit
     );
 
   // helper function for adding a receiver selected request
-  const addReceiverSelectedRequest = (csi, receiveSlot, commit = false) =>
+  const addReceiverSelectedRequest = (csi, receiveSlot, maxFs, commit = false) =>
     mediaRequestManager.addRequest(
       {
         policyInfo: {
@@ -85,7 +98,7 @@ describe('MediaRequestManager', () => {
         receiveSlots: [receiveSlot],
         codecInfo: {
           codec: 'h264',
-          maxFs: RECEIVER_SELECTED_MAX_FS,
+          maxFs: maxFs,
         },
       },
       commit
@@ -116,7 +129,7 @@ describe('MediaRequestManager', () => {
               sinon.match({
                 payloadType: 0x80,
                 h264: sinon.match({
-                  maxFs: ACTIVE_SPEAKER_MAX_FS,
+                  maxFs: expectedRequest.maxFs,
                 }),
               }),
             ],
@@ -134,7 +147,7 @@ describe('MediaRequestManager', () => {
               sinon.match({
                 payloadType: 0x80,
                 h264: sinon.match({
-                  maxFs: RECEIVER_SELECTED_MAX_FS,
+                  maxFs: expectedRequest.maxFs,
                 }),
               }),
             ],
@@ -274,44 +287,79 @@ describe('MediaRequestManager', () => {
 
   it('keeps adding requests with every call to addRequest()', () => {
     // start with 1 request
-    addReceiverSelectedRequest(100, fakeReceiveSlots[0], true);
+    addReceiverSelectedRequest(100, fakeReceiveSlots[0], RECEIVER_SELECTED_MAX_FS, true);
 
     checkMediaRequestsSent([
-      {policy: 'receiver-selected', csi: 100, receiveSlot: fakeWcmeSlots[0]},
+      {
+        policy: 'receiver-selected',
+        csi: 100,
+        receiveSlot: fakeWcmeSlots[0],
+        maxFs: RECEIVER_SELECTED_MAX_FS,
+      },
     ]);
 
     // now add another one
-    addReceiverSelectedRequest(101, fakeReceiveSlots[1], true);
+    addReceiverSelectedRequest(101, fakeReceiveSlots[1], RECEIVER_SELECTED_MAX_FS, true);
 
     checkMediaRequestsSent([
-      {policy: 'receiver-selected', csi: 100, receiveSlot: fakeWcmeSlots[0]},
-      {policy: 'receiver-selected', csi: 101, receiveSlot: fakeWcmeSlots[1]},
+      {
+        policy: 'receiver-selected',
+        csi: 100,
+        receiveSlot: fakeWcmeSlots[0],
+        maxFs: RECEIVER_SELECTED_MAX_FS,
+      },
+      {
+        policy: 'receiver-selected',
+        csi: 101,
+        receiveSlot: fakeWcmeSlots[1],
+        maxFs: RECEIVER_SELECTED_MAX_FS,
+      },
     ]);
 
     // and one more
     addActiveSpeakerRequest(
       1,
       [fakeReceiveSlots[2], fakeReceiveSlots[3], fakeReceiveSlots[4]],
+      ACTIVE_SPEAKER_MAX_FS,
       true
     );
 
     checkMediaRequestsSent([
-      {policy: 'receiver-selected', csi: 100, receiveSlot: fakeWcmeSlots[0]},
-      {policy: 'receiver-selected', csi: 101, receiveSlot: fakeWcmeSlots[1]},
+      {
+        policy: 'receiver-selected',
+        csi: 100,
+        receiveSlot: fakeWcmeSlots[0],
+        maxFs: RECEIVER_SELECTED_MAX_FS,
+      },
+      {
+        policy: 'receiver-selected',
+        csi: 101,
+        receiveSlot: fakeWcmeSlots[1],
+        maxFs: RECEIVER_SELECTED_MAX_FS,
+      },
       {
         policy: 'active-speaker',
         priority: 1,
         receiveSlots: [fakeWcmeSlots[2], fakeWcmeSlots[3], fakeWcmeSlots[4]],
+        maxFs: ACTIVE_SPEAKER_MAX_FS,
       },
     ]);
   });
 
   it('cancels the requests correctly when cancelRequest() is called with commit=true', () => {
     const requestIds = [
-      addActiveSpeakerRequest(255, [fakeReceiveSlots[0], fakeReceiveSlots[1]]),
-      addActiveSpeakerRequest(255, [fakeReceiveSlots[2], fakeReceiveSlots[3]]),
-      addReceiverSelectedRequest(100, fakeReceiveSlots[4]),
-      addReceiverSelectedRequest(200, fakeReceiveSlots[5]),
+      addActiveSpeakerRequest(
+        255,
+        [fakeReceiveSlots[0], fakeReceiveSlots[1]],
+        ACTIVE_SPEAKER_MAX_FS
+      ),
+      addActiveSpeakerRequest(
+        255,
+        [fakeReceiveSlots[2], fakeReceiveSlots[3]],
+        ACTIVE_SPEAKER_MAX_FS
+      ),
+      addReceiverSelectedRequest(100, fakeReceiveSlots[4], RECEIVER_SELECTED_MAX_FS),
+      addReceiverSelectedRequest(200, fakeReceiveSlots[5], RECEIVER_SELECTED_MAX_FS),
     ];
 
     // cancel one of the active speaker requests
@@ -319,9 +367,24 @@ describe('MediaRequestManager', () => {
 
     // expect only the 3 remaining requests to be sent out
     checkMediaRequestsSent([
-      {policy: 'active-speaker', priority: 255, receiveSlots: [fakeWcmeSlots[0], fakeWcmeSlots[1]]},
-      {policy: 'receiver-selected', csi: 100, receiveSlot: fakeWcmeSlots[4]},
-      {policy: 'receiver-selected', csi: 200, receiveSlot: fakeWcmeSlots[5]},
+      {
+        policy: 'active-speaker',
+        priority: 255,
+        receiveSlots: [fakeWcmeSlots[0], fakeWcmeSlots[1]],
+        maxFs: ACTIVE_SPEAKER_MAX_FS,
+      },
+      {
+        policy: 'receiver-selected',
+        csi: 100,
+        receiveSlot: fakeWcmeSlots[4],
+        maxFs: RECEIVER_SELECTED_MAX_FS,
+      },
+      {
+        policy: 'receiver-selected',
+        csi: 200,
+        receiveSlot: fakeWcmeSlots[5],
+        maxFs: RECEIVER_SELECTED_MAX_FS,
+      },
     ]);
 
     // cancel one of the receiver selected requests
@@ -329,8 +392,18 @@ describe('MediaRequestManager', () => {
 
     // expect only the 2 remaining requests to be sent out
     checkMediaRequestsSent([
-      {policy: 'active-speaker', priority: 255, receiveSlots: [fakeWcmeSlots[0], fakeWcmeSlots[1]]},
-      {policy: 'receiver-selected', csi: 100, receiveSlot: fakeWcmeSlots[4]},
+      {
+        policy: 'active-speaker',
+        priority: 255,
+        receiveSlots: [fakeWcmeSlots[0], fakeWcmeSlots[1]],
+        maxFs: ACTIVE_SPEAKER_MAX_FS,
+      },
+      {
+        policy: 'receiver-selected',
+        csi: 100,
+        receiveSlot: fakeWcmeSlots[4],
+        maxFs: RECEIVER_SELECTED_MAX_FS,
+      },
     ]);
   });
 
@@ -338,9 +411,10 @@ describe('MediaRequestManager', () => {
     addActiveSpeakerRequest(
       10,
       [fakeReceiveSlots[0], fakeReceiveSlots[1], fakeReceiveSlots[2]],
+      ACTIVE_SPEAKER_MAX_FS,
       false
     );
-    addReceiverSelectedRequest(123, fakeReceiveSlots[3], false);
+    addReceiverSelectedRequest(123, fakeReceiveSlots[3], RECEIVER_SELECTED_MAX_FS, false);
 
     // nothing should be sent out as we didn't commit the requests
     assert.notCalled(sendMediaRequestsCallback);
@@ -354,8 +428,14 @@ describe('MediaRequestManager', () => {
         policy: 'active-speaker',
         priority: 10,
         receiveSlots: [fakeWcmeSlots[0], fakeWcmeSlots[1], fakeWcmeSlots[2]],
+        maxFs: ACTIVE_SPEAKER_MAX_FS,
+      },
+      {
+        policy: 'receiver-selected',
+        csi: 123,
+        receiveSlot: fakeWcmeSlots[3],
+        maxFs: RECEIVER_SELECTED_MAX_FS,
       },
-      {policy: 'receiver-selected', csi: 123, receiveSlot: fakeWcmeSlots[3]},
     ]);
   });
 
@@ -365,11 +445,12 @@ describe('MediaRequestManager', () => {
       addActiveSpeakerRequest(
         250,
         [fakeReceiveSlots[0], fakeReceiveSlots[1], fakeReceiveSlots[2]],
+        ACTIVE_SPEAKER_MAX_FS,
         false
       ),
-      addReceiverSelectedRequest(98765, fakeReceiveSlots[3], false),
-      addReceiverSelectedRequest(99999, fakeReceiveSlots[4], false),
-      addReceiverSelectedRequest(88888, fakeReceiveSlots[5], true),
+      addReceiverSelectedRequest(98765, fakeReceiveSlots[3], RECEIVER_SELECTED_MAX_FS, false),
+      addReceiverSelectedRequest(99999, fakeReceiveSlots[4], RECEIVER_SELECTED_MAX_FS, false),
+      addReceiverSelectedRequest(88888, fakeReceiveSlots[5], RECEIVER_SELECTED_MAX_FS, true),
     ];
 
     checkMediaRequestsSent([
@@ -377,10 +458,26 @@ describe('MediaRequestManager', () => {
         policy: 'active-speaker',
         priority: 250,
         receiveSlots: [fakeWcmeSlots[0], fakeWcmeSlots[1], fakeWcmeSlots[2]],
+        maxFs: ACTIVE_SPEAKER_MAX_FS,
+      },
+      {
+        policy: 'receiver-selected',
+        csi: 98765,
+        receiveSlot: fakeWcmeSlots[3],
+        maxFs: RECEIVER_SELECTED_MAX_FS,
+      },
+      {
+        policy: 'receiver-selected',
+        csi: 99999,
+        receiveSlot: fakeWcmeSlots[4],
+        maxFs: RECEIVER_SELECTED_MAX_FS,
+      },
+      {
+        policy: 'receiver-selected',
+        csi: 88888,
+        receiveSlot: fakeWcmeSlots[5],
+        maxFs: RECEIVER_SELECTED_MAX_FS,
       },
-      {policy: 'receiver-selected', csi: 98765, receiveSlot: fakeWcmeSlots[3]},
-      {policy: 'receiver-selected', csi: 99999, receiveSlot: fakeWcmeSlots[4]},
-      {policy: 'receiver-selected', csi: 88888, receiveSlot: fakeWcmeSlots[5]},
     ]);
 
     // now cancel 3 of them, but with commit=false => nothing should happen
@@ -394,22 +491,29 @@ describe('MediaRequestManager', () => {
     mediaRequestManager.commit();
 
     checkMediaRequestsSent([
-      {policy: 'receiver-selected', csi: 98765, receiveSlot: fakeWcmeSlots[3]},
+      {
+        policy: 'receiver-selected',
+        csi: 98765,
+        receiveSlot: fakeWcmeSlots[3],
+        maxFs: RECEIVER_SELECTED_MAX_FS,
+      },
     ]);
   });
 
   it('sends the wcme media requests when commit() is called', () => {
     // send some requests, all of them with commit=false
-    addReceiverSelectedRequest(123000, fakeReceiveSlots[0], false);
-    addReceiverSelectedRequest(456000, fakeReceiveSlots[1], false);
+    addReceiverSelectedRequest(123000, fakeReceiveSlots[0], RECEIVER_SELECTED_MAX_FS, false);
+    addReceiverSelectedRequest(456000, fakeReceiveSlots[1], RECEIVER_SELECTED_MAX_FS, false);
     addActiveSpeakerRequest(
       255,
       [fakeReceiveSlots[2], fakeReceiveSlots[3], fakeReceiveSlots[4]],
+      ACTIVE_SPEAKER_MAX_FS,
       false
     );
     addActiveSpeakerRequest(
       254,
       [fakeReceiveSlots[5], fakeReceiveSlots[6], fakeReceiveSlots[7]],
+      ACTIVE_SPEAKER_MAX_FS,
       false
     );
 
@@ -421,56 +525,89 @@ describe('MediaRequestManager', () => {
 
     // check that all requests have been sent out
     checkMediaRequestsSent([
-      {policy: 'receiver-selected', csi: 123000, receiveSlot: fakeWcmeSlots[0]},
-      {policy: 'receiver-selected', csi: 456000, receiveSlot: fakeWcmeSlots[1]},
+      {
+        policy: 'receiver-selected',
+        csi: 123000,
+        receiveSlot: fakeWcmeSlots[0],
+        maxFs: RECEIVER_SELECTED_MAX_FS,
+      },
+      {
+        policy: 'receiver-selected',
+        csi: 456000,
+        receiveSlot: fakeWcmeSlots[1],
+        maxFs: RECEIVER_SELECTED_MAX_FS,
+      },
       {
         policy: 'active-speaker',
         priority: 255,
         receiveSlots: [fakeWcmeSlots[2], fakeWcmeSlots[3], fakeWcmeSlots[4]],
+        maxFs: ACTIVE_SPEAKER_MAX_FS,
       },
       {
         policy: 'active-speaker',
         priority: 254,
         receiveSlots: [fakeWcmeSlots[5], fakeWcmeSlots[6], fakeWcmeSlots[7]],
+        maxFs: ACTIVE_SPEAKER_MAX_FS,
       },
     ]);
   });
 
-  it('clears all the requests on reset()', () => {
+  it('avoids sending duplicate requests and clears all the requests on reset()', () => {
     // send some requests and commit them one by one
-    addReceiverSelectedRequest(1500, fakeReceiveSlots[0], true);
-    addReceiverSelectedRequest(1501, fakeReceiveSlots[1], true);
+    addReceiverSelectedRequest(1500, fakeReceiveSlots[0], RECEIVER_SELECTED_MAX_FS, false);
+    addReceiverSelectedRequest(1501, fakeReceiveSlots[1], RECEIVER_SELECTED_MAX_FS, false);
     addActiveSpeakerRequest(
       255,
       [fakeReceiveSlots[2], fakeReceiveSlots[3], fakeReceiveSlots[4]],
-      true
+      ACTIVE_SPEAKER_MAX_FS,
+      false
     );
     addActiveSpeakerRequest(
       254,
       [fakeReceiveSlots[5], fakeReceiveSlots[6], fakeReceiveSlots[7]],
-      true
+      ACTIVE_SPEAKER_MAX_FS,
+      false
     );
 
-    sendMediaRequestsCallback.resetHistory();
+    // nothing should be sent out as we didn't commit the requests
+    assert.notCalled(sendMediaRequestsCallback);
 
-    // check that when calling commit() all requests are re-sent again
-    mediaRequestManager.commit();
 
+    mediaRequestManager.commit();
     checkMediaRequestsSent([
-      {policy: 'receiver-selected', csi: 1500, receiveSlot: fakeWcmeSlots[0]},
-      {policy: 'receiver-selected', csi: 1501, receiveSlot: fakeWcmeSlots[1]},
+      {
+        policy: 'receiver-selected',
+        csi: 1500,
+        receiveSlot: fakeWcmeSlots[0],
+        maxFs: RECEIVER_SELECTED_MAX_FS,
+      },
+      {
+        policy: 'receiver-selected',
+        csi: 1501,
+        receiveSlot: fakeWcmeSlots[1],
+        maxFs: RECEIVER_SELECTED_MAX_FS,
+      },
       {
         policy: 'active-speaker',
         priority: 255,
         receiveSlots: [fakeWcmeSlots[2], fakeWcmeSlots[3], fakeWcmeSlots[4]],
+        maxFs: ACTIVE_SPEAKER_MAX_FS,
       },
       {
         policy: 'active-speaker',
         priority: 254,
         receiveSlots: [fakeWcmeSlots[5], fakeWcmeSlots[6], fakeWcmeSlots[7]],
+        maxFs: ACTIVE_SPEAKER_MAX_FS,
       },
     ]);
 
+
+    // check that when calling commit()
+    // all requests are not re-sent again (avoid duplicate requests)
+    mediaRequestManager.commit();
+
+    assert.notCalled(sendMediaRequestsCallback);
+
     // now reset everything
     mediaRequestManager.reset();
 
@@ -479,37 +616,201 @@ describe('MediaRequestManager', () => {
     checkMediaRequestsSent([]);
   });
 
-  it('calls resetSourceState() on slots that are stopped being used', () => {
-    const requestIds = [
-      addActiveSpeakerRequest(255, [fakeReceiveSlots[0], fakeReceiveSlots[1]]),
-      addActiveSpeakerRequest(255, [fakeReceiveSlots[2], fakeReceiveSlots[3]]),
-      addReceiverSelectedRequest(100, fakeReceiveSlots[4]),
-      addReceiverSelectedRequest(200, fakeReceiveSlots[5]),
-    ];
+  it('makes sure to call requests correctly after reset was called and another request was added', () => {
+    addReceiverSelectedRequest(1500, fakeReceiveSlots[0], RECEIVER_SELECTED_MAX_FS, false);
+
+    assert.notCalled(sendMediaRequestsCallback);
+
 
     mediaRequestManager.commit();
     checkMediaRequestsSent([
-      {policy: 'active-speaker', priority: 255, receiveSlots: [fakeWcmeSlots[0], fakeWcmeSlots[1]]},
-      {policy: 'active-speaker', priority: 255, receiveSlots: [fakeWcmeSlots[2], fakeWcmeSlots[3]]},
-      {policy: 'receiver-selected', csi: 100, receiveSlot: fakeWcmeSlots[4]},
-      {policy: 'receiver-selected', csi: 200, receiveSlot: fakeWcmeSlots[5]},
+      {
+        policy: 'receiver-selected',
+        csi: 1500,
+        receiveSlot: fakeWcmeSlots[0],
+        maxFs: RECEIVER_SELECTED_MAX_FS,
+      },
     ]);
 
-    // cancel 2 of the requests
-    mediaRequestManager.cancelRequest(requestIds[1], false);
-    mediaRequestManager.cancelRequest(requestIds[3], false);
+    // now reset everything
+    mediaRequestManager.reset();
+
+     // calling commit now should not cause any requests to be sent out
+     mediaRequestManager.commit();
+     checkMediaRequestsSent([]);
 
+     //add new request
+    addReceiverSelectedRequest(1501, fakeReceiveSlots[1], RECEIVER_SELECTED_MAX_FS, false);
+
+    // commit
     mediaRequestManager.commit();
 
-    // expect only the 2 remaining requests to be sent out
+    // check the new request was sent
+    checkMediaRequestsSent([
+      {
+        policy: 'receiver-selected',
+        csi: 1501,
+        receiveSlot: fakeWcmeSlots[1],
+        maxFs: RECEIVER_SELECTED_MAX_FS,
+      },
+    ])
+  })
+
+  it('re-sends media requests after degradation preferences are set', () => {
+    // set max macroblocks limit
+    mediaRequestManager.setDegradationPreferences({maxMacroblocksLimit: 32400});
+    assert.calledOnce(sendMediaRequestsCallback);
+  });
+
+  it('should not degrade max-fs if receive slot sources are not live', () => {
+    // set receive slot source states to "no source"
+    fakeReceiveSlots.forEach((slot) => {
+      slot.sourceState = 'no source';
+    });
+
+    // set max macroblocks limit
+    mediaRequestManager.setDegradationPreferences({maxMacroblocksLimit: 32400});
+    sendMediaRequestsCallback.resetHistory();
+
+    // request 4 "large" 1080p streams, which should degrade to 720p if live
+    addActiveSpeakerRequest(255, fakeReceiveSlots.slice(0, 4), getMaxFs('large'), true);
+
+    // check that resulting requests are 4 "large" 1080p streams
+    checkMediaRequestsSent([
+      {
+        policy: 'active-speaker',
+        priority: 255,
+        receiveSlots: fakeWcmeSlots.slice(0, 4),
+        maxFs: getMaxFs('large'),
+      },
+    ]);
+  });
+
+  it('can degrade max-fs once when request exceeds max macroblocks limit', () => {
+    // set max macroblocks limit
+    mediaRequestManager.setDegradationPreferences({maxMacroblocksLimit: 32400});
+    sendMediaRequestsCallback.resetHistory();
+
+    // request 3 "large" 1080p streams
+    addActiveSpeakerRequest(255, fakeReceiveSlots.slice(0, 3), getMaxFs('large'), false);
+
+    // request additional "large" 1080p stream to exceed max macroblocks limit
+    const additionalRequestId = addReceiverSelectedRequest(
+      123,
+      fakeReceiveSlots[3],
+      getMaxFs('large'),
+      true
+    );
+
+    // check that resulting requests are 4 "medium" 720p streams
+    checkMediaRequestsSent([
+      {
+        policy: 'active-speaker',
+        priority: 255,
+        receiveSlots: fakeWcmeSlots.slice(0, 3),
+        maxFs: getMaxFs('medium'),
+      },
+      {
+        policy: 'receiver-selected',
+        csi: 123,
+        receiveSlot: fakeWcmeSlots[3],
+        maxFs: getMaxFs('medium'),
+      },
+    ]);
+
+    // cancel additional request
+    mediaRequestManager.cancelRequest(additionalRequestId);
+
+    // check that resulting requests are 3 "large" 1080p streams
+    checkMediaRequestsSent([
+      {
+        policy: 'active-speaker',
+        priority: 255,
+        receiveSlots: fakeWcmeSlots.slice(0, 3),
+        maxFs: getMaxFs('large'),
+      },
+    ]);
+  });
+
+  it('can degrade max-fs multiple times when request exceeds max macroblocks limit', () => {
+    // set max macroblocks limit
+    mediaRequestManager.setDegradationPreferences({maxMacroblocksLimit: 32400});
+    sendMediaRequestsCallback.resetHistory();
+
+    // request 10 "large" 1080p streams
+    addActiveSpeakerRequest(255, fakeReceiveSlots.slice(0, 10), getMaxFs('large'), true);
+
+    // check that resulting requests are 10 "small" 360p streams
     checkMediaRequestsSent([
-      {policy: 'active-speaker', priority: 255, receiveSlots: [fakeWcmeSlots[0], fakeWcmeSlots[1]]},
-      {policy: 'receiver-selected', csi: 100, receiveSlot: fakeWcmeSlots[4]},
+      {
+        policy: 'active-speaker',
+        priority: 255,
+        receiveSlots: fakeWcmeSlots.slice(0, 10),
+        maxFs: getMaxFs('small'),
+      },
+    ]);
+  });
+
+  it('can degrade only the largest max-fs when request exceeds max macroblocks limit', () => {
+    // set max macroblocks limit
+    mediaRequestManager.setDegradationPreferences({maxMacroblocksLimit: 32400});
+    sendMediaRequestsCallback.resetHistory();
+
+    // request 5 "large" 1080p streams and 5 "small" 360p streams
+    addActiveSpeakerRequest(255, fakeReceiveSlots.slice(0, 5), getMaxFs('large'), false);
+    addActiveSpeakerRequest(254, fakeReceiveSlots.slice(5, 10), getMaxFs('small'), true);
+
+    // check that resulting requests are 5 "medium" 720p streams and 5 "small" 360p streams
+    checkMediaRequestsSent([
+      {
+        policy: 'active-speaker',
+        priority: 255,
+        receiveSlots: fakeWcmeSlots.slice(0, 5),
+        maxFs: getMaxFs('medium'),
+      },
+      {
+        policy: 'active-speaker',
+        priority: 254,
+        receiveSlots: fakeWcmeSlots.slice(5, 10),
+        maxFs: getMaxFs('small'),
+      },
     ]);
+  });
+
+  it('respects the preferredMaxFs if set', () => {
+    sendMediaRequestsCallback.resetHistory();
+    const clock = FakeTimers.install({now: Date.now()});
+
+    addActiveSpeakerRequest(255, fakeReceiveSlots.slice(0, 10), getMaxFs('large'), true);
+
+    sendMediaRequestsCallback.resetHistory();
+
+
+    const maxFsHandlerCall = fakeReceiveSlots[0].on.getCall(1);
+
+    const maxFsHandler = maxFsHandlerCall.args[1];
+    const eventName = maxFsHandlerCall.args[0];
+
+    assert.equal(eventName, 'maxFsUpdate');
+
+    const preferredFrameSize = 100;
+
+    maxFsHandler({maxFs: preferredFrameSize});
 
-    // and that the receive slots of the 2 cancelled ones had resetSourceState() called
-    assert.calledOnce(fakeReceiveSlots[2].resetSourceState);
-    assert.calledOnce(fakeReceiveSlots[3].resetSourceState);
-    assert.calledOnce(fakeReceiveSlots[5].resetSourceState);
+    clock.tick(999);
+
+    assert.notCalled(sendMediaRequestsCallback);
+
+    clock.tick(1);
+
+    checkMediaRequestsSent([
+      {
+        policy: 'active-speaker',
+        priority: 255,
+        receiveSlots: fakeWcmeSlots.slice(0, 10),
+        maxFs: preferredFrameSize,
+      },
+    ]);
   });
+
 });