CS-1264 Improved Logging to debug Lambda Functions

package.json

@@ -21,6 +21,6 @@
   },
   "scripts": {
     "test": "jest  --watchAll",
-    "lint": "eslint ./src ./__tests__"
+    "lint": "eslint --ext .js src/* __tests__/unit/* --fix"
   }
 }

src/constants/url.js

@@ -9,9 +9,9 @@
  * Copyright (C) SugarCRM Inc. All rights reserved.
  */
 
-module.exports = { 
+module.exports = {
     /**
      * URL Partials
      */
-    HTTPS_PREFIX: 'https://',
+    HTTPS_PREFIX: 'https://'
 };

src/core/data.js

@@ -9,7 +9,7 @@
  * Copyright (C) SugarCRM Inc. All rights reserved.
  */
 
-var Bean = require('./bean.js');
+let Bean = require('./bean.js');
 
 module.exports = {
     createBean: (module, data) => {

src/handlers/lex-get-string.js

@@ -9,7 +9,7 @@
  * Copyright (C) SugarCRM Inc. All rights reserved.
  */
 
-const loggerUtils = require("../utils/logger-utils");
+const loggerUtils = require('../utils/logger-utils');
 
 /**
  * Close dialog with the customer, reporting fulfillmentState Fulfilled
@@ -24,34 +24,34 @@ function close(sessionAttributes, fulfillmentState, message) {
             type: 'Close',
             fulfillmentState,
             message
-        },
+        }
     };
 }
- 
+
 /**
  * Events
  * @param intentRequest
  * @param callback
  */
 function dispatch(intentRequest, callback) {
     const sessionAttributes = intentRequest.sessionAttributes;
-    
+
     // assign the whole user input to sessionAttributes.string
     sessionAttributes.string = intentRequest.inputTranscript;
-    
+
     callback(
         close(
             sessionAttributes,
             'Fulfilled',
             {
                 'contentType': 'PlainText',
                 'content': 'You entered: ' + sessionAttributes.string
-                
+
             }
         )
     );
 }
- 
+
 /**
  * Main handler
  * @param event
@@ -70,4 +70,3 @@ exports.handler = (event, context, callback) => {
         callback(err);
     }
 };
-

src/utils/logger-utils.js

@@ -12,10 +12,10 @@
 /**
  * Log the JSON to AWS Cloudwatch
  *
- * @param {string} title 
- * @param {Object} json 
+ * @param {string} title
+ * @param {Object} json
  */
-function logJson (title, json) {
+function logJson(title, json) {
     console.log(`${title}:\n`, JSON.stringify(json));
 }
 
@@ -59,18 +59,18 @@ function logReturnValue(returnValue) {
 /**
  * Log the S3 event for debugging
  *
- * @param {Object} event 
+ * @param {Object} event
  */
-function logS3Event (event) {
+function logS3Event(event) {
     logJson('S3 Event', event);
 }
 
 /**
  * Log the Sugar API response for debugging
  *
- * @param {string} response 
+ * @param {string} response
  */
-function logSugarApiResponse (response) {
+function logSugarApiResponse(response) {
     logJson('Sugar API Response', response);
 }
 

src/utils/sugar/call-record-utils.js

@@ -23,7 +23,7 @@ const CallsConstants = require('../../constants/sugar-modules/calls');
 const ErrorMessages = require('../../constants/messages/error');
 const UrlConstants = require('../../constants/url');
 
- /**
+/**
   * Return a single call record, false otherwise
   *
   * @param {string} contactId the AWS Connect contact ID
@@ -53,39 +53,39 @@ async function getCallRecord(contactId) {
  * Build the call recording URL based on an AWS Connect URL and not S3. Appropriate
  * AWS Connect user permissions are required to access the URL
  *
- * @param {string} contactId 
+ * @param {string} contactId
  */
-function buildCallRecordingUrl (contactId) {
-  let connectInstance = process.env.awsConnectInstance;
-  let connectDomain = process.env.awsConnectDomain;
-  let callRecordingPartialUrl = process.env.callRecordingPartialUrl;
+function buildCallRecordingUrl(contactId) {
+    let connectInstance = process.env.awsConnectInstance;
+    let connectDomain = process.env.awsConnectDomain;
+    let callRecordingPartialUrl = process.env.callRecordingPartialUrl;
 
-  // return empty if contact ID is not defined
-  if (!contactId) {
-    console.warn('Call Recording URL Warning: Unable to construct a URL without AWS Connect contact ID');
-    return '';
-  }
+    // return empty if contact ID is not defined
+    if (!contactId) {
+        console.warn('Call Recording URL Warning: Unable to construct a URL without AWS Connect contact ID');
+        return '';
+    }
 
-  // return empty if one or more of the required parameters is not defined
-  if (!connectInstance || !connectDomain || !callRecordingPartialUrl) {
-    console.warn('Call Recording URL Warning: Unable to construct a URL with missing required parameter(s). ' +
+    // return empty if one or more of the required parameters is not defined
+    if (!connectInstance || !connectDomain || !callRecordingPartialUrl) {
+        console.warn('Call Recording URL Warning: Unable to construct a URL with missing required parameter(s). ' +
       'Please check the following parameters: AwsConnectInstance, AWSConnectDomain, CallRecordingPartialURL');
-    return '';
-  }
+        return '';
+    }
 
-  let encodedConnectInstance = encodeURIComponent(connectInstance);
-  let encodedContactId = encodeURIComponent(contactId);
+    let encodedConnectInstance = encodeURIComponent(connectInstance);
+    let encodedContactId = encodeURIComponent(contactId);
 
-  connectDomain = connectDomain.trim();
-  callRecordingPartialUrl = callRecordingPartialUrl.trim();
+    connectDomain = connectDomain.trim();
+    callRecordingPartialUrl = callRecordingPartialUrl.trim();
 
-  let url = UrlConstants.HTTPS_PREFIX + encodedConnectInstance + '.' + connectDomain +
+    let url = UrlConstants.HTTPS_PREFIX + encodedConnectInstance + '.' + connectDomain +
     callRecordingPartialUrl + encodedContactId;
 
-  return url;
+    return url;
 }
 
-module.exports = { 
-  getCallRecord,
-  buildCallRecordingUrl
+module.exports = {
+    getCallRecord,
+    buildCallRecordingUrl
 };