API parity with clearTimeout/clearInterval and much housekeeping

.travis.yml

@@ -4,3 +4,7 @@ node_js:
   - '4'
   - '6'
   - '8'
+script:
+  - npm run cov
+after_script:
+  - codecov

index.js

@@ -114,3 +114,9 @@ exports.setInterval = function (cb, interval) {
 	timer.fireEvery(interval);
 	return timer;
 };
+
+exports.clearTimeout = exports.clearInterval = function (timer) {
+	if (timer && typeof timer.clear === 'function') {
+		timer.clear();
+	}
+};

package.json

@@ -1,13 +1,17 @@
 {
   "name": "safe-timers",
-  "version": "1.0.1",
+  "version": "1.1.0",
   "description": "Timers with near-infinite duration support",
   "main": "index.js",
+  "files": [
+    "index.js"
+  ],
   "directories": {
     "test": "test"
   },
   "scripts": {
-    "test": "tape test/*.js | tap-spec"
+    "test": "tape test/*.js | tap-spec",
+    "cov": "nyc npm test && nyc report --reporter=lcov"
   },
   "repository": {
     "type": "git",
@@ -17,8 +21,10 @@
     "setTimeout",
     "setInterval",
     "timer",
+    "timeout",
     "interval",
     "timers",
+    "timeouts",
     "intervals"
   ],
   "author": "Ron Korving <[email protected]>",
@@ -28,6 +34,8 @@
   },
   "homepage": "https://github.com/Wizcorp/safe-timers#readme",
   "devDependencies": {
+    "codecov": "^3.0.0",
+    "nyc": "^11.2.1",
     "tap-spec": "^4.1.1",
     "tape": "^4.5.1"
   }

test/setInterval.js

@@ -145,5 +145,24 @@ test('setInterval', function (t) {
 		}, 10);
 	});
 
+	t.test('clearInterval', function (t) {
+		const interval = timers.setInterval(function () {
+			t.end('interval fired despite clearInterval()');
+		}, 0);
+
+		timers.clearInterval(interval);
+
+		setTimeout(function () {
+			t.pass('interval did not fire');
+			t.end();
+		}, 10);
+	});
+
+	t.test('clearInterval without timer', function (t) {
+		timers.clearInterval(null);
+		t.pass('no error was thrown');
+		t.end();
+	});
+
 	t.end();
 });

test/setTimeout.js

@@ -63,5 +63,24 @@ test('setTimeout', function (t) {
 		}, 10);
 	});
 
+	t.test('clearTimeout', function (t) {
+		const timeout = timers.setTimeout(function () {
+			t.end('timeout fired despite clearTimeout()');
+		}, 0);
+
+		timers.clearTimeout(timeout);
+
+		setTimeout(function () {
+			t.pass('timeout did not fire');
+			t.end();
+		}, 10);
+	});
+
+	t.test('clearTimeout without timer', function (t) {
+		timers.clearTimeout(null);
+		t.pass('no error was thrown');
+		t.end();
+	});
+
 	t.end();
 });

test/setTimeoutAt.js

@@ -4,64 +4,83 @@ const test = require('tape');
 const timers = require('..');
 
 test('setTimeoutAt', function (t) {
-  const originalMaxInterval = timers.maxInterval;
-
-  t.test('Crossing the maxInterval border', function (t) {
-    timers.maxInterval = 2;
-
-    timers.setTimeoutAt(function () {
-      timers.maxInterval = originalMaxInterval;
-
-      t.pass('timeout fired');
-      t.equal(arguments.length, 0);
-      t.end();
-    }, Date.now() + 30);
-  });
-
-  t.test('Crossing the maxInterval border with args', function (t) {
-    timers.maxInterval = 2;
-
-    timers.setTimeoutAt(function (a, b) {
-      timers.maxInterval = originalMaxInterval;
-
-      t.pass('timeout fired');
-      t.equal(arguments.length, 2);
-      t.equal(a, 1);
-      t.equal(b, 2);
-      t.end();
-    }, Date.now() + 30, 1, 2);
-  });
-
-  t.test('Not crossing the maxInterval border', function (t) {
-    timers.setTimeoutAt(function () {
-      t.pass('timeout fired');
-      t.equal(arguments.length, 0);
-      t.end();
-    }, Date.now() + 5);
-  });
-
-  t.test('Not crossing the maxInterval border with args', function (t) {
-    timers.setTimeoutAt(function (a, b) {
-      t.pass('timeout fired');
-      t.equal(arguments.length, 2);
-      t.equal(a, 1);
-      t.equal(b, 2);
-      t.end();
-    }, Date.now() + 5, 1, 2);
-  });
-
-  t.test('clear()', function (t) {
-    const timeout = timers.setTimeoutAt(function () {
-      t.end('timeout fired despite clear()');
-    }, Date.now() + 0);
-
-    timeout.clear();
-
-    setTimeout(function () {
-      t.pass('timeout did not fire');
-      t.end();
-    }, 10);
-  });
-
-  t.end();
+	const originalMaxInterval = timers.maxInterval;
+
+	t.test('Crossing the maxInterval border', function (t) {
+		timers.maxInterval = 2;
+
+		timers.setTimeoutAt(function () {
+			timers.maxInterval = originalMaxInterval;
+
+			t.pass('timeout fired');
+			t.equal(arguments.length, 0);
+			t.end();
+		}, Date.now() + 30);
+	});
+
+	t.test('Crossing the maxInterval border with args', function (t) {
+		timers.maxInterval = 2;
+
+		timers.setTimeoutAt(function (a, b) {
+			timers.maxInterval = originalMaxInterval;
+
+			t.pass('timeout fired');
+			t.equal(arguments.length, 2);
+			t.equal(a, 1);
+			t.equal(b, 2);
+			t.end();
+		}, Date.now() + 30, 1, 2);
+	});
+
+	t.test('Not crossing the maxInterval border', function (t) {
+		timers.setTimeoutAt(function () {
+			t.pass('timeout fired');
+			t.equal(arguments.length, 0);
+			t.end();
+		}, Date.now() + 5);
+	});
+
+	t.test('Not crossing the maxInterval border with args', function (t) {
+		timers.setTimeoutAt(function (a, b) {
+			t.pass('timeout fired');
+			t.equal(arguments.length, 2);
+			t.equal(a, 1);
+			t.equal(b, 2);
+			t.end();
+		}, Date.now() + 5, 1, 2);
+	});
+
+	t.test('clear()', function (t) {
+		const timeout = timers.setTimeoutAt(function () {
+			t.end('timeout fired despite clear()');
+		}, Date.now() + 0);
+
+		timeout.clear();
+
+		setTimeout(function () {
+			t.pass('timeout did not fire');
+			t.end();
+		}, 10);
+	});
+
+	t.test('clearTimeout', function (t) {
+		const timeout = timers.setTimeoutAt(function () {
+			t.end('timeout fired despite clearTimeout()');
+		}, Date.now() + 0);
+
+		timers.clearTimeout(timeout);
+
+		setTimeout(function () {
+			t.pass('timeout did not fire');
+			t.end();
+		}, 10);
+	});
+
+	t.test('clearTimeout without timer', function (t) {
+		timers.clearTimeout(null);
+		t.pass('no error was thrown');
+		t.end();
+	});
+
+	t.end();
 });