Graceful handling of missing newman.run parameters, akin to usage of …

…CLI.

- work gracefully if no options parameter is sent
- throw error even if there is no callback sent to receive it

lib/run/index.js

@@ -90,7 +90,16 @@ module.exports = function (options, callback) {
         (callback = options),
         (options = {})
     );
-    !_.isFunction(callback) && (callback = _.noop);
+
+    // if a callback is not provided, then attach a callback that throws error
+    !_.isFunction(callback) && (callback = function (err) {
+        if (err) { throw err; } // this throw will likely not work with 'catch' as this will be thrown async
+        // the idea bove is to at least have the error message displayed up on the stack in case there is an error
+        // and not be gobbled up and cause confusion for unsuspecting time users.
+    });
+
+    // sanitise object here to avoid repeated property access of undefined error down the line
+    !_.isObject(options) && (options = {});
 
     var emitter = new EventEmitter(), // @todo: create a new inherited constructor
         runner = new runtime.Runner(),

test/unit/newman-programmatic-use.test.js

@@ -0,0 +1,11 @@
+const expect = require('chai').expect,
+    newman = require('../../');
+
+describe('Newman programmatic usage', function () {
+    it('should run without any meaningful parameter and pass error in callback', function (done) {
+        newman.run({}, function (err) {
+            expect(err).to.be.an('error');
+            done();
+        });
+    });
+});