fiftyone.pipeline.cloudrequestengine 4.3.3 → 4.3.4

package/cloudEngine.js

@@ -99,29 +99,30 @@ class CloudEngine extends Engine {
     const engine = this;
 
     return engine.ready().then(function () {
+
       let cloudData = flowData.get('cloud').get('cloud');
 
       cloudData = JSON.parse(cloudData);
+      const result = {};
 
-      if (cloudData.device && cloudData.device[engine.dataKey] === null) {
-        flowData.pipeline.log('warn', engine.dataKey + " not populated. " +
-        cloudData.device[engine.dataKey + 'nullreason'] !== null ? cloudData.device[engine.dataKey + 'nullreason'] : "" +
+      if (cloudData && cloudData[engine.dataKey] === null) {
+        flowData.pipeline.log('error', engine.dataKey + " not populated. " +
+        cloudData[engine.dataKey + 'nullreason'] !== null ? cloudData[engine.dataKey + 'nullreason'] : "" +
         "\n" + "This may be because the provided API key is not authorised for " + engine.dataKey + " queries.");
-      }
+      } 
+      else {
 
       // Loop over cloudData.device properties to check if they have a value
-
-      const result = {};
-
-      Object.entries(cloudData[engine.dataKey]).forEach(function ([key, value]) {
-        result[key] = new AspectPropertyValue();
-
-        if (cloudData[engine.dataKey][key + 'nullreason']) {
-          result[key].noValueMessage = cloudData[engine.dataKey][key + 'nullreason'];
-        } else {
-          result[key].value = value;
-        }
-      });
+        Object.entries(cloudData[engine.dataKey]).forEach(function ([key, value]) {
+          result[key] = new AspectPropertyValue();
+  
+          if (cloudData[engine.dataKey][key + 'nullreason']) {
+            result[key].noValueMessage = cloudData[engine.dataKey][key + 'nullreason'];
+          } else {
+            result[key].value = value;
+          }
+        });
+      }
 
       const data = new AspectDataDictionary(
         {

package/cloudRequestError.js

@@ -24,6 +24,7 @@ class CloudRequestError extends Error {
   constructor(message, responseHeaders, httpStatusCode){
       super(message);
       this.name = this.constructor.name
+	  this.errorMessage = message;
       this.responseHeaders = responseHeaders;
       this.httpStatusCode = httpStatusCode;
       // This clips the constructor invocation from the stack trace.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fiftyone.pipeline.cloudrequestengine",
-  "version": "4.3.3",
+  "version": "4.3.4",
   "description": "Cloud request engine for the 51Degrees Pipeline API",
   "main": "index.js",
   "directories": {
@@ -20,8 +20,8 @@
     "jest": "^24.9.0"
   },
   "dependencies": {
-    "fiftyone.pipeline.core": "^4.3.3",
-    "fiftyone.pipeline.engines": "^4.3.3"
+    "fiftyone.pipeline.core": "^4.3.4",
+    "fiftyone.pipeline.engines": "^4.3.4"
   },
   "contributors": [
     "Filip Hnízdo <filip@octophin.com> (https://octophindigital.com/)",

package/tests/cloudEngine.test.js

@@ -59,15 +59,12 @@ const flowData = pipeline.createFlowData();
 // Check that if an invalid resource key is used, a correct error
 // message will be returned and stored in flowData.errors
 test('missing property service - invalid resource key', done => {
+  pipeline.suppressProcessExceptions = true;
   flowData.process().then(function () {
     var errorFound = false;
-    flowData.errors.CloudEngineBase.forEach(function (errors) {
-      errors.forEach(function (error) {
-        errorFound = errorFound ||
-        (error.message.includes(testResourceKey) &&
-        error.message.includes('not a valid resource key'));
-      });
-    });
+    error = flowData.errors.CloudEngineBase[0];
+    errorFound = error.message.includes(testResourceKey) &&
+    error.message.includes('not a valid resource key');
     expect(errorFound).toBe(true);
 
     done();

package/tests/cloudRequestEngine.test.js

@@ -81,7 +81,6 @@ test('custom end point - default value', done => {
 // Check that values from the http response are populated in the 
 // error object.
 test('HTTP data set in error', done => {
-  let errorThrown = false;
 
   let engine = new CloudRequestEngine({
     resourceKey: testResourceKey
@@ -90,24 +89,22 @@ test('HTTP data set in error', done => {
   const builder = new PipelineBuilder();
   const pipeline = builder.add(engine).build();
   const flowData = pipeline.createFlowData();
-  
-  // When an error occurs, check that the expected values are populated.
+  let expectedError = '\'' + testResourceKey + '\' is not a valid resource key.';      
+ 
+  // When an error occurs, check that the message is logged.
   pipeline.on('error', (e) => {
-    expect(e).toBeDefined();
-    expect(e.message).toBeDefined();
-    let expectedError = '\'' + testResourceKey + '\' is not a valid resource key.';
-    expect(e.message[0].message).toEqual(expectedError); 
-    expect(e.message[0].httpStatusCode).toEqual(400); 
-    expect(e.message[0].responseHeaders.etag).toBeDefined();
-    errorThrown = true;
+    expect(e).toBeDefined();  
+    done();
   });
 
-  // Make sure that the expected error was thrown.
   flowData.process().then(function () {
-    expect(errorThrown).toBe(true);
     done();
+  }).catch((e) => {
+    // When an error occurs, check that the expected values are populated.
+    expect(e[0].errorMessage.indexOf(expectedError) !== -1).toBe(true);     
+    expect(e[0].httpStatusCode).toEqual(400); 
+    expect(e[0].responseHeaders.etag).toBeDefined(); 
   });
-
 });
 
 /**