Merge remote-tracking branch 'ldabiralai/master'

Author: vedharish

lib/Local.js

@@ -19,7 +19,7 @@ function Local(){
 
   this.start = function(options, callback){
     if(typeof callback !== 'function')
-      callback = function(){};
+      callback = function() {};
 
     this.userArgs = [];
     var that = this;
@@ -108,7 +108,7 @@ function Local(){
 
   this.stop = function (callback) {
     if(typeof callback !== 'function')
-      callback = function(){};
+      callback = function() {};
 
     if(!this.pid) return callback();
     this.opcode = 'stop';

test/local.js

@@ -182,9 +182,9 @@ describe('Local', function () {
   });
 
   it('should set proxy', function (done) {
-    bsLocal.start({ 
-      'key': process.env.BROWSERSTACK_ACCESS_KEY, 
-      onlyCommand: true, 
+    bsLocal.start({
+      'key': process.env.BROWSERSTACK_ACCESS_KEY,
+      onlyCommand: true,
       'proxyHost': 'localhost',
       'proxyPort': 8080,
       'proxyUser': 'user',
@@ -208,7 +208,15 @@ describe('Local', function () {
       expect(bsLocal.getBinaryArgs().indexOf('localhost,8000,0')).to.not.equal(-1);
       done();
     });
-  });
+  })
+
+  it('does not error if no callback in stop', function (done) {
+    this.timeout(60000);
+    bsLocal.start({ 'key': process.env.BROWSERSTACK_ACCESS_KEY }, function(){
+      bsLocal.stop();
+      done();
+    });
+  })
 
   afterEach(function (done) {
     this.timeout(60000);