fiftyone.pipeline.core 4.4.7 → 4.4.9

package/elementData.js

@@ -21,7 +21,7 @@
  * ********************************************************************* */
 
 const util = require('util');
-const errorMessages = require('./errorMessages')
+const errorMessages = require('./errorMessages');
 
 /**
  * @typedef {import('./flowElement')} FlowElement
@@ -32,7 +32,6 @@ const errorMessages = require('./errorMessages')
  * Stored in flowData
  */
 class ElementData {
-
   /**
    * Constructor for elementData
    *
@@ -54,18 +53,18 @@ class ElementData {
         // This proxy can end up getting called by anything that tries
         // to access methods & properties on the object.
         // This can include things like the inspector if you do something like
-        // console.log(flowData.location) 
-        // In future, this mechanism will be superseded by a less problematic 
+        // console.log(flowData.location)
+        // In future, this mechanism will be superseded by a less problematic
         // approach. For now, we work around this by only passing string keys
         // to the data getters.
-        if (typeof(key) === 'string' || key == Symbol.iterator) {
+        if (typeof (key) === 'string' || key === Symbol.iterator) {
           try {
             return data[key] || data.get(key);
-          } catch(e) {
-            // If the key was 'inspect' and an error was thrown then we 
+          } catch (e) {
+            // If the key was 'inspect' and an error was thrown then we
             // can ignore it. Otherwise, throw the error back up the stack.
-            if(key != 'inspect') {
-              throw(e);
+            if (key !== 'inspect') {
+              throw (e);
             }
           }
         }
@@ -92,10 +91,12 @@ class ElementData {
    * @returns {mixed} value
    */
   get (key) {
-    let value = this.getInternal(key);
+    const value = this.getInternal(key);
     if (typeof value === 'undefined') {
-      throw util.format(errorMessages.genericMissingProperties, key) + 
-        (typeof this.flowElement === 'undefined' ? '' : ' in data for element "' +
+      throw util.format(errorMessages.genericMissingProperties, key) +
+        (typeof this.flowElement === 'undefined'
+          ? ''
+          : ' in data for element "' +
           this.flowElement.dataKey) + '".' +
         util.format(errorMessages.noReasonUnknown);
     }

package/errorMessages.js

@@ -26,4 +26,4 @@ module.exports = {
   genericMissingProperties: 'Property "%s" not found',
   noReasonUnknown: ' Please check that the element and property names ' +
   'are correct.'
-}
+};

package/examples/examples.test.js

@@ -48,7 +48,7 @@ const testExample = function ({ file, portNumber }) {
 
   code += serverClose;
 
-  () => jest.fn(eval(code));
+  () => jest.fn(eval(code)); // eslint-disable-line
 };
 
 test('pipeline functionality example', () => {

package/examples/flowElementsForExamples/async.js

@@ -52,7 +52,7 @@ class Async extends FlowElement {
         const contents = { string: 'hello' };
 
         const data = new ElementDataDictionary(
-          { flowElement: flowElement, contents: contents }
+          { flowElement, contents }
         );
 
         flowData.setElementData(data);

package/examples/flowElementsForExamples/sync.js

@@ -63,7 +63,7 @@ class Sync extends FlowElement {
     }
 
     const data = new ElementDataDictionary(
-      { flowElement: this, contents: contents }
+      { flowElement: this, contents }
     );
 
     flowData.setElementData(data);

package/flowData.js

@@ -69,12 +69,11 @@ class FlowData {
         // to the data getters.
         if (typeof key === 'string' || key === Symbol.iterator) {
           try {
-            let value = data[key]; 
+            let value = data[key];
             if (typeof value === 'undefined') {
               value = data.get(key);
             }
             return value;
-
           } catch (e) {
             // If the key was 'then' and an error was thrown then we
             // can ignore it. Otherwise, throw the error back up the stack.
@@ -142,14 +141,13 @@ class FlowData {
 
     this.errors[errorKey] = error;
 
-    let logMessage = "Error occurred during processing";
+    let logMessage = 'Error occurred during processing';
 
     if (errorKey !== undefined) {
-        logMessage = logMessage + " of " + errorKey + ". '" + error + "'";
+      logMessage = logMessage + ' of ' + errorKey + ". '" + error + "'";
     }
 
     this.pipeline.log('error', logMessage);
-
   }
 
   /**
@@ -187,7 +185,7 @@ class FlowData {
 
     const elementData = this.data[flowElementDataKey];
     if (typeof elementData === 'undefined') {
-      var message = util.format(errorMessages.noElementData,
+      const message = util.format(errorMessages.noElementData,
         flowElementDataKey, Object.keys(this.data).join(', '));
       throw message;
     }

package/flowElement.js

@@ -89,6 +89,7 @@ class FlowElement {
   /**
    * Function called to check if a FlowElement is ready
    * Used when there are asynchronous initialisation steps
+   *
    * @returns {Promise}
    * */
   ready () {
@@ -148,11 +149,11 @@ class FlowElement {
     return this.properties;
   }
 
-  _log(type, message) {
+  _log (type, message) {
     if (this.pipelines) {
       this.pipelines.forEach(pipeline => {
         pipeline.log(type, message);
-      })
+      });
     }
   }
 }

package/helpers.js

@@ -21,22 +21,24 @@
  * ********************************************************************* */
 
 class Helpers {
-    
-	/**
-	 * Set response headers in the response object (e.g. Accept-CH)
-     * @param response: The response to set the headers in.
-	 * @param flowData: A processed FlowData instance to get the response header values
-	 * from.
-	 */
-	static setResponseHeaders(response, flowData) {
-        for (const [key, value] of Object.entries(flowData['set-headers']['responseheadersdictionary'])) {
-			if (response.hasHeader(key)) {
-				response.setHeader(key, `${response.getHeader(key)},${value}`);
-			} else {
-            	response.setHeader(key, value);
-			}
-        }
+  /**
+   * Set response headers in the response object (e.g. Accept-CH)
+   *
+   * @param response: The response to set the headers in.
+   * @param flowData: A processed FlowData instance to get the response header values
+   * from.
+   * @param response
+   * @param flowData
+   */
+  static setResponseHeaders (response, flowData) {
+    for (const [key, value] of Object.entries(flowData['set-headers'].responseheadersdictionary)) {
+      if (response.hasHeader(key)) {
+        response.setHeader(key, `${response.getHeader(key)},${value}`);
+      } else {
+        response.setHeader(key, value);
+      }
     }
+  }
 }
 
-module.exports = Helpers;
+module.exports = Helpers;

package/javascriptbuilder.js

@@ -32,7 +32,7 @@ const template = fs.readFileSync(
 const FlowElement = require('./flowElement.js');
 const EvidenceKeyFilter = require('./evidenceKeyFilter.js');
 const ElementDataDictionary = require('./elementDataDictionary.js');
-var uglifyJS = require('uglify-js');
+const uglifyJS = require('uglify-js');
 
 /**
  * An instance of EvidenceKeyFilter which removes all but header
@@ -86,12 +86,12 @@ class JavaScriptBuilderElement extends FlowElement {
     super(...arguments);
 
     this.settings = {
-      objName: objName,
-      protocol: protocol,
-      host: host,
-      endPoint: endPoint,
-      enableCookies: enableCookies,
-      minify: minify
+      objName,
+      protocol,
+      host,
+      endPoint,
+      enableCookies,
+      minify
     };
 
     this.dataKey = 'javascriptbuilder';

package/jsonbundler.js

@@ -24,8 +24,7 @@ const FlowElement = require('./flowElement.js');
 const ElementDataDictionary = require('./elementDataDictionary.js');
 const BasicListEvidenceKeyFilter = require('./basicListEvidenceKeyFilter.js');
 
-const elementBlacklist = [ 'jsonbundler', 'javascriptbuilder', 'sequence', 'set-headers'];
-
+const elementBlacklist = ['jsonbundler', 'javascriptbuilder', 'sequence', 'set-headers'];
 
 /**
  * The JSONBundler aggregates all properties from FlowElements
@@ -121,7 +120,7 @@ class JSONBundlerElement extends FlowElement {
         }
 
         this.propertyCache[flowElement.dataKey] = {
-          delayExecutionList: delayExecutionList,
+          delayExecutionList,
           evidenceProperties: delayedEvidenceProperties
         };
       }
@@ -182,7 +181,7 @@ class JSONBundlerElement extends FlowElement {
           !flowData.evidence.get('query.sequence') ||
           flowData.evidence.get('query.sequence') < 10
         ) {
-          var type =
+          const type =
             propertyObject[
               Object.keys(propertyObject).find(
                 (key) => key.toLowerCase() === 'type'

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fiftyone.pipeline.core",
-  "version": "4.4.7",
+  "version": "4.4.9",
   "description": "Core library for the 51Degrees Pipeline API",
   "directories": {
     "test": "tests"
@@ -12,21 +12,21 @@
   },
   "author": "51Degrees <support@51Degrees.com>",
   "dependencies": {
-    "mustache": "^4.0.1",
-    "uglify-js": "^3.8.1"
+    "mustache": "^4.2.0",
+    "uglify-js": "^3.17.4"
   },
   "devDependencies": {
-    "@types/node": "^15.12.2",
+    "@types/node": "^15.14.9",
     "eslint": "^6.8.0",
     "eslint-config-standard": "^14.1.1",
-    "eslint-plugin-import": "^2.20.2",
-    "eslint-plugin-jsdoc": "^25.4.0",
+    "eslint-plugin-import": "^2.27.5",
+    "eslint-plugin-jsdoc": "^25.4.3",
     "eslint-plugin-node": "^11.1.0",
-    "eslint-plugin-promise": "^4.2.1",
-    "eslint-plugin-standard": "^4.0.1",
-    "jest": "^27.0.6",
-    "jest-each": "^27.0.6",
-    "node-mocks-http": "^1.10.1"
+    "eslint-plugin-promise": "^4.3.1",
+    "eslint-plugin-standard": "^4.1.0",
+    "jest": "^27.5.1",
+    "jest-each": "^27.5.1",
+    "node-mocks-http": "^1.12.2"
   },
   "contributors": [
     "Filip Hnízdo <filip@octophin.com> (https://octophindigital.com/)",

package/pipeline.js

@@ -38,8 +38,9 @@ class Pipeline {
    *
    * @param {FlowElement[]} flowElements list of FlowElements to
    * add to the Pipeline
-   * @param {Boolean} suppressProcessExceptions If true then pipeline 
+   * @param {boolean} suppressProcessExceptions If true then pipeline
    * will suppress exceptions added to FlowData.
+   * @param dataFileUpdateService
    */
   constructor (flowElements = [], suppressProcessExceptions = false, dataFileUpdateService = null) {
     const pipeline = this;
@@ -137,10 +138,10 @@ class Pipeline {
               })
               .catch(setError);
 
-            if (flowData.errors !== undefined && Object.entries(flowData.errors).length !== 0  && !pipeline.suppressProcessExceptions) {              
-              throw Object.values(flowData.errors)[0];    
+            if (flowData.errors !== undefined && Object.entries(flowData.errors).length !== 0 && !pipeline.suppressProcessExceptions) {
+              throw Object.values(flowData.errors)[0];
             }
-          });         
+          });
         };
       };
 
@@ -217,8 +218,8 @@ class Pipeline {
   }
 
   /**
-   * 
-   * @param {FlowElement} flowElement 
+   *
+   * @param {FlowElement} flowElement
    * @returns {void}
    */
   updatePropertyDataBaseForElement (flowElement) {
@@ -267,7 +268,7 @@ class Pipeline {
             }
 
             pipeline.propertyDatabase[metaKey][metaValue][propertyKey] = {
-              propertyName: propertyName,
+              propertyName,
               flowElementKey: flowElementDataKey
             };
           });

package/pipelineBuilder.js

@@ -61,7 +61,7 @@ class PipelineBuilder {
 
     if (settings.dataFileUpdateService) {
       this.dataFileUpdateService = settings.dataFileUpdateService;
-      }
+    }
 
     if (typeof settings.addJavaScriptBuilder !== 'undefined') {
       this.addJavaScriptBuilder = settings.addJavaScriptBuilder;
@@ -131,7 +131,7 @@ class PipelineBuilder {
     return new Pipeline(flowElements, false, this.dataFileUpdateService);
   }
 
-  addRequiredElements(flowElements) {
+  addRequiredElements (flowElements) {
     return flowElements
       .concat(this.getJavaScriptElements())
       .concat(this.getHttpElements());
@@ -178,7 +178,7 @@ class PipelineBuilder {
    * @returns {FlowElement[]} list of HTTP related
    * FlowElements
    */
-   getHttpElements () {
+  getHttpElements () {
     const flowElements = [];
 
     if (this.useSetHeaderProperties) {