Close #135 PR: add failing test demonstrating long running onExit() h…

…ook in child process. Fixes #134

package.json

@@ -90,6 +90,7 @@
   "devDependencies": {
     "coveralls": "^2.11.4",
     "nyc": "^3.2.2",
+    "signal-exit": "^2.1.2",
     "tap-dot": "^1.0.0",
     "tape": "^4.0.0",
     "xo": "*"

test/fixture/long-running.js

@@ -1,12 +1,25 @@
 'use strict';
 const test = require('../../');
+var onExit = require('signal-exit');
 
 test('long running', function (t) {
 	t.plan(1);
 
+	onExit(function () {
+		// simulate an exit hook that lasts a short while
+		var start = Date.now();
+		while(Date.now() - start < 2000) {
+			//synchronously wait for 2 seconds
+		}
+		process.send({name:'cleanup-completed', data: {completed: true}});
+	}, {alwaysLast: true});
+
+	setTimeout(function () {
+		t.ok(true);
+	});
+
 	setTimeout(function () {
+		// this would keep the process running for a long time.
 		console.log('I\'m gonna live forever!!');
 	}, 15000);
-
-	t.ok(true);
 });

test/fork.js

@@ -41,12 +41,17 @@ test('rejects on error and streams output', function (t) {
 });
 
 test('exit after tests are finished', function (t) {
-	t.plan(1);
+	t.plan(2);
 
 	var start = Date.now();
+	var cleanupCompleted = false;
 
 	fork(fixture('long-running.js'))
 		.on('exit', function () {
-			t.ok(Date.now() - start < 10000);
+			t.ok(Date.now() - start < 10000, 'did NOT wait for setTimeout(fn, 15000');
+			t.ok(cleanupCompleted, 'did wait for onExit(fn) to complete');
+		})
+		.on('cleanup-completed', function (event) {
+			cleanupCompleted = event.completed;
 		});
 });