Revert timeout and slow as string flags so that they can accept human…

… readable values

lib/cli/options.js

@@ -124,7 +124,7 @@ const createErrorForNumericPositionalArg = (
     return (
       isMochaFlag(arg) &&
       allArgs[index + 1] === String(numericArg) &&
-      parsedResult[normalizedArg] !== numericArg
+      parsedResult[normalizedArg] !== String(numericArg)
     );
   });
 
@@ -157,7 +157,7 @@ const parse = (args = [], defaultValues = {}, ...configObjects) => {
   //    are ALL boolean flags.
   // 4. we can then reapply the values after yargs-parser is done.
   const allArgs = Array.isArray(args) ? args : args.split(' ');
-  const nodeArgs = allArgs.reduce((acc, arg, index, allArgs) => {
+  const nodeArgs = allArgs.reduce((acc, arg) => {
     const pair = arg.split('=');
     let flag = pair[0];
     if (isNodeFlag(flag, false)) {

lib/cli/run-option-metadata.js

@@ -50,8 +50,17 @@ const TYPES = (exports.types = {
     'sort',
     'watch'
   ],
-  number: ['retries', 'jobs', 'slow', 'timeout'],
-  string: ['config', 'fgrep', 'grep', 'package', 'reporter', 'ui']
+  number: ['retries', 'jobs'],
+  string: [
+    'config',
+    'fgrep',
+    'grep',
+    'package',
+    'reporter',
+    'ui',
+    'slow',
+    'timeout'
+  ]
 });
 
 /**

test/node-unit/cli/options.spec.js

@@ -527,7 +527,7 @@ describe('options', function () {
           loadOptions('--timeout 500'),
           'to have property',
           'timeout',
-          500
+          '500'
         );
       });