@webex/calling 3.8.0 → 3.8.1-next.2

package/dist/CallingClient/calling/call.test.js

@@ -25,6 +25,7 @@ var _testUtil = require("../../common/testUtil");
 var _callManager = require("./callManager");
 var _Logger = _interopRequireDefault(require("../../Logger"));
 var _Errors = require("../../Errors");
+var _constants2 = require("../../common/constants");
 function _getRequireWildcardCache(e) { if ("function" != typeof _WeakMap) return null; var r = new _WeakMap(), t = new _WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(e) { return e ? t : r; })(e); }
 function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != _typeof(e) && "function" != typeof e) return { default: e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = _Object$defineProperty && _Object$getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && Object.prototype.hasOwnProperty.call(e, u)) { var i = a ? _Object$getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? _Object$defineProperty(n, u, i) : n[u] = e[u]; } return n.default = e, t && t.set(e, n), n; }
 /* eslint-disable @typescript-eslint/no-explicit-any */
@@ -32,6 +33,7 @@ function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e;
 /* eslint-disable @typescript-eslint/no-shadow */
 
 jest.mock('@webex/internal-media-core');
+var uploadLogsSpy = jest.spyOn(Utils, 'uploadLogs').mockResolvedValue(undefined);
 var webex = (0, _testUtil.getTestUtilsWebex)();
 var mockInternalMediaCoreModule = InternalMediaCoreModule;
 var defaultServiceIndicator = _types3.ServiceIndicator.CALLING;
