@webex/plugin-meetings 2.60.0 → 2.60.1-next.2

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

@@ -1,8 +1,17 @@
 import {assert} from '@webex/test-helper-chai';
+import sinon from 'sinon';
 
 import MeetingsUtil from '@webex/plugin-meetings/src/meetings/util';
+import Metrics from '@webex/plugin-meetings/src/metrics';
+import BEHAVIORAL_METRICS from '@webex/plugin-meetings/src/metrics/constants';
 
 describe('plugin-meetings', () => {
+  beforeEach(() => {
+    sinon.stub(Metrics, 'sendBehavioralMetric');
+  });
+  afterEach(() => {
+    sinon.restore();
+  });
   describe('Meetings utils function', () => {
     describe('#parseDefaultSiteFromMeetingPreferences', () => {
       it('should return the default true site from user preferences', () => {
@@ -33,7 +42,7 @@ describe('plugin-meetings', () => {
         );
       });
 
-      it('should work fine if no default true site', () => {
+      it('should take the first site if none are default', () => {
         const userPreferences = {
           sites: [
             {
@@ -51,7 +60,10 @@ describe('plugin-meetings', () => {
           ],
         };
 
-        assert.equal(MeetingsUtil.parseDefaultSiteFromMeetingPreferences(userPreferences), '');
+        assert.equal(
+          MeetingsUtil.parseDefaultSiteFromMeetingPreferences(userPreferences),
+          'site1-example.webex.com'
+        );
       });
 
       it('should work fine if sites an empty array', () => {
@@ -62,5 +74,193 @@ describe('plugin-meetings', () => {
         assert.equal(MeetingsUtil.parseDefaultSiteFromMeetingPreferences(userPreferences), '');
       });
     });
+
+    describe('#getThisDevice', () => {
+      it('return null if no devices in self', () => {
+        const newLocus = {};
+        assert.equal(MeetingsUtil.getThisDevice(newLocus, '123'), null);
+      });
+      it('return null if no matched device in self', () => {
+        const newLocus = {
+          self: {
+            devices: [{state: 'JOINED', url: '456'}]
+          }
+        };
+        assert.equal(MeetingsUtil.getThisDevice(newLocus, '123'), null);
+      });
+      it('return the device match with current device', () => {
+        const newLocus = {
+          self: {
+            devices: [{state: 'JOINED', url: '123'}]
+          }
+        };
+        assert.deepEqual(MeetingsUtil.getThisDevice(newLocus, '123'), {state: 'JOINED', url: '123'});
+      })
+    });
+
+    describe('#isBreakoutLocusDTO', () => {
+      it('returns false is no breakout in locus.controls', () => {
+        const newLocus = {
+          controls: {}
+        };
+
+        assert.equal(MeetingsUtil.isBreakoutLocusDTO(newLocus), false);
+      });
+
+      it('returns is breakout locus DTO if sessionType is BREAKOUT', () => {
+        const newLocus = {
+          controls: {
+            breakout: {
+              sessionType: 'BREAKOUT',
+            },
+          },
+        };
+        assert.equal(MeetingsUtil.isBreakoutLocusDTO(newLocus), true);
+      });
+
+      it('returns is not breakout locus DTO if sessionType is MAIN', () => {
+        const newLocus = {
+          controls: {
+            breakout: {
+              sessionType: 'MAIN',
+            },
+          },
+        };
+        assert.equal(MeetingsUtil.isBreakoutLocusDTO(newLocus), false);
+      });
+    });
+
+    describe('#joinedOnThisDevice', () => {
+      it('return false if no devices in self', () => {
+        const newLocus = {};
+        assert.equal(MeetingsUtil.joinedOnThisDevice(null, newLocus, '123'), false);
+      });
+      it('return true if joined on this device', () => {
+        const newLocus = {
+          self: {
+            devices: [{state: 'JOINED', correlationId: '111', url: '123'}]
+          }
+        };
+        const meeting = {
+          correlationId: '111'
+        };
+
+        assert.equal(MeetingsUtil.joinedOnThisDevice(meeting, newLocus, '123'), true);
+      });
+      it('return true if selfMoved on this device', () => {
+        const newLocus = {
+          self: {
+            devices: [{state: 'LEFT', reason: 'MOVED', correlationId: '111', url: '123'}]
+          }
+        };
+        const meeting = {
+          correlationId: '111'
+        };
+
+        assert.equal(MeetingsUtil.joinedOnThisDevice(meeting, newLocus, '123'), true);
+      });
+    });
+
+    describe("#handleRoapMercury", () => {
+      it('it sends the correct behaviour metric', () => {
+        const roapMessageReceived = sinon.stub();
+        const envelope = {
+          data: {
+            message:{
+              seq: "seq",
+              messageType: 'messageType',
+              tieBreaker: 'tieBreaker',
+              errorType: 'errorType',
+              errorCause: 'errorCause',
+              sdps: [{id:'sdp-1'}]
+            },
+            correlationId: 'correlationId',
+            eventType: 'locus.message.roap',
+          }
+        };
+        const meetingCollection = {
+          getByKey: () => ({
+            id: 'meeting-id',
+            roapMessageReceived
+          })
+        };
+
+        MeetingsUtil.handleRoapMercury(envelope, meetingCollection);
+        assert.calledWith(Metrics.sendBehavioralMetric, BEHAVIORAL_METRICS.ROAP_MERCURY_EVENT_RECEIVED,  {
+          correlation_id: 'correlationId',
+          seq: "seq",
+          message_type: 'messageType',
+          error_type: 'errorType',
+          error_cause: 'errorCause',
+        })
+        assert.calledWith(roapMessageReceived, {
+          seq: "seq",
+          messageType: 'messageType',
+          errorType: 'errorType',
+          tieBreaker: 'tieBreaker',
+          errorCause: 'errorCause',
+          sdp: {id:'sdp-1'}
+        })
+
+      });
+    })
+  });
+
+  describe('#isValidBreakoutLocus', () => {
+    it('returns false if is not breakout locus', () => {
+      const newLocus = {
+        controls: {}
+      };
+
+      assert.equal(MeetingsUtil.isValidBreakoutLocus(newLocus), false);
+    });
+
+    it('returns false if fullState is inactive', () => {
+      const newLocus = {
+        controls: {
+          breakout: {
+            sessionType: 'BREAKOUT',
+          },
+        },
+        fullState: {
+          state: 'INACTIVE'
+        }
+      };
+      assert.equal(MeetingsUtil.isValidBreakoutLocus(newLocus), false);
+    });
+
+    it('returns false if self is not joined', () => {
+      const newLocus = {
+        controls: {
+          breakout: {
+            sessionType: 'BREAKOUT',
+          },
+        },
+        fullState: {
+          state: 'ACTIVE'
+        },
+        self: {
+          state: 'LEFT'
+        }
+      };
+      assert.equal(MeetingsUtil.isValidBreakoutLocus(newLocus), false);
+    });
+
+    it('returns true if self is JOINED and fullState is active', () => {
+      const newLocus = {
+        controls: {
+          breakout: {
+            sessionType: 'BREAKOUT',
+          },
+        },
+        fullState: {
+          state: 'ACTIVE'
+        },
+        self: {
+          state: 'JOINED'
+        }
+      };
+      assert.equal(MeetingsUtil.isValidBreakoutLocus(newLocus), true);
+    });
   });
 });

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

@@ -13,41 +13,64 @@ describe('member', () => {
     sinon.restore();
   });
 
-  describe('#processParticipant', ()=>{
+  it('checks member properties', () => {
+    assert.exists(member.supportsInterpretation);
+    assert.exists(member.supportsBreakouts);
+    assert.exists(member.supportLiveAnnotation);
+    assert.exists(member.canReclaimHost);
+  });
+
+  describe('roles', () => {
+    it('checks that processParticipant calls processRoles', () => {
+      sinon.spy(member, 'processRoles');
+      member.processParticipant(participant);
+
+      assert.calledOnceWithExactly(member.processRoles, participant);
+    });
+
+    it('checks that processRoles calls extractControlRoles', () => {
+      sinon.spy(MemberUtil, 'extractControlRoles');
+      member.processParticipant(participant);
+
+      assert.calledOnceWithExactly(MemberUtil.extractControlRoles, participant);
+    });
+  });
+
+  describe('#processParticipant', () => {
     it('checks that processParticipant calls isHandRaised', () => {
       sinon.spy(MemberUtil, 'isHandRaised');
       member.processParticipant(participant);
-  
+
       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', ()=>{
+  describe('#processMember', () => {
     it('checks that processMember calls isRemovable', () => {
       sinon.spy(MemberUtil, 'isRemovable');
       member.processMember();
-  
+
       assert.calledOnce(MemberUtil.isRemovable);
     });
-  
+
     it('checks that processMember calls isMutable', () => {
       sinon.spy(MemberUtil, 'isMutable');
       member.processMember();
-  
+
       assert.calledOnce(MemberUtil.isMutable);
     });
-  
+
     it('checks that processMember calls extractMediaStatus', () => {
       sinon.spy(MemberUtil, 'extractMediaStatus');
       member.processMember();
-  
+
       assert.calledOnceWithExactly(MemberUtil.extractMediaStatus, participant);
     });
   })