Throw an exception when timeout too large.

closes mochajs#1644

Author: callumacrae

lib/runnable.js

@@ -65,6 +65,7 @@ Runnable.prototype.__proto__ = EventEmitter.prototype;
 Runnable.prototype.timeout = function(ms){
   if (0 == arguments.length) return this._timeout;
   if (ms === 0) this._enableTimeouts = false;
+  if (ms > Math.pow(2, 31)) throw new RangeError('Interval too large, must be less than 2^31');
   if ('string' == typeof ms) ms = milliseconds(ms);
   debug('timeout %d', ms);
   this._timeout = ms;

test/runnable.js

@@ -40,6 +40,13 @@ describe('Runnable(title, fn)', function(){
     })
   })
 
+  describe('#timeout(ms) when ms>2^31', function(){
+    it('should throw an error', function () {
+      var run = new Runnable;
+      run.timeout.bind(run, 1e10).should.throw(/Interval too large/);
+    });
+  });
+
   describe('#enableTimeouts(enabled)', function(){
     it('should set enabled', function(){
       var run = new Runnable;