@webex/plugin-meetings 2.60.1-next.9 → 2.60.2

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

@@ -1,17 +1,8 @@
 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', () => {
@@ -42,7 +33,7 @@ describe('plugin-meetings', () => {
         );
       });
 
-      it('should take the first site if none are default', () => {
+      it('should work fine if no default true site', () => {
         const userPreferences = {
           sites: [
             {
@@ -60,10 +51,7 @@ describe('plugin-meetings', () => {
           ],
         };
 
-        assert.equal(
-          MeetingsUtil.parseDefaultSiteFromMeetingPreferences(userPreferences),
-          'site1-example.webex.com'
-        );
+        assert.equal(MeetingsUtil.parseDefaultSiteFromMeetingPreferences(userPreferences), '');
       });
 
       it('should work fine if sites an empty array', () => {
@@ -74,193 +62,5 @@ 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,64 +13,41 @@ describe('member', () => {
     sinon.restore();
   });
 
-  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', () => {
+  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);
     });
   })