From 64d7ea9ce484e35a13d612803a6aeaf7aaa531f3 Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Wed, 9 Nov 2016 22:09:56 -0800
Subject: [PATCH] test: refactor test-next-tick-error-spin

* use common.mustCall()
* setTimeout() -> setImmediate()
* assert() -> assert.strictEqual()
* var -> const
* remove unneeded console.log()
* remove commented-out code

PR-URL: https://github.com/nodejs/node/pull/9537
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
---
 test/sequential/test-next-tick-error-spin.js | 25 +++++++++-----------
 1 file changed, 11 insertions(+), 14 deletions(-)

diff --git a/test/sequential/test-next-tick-error-spin.js b/test/sequential/test-next-tick-error-spin.js
index 7a76ab1e12..911d99d741 100644
--- a/test/sequential/test-next-tick-error-spin.js
+++ b/test/sequential/test-next-tick-error-spin.js
@@ -1,24 +1,23 @@
 'use strict';
-var common = require('../common');
-var assert = require('assert');
+const common = require('../common');
+const assert = require('assert');
 
 if (process.argv[2] !== 'child') {
-  var spawn = require('child_process').spawn;
-  var child = spawn(process.execPath, [__filename, 'child'], {
+  const spawn = require('child_process').spawn;
+  const child = spawn(process.execPath, [__filename, 'child'], {
     stdio: 'pipe'//'inherit'
   });
-  var timer = setTimeout(function() {
+  const timer = setTimeout(function() {
     throw new Error('child is hung');
   }, common.platformTimeout(3000));
-  child.on('exit', function(code) {
-    console.error('ok');
-    assert(!code);
+  child.on('exit', common.mustCall(function(code) {
+    assert.strictEqual(code, 0);
     clearTimeout(timer);
-  });
+  }));
 } else {
 
-  var domain = require('domain');
-  var d = domain.create();
+  const domain = require('domain');
+  const d = domain.create();
   process.maxTickDepth = 10;
 
   // in the error handler, we trigger several MakeCallback events
@@ -40,10 +39,8 @@ if (process.argv[2] !== 'child') {
   }
 
   f();
-  setTimeout(function() {
+  setImmediate(function() {
     console.error('broke in!');
-    //process.stdout.close();
-    //process.stderr.close();
     process.exit(0);
   });
 }