@webex/plugin-meetings 1.153.1 → 1.154.0

package/src/metrics/index.js

@@ -14,7 +14,7 @@ import BrowserDetection from '../common/browser-detection';
 
 import {
   error, eventType, errorCodes as ERROR_CODE, OS_NAME, UNKNOWN, CLIENT_NAME,
-  mediaType
+  mediaType, PLATFORM
 } from './config';
 
 const OSMap = {
@@ -489,11 +489,11 @@ class Metrics {
   }
 
   /**
-   * Uploads given metric to the Metrics service as an operational metric.
+   * Uploads given metric to the Metrics service as an Behavioral metric.
    * Metadata about the environment such as browser, OS, SDK and their versions
    * are automatically added when the metric is sent.
    *
-   * The Metrics service will send an operational metric to InfluxDB for
+   * The Metrics service will send an Behavioral metric to InfluxDB for
    * aggregation.
    * See https://confluence-eng-gpk2.cisco.com/conf/display/WBXT/Getting+started+with+Metrics+Service.
    *
@@ -503,12 +503,24 @@ class Metrics {
    *
    * @returns {void}
    */
-  sendOperationalMetric(metricName, metricFields = {}, metricTags = {}) {
+  sendBehavioralMetric(metricName, metricFields = {}, metricTags = {}) {
     const fields = {
       ...metricFields,
       browser_version: getBrowserVersion(),
       os_version: getOSVersion(),
-      sdk_version: this.webex.version
+      sdk_version: this.webex.version,
+      platform: PLATFORM
+    };
+
+    const context = {
+      app: {
+        version: this.webex.version
+      },
+      locale: 'en-US',
+      os: {
+        name: getOSName(),
+        version: getOSVersion()
+      }
     };
 
     const tags = {
@@ -517,17 +529,19 @@ class Metrics {
       org_id: this.webex.credentials.getOrgId(),
       os: getOSName(),
       domain: window.location.hostname,
-      client_id: this.webex.credentials.config.client_id
+      client_id: this.webex.credentials.config.client_id,
+      user_id: this.webex.internal.device.userId
     };
 
     if (!metricName) {
-      throw Error('Missing operational metric name. Please provide one');
+      throw Error('Missing behavioral metric name. Please provide one');
     }
 
     this.webex.internal.metrics.submitClientMetrics(metricName, {
-      type: ['operational'],
+      type: ['behavioral', 'operational'],
       fields,
-      tags
+      tags,
+      context
     });
   }
 }

package/src/peer-connection-manager/index.js

@@ -10,9 +10,8 @@ import Metrics from '../metrics';
 import LoggerProxy from '../common/logs/logger-proxy';
 import StaticConfig from '../common/config';
 import {
-  RETRY_TIMEOUT,
-  ICE_TIMEOUT,
-  HOST,
+  COMPLETE,
+  GATHERING,
   AUDIO,
   SDP,
   ICE_STATE,
@@ -22,13 +21,12 @@ import {
   OFFER,
   QUALITY_LEVELS,
   MAX_FRAMESIZES,
-  METRICS_OPERATIONAL_MEASURES,
-  IPV4_REGEX
+  BEHAVIORAL_METRICS
 } from '../constants';
 import {error, eventType} from '../metrics/config';
 import MediaError from '../common/errors/media';
 import ParameterError from '../common/errors/parameter';
-import {InvalidSdpError} from '../common/errors/webex-errors';
+import {InvalidSdpError, IceGatheringFailed} from '../common/errors/webex-errors';
 import BrowserDetection from '../common/browser-detection';
 
 import PeerConnectionUtils from './util';
@@ -122,18 +120,10 @@ const isSdpInvalid = (sdp) => {
   const parsedSdp = sdpTransform.parse(sdp);
 
   for (const mediaLine of parsedSdp.media) {
-    if (mediaLine.candidates && mediaLine.candidates.length === 0) {
+    if (!mediaLine.candidates || mediaLine.candidates?.length === 0) {
       LoggerProxy.logger.error('PeerConnectionManager:index#isSdpInvalid --> iceCandidate: Ice candadate never completed');
 
-      return 'iceCandidate: Ice candadate never completed';
-    }
-    // Sometimes the candidates might be there but only IPV6 we need to makes sure we have IPV4
-    const hostCandidate = mediaLine.candidates.filter((candidate) => !!(candidate.type === HOST && candidate.ip.match(IPV4_REGEX)));
-
-    if (hostCandidate.length === 0) {
-      LoggerProxy.logger.error('PeerConnectionManager:index#isSdpInvalid --> iceCandidate: no IPV4 candidate present');
-
-      return 'iceCandidate: no IPV4 candidate present';
+      return 'iceCandidate: Ice gathering never completed';
     }
 
     if (SDP.BAD_MEDIA_PORTS.includes(mediaLine.port)) {
@@ -193,45 +183,52 @@ pc.setContentSlides = (screenPc) => {
  */
 pc.iceCandidate = (peerConnection, {remoteQualityLevel}) =>
   new Promise((resolve, reject) => {
-    // TODO: we dont need timeout as we can check the api state and validate.
-    const timeout = setTimeout(() => {
+    const now = Date.now();
+    const doneGatheringIceCandidate = () => {
+      const miliseconds = parseInt(Math.abs(Date.now() - now), 4);
+
       peerConnection.sdp = limitBandwidth(peerConnection.localDescription.sdp);
       peerConnection.sdp = setMaxFs(peerConnection.sdp, remoteQualityLevel);
       peerConnection.sdp = PeerConnectionUtils.convertCLineToIpv4(peerConnection.sdp);
 
-      if (isSdpInvalid(peerConnection.sdp)) {
-        setTimeout(() => {
-          // peerconnection does gather ice candidate IP but in some cases due to firewall
-          // or proxy the ice candidate does not get gathered so we need to wait and then retry
-          // if still not valid then throw an error saying missing ice candidate
-          // if ice candidate still not present after retry
-          const invalidSdpPresent = isSdpInvalid(peerConnection.sdp);
+      const invalidSdpPresent = isSdpInvalid(peerConnection.sdp);
 
-          if (!invalidSdpPresent) {
-            resolve(peerConnection);
-          }
-          else {
-            LoggerProxy.logger.error('PeerConnectionManager:index#iceCandidate --> SDP not valid after waiting.');
-            reject(new InvalidSdpError(invalidSdpPresent));
-          }
-        }, RETRY_TIMEOUT);
+      if (invalidSdpPresent) {
+        LoggerProxy.logger.error('PeerConnectionManager:index#iceCandidate --> SDP not valid after waiting.');
+        reject(new InvalidSdpError(invalidSdpPresent));
       }
-      else {
-        resolve(peerConnection);
+      LoggerProxy.logger.log(`PeerConnectionManager:index#iceCandidate --> Time to gather ice candidate ${miliseconds} miliseconds`);
+
+
+      resolve();
+    };
+
+    // If ice has already been gathered
+    if (peerConnection.iceGatheringState === COMPLETE) {
+      doneGatheringIceCandidate();
+    }
+
+    peerConnection.onIceGatheringStateChange = () => {
+      if (peerConnection.iceGatheringState === COMPLETE) {
+        doneGatheringIceCandidate(peerConnection);
       }
-    }, ICE_TIMEOUT);
+      if (peerConnection.iceGatheringState === GATHERING) {
+        LoggerProxy.logger.log('PeerConnectionManager:index#onIceGatheringStateChange --> Ice state changed to gathering');
+      }
+    };
 
     peerConnection.onicecandidate = (evt) => {
-      if (!evt.candidate && !peerConnection.sdp) {
-        peerConnection.sdp = limitBandwidth(peerConnection.localDescription.sdp);
-        peerConnection.sdp = setMaxFs(peerConnection.sdp, remoteQualityLevel);
-        peerConnection.sdp = PeerConnectionUtils.convertCLineToIpv4(peerConnection.sdp);
-
-        if (evt.candidate === null && !isSdpInvalid(peerConnection.sdp)) {
-          clearTimeout(timeout);
-          resolve(peerConnection);
-        }
+      if (evt.candidate === null) {
+        doneGatheringIceCandidate(peerConnection);
       }
+      else {
+        LoggerProxy.logger.log(`PeerConnectionManager:index#onicecandidate --> Candidate ${evt.candidate?.type} ${evt.candidate?.protocol} ${evt.candidate?.address}:${evt.candidate?.port}`);
+      }
+    };
+
+    peerConnection.onicecandidateerror = (event) => {
+      LoggerProxy.logger.error('PeerConnectionManager:index#onicecandidateerror --> Failed to gather ice candidate.', event);
+      reject(new IceGatheringFailed());
     };
   });
 
@@ -345,7 +342,7 @@ pc.setRemoteSessionDetails = (
         LoggerProxy.logger.error(`Peer-connection-manager:index#setRemoteDescription --> ${error} missing remotesdp`);
 
 
-        const metricName = METRICS_OPERATIONAL_MEASURES.PEERCONNECTION_FAILURE;
+        const metricName = BEHAVIORAL_METRICS.PEERCONNECTION_FAILURE;
         const data = {
           correlation_id: meetingId,
           reason: error.message,
@@ -355,7 +352,7 @@ pc.setRemoteSessionDetails = (
           type: error.name
         };
 
-        Metrics.sendOperationalMetric(metricName, data, metadata);
+        Metrics.sendBehavioralMetric(metricName, data, metadata);
 
         return Metrics.postEvent({
           event: eventType.REMOTE_SDP_RECEIVED,
@@ -406,9 +403,6 @@ pc.createOffer = (peerConnection, {
     })
     .then(() => pc.iceCandidate(peerConnection, {remoteQualityLevel}))
     .then(() => {
-      peerConnection.sdp = limitBandwidth(peerConnection.localDescription.sdp);
-      peerConnection.sdp = setMaxFs(peerConnection.sdp, remoteQualityLevel);
-      peerConnection.sdp = PeerConnectionUtils.convertCLineToIpv4(peerConnection.sdp);
       if (!checkH264Support(peerConnection.sdp)) {
         throw new MediaError('openH264 is downloading please Wait. Upload logs if not working on second try');
       }
@@ -429,8 +423,8 @@ pc.createOffer = (peerConnection, {
     .catch((error) => {
       LoggerProxy.logger.error(`Peer-connection-manager:index#createOffer --> ${error}`);
       if (error instanceof InvalidSdpError) {
-        Metrics.sendOperationalMetric(
-          METRICS_OPERATIONAL_MEASURES.INVALID_ICE_CANDIDATE,
+        Metrics.sendBehavioralMetric(
+          BEHAVIORAL_METRICS.INVALID_ICE_CANDIDATE,
           {
             correlation_id: meetingId,
             code: error.code,
@@ -439,7 +433,7 @@ pc.createOffer = (peerConnection, {
         );
       }
       else {
-        const metricName = METRICS_OPERATIONAL_MEASURES.PEERCONNECTION_FAILURE;
+        const metricName = BEHAVIORAL_METRICS.PEERCONNECTION_FAILURE;
         const data = {
           correlation_id: meetingId,
           reason: error.message,
@@ -449,7 +443,7 @@ pc.createOffer = (peerConnection, {
           type: error.name
         };
 
-        Metrics.sendOperationalMetric(metricName, data, metadata);
+        Metrics.sendBehavioralMetric(metricName, data, metadata);
       }
 
       Metrics.postEvent({
@@ -544,15 +538,15 @@ pc.createAnswer = (params, {meetingId, remoteQualityLevel}) => {
     })
     .catch((error) => {
       if (error instanceof InvalidSdpError) {
-        Metrics.sendOperationalMetric(
-          METRICS_OPERATIONAL_MEASURES.INVALID_ICE_CANDIDATE,
+        Metrics.sendBehavioralMetric(
+          BEHAVIORAL_METRICS.INVALID_ICE_CANDIDATE,
           {
             correlation_id: meetingId
           }
         );
       }
       else {
-        const metricName = METRICS_OPERATIONAL_MEASURES.PEERCONNECTION_FAILURE;
+        const metricName = BEHAVIORAL_METRICS.PEERCONNECTION_FAILURE;
         const data = {
           correlation_id: meetingId,
           reason: error.message,
@@ -562,7 +556,7 @@ pc.createAnswer = (params, {meetingId, remoteQualityLevel}) => {
           type: error.name
         };
 
-        Metrics.sendOperationalMetric(metricName, data, metadata);
+        Metrics.sendBehavioralMetric(metricName, data, metadata);
       }
 
       LoggerProxy.logger.error(`PeerConnectionManager:index#setRemoteSessionDetails --> Error creating remote session, error: ${error}`);
@@ -625,8 +619,8 @@ pc.setPeerConnectionEvents = (meeting) => {
       function: 'connectionFailed'
     });
 
-    Metrics.sendOperationalMetric(
-      METRICS_OPERATIONAL_MEASURES.CONNECTION_FAILURE,
+    Metrics.sendBehavioralMetric(
+      BEHAVIORAL_METRICS.CONNECTION_FAILURE,
       {
         correlation_id: meeting.correlationId,
         locus_id: meeting.locusId
@@ -648,6 +642,13 @@ pc.setPeerConnectionEvents = (meeting) => {
         // Ice connection state goes to connected when both client and server sends STUN packets and
         // Established connected between them. Firefox does not trigger COMPLETED and only trigger CONNECTED
         Metrics.postEvent({event: eventType.ICE_END, meeting});
+        Metrics.sendBehavioralMetric(
+          BEHAVIORAL_METRICS.CONNECTION_SUCCESS,
+          {
+            correlation_id: meeting.correlationId,
+            locus_id: meeting.locusId
+          }
+        );
         meeting.setNetworkStatus(NETWORK_STATUS.CONNECTED);
         meeting.reconnectionManager.iceReconnected();
         LoggerProxy.logger.info('PeerConnectionManager:index#setPeerConnectionEvents --> ICE STATE CONNECTED.');

package/src/reconnection-manager/index.js

@@ -8,7 +8,7 @@ import LoggerProxy from '../common/logs/logger-proxy';
 import Trigger from '../common/events/trigger-proxy';
 import {
   EVENT_TRIGGERS,
-  METRICS_OPERATIONAL_MEASURES,
+  BEHAVIORAL_METRICS,
   RECONNECTION,
   SHARE_STATUS,
   SHARE_STOPPED_REASON,
@@ -424,8 +424,8 @@ export default class ReconnectionManager {
       }
       else {
         LoggerProxy.logger.error('ReconnectionManager:index#rejoinMeeting --> Unable to rejoin meeting after max attempts.', joinError);
-        Metrics.sendOperationalMetric(
-          METRICS_OPERATIONAL_MEASURES.MEETING_MAX_REJOIN_FAILURE,
+        Metrics.sendBehavioralMetric(
+          BEHAVIORAL_METRICS.MEETING_MAX_REJOIN_FAILURE,
           {
             locus_id: this.meeting.locusUrl.split('/').pop(),
             reason: joinError.message,

package/src/roap/handler.js

@@ -2,7 +2,7 @@
 import {StatelessWebexPlugin} from '@webex/webex-core';
 
 import LoggerProxy from '../common/logs/logger-proxy';
-import {ROAP, _OFFER_, METRICS_OPERATIONAL_MEASURES} from '../constants';
+import {ROAP, _OFFER_, BEHAVIORAL_METRICS} from '../constants';
 import Metrics from '../metrics';
 
 import RoapUtil from './util';
@@ -44,14 +44,14 @@ const handleSessionStep = ({
   if (session.OFFER && messageType === _OFFER_) {
     session.GLARE_OFFER = roap.msg;
     session.GLARE_OFFER.remote = !!roap.remote;
-    const metricName = METRICS_OPERATIONAL_MEASURES.ROAP_GLARE_CONDITION;
+    const metricName = BEHAVIORAL_METRICS.ROAP_GLARE_CONDITION;
     const data = {
       correlation_id: correlationId,
       locus_id: locusUrl.split('/').pop(),
       sequence: sequenceId
     };
 
-    Metrics.sendOperationalMetric(metricName, data);
+    Metrics.sendBehavioralMetric(metricName, data);
 
     LoggerProxy.logger.warn(`Roap:handler#handleSessionStep --> Glare condition occurred with new mercury event, sequenceId: ${sequenceId}`);
   }
@@ -112,7 +112,7 @@ export default class RoapHandler extends StatelessWebexPlugin {
               });
             })
             .catch((error) => {
-              const metricName = METRICS_OPERATIONAL_MEASURES.ROAP_ANSWER_FAILURE;
+              const metricName = BEHAVIORAL_METRICS.ROAP_ANSWER_FAILURE;
               const data = {
                 correlation_id: meeting.correlationId,
                 locus_id: meeting.locusUrl.split('/').pop(),
@@ -123,7 +123,7 @@ export default class RoapHandler extends StatelessWebexPlugin {
                 type: error.name
               };
 
-              Metrics.sendOperationalMetric(metricName, data, metadata);
+              Metrics.sendBehavioralMetric(metricName, data, metadata);
               LoggerProxy.logger.error(`Roap:handler#perform --> Error occured during wait receive answer, continuing, ${error}`);
             });
         }

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

@@ -30,7 +30,7 @@ import Metrics from '@webex/plugin-meetings/src/metrics';
 import {
   FLOOR_ACTION,
   SHARE_STATUS,
-  METRICS_OPERATIONAL_MEASURES,
+  BEHAVIORAL_METRICS,
   MEETING_INFO_FAILURE_REASON,
   PASSWORD_STATUS,
   EVENTS,
@@ -48,6 +48,7 @@ import WebExMeetingsErrors from '../../../../src/common/errors/webex-meetings-er
 import ParameterError from '../../../../src/common/errors/parameter';
 import PasswordError from '../../../../src/common/errors/password-error';
 import CaptchaError from '../../../../src/common/errors/captcha-error';
+import IntentToJoinError from '../../../../src/common/errors/intent-to-join';
 import DefaultSDKConfig from '../../../../src/config';
 import testUtils from '../../../utils/testUtils';
 import {MeetingInfoV2CaptchaError, MeetingInfoV2PasswordError} from '../../../../src/meeting-info/meeting-info-v2';
@@ -609,6 +610,17 @@ describe('plugin-meetings', () => {
         });
       });
       describe('#join', () => {
+        let sandbox = null;
+
+        beforeEach(() => {
+          sandbox = sinon.createSandbox();
+        });
+
+        afterEach(() => {
+          sandbox.restore();
+          sandbox = null;
+        });
+
         it('should have #join', () => {
           assert.exists(meeting.join);
         });
@@ -618,8 +630,9 @@ describe('plugin-meetings', () => {
         });
         describe('successful', () => {
           beforeEach(() => {
-            MeetingUtil.joinMeeting = sinon.stub().returns(Promise.resolve());
+            sandbox.stub(MeetingUtil, 'joinMeeting').returns(Promise.resolve());
           });
+
           it('should join the meeting and return promise', async () => {
             const join = meeting.join();
 
@@ -649,9 +662,10 @@ describe('plugin-meetings', () => {
         });
         describe('failure', () => {
           beforeEach(() => {
-            MeetingUtil.joinMeeting = sinon.stub().returns(Promise.reject());
+            sandbox.stub(MeetingUtil, 'joinMeeting').returns(Promise.reject());
             meeting.logger.log = sinon.stub().returns(true);
           });
+
           describe('guest join', () => {
             beforeEach(() => {
               MeetingUtil.isPinOrGuest = sinon.stub().returns(true);
@@ -662,6 +676,24 @@ describe('plugin-meetings', () => {
                 assert.calledOnce(MeetingUtil.joinMeeting);
               });
             });
+            it('should succeed when called again after IntentToJoinError error', async () => {
+              let joinSucceeded = false;
+
+              try {
+                await meeting.join();
+                joinSucceeded = true;
+              }
+              catch (e) {
+                assert.instanceOf(e, IntentToJoinError);
+              }
+              assert.isFalse(joinSucceeded);
+
+              // IntentToJoinError means that client should call join() again
+              // with moderator and pin explicitly set
+              MeetingUtil.joinMeeting = sinon.stub().returns(Promise.resolve());
+              await meeting.join({pin: '1234', moderator: false});
+              assert.calledWith(MeetingUtil.joinMeeting, meeting, {moderator: false, pin: '1234'});
+            });
           });
           it('should fail if password is required', async () => {
             meeting.passwordStatus = PASSWORD_STATUS.REQUIRED;
@@ -2401,12 +2433,12 @@ describe('plugin-meetings', () => {
           });
 
           it('should send metrics on reconnect failure', async () => {
-            sandbox.stub(Metrics, 'sendOperationalMetric');
+            sandbox.stub(Metrics, 'sendBehavioralMetric');
             await assert.isRejected(meeting.reconnect());
-            assert(Metrics.sendOperationalMetric.calledOnce);
+            assert(Metrics.sendBehavioralMetric.calledOnce);
             assert.calledWith(
-              Metrics.sendOperationalMetric,
-              METRICS_OPERATIONAL_MEASURES.MEETING_RECONNECT_FAILURE,
+              Metrics.sendBehavioralMetric,
+              BEHAVIORAL_METRICS.MEETING_RECONNECT_FAILURE,
               {
                 correlation_id: meeting.correlationId,
                 locus_id: meeting.locusUrl.split('/').pop(),

package/test/unit/spec/meeting/request.js

@@ -19,10 +19,18 @@ describe('plugin-meetings', () => {
       regionCode: 'WEST-COAST'
     };
 
+    webex.internal = {
+      services: {
+        get: sinon.mock().returns('locusUrl'),
+        waitForCatalog: sinon.mock().returns(Promise.resolve({}))
+      }
+    };
+
     meetingsRequest = new MeetingRequest({}, {
       parent: webex
     });
 
+
     meetingsRequest.request = sinon.mock().returns(Promise.resolve({}));
   });
 
@@ -112,12 +120,15 @@ describe('plugin-meetings', () => {
         const deviceUrl = 'deviceUrl';
         const correlationId = 'random-uuid';
         const roapMessage = 'roap-message';
+        const permissionToken = 'permission-token';
 
         await meetingsRequest.joinMeeting({
           locusUrl,
           deviceUrl,
           correlationId,
-          roapMessage
+          roapMessage,
+          permissionToken
+
         });
         const requestParams = meetingsRequest.request.getCall(0).args[0];
 
@@ -125,8 +136,49 @@ describe('plugin-meetings', () => {
         assert.equal(requestParams.uri, `${locusUrl}/participant?alternateRedirect=true`);
         assert.equal(requestParams.body.device.url, deviceUrl);
         assert.equal(requestParams.body.device.countryCode, 'US');
+        assert.equal(requestParams.body.permissionToken, 'permission-token');
         assert.equal(requestParams.body.device.regionCode, 'WEST-COAST');
       });
+
+      it('sends /call with meetingNumber if inviteeAddress does not exist', async () => {
+        const deviceUrl = 'deviceUrl';
+        const correlationId = 'random-uuid';
+        const roapMessage = 'roap-message';
+        const meetingNumber = 'meetingNumber';
+
+        await meetingsRequest.joinMeeting({
+          deviceUrl,
+          correlationId,
+          roapMessage,
+          meetingNumber
+        });
+        const requestParams = meetingsRequest.request.getCall(0).args[0];
+
+        assert.equal(requestParams.method, 'POST');
+        assert.equal(requestParams.uri, 'locusUrl/loci/call?alternateRedirect=true');
+        assert.equal(requestParams.body.invitee.address, 'wbxmn:meetingNumber');
+      });
+
+      it('sends /call with inviteeAddress over meetingNumber as preference', async () => {
+        const deviceUrl = 'deviceUrl';
+        const correlationId = 'random-uuid';
+        const roapMessage = 'roap-message';
+        const meetingNumber = 'meetingNumber';
+        const inviteeAddress = 'sipUrl';
+
+        await meetingsRequest.joinMeeting({
+          deviceUrl,
+          correlationId,
+          roapMessage,
+          meetingNumber,
+          inviteeAddress
+        });
+        const requestParams = meetingsRequest.request.getCall(0).args[0];
+
+        assert.equal(requestParams.method, 'POST');
+        assert.equal(requestParams.uri, 'locusUrl/loci/call?alternateRedirect=true');
+        assert.equal(requestParams.body.invitee.address, 'sipUrl');
+      });
     });
 
     describe('#pstn', () => {

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

@@ -4,6 +4,7 @@ import MeetingUtil from '@webex/plugin-meetings/src/meeting/util';
 import LoggerProxy from '@webex/plugin-meetings/src/common/logs/logger-proxy';
 import LoggerConfig
   from '@webex/plugin-meetings/src/common/logs/logger-config';
+import Metrics from '@webex/plugin-meetings/src/metrics/index';
 
 describe('plugin-meetings', () => {
   describe('Meeting utils function', () => {
@@ -11,6 +12,7 @@ describe('plugin-meetings', () => {
     const meeting = {};
 
     beforeEach(() => {
+      Metrics.postEvent = sinon.stub();
       const logger = {
         info: sandbox.stub(),
         log: sandbox.stub(),
@@ -132,6 +134,45 @@ describe('plugin-meetings', () => {
         });
       });
     });
+
+    describe('joinMeeting', () => {
+      it('#Should call `meetingRequest.joinMeeting', async () => {
+        const meeting = {meetingJoinUrl: 'meetingJoinUrl', locusUrl: 'locusUrl', meetingRequest: {joinMeeting: sinon.stub().returns(Promise.resolve({body: {}, headers: {}}))}};
+
+        MeetingUtil.parseLocusJoin = sinon.stub();
+        await MeetingUtil.joinMeeting(meeting, {});
+
+        assert.calledOnce(meeting.meetingRequest.joinMeeting);
+        const parameter = meeting.meetingRequest.joinMeeting.getCall(0).args[0];
+
+        assert.equal(parameter.inviteeAddress, 'meetingJoinUrl');
+      });
+
+      it('#Should fallback sipUrl if meetingJoinUrl does not exists', async () => {
+        const meeting = {sipUri: 'sipUri', locusUrl: 'locusUrl', meetingRequest: {joinMeeting: sinon.stub().returns(Promise.resolve({body: {}, headers: {}}))}};
+
+        MeetingUtil.parseLocusJoin = sinon.stub();
+        await MeetingUtil.joinMeeting(meeting, {});
+
+        assert.calledOnce(meeting.meetingRequest.joinMeeting);
+        const parameter = meeting.meetingRequest.joinMeeting.getCall(0).args[0];
+
+        assert.equal(parameter.inviteeAddress, 'sipUri');
+      });
+
+      it('#Should fallback to meetingNumber if meetingJoinUrl/sipUrl  does not exists', async () => {
+        const meeting = {meetingNumber: 'meetingNumber', locusUrl: 'locusUrl', meetingRequest: {joinMeeting: sinon.stub().returns(Promise.resolve({body: {}, headers: {}}))}};
+
+        MeetingUtil.parseLocusJoin = sinon.stub();
+        await MeetingUtil.joinMeeting(meeting, {});
+
+        assert.calledOnce(meeting.meetingRequest.joinMeeting);
+        const parameter = meeting.meetingRequest.joinMeeting.getCall(0).args[0];
+
+        assert.isUndefined(parameter.inviteeAddress);
+        assert.equal(parameter.meetingNumber, 'meetingNumber');
+      });
+    });
   });
 });
 

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

@@ -468,7 +468,7 @@ skipInBrowser(describe)('plugin-meetings', () => {
         });
         describe('successful MeetingInfo.#fetchMeetingInfo', () => {
           beforeEach(() => {
-            webex.meetings.meetingInfo.fetchMeetingInfo = sinon.stub().returns(Promise.resolve(true));
+            webex.meetings.meetingInfo.fetchMeetingInfo = sinon.stub().returns(Promise.resolve({body: {permissionToken: 'PT', meetingJoinUrl: 'meetingJoinUrl'}}));
           });
           it('creates the meeting from a successful meeting info fetch promise testing', async () => {
             const meeting = webex.meetings.createMeeting('test', 'test');
@@ -482,6 +482,8 @@ skipInBrowser(describe)('plugin-meetings', () => {
             assert.calledWith(webex.meetings.meetingInfo.fetchMeetingInfo, 'test');
             assert.calledWith(MeetingsUtil.extractDestination, 'test', 'test');
             assert.calledWith(MeetingsUtil.getMeetingAddedType, 'test');
+            assert.equal(meeting.permissionToken, 'PT');
+            assert.equal(meeting.meetingJoinUrl, 'meetingJoinUrl');
           });
 
           it('creates the meeting from a successful meeting info fetch meeting resolve testing', async () => {

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

@@ -90,9 +90,9 @@ browserOnly(describe)('Meeting metrics', () => {
     });
   });
 
-  describe('#sendOperationalMetric', () => {
+  describe('#sendBehavioralMetric', () => {
     it('sends client metric via Metrics plugin', () => {
-      metrics.sendOperationalMetric('myMetric');
+      metrics.sendBehavioralMetric('myMetric');
 
       assert.calledOnce(mockSubmitMetric);
     });
@@ -101,13 +101,13 @@ browserOnly(describe)('Meeting metrics', () => {
       const data = {value: 567};
       const metadata = {test: true};
 
-      metrics.sendOperationalMetric('myMetric', data, metadata);
+      metrics.sendBehavioralMetric('myMetric', data, metadata);
 
       assert.calledWithMatch(
         mockSubmitMetric,
         'myMetric',
         {
-          type: ['operational'],
+          type: ['behavioral'],
           fields: {
             browser_version: getBrowserVersion(),
             os_version: getOSVersion(),
@@ -127,8 +127,8 @@ browserOnly(describe)('Meeting metrics', () => {
 
     it('throws error if no metric name is given', () => {
       assert.throws(
-        () => metrics.sendOperationalMetric(),
-        'Missing operational metric name. Please provide one'
+        () => metrics.sendBehavioralMetric(),
+        'Missing behavioral metric name. Please provide one'
       );
     });
   });