lint errors

Author: vedharish

bin/runner.js

@@ -299,8 +299,6 @@ var statusPoller = {
               if (!worker.isAckd) {
                 logger.trace('[%s] worker.activityTimeout', worker.id);
 
-                var subject = 'Worker inactive for too long: ' + worker.string;
-                var content = 'Worker details:\n' + JSON.stringify(worker.config, null, 4);
                 delete workers[workerData.key];
                 delete workerKeys[worker.id];
                 config.status += 1;
@@ -327,8 +325,6 @@ var statusPoller = {
               if (worker.isAckd) {
                 logger.trace('[%s] worker.testActivityTimeout', worker.id);
 
-                var subject = 'Tests timed out on: ' + worker.string;
-                var content = 'Worker details:\n' + JSON.stringify(worker.config, null, 4);
                 delete workers[workerData.key];
                 delete workerKeys[worker.id];
                 config.status += 1;
@@ -422,5 +418,5 @@ exports.test = function(config_file, callback) {
   } catch (e) {
     callback(e);
   }
-}
+};
 exports.cleanUpAndExit = cleanUpAndExit;

lib/config.js

@@ -46,14 +46,17 @@ exports.config = function(config_path) {
 
   if (process.env.BROWSERSTACK_KEY) {
     this.key = process.env.BROWSERSTACK_KEY;
+    delete config.key;
   }
 
   if (process.env.BROWSERSTACK_ACCESS_KEY) {
     this.key = process.env.BROWSERSTACK_ACCESS_KEY;
+    delete config.key;
   }
 
   if (process.env.BROWSERSTACK_USERNAME) {
     this.username = process.env.BROWSERSTACK_USERNAME;
+    delete config.username;
   }
 
   if (!config.project) {
@@ -72,8 +75,9 @@ exports.config = function(config_path) {
     this.build = commit_id ? 'Commit-' + commit_id.slice(0, commit_id.length / 2) : 'Local run, ' + new Date().toISOString();
   }
 
+  var that = this;
   ['username', 'key', 'browsers', 'test_path'].forEach(function(param) {
-    if (typeof config[param] === 'undefined' && typeof this[param] === 'undefined') {
+    if (typeof config[param] === 'undefined' && typeof that[param] === 'undefined') {
       console.error('Configuration parameter `%s` is required.', param);
       process.exit(1);
     }
@@ -100,4 +104,4 @@ exports.config = function(config_path) {
   for (var key in config) {
     this[key] = config[key];
   }
-}
+};

lib/local.js

@@ -4,8 +4,7 @@ var Log = require('./logger'),
   fs = require('fs'),
   https = require('https'),
   windows = ((process.platform.match(/win32/) || process.platform.match(/win64/)) !== null),
-  localBinary = __dirname + '/BrowserStackLocal' + (windows ? '.exe' : ''),
-  utils = require('./utils');
+  localBinary = __dirname + '/BrowserStackLocal' + (windows ? '.exe' : '');
 
 var Tunnel = function Tunnel(key, port, uniqueIdentifier, config, callback) {
   var that = {};

lib/utils.js

@@ -1,10 +1,3 @@
-var Log = require('./logger'),
-    logger = new Log(global.logLevel),
-    http = require('http'),
-    url = require('url'),
-    querystring = require('querystring'),
-    config;
-
 String.prototype.escapeSpecialChars = function() {
   return this.replace(/\n/g, '\\n')
   .replace(/\r/g, '\\r')