@webex/plugin-meetings 3.0.0-stream-classes.4 → 3.0.0

package/test/unit/spec/meetings/utils.js

@@ -162,9 +162,24 @@ describe('plugin-meetings', () => {
     });
 
     describe("#handleRoapMercury", () => {
-      it('it sends the correct behaviour metric', () => {
-        const roapMessageReceived = sinon.stub();
-        const envelope = {
+      let envelope;
+      let meetingCollection;
+      let roapMessageReceived;
+      let handleTurnDiscoveryResponse;
+      let meeting;
+
+      beforeEach(() => {
+        roapMessageReceived = sinon.stub();
+        handleTurnDiscoveryResponse = sinon.stub();
+
+        meeting = {
+          id: 'meeting-id',
+          roapMessageReceived,
+          roap: {
+            turnDiscovery: {handleTurnDiscoveryResponse}
+          }
+        };
+        envelope = {
           data: {
             message:{
               seq: "seq",
@@ -178,17 +193,12 @@ describe('plugin-meetings', () => {
             eventType: 'locus.message.roap',
           }
         };
-        const meetingCollection = {
-          getByKey: () => ({
-            id: 'meeting-id',
-            mediaProperties: {
-              webrtcMediaConnection: {
-                roapMessageReceived
-              }
-            }
-          })
+        meetingCollection = {
+          getByKey: () => meeting
         };
+      });
 
+      it('it sends the correct behaviour metric', () => {
         MeetingsUtil.handleRoapMercury(envelope, meetingCollection);
         assert.calledWith(Metrics.sendBehavioralMetric, BEHAVIORAL_METRICS.ROAP_MERCURY_EVENT_RECEIVED,  {
           correlation_id: 'correlationId',
@@ -207,6 +217,19 @@ describe('plugin-meetings', () => {
         })
 
       });
+
+      it('calls handleTurnDiscoveryResponse for TURN_DISCOVERY_RESPONSE', () => {
+        envelope.data.message.messageType = 'TURN_DISCOVERY_RESPONSE';
+        delete envelope.data.message.sdps;
+        MeetingsUtil.handleRoapMercury(envelope, meetingCollection);
+        assert.calledWith(meeting.roap.turnDiscovery.handleTurnDiscoveryResponse, {
+          seq: "seq",
+          messageType: 'TURN_DISCOVERY_RESPONSE',
+          errorType: 'errorType',
+          tieBreaker: 'tieBreaker',
+          errorCause: 'errorCause',
+        }, 'from mercury')
+      });
     })
   });
 

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

@@ -17,13 +17,7 @@ describe('member', () => {
     assert.exists(member.supportsInterpretation);
     assert.exists(member.supportsBreakouts);
     assert.exists(member.supportLiveAnnotation);
-  });
-
-  it('checks that processParticipant calls isHandRaised', () => {
-    sinon.spy(MemberUtil, 'isHandRaised');
-    member.processParticipant(participant);
-
-    assert.calledOnceWithExactly(MemberUtil.isHandRaised, participant);
+    assert.exists(member.canReclaimHost);
   });
 
   describe('roles', () => {
@@ -49,6 +43,13 @@ describe('member', () => {
 
       assert.calledOnceWithExactly(MemberUtil.isHandRaised, participant);
     });
+
+    it('checks that processParticipant calls canReclaimHost', () => {
+      sinon.spy(MemberUtil, 'canReclaimHost');
+      member.processParticipant(participant);
+  
+      assert.calledOnceWithExactly(MemberUtil.canReclaimHost, participant);
+    });
   })
 
   describe('#processMember', () => {

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

@@ -50,6 +50,38 @@ describe('plugin-meetings', () => {
     });
   });
 
+  describe('MemberUtil.canReclaimHost', () => {
+    it('throws error when there is no participant', () => {
+      assert.throws(() => {
+        MemberUtil.canReclaimHost();
+      }, 'canReclaimHostRole could not be processed, participant is undefined.');
+    });
+
+    it('returns true when canReclaimHostRole is true', () => {
+      const participant = {
+        canReclaimHostRole: true,
+      };
+
+      assert.isTrue(MemberUtil.canReclaimHost(participant));
+    });
+
+    it('returns false when canReclaimHostRole is false', () => {
+      const participant = {
+        canReclaimHostRole: false,
+      };
+
+      assert.isFalse(MemberUtil.canReclaimHost(participant));
+    });
+
+    it('returns false when canReclaimHostRole is falsy', () => {
+      const participant = {
+        canReclaimHostRole: undefined,
+      };
+
+      assert.isFalse(MemberUtil.canReclaimHost(participant));
+    });
+  });
+
   describe('MemberUtil.extractControlRoles', () => {
     it('happy path extract control roles', () => {
       const participant = {

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

@@ -17,6 +17,12 @@ import MembersUtil from '@webex/plugin-meetings/src/members/util';
 import * as MembersRequestImport from '@webex/plugin-meetings/src/members/request';
 import Trigger from '@webex/plugin-meetings/src/common/events/trigger-proxy';
 import {EVENT_TRIGGERS} from '@webex/plugin-meetings/src/constants';
+import {
+  ReclaimHostEmptyWrongKeyError,
+  ReclaimHostIsHostAlreadyError,
+  ReclaimHostNotAllowedError,
+  ReclaimHostNotSupportedError,
+} from '../../../../src/common/errors/reclaim-host-role-errors';
 
 const {assert} = chai;
 
@@ -337,7 +343,17 @@ describe('plugin-meetings', () => {
     });
 
     describe('#assignRoles', () => {
-      const setup = (locusUrl) => {
+      const fakeRoles = [
+        {type: 'PRESENTER', hasRole: true},
+        {type: 'MODERATOR', hasRole: false},
+        {type: 'COHOST', hasRole: true},
+      ];
+
+      const resolvedValue = "it worked";
+
+      const genericMessage = 'Generic error from the API';
+
+      const setup = (locusUrl, errorCode) => {
         const members = createMembers({url: locusUrl});
 
         const spies = {
@@ -345,9 +361,14 @@ describe('plugin-meetings', () => {
             MembersUtil,
             'generateRoleAssignmentMemberOptions'
           ),
-          assignRolesMember: sandbox.spy(members.membersRequest, 'assignRolesMember'),
         };
 
+        if (errorCode) {
+          spies.assignRolesMember = sandbox.stub(members.membersRequest, 'assignRolesMember').rejects({body: {errorCode}, message: genericMessage});
+        } else {
+          spies.assignRolesMember = sandbox.stub(members.membersRequest, 'assignRolesMember').resolves(resolvedValue);
+        }
+
         return {members, spies};
       };
 
@@ -357,6 +378,21 @@ describe('plugin-meetings', () => {
         assert.notCalled(spies.assignRolesMember);
       };
 
+      const checkError = async (error, expectedMemberId, expectedRoles, expectedLocusUrl, resultPromise, expectedMessage, spies) => {
+        await assert.isRejected(resultPromise, error, expectedMessage);
+        assert.calledOnceWithExactly(
+          spies.generateRoleAssignmentMemberOptions,
+          expectedMemberId,
+          expectedRoles,
+          expectedLocusUrl
+        );
+        assert.calledOnceWithExactly(spies.assignRolesMember, {
+          memberId: expectedMemberId,
+          roles: expectedRoles,
+          locusUrl: expectedLocusUrl,
+        });
+      };
+
       const checkValid = async (
         resultPromise,
         spies,
@@ -364,7 +400,7 @@ describe('plugin-meetings', () => {
         expectedRoles,
         expectedLocusUrl
       ) => {
-        await assert.isFulfilled(resultPromise);
+        const resolvedValue = await assert.isFulfilled(resultPromise);
         assert.calledOnceWithExactly(
           spies.generateRoleAssignmentMemberOptions,
           expectedMemberId,
@@ -376,7 +412,7 @@ describe('plugin-meetings', () => {
           roles: expectedRoles,
           locusUrl: expectedLocusUrl,
         });
-        assert.strictEqual(resultPromise, spies.assignRolesMember.getCall(0).returnValue);
+        assert.strictEqual(resolvedValue, resolvedValue);
       };
 
       it('should not make a request if there is no member id', async () => {
@@ -387,7 +423,7 @@ describe('plugin-meetings', () => {
         await checkInvalid(
           resultPromise,
           'The member id must be defined to assign the roles to a member.',
-          spies
+          spies,
         );
       });
 
@@ -399,7 +435,92 @@ describe('plugin-meetings', () => {
         await checkInvalid(
           resultPromise,
           'The associated locus url for this meetings members object must be defined.',
-          spies
+          spies,
+        );
+      });
+
+      it('should not make a request if locus throws ReclaimHostNotSupportedError', async () => {
+        const memberId = uuid.v4();
+        const {members, spies} = setup(url1, 2400127);
+
+        const resultPromise = members.assignRoles(memberId, fakeRoles);
+
+        await checkError(
+          ReclaimHostNotSupportedError,
+          memberId,
+          fakeRoles,
+          url1,
+          resultPromise,
+          'Non converged meetings, PSTN or SIP users in converged meetings are not supported currently.',
+          spies,
+        );
+      });
+
+      it('should not make a request if locus throws ReclaimHostNotAllowedError', async () => {
+        const memberId = uuid.v4();
+        const {members, spies} = setup(url1, 2403135);
+
+        const resultPromise = members.assignRoles(memberId, fakeRoles);
+
+        await checkError(
+          ReclaimHostNotAllowedError,
+          memberId,
+          fakeRoles,
+          url1,
+          resultPromise,
+          'Reclaim Host Role Not Allowed For Other Participants. Participants cannot claim host role in PMR meeting, space instant meeting or escalated instant meeting. However, the original host still can reclaim host role when it manually makes another participant to be the host.',
+          spies,
+        );
+      });
+
+      it('should not make a request if locus throws ReclaimHostEmptyWrongKeyError', async () => {
+        const memberId = uuid.v4();
+        const {members, spies} = setup(url1, 2403136);
+
+        const resultPromise = members.assignRoles(memberId, fakeRoles);
+
+        await checkError(
+          ReclaimHostEmptyWrongKeyError,
+          memberId,
+          fakeRoles,
+          url1,
+          resultPromise,
+          'Host Key Not Specified Or Matched. The original host can reclaim the host role without entering the host key. However, any other person who claims the host role must enter the host key to get it.',
+          spies,
+        );
+      });
+
+      it('should not make a request if locus throws ReclaimHostIsHostAlreadyError', async () => {
+        const memberId = uuid.v4();
+        const {members, spies} = setup(url1, 2409150);
+
+        const resultPromise = members.assignRoles(memberId, fakeRoles);
+
+        await checkError(
+          ReclaimHostIsHostAlreadyError,
+          memberId,
+          fakeRoles,
+          url1,
+          resultPromise,
+          'Participant Having Host Role Already. Participant who sends request to reclaim host role has already a host role.',
+          spies,
+        );
+      });
+
+      it('should not make a request if locus throws a different error', async () => {
+        const memberId = uuid.v4();
+        const {members, spies} = setup(url1, 1234);
+
+        const resultPromise = members.assignRoles(memberId, fakeRoles);
+
+        await checkError(
+          {body: {errorCode: 1234}, message: genericMessage},
+          memberId,
+          fakeRoles,
+          url1,
+          resultPromise,
+          genericMessage,
+          spies,
         );
       });
 
@@ -407,22 +528,14 @@ describe('plugin-meetings', () => {
         const memberId = uuid.v4();
         const {members, spies} = setup(url1);
 
-        const resultPromise = members.assignRoles(memberId, [
-          {type: 'PRESENTER', hasRole: true},
-          {type: 'MODERATOR', hasRole: false},
-          {type: 'COHOST', hasRole: true},
-        ]);
+        const resultPromise = members.assignRoles(memberId, fakeRoles);
 
         await checkValid(
           resultPromise,
           spies,
           memberId,
-          [
-            {type: 'PRESENTER', hasRole: true},
-            {type: 'MODERATOR', hasRole: false},
-            {type: 'COHOST', hasRole: true},
-          ],
-          url1
+          fakeRoles,
+          url1,
         );
       });
     });

package/test/unit/spec/members/utils.js

@@ -59,6 +59,32 @@ describe('plugin-meetings', () => {
           },
         });
       });
+
+      it('returns the correct request params with a hostKey', () => {
+        const format = {
+          locusUrl: 'locusUrl',
+          memberId: 'test',
+          roles: [
+            {type: 'PRESENTER', hasRole: true, hostKey: '123456'},
+            {type: 'MODERATOR', hasRole: false, hostKey: '123456'},
+            {type: 'COHOST', hasRole: true, hostKey: '123456'},
+          ],
+        };
+
+        assert.deepEqual(MembersUtil.getRoleAssignmentMemberRequestParams(format), {
+          method: 'PATCH',
+          uri: `locusUrl/${PARTICIPANT}/test/${CONTROLS}`,
+          body: {
+            role: {
+              roles: [
+                {type: 'PRESENTER', hasRole: true, hostKey: '123456'},
+                {type: 'MODERATOR', hasRole: false, hostKey: '123456'},
+                {type: 'COHOST', hasRole: true, hostKey: '123456'},
+              ],
+            },
+          },
+        });
+      });
     });
 
     describe('#generateRaiseHandMemberOptions', () => {

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

@@ -5,7 +5,6 @@ import 'jsdom-global/register';
 import sinon from 'sinon';
 import Metrics from '@webex/internal-plugin-metrics';
 import MockWebex from '@webex/test-helper-mock-webex';
-import {browserOnly} from '@webex/test-helper-mocha';
 import {assert} from '@webex/test-helper-chai';
 
 import metrics from '@webex/plugin-meetings/src/metrics';
@@ -14,7 +13,7 @@ import metrics from '@webex/plugin-meetings/src/metrics';
  * Meeting can only run in a browser, so we can only send metrics for
  * browser usage.
  */
-browserOnly(describe)('Meeting metrics', () => {
+describe('Meeting metrics', () => {
   let webex, mockSubmitMetric, sandbox;
 
   beforeEach(() => {

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

@@ -951,6 +951,7 @@ describe('MediaRequestManager', () => {
         maxMbps: 3000,
       },
     ]);
+    clock.uninstall()
   });
 
   describe('maxPayloadBitsPerSecond', () => {

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

@@ -747,7 +747,11 @@ describe('RemoteMediaManager', () => {
 
       assert.calledWith(
         logger.log,
-        'RemoteMediaManager#updateVideoReceiveSlots --> receive slots updated: unused=0, activeSpeaker=6, receiverSelected=4\ngroup: thumbnails\nfake video slot fake video slot fake video slot fake video slot fake video slot fake video slot\nreceiverSelected:\n stage-1: fake video slot\n stage-2: fake video slot\n stage-3: fake video slot\n stage-4: fake video slot\n'
+        'RemoteMediaManager#setLayout --> new layout selected: Stage'
+      );
+      assert.calledWith(
+        logger.log,
+        'RemoteMediaManager#logMainVideoReceiveSlots --> MAIN VIDEO receive slots: unused=0, activeSpeaker=6, receiverSelected=4\ngroup: thumbnails\nfake video slot, fake video slot, fake video slot, fake video slot, fake video slot, fake video slot\nreceiverSelected:\n stage-1: fake video slot\n stage-2: fake video slot\n stage-3: fake video slot\n stage-4: fake video slot\n'
       );
     });
 
@@ -769,7 +773,11 @@ describe('RemoteMediaManager', () => {
 
       assert.calledWith(
         logger.log,
-        'RemoteMediaManager#updateVideoReceiveSlots --> receive slots updated: unused=0, activeSpeaker=9, receiverSelected=0\ngroup: main\nfake video slot fake video slot fake video slot fake video slot fake video slot fake video slot fake video slot fake video slot fake video slot\nreceiverSelected:\n'
+        'RemoteMediaManager#setLayout --> new layout selected: AllEqual'
+      );
+      assert.calledWith(
+        logger.log,
+        'RemoteMediaManager#logMainVideoReceiveSlots --> MAIN VIDEO receive slots: unused=0, activeSpeaker=9, receiverSelected=0\ngroup: main\nfake video slot, fake video slot, fake video slot, fake video slot, fake video slot, fake video slot, fake video slot, fake video slot, fake video slot\nreceiverSelected:\n'
       );
     });