fixed karma and jshint yargs checks

Author: benholloway

lib/test/karma.js

@@ -128,8 +128,8 @@ function karmaCreateConfig(reporters, configFileName) {
     else {
       //non-javascript files, such as source maps, should be included but not tested
       files.push({
-        pattern: file.path,
-        included: false,
+        pattern : file.path,
+        included: false
       });
     }
     transformDone();
@@ -164,8 +164,8 @@ function karmaCreateConfig(reporters, configFileName) {
 
     function specTransformFn(file, encoding, specFileDone) {
       files.push({
-        pattern: file.path,
-        included: false,
+        pattern : file.path,
+        included: false
       });
       specFileDone();
     }
@@ -219,8 +219,8 @@ function karmaRun(reporters, bannerWidth) {
         //NOTE this workaround is necessary, see issue:
         //https://github.com/sindresorhus/supports-color/issues/13
         //TODO @bguiz remove workaround when issue has been resolved
-        SUPPORTS_COLOR: true,
-      }),
+        SUPPORTS_COLOR: true
+      })
     });
     karmaBackground.on('close', function(exitCode) {
       done();
@@ -229,22 +229,22 @@ function karmaRun(reporters, bannerWidth) {
   });
 };
 
-var yargsOptionDefiniton = {
+var yargsOptionDefinition = {
   key: 'karma-reporter',
   value: {
-    describe: 'Specify a custom Karma reporter to use. ' +
-      'Either a locally npm installed module, or an asolute path to one.',
-    alias: ['k'],
-    default: defaultReporterName,
-    string:true,
+    describe: 'Specify a custom Karma reporter to use. Either a locally npm installed module, or an asolute path to ' +
+    'one.',
+    alias   : ['k'],
+    default : defaultReporterName,
+    string  : true
   }
 };
 var checkKarmaReporter = yargs.createCheck()
   .withGate(function (argv) {
     return !argv.help;
   })
   .withTest({
-    karmareporter: function(value) {
+    'karma-reporter': function(value) {
       if (typeof value !== 'undefined') {
         try {
           getKarmaReporterPluginPath(value);
@@ -253,13 +253,13 @@ var checkKarmaReporter = yargs.createCheck()
           return 'Illegal value for "reporter"\n' + ex;
         }
       }
-    },
+    }
   })
   .commit();
 
 module.exports = {
   createConfig: karmaCreateConfig,
-  run: karmaRun,
-  yargsCheck: checkKarmaReporter,
-  yargsOption: yargsOptionDefiniton,
+  run         : karmaRun,
+  yargsCheck  : checkKarmaReporter,
+  yargsOption : yargsOptionDefinition
 };

lib/util/jshint-reporter.js

@@ -92,7 +92,7 @@ var checkJsHintReporter = yargs.createCheck()
     return !argv.help;
   })
   .withTest({
-    reporter: function(value) {
+    'jshint-reporter': function(value) {
       if (typeof value === 'undefined') {
         return;
       }