fix timeout/slow string values and duplicate arguments

lib/cli/run-option-metadata.js

@@ -44,8 +44,8 @@ exports.types = {
     'sort',
     'watch'
   ],
-  number: ['retries', 'slow', 'timeout'],
-  string: ['fgrep', 'grep', 'package', 'reporter', 'ui']
+  number: ['retries'],
+  string: ['fgrep', 'grep', 'package', 'reporter', 'ui', 'slow', 'timeout']
 };
 
 /**

lib/cli/run.js

@@ -258,6 +258,16 @@ exports.builder = yargs =>
         }
       });
 
+      types.boolean.concat(types.string, types.number).forEach(opt => {
+        if (Array.isArray(argv[opt])) {
+          throw createInvalidArgumentValueError(
+            `Argument "${opt}" can only be provided once`,
+            opt,
+            argv[opt]
+          );
+        }
+      });
+
       // yargs.implies() isn't flexible enough to handle this
       if (argv.invert && !('fgrep' in argv || 'grep' in argv)) {
         throw createMissingArgumentError(

test/integration/fixtures/options/slow-test.fixture.js

@@ -0,0 +1,11 @@
+'use strict';
+
+describe('a suite', function() {
+  it('should succeed, slowly', function(done) {
+    setTimeout(done, 500);
+  });
+
+  it('should succeed, very slowly', function(done) {
+    setTimeout(done, 1500);
+  });
+});

test/integration/invalid-arguments.spec.js

@@ -3,18 +3,37 @@
 var invokeMocha = require('./helpers').invokeMocha;
 
 describe('invalid arguments', function() {
-  it('should exit with failure if arguments are invalid', function(done) {
-    invokeMocha(
-      ['--ui'],
-      function(err, result) {
-        if (err) {
-          return done(err);
-        }
-        expect(result, 'to have failed');
-        expect(result.output, 'to match', /not enough arguments/i);
-        done();
-      },
-      {stdio: 'pipe'}
-    );
+  describe('when argument is missing required value', function() {
+    it('should exit with failure', function(done) {
+      invokeMocha(
+        ['--ui'],
+        function(err, result) {
+          if (err) {
+            return done(err);
+          }
+          expect(result, 'to have failed');
+          expect(result.output, 'to match', /not enough arguments/i);
+          done();
+        },
+        {stdio: 'pipe'}
+      );
+    });
+  });
+
+  describe('when argument is provided twice', function() {
+    it('should exit with failure', function(done) {
+      invokeMocha(
+        ['--timeout', '600', '--timeout', '600'],
+        function(err, result) {
+          if (err) {
+            return done(err);
+          }
+          expect(result, 'to have failed');
+          expect(result.output, 'to match', /can only be provided once/i);
+          done();
+        },
+        {stdio: 'pipe'}
+      );
+    });
   });
 });

test/integration/options/timeout.spec.js

@@ -0,0 +1,35 @@
+'use strict';
+
+var helpers = require('../helpers');
+var runMochaJSON = helpers.runMochaJSON;
+
+describe('--timeout', function() {
+  it('should allow human-readable string value', function(done) {
+    runMochaJSON('options/slow-test', ['--timeout', '1s'], function(err, res) {
+      if (err) {
+        done(err);
+        return;
+      }
+      expect(res, 'to have failed')
+        .and('to have passed test count', 1)
+        .and('to have failed test count', 1);
+      done();
+    });
+  });
+
+  it('should allow numeric value', function(done) {
+    runMochaJSON('options/slow-test', ['--timeout', '1000'], function(
+      err,
+      res
+    ) {
+      if (err) {
+        done(err);
+        return;
+      }
+      expect(res, 'to have failed')
+        .and('to have passed test count', 1)
+        .and('to have failed test count', 1);
+      done();
+    });
+  });
+});

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

@@ -624,7 +624,7 @@ describe('options', function() {
           findupSync
         });
 
-        expect(loadOptions(), 'to satisfy', {timeout: 800, require: ['foo']});
+        expect(loadOptions(), 'to satisfy', {timeout: '800', require: ['foo']});
       });
 
       it('should prioritize package.json over mocha.opts', function() {
@@ -687,7 +687,7 @@ describe('options', function() {
           loadOptions('--timeout 500'),
           'to have property',
           'timeout',
-          500
+          '500'
         );
       });
     });