Add minimist to the project and update the test (#65)

Author: alexcanessa

package.json

@@ -40,8 +40,9 @@
     "connectivity": "^1.0.0",
     "es6-promise": "^3.2.1",
     "github-api": "^3.0.0",
-    "require-yaml": "0.0.1",
-    "object-assign-deep": "0.0.4"
+    "minimist": "^1.2.0",
+    "object-assign-deep": "0.0.4",
+    "require-yaml": "0.0.1"
   },
   "devDependencies": {
     "eslint": "^3.6.0",

src/utils.js

@@ -2,6 +2,7 @@
 
 var chalk = require('chalk');
 var fs = require('fs');
+var minimist = require('minimist');
 require('require-yaml');
 
 /**
@@ -130,17 +131,7 @@ function dashToCamelCase(value) {
 * @return {Object}     The object containg the key/value options
 */
 function getBashOptions(args) {
-    var settings = {};
-
-    for (var i = 2; i < args.length; i++) {
-        var paramArray = args[i].split('=');
-        var key = paramArray[0].replace('--', '');
-        var value = paramArray[1];
-
-        settings[dashToCamelCase(key)] = value || true;
-    }
-
-    return settings;
+    return minimist(args.slice(2));
 }
 
 /**

test/utils.js

@@ -3,41 +3,42 @@
 var utils = require('../src/utils');
 
 exports['utils'] = {
-   'Should return the string of the formatted date': function (test) {
-      test.expect(1);
-
-      test.deepEqual(utils.formatDate(new Date(0)), '01/01/1970', 'Given a date object.');
-      test.done();
-   },
-   'Should return the options in a key/value format': function (test) {
-      test.expect(1);
-
-      let bashOptions = utils.getBashOptions([null, null, '--key=value', '--key2=value2']);
-
-      test.deepEqual(JSON.stringify(bashOptions), JSON.stringify({
-         key: 'value',
-         key2: 'value2'
-      }), 'Given an array of node arguments.');
-      test.done();
-   },
-   'Should return a camelCase string': function (test) {
-      test.expect(2);
-
-      test.deepEqual(utils.dashToCamelCase('this-is-a-string'), 'thisIsAString', 'Given a string with dashes.');
-      test.deepEqual(utils.dashToCamelCase('tHIs-Is-a-sTriNg'), 'thisIsAString', 'Given a string with random capital letters');
-      test.done();
-   },
-   'Should return if a number is in between a range': function (test) {
-      test.expect(7);
-
-      test.deepEqual(utils.isInRange(2, 1, 3), true, 'Given a number in range');
-      test.deepEqual(utils.isInRange(1, 2, 3), false, 'Given a number below range');
-      test.deepEqual(utils.isInRange(4, 1, 3), false, 'Given a number above range');
-      test.deepEqual(utils.isInRange(-1, 1, 3), false, 'Given a number above range, negative');
-      test.deepEqual(utils.isInRange(-1, -3, 0), true, 'Given a number in range, negative');
-      test.deepEqual(utils.isInRange(2, 2, 5), true, 'Given same number as first range value');
-      test.deepEqual(utils.isInRange(5, 2, 5), false, 'Given same number as last range value');
-
-      test.done();
-   }
+    'Should return the string of the formatted date': function (test) {
+        test.expect(1);
+
+        test.deepEqual(utils.formatDate(new Date(0)), '01/01/1970', 'Given a date object.');
+        test.done();
+    },
+    'Should return the options in a key/value format': function (test) {
+        test.expect(1);
+
+        let bashOptions = utils.getBashOptions([null, null, '--key=value', '--key2=value2']);
+
+        test.deepEqual(JSON.stringify(bashOptions), JSON.stringify({
+            _: [],
+            key: 'value',
+            key2: 'value2'
+        }), 'Given an array of node arguments.');
+        test.done();
+    },
+    'Should return a camelCase string': function (test) {
+        test.expect(2);
+
+        test.deepEqual(utils.dashToCamelCase('this-is-a-string'), 'thisIsAString', 'Given a string with dashes.');
+        test.deepEqual(utils.dashToCamelCase('tHIs-Is-a-sTriNg'), 'thisIsAString', 'Given a string with random capital letters');
+        test.done();
+    },
+    'Should return if a number is in between a range': function (test) {
+        test.expect(7);
+
+        test.deepEqual(utils.isInRange(2, 1, 3), true, 'Given a number in range');
+        test.deepEqual(utils.isInRange(1, 2, 3), false, 'Given a number below range');
+        test.deepEqual(utils.isInRange(4, 1, 3), false, 'Given a number above range');
+        test.deepEqual(utils.isInRange(-1, 1, 3), false, 'Given a number above range, negative');
+        test.deepEqual(utils.isInRange(-1, -3, 0), true, 'Given a number in range, negative');
+        test.deepEqual(utils.isInRange(2, 2, 5), true, 'Given same number as first range value');
+        test.deepEqual(utils.isInRange(5, 2, 5), false, 'Given same number as last range value');
+
+        test.done();
+    }
 };