Handle exit codes better and reject promise if so.

Author: paulrutter

CHANGELOG.md

@@ -1,3 +1,6 @@
+06-10-2017 - Paul Rütter
+- 1.0.9 - Handle exit codes better and reject promise if so.
+
 05-10-2017 - Paul Rütter
 - 1.0.8 - Add CPU profile functionality.
 

lib/cpuProfileWorker.js

@@ -9,8 +9,8 @@ console.error('Started CPU profile (duration: %sms) %s worker on DevTools port \
 
 let CDP = require('chrome-remote-interface');
 let writeStream = fs.createWriteStream(path);
-let handleError = function () {
-    console.error(arguments);
+let handleError = function (arg1, arg2) {
+    console.error("Error occurred while creating CPU profile", arg1, arg2 || "");
     writeStream.end();
     process.exit(-1);
 };

lib/heapdumpWorker.js

@@ -9,14 +9,15 @@ console.error('Started heapdump %sworker for \'%s\' on DevTools port \'%s\'.', l
 
 let CDP = require('chrome-remote-interface');
 let writeStream = fs.createWriteStream(path);
-let handleError = function () {
-    console.error(arguments);
+let handleError = function (arg1, arg2) {
+    console.error("Error occurred while creating heapdump", arg1, arg2 || "");
     writeStream.end();
     process.exit(-1);
 };
 writeStream.on('error', (err) => {
     handleError("Heapdump path not valid or writable", err);
 });
+
 CDP({
     host: 'localhost',
     port: devToolsPort,

lib/index.js

@@ -76,9 +76,13 @@ class NodeOomHeapDumpImpl {
         error = err;
         reject(err);
       });
-      child.on('exit', () => {
+      child.on('exit', (code) => {
         if (!error) {
-          resolve();
+          if (code === 0) {
+            resolve();
+          } else {
+            reject(new Error("Worker exited with statusCode: " + code));
+          }
         }
         this._busy = false;
       });

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "node-oom-heapdump",
-  "version": "1.0.8",
+  "version": "1.0.9",
   "description": "Create a V8 heap snapshot right before an \"Out of Memory\" error occurs, or create a heap snapshot on request.",
   "main": "index.js",
   "scripts": {