@webex/calling 3.8.0 → 3.8.1-next.2

package/dist/Contacts/ContactsClient.test.js

@@ -36,6 +36,7 @@ describe('ContactClient Tests', function () {
   // eslint-disable-next-line no-underscore-dangle
   var contactServiceGroupUrl = "".concat(webex.internal.services._serviceUrls.contactsService, "/").concat(_constants2.ENCRYPT_FILTER, "/").concat(_constants2.USERS, "/").concat(_constants2.GROUP_FILTER);
   var serviceErrorCodeHandlerSpy = jest.spyOn(utils, 'serviceErrorCodeHandler');
+  var uploadLogsSpy = jest.spyOn(utils, 'uploadLogs').mockResolvedValue();
   var failureResponsePayload = {
     statusCode: 503,
     body: {}
@@ -47,6 +48,12 @@ describe('ContactClient Tests', function () {
     });
     expect(contactClient).toBeTruthy();
     expect(contactClient.getSDKConnector().getWebex()).toBeTruthy();
+
+    // Set up log spies for each test
+    jest.spyOn(_Logger.default, 'info');
+    jest.spyOn(_Logger.default, 'log');
+    jest.spyOn(_Logger.default, 'warn');
+    jest.spyOn(_Logger.default, 'error');
   });
   afterEach(function () {
     webex.request.mockClear();
@@ -211,11 +218,28 @@ describe('ContactClient Tests', function () {
                   }
                 });
               }
+              expect(_Logger.default.info).toHaveBeenCalledWith(_constants.METHOD_START_MESSAGE, {
+                file: _constants2.CONTACTS_FILE,
+                method: _constants2.METHODS.GET_CONTACTS
+              });
+              if (codeObj.payloadData) {
+                expect(_Logger.default.log).toHaveBeenCalledWith('Successfully fetched contacts and groups', {
+                  file: _constants2.CONTACTS_FILE,
+                  method: _constants2.METHODS.GET_CONTACTS
+                });
+              } else {
+                expect(_Logger.default.error).toHaveBeenCalled();
+              }
             } else {
               expect(webex.request).toBeCalledOnceWith({
                 uri: contactServiceUrl,
                 method: _types.HTTP_METHODS.GET
               });
+              expect(_Logger.default.info).toHaveBeenCalledWith(_constants.METHOD_START_MESSAGE, {
+                file: _constants2.CONTACTS_FILE,
+                method: _constants2.METHODS.GET_CONTACTS
+              });
+              expect(_Logger.default.error).toHaveBeenCalled();
             }
             expect(contactsResponse).toEqual({
               data: expect.any(Object),
@@ -227,7 +251,7 @@ describe('ContactClient Tests', function () {
             } else {
               expect(serviceErrorCodeHandlerSpy).toBeCalledOnceWith(codeObj.payloadData ? respPayload : expect.any(Error), {
                 file: _constants2.CONTACTS_FILE,
-                method: 'getContacts'
+                method: _constants2.METHODS.GET_CONTACTS
               });
             }
           case 8:
@@ -291,7 +315,27 @@ describe('ContactClient Tests', function () {
           });
           expect(contactClient['groups'].length).toEqual(2);
           expect(contactClient['groups'][1].displayName).toEqual('Top Contacts');
-        case 19:
+          expect(_Logger.default.info).toHaveBeenCalledWith("".concat(_constants.METHOD_START_MESSAGE, " with displayName: Top Contacts"), {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.CREATE_CONTACT_GROUP
+          });
+          expect(_Logger.default.info).toHaveBeenCalledWith(_constants.METHOD_START_MESSAGE, {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.CREATE_NEW_ENCRYPTION_KEY_URL
+          });
+          expect(_Logger.default.info).toHaveBeenCalledWith('Requesting kms for a new KRO and key', {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.CREATE_NEW_ENCRYPTION_KEY_URL
+          });
+          expect(_Logger.default.log).toHaveBeenCalledWith("Creating a default group: ".concat(_constants2.DEFAULT_GROUP_NAME), {
+            file: _constants2.CONTACTS_FILE,
+            method: 'fetchEncryptionKeyUrl'
+          });
+          expect(_Logger.default.log).toHaveBeenCalledWith("Contact group Top Contacts successfully created", {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.CREATE_CONTACT_GROUP
+          });
+        case 24:
         case "end":
           return _context2.stop();
       }
@@ -299,7 +343,7 @@ describe('ContactClient Tests', function () {
   })));
   it('create a contact group with existing key info', /*#__PURE__*/(0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee3() {
     var _contactsResponse$dat2;
-    var successResponsePayload, logInfoSpy, contactsResponse;
+    var successResponsePayload, infoSpy, contactsResponse;
     return _regenerator.default.wrap(function _callee3$(_context3) {
       while (1) switch (_context3.prev = _context3.next) {
         case 0:
@@ -310,16 +354,24 @@ describe('ContactClient Tests', function () {
           contactClient['groups'] = _contactFixtures.mockContactGroupListOne;
           webex.request.mockResolvedValue(successResponsePayload);
           webex.internal.encryption.encryptText.mockResolvedValue('Encrypted Top Contacts');
-          logInfoSpy = jest.spyOn(_Logger.default, 'info');
+          infoSpy = jest.spyOn(_Logger.default, 'info');
           _context3.next = 7;
           return contactClient.createContactGroup('Top Contacts');
         case 7:
           contactsResponse = _context3.sent;
           expect(contactsResponse.statusCode).toEqual(201);
           expect((_contactsResponse$dat2 = contactsResponse.data.group) === null || _contactsResponse$dat2 === void 0 ? void 0 : _contactsResponse$dat2.groupId).toBe(mockGroupResponse.groupId);
-          expect(logInfoSpy).not.toBeCalledWith('Requesting kms for a new KRO and key', {
+          expect(infoSpy).toBeCalledWith("".concat(_constants.METHOD_START_MESSAGE, " with displayName: Top Contacts"), {
             file: _constants2.CONTACTS_FILE,
-            method: 'createNewEncryptionKeyUrl'
+            method: _constants2.METHODS.CREATE_CONTACT_GROUP
+          });
+          expect(_Logger.default.log).toBeCalledWith("Contact group Top Contacts successfully created", {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.CREATE_CONTACT_GROUP
+          });
+          expect(infoSpy).not.toBeCalledWith(_constants.METHOD_START_MESSAGE, {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.CREATE_NEW_ENCRYPTION_KEY_URL
           });
           expect(webex.request).toBeCalledOnceWith({
             uri: contactServiceGroupUrl,
@@ -333,7 +385,7 @@ describe('ContactClient Tests', function () {
           });
           expect(contactClient['groups'].length).toEqual(2);
           expect(contactClient['groups'][1].displayName).toEqual('Top Contacts');
-        case 14:
+        case 16:
         case "end":
           return _context3.stop();
       }
@@ -356,17 +408,21 @@ describe('ContactClient Tests', function () {
           expect(contactsResponse.statusCode).toBe(400);
           expect(logSpy).toBeCalledOnceWith("Group name ".concat(mockGroupResponse.displayName, " already exists."), {
             file: _constants2.CONTACTS_FILE,
-            method: 'createContactGroup'
+            method: _constants2.METHODS.CREATE_CONTACT_GROUP
+          });
+          expect(_Logger.default.info).toBeCalledWith("".concat(_constants.METHOD_START_MESSAGE, " with displayName: ").concat(mockGroupResponse.displayName), {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.CREATE_CONTACT_GROUP
           });
           expect(contactClient['groups']).toEqual(_contactFixtures.mockContactResponseBodyOne.groups);
-        case 11:
+        case 12:
         case "end":
           return _context4.stop();
       }
     }, _callee4);
   })));
   it('create a contact group - service unavailable', /*#__PURE__*/(0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee5() {
-    var loggerContext, warnSpy, contactsResponse;
+    var loggerContext, warnSpy, errorSpy, contactsResponse;
     return _regenerator.default.wrap(function _callee5$(_context5) {
       while (1) switch (_context5.prev = _context5.next) {
         case 0:
@@ -380,9 +436,10 @@ describe('ContactClient Tests', function () {
           webex.internal.encryption.kms.createResource.mockResolvedValue(_contactFixtures.mockKmsKey);
           webex.internal.encryption.encryptText.mockResolvedValueOnce('Encrypted group name');
           warnSpy = jest.spyOn(_Logger.default, 'warn');
-          _context5.next = 9;
+          errorSpy = jest.spyOn(_Logger.default, 'error');
+          _context5.next = 10;
           return contactClient.createContactGroup('New group');
-        case 9:
+        case 10:
           contactsResponse = _context5.sent;
           expect(contactsResponse.statusCode).toBe(503);
           expect(webex.request).toBeCalledOnceWith({
@@ -395,19 +452,21 @@ describe('ContactClient Tests', function () {
               schemas: 'urn:cisco:codev:identity:contact:core:1.0'
             }
           });
-          expect(warnSpy).toBeCalledTimes(2);
-          expect(warnSpy).toHaveBeenNthCalledWith(1, 'Unable to create contact group.', loggerContext);
-          expect(warnSpy).toHaveBeenNthCalledWith(2, '503 Unable to establish a connection with the server', loggerContext);
+          expect(_Logger.default.info).toBeCalledWith("".concat(_constants.METHOD_START_MESSAGE, " with displayName: New group"), loggerContext);
+          expect(warnSpy).toBeCalledTimes(1);
+          expect(warnSpy).toHaveBeenNthCalledWith(1, '503 Unable to establish a connection with the server', loggerContext);
+          expect(errorSpy).toBeCalledTimes(1);
+          expect(errorSpy).toHaveBeenNthCalledWith(1, Error("Unable to create contact group: ".concat(failureResponsePayload)), loggerContext);
           expect(contactClient['groups']).toEqual(_contactFixtures.mockContactGroupListOne);
           expect(serviceErrorCodeHandlerSpy).toBeCalledOnceWith(failureResponsePayload, loggerContext);
-        case 17:
+        case 20:
         case "end":
           return _context5.stop();
       }
     }, _callee5);
   })));
   it('delete a contact group - service unavailable', /*#__PURE__*/(0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee6() {
-    var loggerContext, warnSpy, contactsResponse;
+    var loggerContext, warnSpy, errorSpy, contactsResponse;
     return _regenerator.default.wrap(function _callee6$(_context6) {
       while (1) switch (_context6.prev = _context6.next) {
         case 0:
@@ -420,21 +479,26 @@ describe('ContactClient Tests', function () {
           webex.internal.encryption.kms.createUnboundKeys.mockResolvedValue([_contactFixtures.mockKmsKey]);
           webex.internal.encryption.kms.createResource.mockResolvedValue(_contactFixtures.mockKmsKey);
           warnSpy = jest.spyOn(_Logger.default, 'warn');
-          _context6.next = 8;
+          errorSpy = jest.spyOn(_Logger.default, 'error');
+          _context6.next = 9;
           return contactClient.deleteContactGroup(mockGroupResponse.groupId);
-        case 8:
+        case 9:
           contactsResponse = _context6.sent;
           expect(contactsResponse.statusCode).toBe(503);
           expect(webex.request).toBeCalledOnceWith({
             method: _types.HTTP_METHODS.DELETE,
             uri: "".concat(contactServiceGroupUrl, "/").concat(mockGroupResponse.groupId)
           });
-          expect(warnSpy).toBeCalledTimes(2);
-          expect(warnSpy).toHaveBeenNthCalledWith(1, "Unable to delete contact group ".concat(mockGroupResponse.groupId), loggerContext);
-          expect(warnSpy).toHaveBeenNthCalledWith(2, '503 Unable to establish a connection with the server', loggerContext);
+          expect(_Logger.default.info).toBeCalledWith("".concat(_constants.METHOD_START_MESSAGE, " with groupId: ").concat(mockGroupResponse.groupId), loggerContext);
+          expect(_Logger.default.info).toBeCalledWith("Deleting contact group: ".concat(mockGroupResponse.groupId), loggerContext);
+          expect(warnSpy).toBeCalledTimes(1);
+          expect(errorSpy).toBeCalledTimes(1);
+          expect(uploadLogsSpy).toBeCalledTimes(1);
+          expect(errorSpy).toHaveBeenNthCalledWith(1, Error("Unable to delete contact group ".concat(mockGroupResponse.groupId, ": ").concat(failureResponsePayload)), loggerContext);
+          expect(warnSpy).toHaveBeenNthCalledWith(1, '503 Unable to establish a connection with the server', loggerContext);
           expect(contactClient['groups']).toEqual(_contactFixtures.mockContactGroupListOne);
           expect(serviceErrorCodeHandlerSpy).toBeCalledOnceWith(failureResponsePayload, loggerContext);
-        case 16:
+        case 21:
         case "end":
           return _context6.stop();
       }
@@ -460,7 +524,15 @@ describe('ContactClient Tests', function () {
             method: _types.HTTP_METHODS.DELETE
           });
           expect(contactClient['groups']).toEqual([]);
-        case 9:
+          expect(_Logger.default.info).toBeCalledWith("Deleting contact group: ".concat(_contactFixtures.mockContactGroupListOne[0].groupId), {
+            file: _constants2.CONTACTS_FILE,
+            method: 'deleteContactGroup'
+          });
+          expect(_Logger.default.log).toBeCalledWith("Contact group ".concat(_contactFixtures.mockContactGroupListOne[0].groupId, " successfully deleted"), {
+            file: _constants2.CONTACTS_FILE,
+            method: 'deleteContactGroup'
+          });
+        case 11:
         case "end":
           return _context7.stop();
       }
@@ -468,7 +540,7 @@ describe('ContactClient Tests', function () {
   })));
   it('create a contact with an existing group', /*#__PURE__*/(0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee8() {
     var _res$data$contact, _result$data$contact;
-    var mockContactResponse, successResponsePayload, logSpy, contact, res, result;
+    var mockContactResponse, successResponsePayload, infoSpy, logSpy, contact, res, result;
     return _regenerator.default.wrap(function _callee8$(_context8) {
       while (1) switch (_context8.prev = _context8.next) {
         case 0:
@@ -479,17 +551,26 @@ describe('ContactClient Tests', function () {
           };
           webex.request.mockResolvedValue(successResponsePayload);
           webex.internal.encryption.encryptText.mockResolvedValue('Encrypted contact name');
-          logSpy = jest.spyOn(_Logger.default, 'info');
+          infoSpy = jest.spyOn(_Logger.default, 'info');
+          logSpy = jest.spyOn(_Logger.default, 'log');
           contactClient['groups'] = _contactFixtures.mockContactGroupListOne;
           contactClient['encryptionKeyUrl'] = _contactFixtures.mockContactGroupListOne[0].encryptionKeyUrl;
           contact = _contactFixtures.mockContactListTwo.slice()[0];
           contact.groups = [];
-          _context8.next = 11;
+          _context8.next = 12;
           return contactClient.createContact(contact);
-        case 11:
+        case 12:
           res = _context8.sent;
           expect(res.statusCode).toEqual(201);
           expect((_res$data$contact = res.data.contact) === null || _res$data$contact === void 0 ? void 0 : _res$data$contact.contactId).toBe(mockContactResponse.contactId);
+          expect(infoSpy).toBeCalledWith("".concat(_constants.METHOD_START_MESSAGE, " with contactType: ").concat(contact.contactType), {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.CREATE_CONTACT
+          });
+          expect(logSpy).toBeCalledWith("Contact successfully created", {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.CREATE_CONTACT
+          });
           expect(logSpy).not.toBeCalledWith('Created a KRO and encryptionKeyUrl', {
             file: _constants2.CONTACTS_FILE,
             method: 'createNewEncryptionKeyUrl'
@@ -498,7 +579,7 @@ describe('ContactClient Tests', function () {
             file: _constants2.CONTACTS_FILE,
             method: 'createNewEncryptionKeyUrl'
           });
-          expect(logSpy).not.toBeCalledWith("Creating a default group: ".concat(_constants2.DEFAULT_GROUP_NAME), {
+          expect(infoSpy).not.toBeCalledWith("Creating a default group: ".concat(_constants2.DEFAULT_GROUP_NAME), {
             file: _constants2.CONTACTS_FILE,
             method: 'fetchEncryptionKeyUrl'
           });
@@ -515,16 +596,16 @@ describe('ContactClient Tests', function () {
           logSpy.mockClear();
 
           /* for coverage */
-          _context8.next = 22;
+          _context8.next = 25;
           return contactClient.createContact(contact);
-        case 22:
+        case 25:
           result = _context8.sent;
           expect((_result$data$contact = result.data.contact) === null || _result$data$contact === void 0 ? void 0 : _result$data$contact.contactId).toBe(mockContactResponse.contactId);
           expect(logSpy).not.toBeCalledWith("Creating a default group: ".concat(_constants2.DEFAULT_GROUP_NAME), {
             file: _constants2.CONTACTS_FILE,
             method: 'fetchEncryptionKeyUrl'
           });
-        case 25:
+        case 28:
         case "end":
           return _context8.stop();
       }
@@ -588,7 +669,27 @@ describe('ContactClient Tests', function () {
             keyUris: [_contactFixtures.mockKmsKey.uri]
           });
           expect((_res$data$contact2 = res.data.contact) === null || _res$data$contact2 === void 0 ? void 0 : _res$data$contact2.contactId).toBe(mockContactResponse.contactId);
-        case 21:
+          expect(_Logger.default.info).toBeCalledWith("".concat(_constants.METHOD_START_MESSAGE, " with contactType: CUSTOM"), {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.CREATE_CONTACT
+          });
+          expect(_Logger.default.info).toBeCalledWith(_constants.METHOD_START_MESSAGE, {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.CREATE_NEW_ENCRYPTION_KEY_URL
+          });
+          expect(_Logger.default.info).toBeCalledWith('Requesting kms for a new KRO and key', {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.CREATE_NEW_ENCRYPTION_KEY_URL
+          });
+          expect(_Logger.default.log).toBeCalledWith("Creating a default group: ".concat(_constants2.DEFAULT_GROUP_NAME), {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.FETCH_ENCRYPTION_KEY_URL
+          });
+          expect(_Logger.default.log).toBeCalledWith("Contact successfully created", {
+            file: _constants2.CONTACTS_FILE,
+            method: 'createContact'
+          });
+        case 26:
         case "end":
           return _context9.stop();
       }
@@ -623,10 +724,14 @@ describe('ContactClient Tests', function () {
           res = _context10.sent;
           expect(res.statusCode).toEqual(400);
           expect(res.data.error).toEqual('contactId is required for contactType:CLOUD.');
+          expect(_Logger.default.info).toBeCalledWith("".concat(_constants.METHOD_START_MESSAGE, " with contactType: CLOUD"), {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.CREATE_CONTACT
+          });
           contact.contactId = mockContactResponse.contactId;
-          _context10.next = 17;
+          _context10.next = 18;
           return contactClient.createContact(contact);
-        case 17:
+        case 18:
           res = _context10.sent;
           expect(res.statusCode).toEqual(201);
           expect((_res$data$contact3 = res.data.contact) === null || _res$data$contact3 === void 0 ? void 0 : _res$data$contact3.contactId).toBe(mockContactResponse.contactId);
@@ -660,7 +765,11 @@ describe('ContactClient Tests', function () {
               'spark-user-agent': 'webex-calling/beta'
             }
           });
-        case 24:
+          expect(_Logger.default.log).toBeCalledWith("Contact successfully created", {
+            file: _constants2.CONTACTS_FILE,
+            method: 'createContact'
+          });
+        case 26:
         case "end":
           return _context10.stop();
       }
@@ -693,10 +802,18 @@ describe('ContactClient Tests', function () {
           });
           expect(serviceErrorCodeHandlerSpy).toBeCalledOnceWith(failureResponsePayload, {
             file: _constants2.CONTACTS_FILE,
-            method: 'createContact'
+            method: _constants2.METHODS.CREATE_CONTACT
           });
           expect(res.statusCode).toEqual(503);
-        case 10:
+          expect(_Logger.default.info).toBeCalledWith("".concat(_constants.METHOD_START_MESSAGE, " with contactType: CLOUD"), {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.CREATE_CONTACT
+          });
+          expect(_Logger.default.error).toBeCalledWith(Error("Failed to create contact: ".concat(failureResponsePayload)), {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.CREATE_CONTACT
+          });
+        case 12:
         case "end":
           return _context11.stop();
       }
@@ -722,7 +839,15 @@ describe('ContactClient Tests', function () {
             method: _types.HTTP_METHODS.DELETE
           });
           expect(contactClient['contacts']).toEqual([]);
-        case 9:
+          expect(_Logger.default.info).toBeCalledWith("".concat(_constants.METHOD_START_MESSAGE, " with contactId: ").concat(_contactFixtures.mockContactListOne[0].contactId), {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.DELETE_CONTACT
+          });
+          expect(_Logger.default.info).toBeCalledWith("Deleting contact : ".concat(_contactFixtures.mockContactListOne[0].contactId), {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.DELETE_CONTACT
+          });
+        case 11:
         case "end":
           return _context12.stop();
       }
@@ -746,10 +871,18 @@ describe('ContactClient Tests', function () {
           });
           expect(serviceErrorCodeHandlerSpy).toBeCalledOnceWith(failureResponsePayload, {
             file: _constants2.CONTACTS_FILE,
-            method: 'deleteContact'
+            method: _constants2.METHODS.DELETE_CONTACT
           });
           expect(contactClient['contacts']).toEqual(_contactFixtures.mockContactListOne);
-        case 9:
+          expect(_Logger.default.info).toBeCalledWith("".concat(_constants.METHOD_START_MESSAGE, " with contactId: ").concat(_contactFixtures.mockContactListOne[0].contactId), {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.DELETE_CONTACT
+          });
+          expect(_Logger.default.info).toBeCalledWith("Deleting contact : ".concat(_contactFixtures.mockContactListOne[0].contactId), {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.DELETE_CONTACT
+          });
+        case 11:
         case "end":
           return _context13.stop();
       }
@@ -813,13 +946,18 @@ describe('ContactClient Tests', function () {
     }]);
   });
   it('test resolveContacts function encountering an error', function () {
+    var warnSpy = jest.spyOn(_Logger.default, 'warn');
     var contact = contactClient['resolveCloudContacts']({
       userId: _contactFixtures.mockContactMinimum
     }, _contactFixtures.mockSCIMMinListResponse);
     expect(contact).toEqual(null);
+    expect(warnSpy).toHaveBeenCalledWith('Error occurred while parsing resolved contacts', {
+      file: _constants2.CONTACTS_FILE,
+      method: 'resolveCloudContacts'
+    });
   });
   it('logs error for chunk when scimQuery API call fails in the loop for getContacts', /*#__PURE__*/(0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee14() {
-    var mockData, respPayload;
+    var mockData, respPayload, warnSpy, infoSpy, logSpy;
     return _regenerator.default.wrap(function _callee14$(_context14) {
       while (1) switch (_context14.prev = _context14.next) {
         case 0:
@@ -836,17 +974,27 @@ describe('ContactClient Tests', function () {
           mockData.decryptTextList.forEach(function (text) {
             webex.internal.encryption.decryptText.mockResolvedValueOnce(text);
           });
-          jest.spyOn(_Logger.default, 'warn');
-          _context14.next = 7;
+          warnSpy = jest.spyOn(_Logger.default, 'warn');
+          infoSpy = jest.spyOn(_Logger.default, 'info');
+          logSpy = jest.spyOn(_Logger.default, 'log');
+          _context14.next = 9;
           return contactClient.getContacts();
-        case 7:
+        case 9:
           expect(webex.request).toBeCalledTimes(2);
-          expect(_Logger.default.warn).toBeCalledTimes(1);
-          expect(_Logger.default.warn).toBeCalledWith('Error processing contact chunk 0-50', {
+          expect(warnSpy).toBeCalledTimes(1);
+          expect(warnSpy).toBeCalledWith('Error processing contact chunk 0-50', {
             file: 'Contacts',
-            method: 'getContacts'
+            method: _constants2.METHODS.GET_CONTACTS
           });
-        case 10:
+          expect(infoSpy).toBeCalledWith(_constants.METHOD_START_MESSAGE, {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.GET_CONTACTS
+          });
+          expect(logSpy).toBeCalledWith('Successfully fetched contacts and groups', {
+            file: _constants2.CONTACTS_FILE,
+            method: _constants2.METHODS.GET_CONTACTS
+          });
+        case 14:
         case "end":
           return _context14.stop();
       }