Handing timeout SIGTERMs after explicit root test end

Previously, it'd just throw a 'test after end' exception because the
onTimeout method emits a fail().  This prints something at least a
little useful to stderr before exiting in failure.

Author: isaacs

lib/root.js

@@ -133,5 +133,20 @@ onExit(function (code, signal) {
     })
   }
 
-  tap._onTimeout(extra)
+  if (!tap._ended) {
+    tap._onTimeout(extra)
+  } else {
+    console.error('possible timeout: SIGTERM received after tap end')
+    if (extra.handles || extra.requests) {
+      delete extra.signal
+      if (!extra.at) {
+        delete extra.at
+      }
+      var yaml = require('js-yaml')
+      console.error('  ---\n  ' +
+               yaml.safeDump(extra).split('\n').join('\n  ').trim() +
+               '\n  ...\n')
+    }
+    process.exit(1)
+  }
 })

test/test/child-sigterm-after-end-bail.tap

@@ -0,0 +1,16 @@
+TAP version 13
+# child start
+TAP version 13
+ok 1 - this is fine
+1..1
+___/# time=[0-9.]+(ms)?/~~~
+possible timeout: SIGTERM received after tap end
+  ---
+  {"handles":[{"type":"Timer"}]}
+  ...
+
+# child end code=1 signal=null
+ok 1 - should not be equal
+1..1
+___/# time=[0-9.]+(ms)?/~~~
+

test/test/child-sigterm-after-end.js

@@ -0,0 +1,22 @@
+var t = require('../..')
+if (process.argv[2] === 'child') {
+  setInterval(function () {}, 10000)
+  t.pass('this is fine')
+  t.end()
+  // use ipc so that we're not waiting longer than necessary
+  process.send({ do: 'it' })
+} else {
+  t.comment('child start')
+  var spawn = require('child_process').spawn
+  var child = spawn(process.execPath, [__filename, 'child'], {
+    stdio: [ 0, 1, 1, 'ipc']
+  })
+  child.on('message', function () {
+    child.kill('SIGTERM')
+  })
+  child.on('exit', function (code, signal) {
+    t.comment('child end code=%j signal=%j', code, signal)
+    t.notEqual(code, 0)
+    t.end()
+  })
+}

test/test/child-sigterm-after-end.tap

@@ -0,0 +1,16 @@
+TAP version 13
+# child start
+TAP version 13
+ok 1 - this is fine
+1..1
+___/# time=[0-9.]+(ms)?/~~~
+possible timeout: SIGTERM received after tap end
+  ---
+  {"handles":[{"type":"Timer"}]}
+  ...
+
+# child end code=1 signal=null
+ok 1 - should not be equal
+1..1
+___/# time=[0-9.]+(ms)?/~~~
+