@@ -134,7 +136,7 @@ describe('Call Tests', function () {
     expect(call).toBeTruthy();
   });
   it('should log a warning when sending a digit fails', /*#__PURE__*/(0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee() {
-    var tone, errorMessage, mockMediaConnection, callManager, call, realMediaConnection, logWarnSpy;
+    var tone, errorMessage, mockMediaConnection, callManager, call, realMediaConnection, warnSpy, infoSpy;
     return _regenerator.default.wrap(function _callee$(_context) {
       while (1) switch (_context.prev = _context.next) {
         case 0:
@@ -150,12 +152,19 @@ describe('Call Tests', function () {
           realMediaConnection = call.mediaConnection; // Set the mock mediaConnection object
           call.mediaConnection = mockMediaConnection;
 
-          // Spy on the log.warn method
-          logWarnSpy = jest.spyOn(_Logger.default, 'warn'); // Call the sendDigit method
+          // Spy on the log methods
+          warnSpy = jest.spyOn(_Logger.default, 'warn');
+          infoSpy = jest.spyOn(_Logger.default, 'info'); // Call the sendDigit method
           call.sendDigit(tone);
 
+          // Expect log.info to be called with the digit being sent
+          expect(infoSpy).toHaveBeenCalledWith("".concat(_constants2.METHOD_START_MESSAGE, " with: ").concat(tone), {
+            file: 'call',
+            method: 'sendDigit'
+          });
+
           // Expect the log.warn method to be called with the error message
-          expect(logWarnSpy).toHaveBeenLastCalledWith("Unable to send digit on call: ".concat(errorMessage), {
+          expect(warnSpy).toHaveBeenCalledWith("Unable to send digit on call: ".concat(errorMessage), {
             file: 'call',
             method: 'sendDigit'
           });
@@ -163,21 +172,21 @@ describe('Call Tests', function () {
           // Restore the real mediaConnection object
           call.mediaConnection = realMediaConnection;
           call.end();
-          _context.next = 14;
+          _context.next = 16;
           return (0, _Utils.waitForMsecs)(50);
-        case 14:
+        case 16:
           // Need to add a small delay for Promise and callback to finish.
 
           /* After call ends, call manager should have 0 record */
           expect((0, _keys.default)(callManager.getActiveCalls()).length).toBe(0);
-        case 15:
+        case 17:
         case "end":
           return _context.stop();
       }
     }, _callee);
   })));
   it('delete call object when ending the call', /*#__PURE__*/(0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee2() {
-    var callManager, mockStream, localAudioStream, call;
+    var callManager, mockStream, localAudioStream, call, infoSpy;
     return _regenerator.default.wrap(function _callee2$(_context2) {
       while (1) switch (_context2.prev = _context2.next) {
         case 0:
@@ -198,19 +207,28 @@ describe('Call Tests', function () {
           };
           localAudioStream = mockStream;
           call = callManager.createCall(_types3.CallDirection.OUTBOUND, deviceId, mockLineId, dest);
+          infoSpy = jest.spyOn(_Logger.default, 'info');
           expect(call).toBeTruthy();
           /* After creation , call manager should have 1 record */
           expect((0, _keys.default)(callManager.getActiveCalls()).length).toBe(1);
           call.mute(localAudioStream);
           expect(call.isMuted()).toEqual(true);
           expect(mockStream.setUserMuted).toBeCalledOnceWith(true);
+          expect(infoSpy).toHaveBeenCalledWith('invoking with: user mute', {
+            file: 'call',
+            method: 'mute'
+          });
           call.mute(localAudioStream);
           expect(mockStream.setUserMuted).toBeCalledWith(false);
           expect(call.isMuted()).toEqual(false);
           call.end();
-          _context2.next = 16;
+          expect(infoSpy).toHaveBeenCalledWith(_constants2.METHOD_START_MESSAGE, {
+            file: 'call',
+            method: 'end'
+          });
+          _context2.next = 19;
           return (0, _Utils.waitForMsecs)(50);
-        case 16:
+        case 19:
           // Need to add a small delay for Promise and callback to finish.
           expect(parseMediaQualityStatisticsMock).toHaveBeenCalledTimes(1);
           expect(webex.request.mock.calls[0][0].body.metrics).toStrictEqual(disconnectStats);
@@ -219,7 +237,7 @@ describe('Call Tests', function () {
 
           /* After call ends, call manager should have 0 record */
           expect((0, _keys.default)(callManager.getActiveCalls()).length).toBe(0);
-        case 21:
+        case 24:
         case "end":
           return _context2.stop();
       }
@@ -330,7 +348,7 @@ describe('Call Tests', function () {
     }, _callee6, null, [[1, 7]]);
   })));
   it('dial functionality tests for coverage', /*#__PURE__*/(0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee7() {
-    var mockStream, localAudioStream, warnSpy, call, bnrMetricSpy;
+    var mockStream, localAudioStream, warnSpy, call, bnrMetricSpy, logSpy;
     return _regenerator.default.wrap(function _callee7$(_context7) {
       while (1) switch (_context7.prev = _context7.next) {
         case 0:
@@ -347,10 +365,12 @@ describe('Call Tests', function () {
           warnSpy = jest.spyOn(_Logger.default, 'warn');
           call = (0, _call.createCall)(activeUrl, webex, _types3.CallDirection.OUTBOUND, deviceId, mockLineId, deleteCallFromCollection, defaultServiceIndicator, dest);
           bnrMetricSpy = jest.spyOn(call['metricManager'], 'submitBNRMetric');
+          logSpy = jest.spyOn(_Logger.default, 'log');
           call.dial(localAudioStream);
           expect(mockTrack.enabled).toEqual(true);
           expect(mockInternalMediaCoreModule.RoapMediaConnection).toBeCalledOnceWith(roapMediaConnectionConfig, roapMediaConnectionOptions, expect.any(String), expect.any(Function), expect.any(Function), expect.any(Function));
           expect(call['mediaStateMachine'].state.value).toBe('S_SEND_ROAP_OFFER');
+          expect(logSpy).toHaveBeenCalled();
           expect(bnrMetricSpy).toBeCalledOnceWith(_types4.METRIC_EVENT.BNR_ENABLED, _types4.METRIC_TYPE.BEHAVIORAL, call.getCallId(), call.getCorrelationId());
 
           /* Now change the state and recall to check for error */
@@ -361,14 +381,14 @@ describe('Call Tests', function () {
             file: 'call',
             method: 'dial'
           });
-        case 14:
+        case 16:
         case "end":
           return _context7.stop();
       }
     }, _callee7);
   })));
   it('answer functionality tests for coverage', /*#__PURE__*/(0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee8() {
-    var mockStream, localAudioStream, warnSpy, call, bnrMetricSpy;
+    var mockStream, localAudioStream, warnSpy, call, bnrMetricSpy, infoSpy;
     return _regenerator.default.wrap(function _callee8$(_context8) {
       while (1) switch (_context8.prev = _context8.next) {
         case 0:
@@ -386,6 +406,7 @@ describe('Call Tests', function () {
           call = (0, _call.createCall)(activeUrl, webex, _types3.CallDirection.OUTBOUND, deviceId, mockLineId, deleteCallFromCollection, defaultServiceIndicator, dest);
           /** Cannot answer in idle state */
           bnrMetricSpy = jest.spyOn(call['metricManager'], 'submitBNRMetric');
+          infoSpy = jest.spyOn(_Logger.default, 'info');
           call.answer(localAudioStream);
           expect(mockTrack.enabled).toEqual(true);
           expect(mockInternalMediaCoreModule.RoapMediaConnection).toBeCalledOnceWith(roapMediaConnectionConfig, roapMediaConnectionOptions, expect.any(String), expect.any(Function), expect.any(Function), expect.any(Function));
@@ -394,13 +415,14 @@ describe('Call Tests', function () {
             file: 'call',
             method: 'answer'
           });
+          expect(infoSpy).toHaveBeenCalled();
 
           /* Now change the state and recall to check for correct flow */
           call['callStateMachine'].state.value = 'S_SEND_CALL_PROGRESS';
           call.answer(localAudioStream);
           expect(call['callStateMachine'].state.value).toBe('S_SEND_CALL_CONNECT');
           expect(bnrMetricSpy).toBeCalledOnceWith(_types4.METRIC_EVENT.BNR_ENABLED, _types4.METRIC_TYPE.BEHAVIORAL, call.getCallId(), call.getCorrelationId());
-        case 14:
+        case 16:
         case "end":
           return _context8.stop();
       }
@@ -601,6 +623,7 @@ describe('Call Tests', function () {
   it('update media with invalid stream', function () {
     var callManager = (0, _callManager.getCallManager)(webex, defaultServiceIndicator);
     var warnSpy = jest.spyOn(_Logger.default, 'warn');
+    var infoSpy = jest.spyOn(_Logger.default, 'info');
     var mockStream = {
       outputStream: {
         getAudioTracks: jest.fn().mockReturnValue([mockTrack])
@@ -620,10 +643,14 @@ describe('Call Tests', function () {
     var localAudioStream2 = errorStream;
     call.updateMedia(localAudioStream2);
     expect(call['mediaConnection'].updateLocalTracks).not.toBeCalled();
-    expect(warnSpy).toBeCalledOnceWith("Did not find a local track while updating media for call ".concat(call.getCorrelationId(), ". Will not update media"), {
+    expect(warnSpy).toHaveBeenCalledWith("Did not find a local track while updating media for call ".concat(call.getCorrelationId(), ". Will not update media"), {
       file: 'call',
       method: 'updateMedia'
     });
+    expect(infoSpy).toHaveBeenCalledWith('invoking with stream', {
+      file: 'call',
+      method: 'dial'
+    });
   });
   it('test system mute and user mute different scnearios', /*#__PURE__*/(0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee12() {
     var logSpy, callManager, mockStream, localAudioStream, call;
@@ -661,7 +688,7 @@ describe('Call Tests', function () {
           call.mute(localAudioStream, _types5.MUTE_TYPE.USER);
           expect(call.isMuted()).toEqual(true);
           expect(mockStream.setUserMuted).not.toBeCalledOnceWith(true);
-          expect(logSpy).toBeCalledOnceWith("Call is muted on the system - ".concat(call.getCorrelationId(), "."), {
+          expect(logSpy).toHaveBeenCalledWith("Call is muted on the system - ".concat(call.getCorrelationId(), "."), {
             file: 'call',
             method: 'mute'
           });
@@ -682,7 +709,7 @@ describe('Call Tests', function () {
           mockStream.systemMuted = true;
           call.mute(localAudioStream, _types5.MUTE_TYPE.SYSTEM);
           expect(call.isMuted()).toEqual(true);
-          expect(logSpy).toBeCalledOnceWith("Call is muted by the user already - ".concat(call.getCorrelationId(), "."), {
+          expect(logSpy).toHaveBeenCalledWith("Call is muted by the user already - ".concat(call.getCorrelationId(), "."), {
             file: 'call',
             method: 'mute'
           });
@@ -817,7 +844,11 @@ describe('State Machine handler tests', function () {
             file: 'call',
             method: 'handleCallEstablished'
           });
-        case 14:
+          expect(logSpy).toHaveBeenCalledWith("".concat(_constants2.METHOD_START_MESSAGE, " with: ").concat(call.getCorrelationId()), {
+            file: 'call',
+            method: 'handleCallEstablished'
+          });
+        case 15:
         case "end":
           return _context15.stop();
       }
@@ -967,11 +998,12 @@ describe('State Machine handler tests', function () {
     }, _callee18);
   })));
   it('state changes during unsuccessful incoming call due error in call connect', /*#__PURE__*/(0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee19() {
-    var warnSpy, stateMachineSpy, statusPayload, roapMessage;
+    var warnSpy, errorSpy, stateMachineSpy, statusPayload, roapMessage;
     return _regenerator.default.wrap(function _callee19$(_context19) {
       while (1) switch (_context19.prev = _context19.next) {
         case 0:
           warnSpy = jest.spyOn(_Logger.default, 'warn');
+          errorSpy = jest.spyOn(_Logger.default, 'error');
           stateMachineSpy = jest.spyOn(call, 'sendCallStateMachineEvt');
           statusPayload = {
             statusCode: 200,
@@ -995,22 +1027,23 @@ describe('State Machine handler tests', function () {
           webex.request.mockRejectedValueOnce({
             statusCode: 403
           }).mockResolvedValue(statusPayload);
-          _context19.next = 12;
+          _context19.next = 13;
           return call['handleOutgoingCallConnect']({
             type: 'E_SEND_CALL_CONNECT'
           });
-        case 12:
+        case 13:
           expect(call['callStateMachine'].state.value).toBe('S_UNKNOWN');
           expect(stateMachineSpy).toBeCalledTimes(3);
-          expect(warnSpy).toBeCalledTimes(4);
-        case 15:
+          expect(warnSpy).toBeCalledTimes(3);
+          expect(errorSpy).toBeCalledTimes(1);
+        case 17:
         case "end":
           return _context19.stop();
       }
     }, _callee19);
   })));
   it('state changes during successful outgoing call', /*#__PURE__*/(0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee20() {
-    var statusPayload, dummyEvent, postMediaSpy, dummyOkEvent;
+    var statusPayload, dummyEvent, postMediaSpy, logSpy, dummyOkEvent;
     return _regenerator.default.wrap(function _callee20$(_context20) {
       while (1) switch (_context20.prev = _context20.next) {
         case 0:
@@ -1023,9 +1056,11 @@ describe('State Machine handler tests', function () {
             data: undefined
           };
           postMediaSpy = jest.spyOn(call, 'postMedia');
+          logSpy = jest.spyOn(_Logger.default, 'log');
           webex.request.mockReturnValue(statusPayload);
           call.sendCallStateMachineEvt(dummyEvent);
           expect(call['callStateMachine'].state.value).toBe('S_SEND_CALL_SETUP');
+          expect(logSpy).toHaveBeenCalled();
           dummyEvent.type = 'E_RECV_CALL_PROGRESS';
           call.sendCallStateMachineEvt(dummyEvent);
           expect(call['callStateMachine'].state.value).toBe('S_RECV_CALL_PROGRESS');
@@ -1084,7 +1119,7 @@ describe('State Machine handler tests', function () {
             type: 'E_SEND_CALL_DISCONNECT'
           });
           expect(call['callStateMachine'].state.value).toBe('S_SEND_CALL_DISCONNECT');
-        case 40:
+        case 42:
         case "end":
           return _context20.stop();
       }
@@ -1212,7 +1247,7 @@ describe('State Machine handler tests', function () {
     }, _callee22);
   })));
   it('state changes during unsuccessful outgoing call due to error in call setup', /*#__PURE__*/(0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee23() {
-    var statusPayload, dummyEvent;
+    var statusPayload, dummyEvent, errorSpy;
     return _regenerator.default.wrap(function _callee23$(_context23) {
       while (1) switch (_context23.prev = _context23.next) {
         case 0:
@@ -1229,19 +1264,25 @@ describe('State Machine handler tests', function () {
             }
           };
           webex.request.mockRejectedValueOnce(statusPayload);
+          errorSpy = jest.spyOn(_Logger.default, 'error');
           call.sendCallStateMachineEvt(dummyEvent);
-          _context23.next = 6;
+          _context23.next = 7;
           return (0, _testUtil.flushPromises)(3);
-        case 6:
-          expect(call['callStateMachine'].state.value).toBe('S_UNKNOWN');
         case 7:
+          expect(call['callStateMachine'].state.value).toBe('S_UNKNOWN');
+          expect(errorSpy).toHaveBeenCalled();
+          expect(uploadLogsSpy).toHaveBeenCalledWith({
+            correlationId: call.getCorrelationId(),
+            callId: call.getCallId()
+          });
+        case 10:
         case "end":
           return _context23.stop();
       }
     }, _callee23);
   })));
   it('state changes during unsuccessful outgoing call due to error in media ok', /*#__PURE__*/(0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee24() {
-    var statusPayload, dummyEvent;
+    var statusPayload, dummyEvent, warnSpy;
     return _regenerator.default.wrap(function _callee24$(_context24) {
       while (1) switch (_context24.prev = _context24.next) {
         case 0:
@@ -1260,16 +1301,26 @@ describe('State Machine handler tests', function () {
           call['earlyMedia'] = true;
           call['mediaStateMachine'].state.value = 'S_RECV_ROAP_ANSWER';
           webex.request.mockRejectedValue(statusPayload);
-          _context24.next = 8;
-          return call['handleRoapEstablished']({}, dummyEvent);
-        case 8:
+          warnSpy = jest.spyOn(_Logger.default, 'warn');
+          jest.spyOn(Utils, 'uploadLogs').mockResolvedValue(undefined);
           _context24.next = 10;
-          return (0, _testUtil.flushPromises)(2);
+          return call['handleRoapEstablished']({}, dummyEvent);
         case 10:
+          _context24.next = 12;
+          return (0, _testUtil.flushPromises)(2);
+        case 12:
           expect(call.isConnected()).toBe(false);
           expect(call['mediaStateMachine'].state.value).toBe('S_ROAP_ERROR');
           expect(call['callStateMachine'].state.value).toBe('S_UNKNOWN');
-        case 13:
+          expect(warnSpy).toHaveBeenCalledWith('Failed to process MediaOk request', {
+            file: 'call',
+            method: 'handleRoapEstablished'
+          });
+          expect(uploadLogsSpy).toHaveBeenCalledWith({
+            correlationId: call.getCorrelationId(),
+            callId: call.getCallId()
+          });
+        case 17:
         case "end":
           return _context24.stop();
       }
@@ -1991,7 +2042,7 @@ describe('State Machine handler tests', function () {
           dummyEvent.data = undefined;
           call.sendCallStateMachineEvt(dummyEvent);
           expect(call['callStateMachine'].state.value).toBe('S_CALL_HOLD');
-          expect(infoSpy).toHaveBeenLastCalledWith("handleCallHold: ".concat(call.getCorrelationId(), "  "), {
+          expect(infoSpy).toHaveBeenLastCalledWith("".concat(_constants2.METHOD_START_MESSAGE, " with: ").concat(call.getCorrelationId()), {
             file: 'call',
             method: 'handleCallHold'
           });
@@ -2949,7 +3000,7 @@ describe('Supplementary Services tests', function () {
       return _regenerator.default.wrap(function _callee63$(_context63) {
         while (1) switch (_context63.prev = _context63.next) {
           case 0:
-            expect.assertions(10);
+            expect.assertions(12); // Updated to match actual assertion count
             responsePayload = {
               statusCode: 200,
               body: mockResponseBody
@@ -3012,7 +3063,9 @@ describe('Supplementary Services tests', function () {
             expect(uploadLogsSpy).not.toBeCalled();
             expect(infoSpy).toHaveBeenCalledWith("Initiating Consult transfer between : ".concat(call.getCallId(), " and ").concat(secondCall.getCallId()), transferLoggingContext);
             expect(warnSpy).not.toHaveBeenCalledWith("Consult Transfer failed for correlationId ".concat(call.getCorrelationId()), transferLoggingContext);
-          case 22:
+            expect(infoSpy).toHaveBeenCalledWith("Initiating Consult transfer between : ".concat(call.getCallId(), " and ").concat(secondCall.getCallId()), transferLoggingContext);
+            expect(warnSpy).not.toHaveBeenCalledWith("Consult Transfer failed for correlationId ".concat(call.getCorrelationId()), transferLoggingContext);
+          case 24:
           case "end":
             return _context63.stop();
         }
@@ -3023,7 +3076,7 @@ describe('Supplementary Services tests', function () {
       return _regenerator.default.wrap(function _callee65$(_context65) {
         while (1) switch (_context65.prev = _context65.next) {
           case 0:
-            expect.assertions(8);
+            expect.assertions(10); // Updated to match actual assertion count
             responsePayload = {
               statusCode: 200,
               body: mockResponseBody
@@ -3066,7 +3119,9 @@ describe('Supplementary Services tests', function () {
             expect(uploadLogsSpy).not.toBeCalled();
             expect(infoSpy).toHaveBeenCalledWith("Initiating Blind transfer with : ".concat(transfereeNumber), transferLoggingContext);
             expect(warnSpy).not.toHaveBeenCalledWith("Blind Transfer failed for correlationId ".concat(call.getCorrelationId()), transferLoggingContext);
-          case 19:
+            expect(infoSpy).toHaveBeenCalledWith("Initiating Blind transfer with : ".concat(transfereeNumber), transferLoggingContext);
+            expect(warnSpy).not.toHaveBeenCalledWith("Blind Transfer failed for correlationId ".concat(call.getCorrelationId()), transferLoggingContext);
+          case 21:
           case "end":
             return _context65.stop();
         }