fiftyone.pipeline.core 4.3.3 → 4.3.4

package/flowData.js

@@ -140,13 +140,16 @@ class FlowData {
       errorKey = 'other';
     }
 
-    if (!this.errors[errorKey]) {
-      this.errors[errorKey] = [];
+    this.errors[errorKey] = error;
+
+    let logMessage = "Error occurred during processing";
+
+    if (errorKey !== undefined) {
+        logMessage = logMessage + " of " + errorKey + ". '" + error + "'";
     }
 
-    this.pipeline.log('error', { message: error, source: errorKey });
+    this.pipeline.log('error', logMessage);
 
-    this.errors[errorKey].push(error);
   }
 
   /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fiftyone.pipeline.core",
-  "version": "4.3.3",
+  "version": "4.3.4",
   "description": "Core library for the 51Degrees Pipeline API",
   "directories": {
     "test": "tests"

package/pipeline.js

@@ -38,13 +38,18 @@ class Pipeline {
    *
    * @param {FlowElement[]} flowElements list of FlowElements to
    * add to the Pipeline
+   * @param {Boolean} suppressProcessExceptions If true then pipeline 
+   * will suppress exceptions added to FlowData.
    */
-  constructor (flowElements = []) {
+  constructor (flowElements = [], suppressProcessExceptions = false) {
     const pipeline = this;
 
     // The chain of flowElements to run, including arrays of parallel elements
     this.flowElementsChain = flowElements;
 
+    // If true then pipeline will suppress exceptions added to FlowData.
+    this.suppressProcessExceptions = suppressProcessExceptions;
+
     // A logger for emitting messages
     this.eventEmitter = new EventEmitter();
 
@@ -126,7 +131,11 @@ class Pipeline {
                 resolve(flowData);
               })
               .catch(setError);
-          });
+
+            if (flowData.errors !== undefined && Object.entries(flowData.errors).length !== 0  && !pipeline.suppressProcessExceptions) {              
+              throw Object.values(flowData.errors)[0];    
+            }
+          });         
         };
       };
 

package/tests/logs-errors-and-stops.test.js

@@ -35,8 +35,9 @@ const syncPipeline = new PipelineBuilder()
 
 const syncFlowData = syncPipeline.createFlowData();
 test('error data is populated', done => {
+  syncPipeline.suppressProcessExceptions = true;
   syncFlowData.process().then(function () {
-    expect(syncFlowData.errors.error[0]).toBe('Something went wrong');
+    expect(syncFlowData.errors.error).toBe('Something went wrong');
 
     done();
   });
@@ -45,17 +46,28 @@ test('error data is populated', done => {
 let log;
 
 syncPipeline.on('error', function (error) {
-  log = error.message;
+  console.log(error);
+  log = error;
 });
 
 test('logging', done => {
+  syncPipeline.suppressProcessExceptions = true;
   syncFlowData.process().then(function () {
-    expect(log).toBe('Something went wrong');
+    expect(log).toBe("Error occurred during processing of error. 'Something went wrong'");
 
     done();
   });
 });
 
+test("Don't Suppress Exception", done => {
+  syncFlowData.process().then(function () {
+    done();
+  }).catch((e) => {
+    // When an error occurs, check that the expected values are populated.
+    expect(e.indexOf('Something went wrong') !== -1).toBe(true);
+  });
+});
+
 test('stop flag works', done => {
   syncFlowData.process().then(function () {
     try {