treat negative timeouts as infinite

when a negative timeout is supplied, CT should not set a new timeout and instead keep the content up indefinitely. this commit also adds some utilities and documentation to improve local testrunning.

Makefile

@@ -0,0 +1,5 @@
+test:
+	./node_modules/.bin/mocha
+
+
+.PHONY: test

README.md

@@ -62,3 +62,9 @@ The main file is `crimsontwins.js`. Run that like
 ```
 
 You should now have a running crimsontwins instance.
+
+
+Development
+===========
+
+Make a feature branch and hack away. When you're ready, open a pull request. Use `make test` to run tests locally.

app/manager.js

@@ -87,6 +87,7 @@ ScreenManager.prototype.sendUrl = function(url, screenName) {
       }
       screen.content = content;
       p.resolve(content);
+      self.makeTimeout(screen.name);
       self.emit('screenChanged', screen);
     },
     function fail(error) {
@@ -108,6 +109,7 @@ ScreenManager.prototype.onContentLoad = function(first_argument) {
       screen.content = self.contentManager.next();
       self.emit('screenChanged', screen);
 
+      // ensure all screens don't change at once
       self.makeTimeout(screen.name, skew);
       skew += skewIncr;
     }
@@ -123,13 +125,17 @@ ScreenManager.prototype.cycleScreen = function(name) {
 
 ScreenManager.prototype.makeTimeout = function(name, time) {
   var oldTimeout = this.timeouts[name];
+  if (arguments.length < 2) {
+    time = config.resetTime;
+  }
+
   if (oldTimeout) {
     clearTimeout(oldTimeout);
   }
-  if (time === undefined) {
-    time = config.resetTime;
+
+  if (time > -1) {
+    this.timeouts[name] = setTimeout(this.cycleScreen.bind(this, name), time);
   }
-  this.timeouts[name] = setTimeout(this.cycleScreen.bind(this, name), time);
 };
 
 ScreenManager.prototype.reset = function(screenName) {

test/manager.js

@@ -196,6 +196,16 @@ describe('ScreenManager', function() {
       clock.tick(60);
       assert.notEqual(screen.content.url, before);
     });
+
+    it('should not timeout when passed a negative time', function() {
+      var screen = screenMan.screens[0];
+      var before = screen.content.url;
+      screenMan.makeTimeout(screen.name, -1);
+      clock.tick(mockConfig.resetTime);
+      assert.equal(screen.content.url, before);
+      clock.tick(mockConfig.resetTime + 1); // induction step
+      assert.equal(screen.content.url, before);
+    })
   });
 
   describe('#reset', function() {
@@ -384,4 +394,4 @@ describe('ContentManager', function() {
       cm.load();
     });
   });
-});
+});