only pass string values to args

Author: Blackbaud-PatrickOFriel

lib/Local.js

@@ -125,7 +125,7 @@ function Local(){
         if(value.toString() !== 'true')
           this.verboseFlag = value;
         else {
-          this.verboseFlag = 1;
+          this.verboseFlag = '1';
         }
         break;
 
@@ -245,7 +245,7 @@ function Local(){
     }
     if(this.parallelRunsFlag){
       args.push('--parallel-runs');
-      args.push(this.parallelRunsFlag);
+      args.push(this.parallelRunsFlag.toString());
     }
     if(this.onlyHosts) {
       args.push('--only');
@@ -275,7 +275,7 @@ function Local(){
       args.push(this.forceFlag);
     if(this.verboseFlag){
       args.push('--verbose');
-      args.push(this.verboseFlag);
+      args.push(this.verboseFlag.toString());
     }
     for(var i in this.userArgs){
       args.push(this.userArgs[i]);

test/local.js

@@ -52,15 +52,23 @@ describe('Local', function () {
   it('should enable verbose', function (done) {
     bsLocal.start({ 'key': process.env.BROWSERSTACK_ACCESS_KEY, onlyCommand: true, 'verbose': true }, function(){
       expect(bsLocal.getBinaryArgs().indexOf('--verbose')).to.not.equal(-1);
-      expect(bsLocal.getBinaryArgs().indexOf(1)).to.not.equal(-1);
+      expect(bsLocal.getBinaryArgs().indexOf('1')).to.not.equal(-1);
       done();
     });
   });
 
   it('should enable verbose with log level', function (done) {
     bsLocal.start({ 'key': process.env.BROWSERSTACK_ACCESS_KEY, onlyCommand: true, 'verbose': 2 }, function(){
       expect(bsLocal.getBinaryArgs().indexOf('--verbose')).to.not.equal(-1);
-      expect(bsLocal.getBinaryArgs().indexOf(2)).to.not.equal(-1);
+      expect(bsLocal.getBinaryArgs().indexOf('2')).to.not.equal(-1);
+      done();
+    });
+  });
+
+  it('should enable verbose with log level string', function (done) {
+    bsLocal.start({ 'key': process.env.BROWSERSTACK_ACCESS_KEY, onlyCommand: true, 'verbose': '2' }, function(){
+      expect(bsLocal.getBinaryArgs().indexOf('--verbose')).to.not.equal(-1);
+      expect(bsLocal.getBinaryArgs().indexOf('2')).to.not.equal(-1);
       done();
     });
   });
@@ -157,6 +165,22 @@ describe('Local', function () {
     });
   });
 
+  it('should set parallelRuns', function (done) {
+    bsLocal.start({ 'key': process.env.BROWSERSTACK_ACCESS_KEY, onlyCommand: true, 'parallelRuns': '10' }, function(){
+      expect(bsLocal.getBinaryArgs().indexOf('-parallel-runs')).to.not.equal(-1);
+      expect(bsLocal.getBinaryArgs().indexOf('10')).to.not.equal(-1);
+      done();
+    });
+  });
+
+  it('should set parallelRuns with integer value', function (done) {
+    bsLocal.start({ 'key': process.env.BROWSERSTACK_ACCESS_KEY, onlyCommand: true, 'parallelRuns': 10 }, function(){
+      expect(bsLocal.getBinaryArgs().indexOf('-parallel-runs')).to.not.equal(-1);
+      expect(bsLocal.getBinaryArgs().indexOf('10')).to.not.equal(-1);
+      done();
+    });
+  });
+
   it('should set proxy', function (done) {
     bsLocal.start({ 
       'key': process.env.BROWSERSTACK_ACCESS_KEY,