test: refactor test-async-wrap-*

Trott · addaleax · commit 9e4ce6fc8ee5 · 2016-12-08T02:33:55.000+01:00
* `assert.equal()` -> `assert.strictEqual()` * add duration to `setTimeout()` PR-URL: #9663 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Prince John Wesley <princejohnwesley@gmail.com>
diff --git a/test/parallel/test-async-wrap-check-providers.js b/test/parallel/test-async-wrap-check-providers.js
@@ -41,7 +41,7 @@ async_wrap.setupHooks({ init });
 async_wrap.enable();
 
 
-setTimeout(function() { });
+setTimeout(function() { }, 1);
 
 fs.stat(__filename, noop);
 
diff --git a/test/parallel/test-async-wrap-disabled-propagate-parent.js b/test/parallel/test-async-wrap-disabled-propagate-parent.js
@@ -18,12 +18,12 @@ function init(uid, type, parentUid, parentHandle) {
     cntr++;
     // Cannot assert in init callback or will abort.
     process.nextTick(() => {
-      assert.equal(providers[type], 'TCPWRAP');
-      assert.equal(parentUid, server._handle[uidSymbol],
-                   'server uid doesn\'t match parent uid');
-      assert.equal(parentHandle, server._handle,
-                   'server handle doesn\'t match parent handle');
-      assert.equal(this, client._handle, 'client doesn\'t match context');
+      assert.strictEqual(providers[type], 'TCPWRAP');
+      assert.strictEqual(parentUid, server._handle[uidSymbol],
+                         'server uid doesn\'t match parent uid');
+      assert.strictEqual(parentHandle, server._handle,
+                         'server handle doesn\'t match parent handle');
+      assert.strictEqual(this, client._handle, 'client doesn\'t match context');
     });
   }
 }
@@ -48,5 +48,5 @@ async_wrap.disable();
 
 process.on('exit', function() {
   // init should have only been called once with a parent.
-  assert.equal(cntr, 1);
+  assert.strictEqual(cntr, 1);
 });
diff --git a/test/parallel/test-async-wrap-propagate-parent.js b/test/parallel/test-async-wrap-propagate-parent.js
@@ -18,12 +18,12 @@ function init(uid, type, parentUid, parentHandle) {
     cntr++;
     // Cannot assert in init callback or will abort.
     process.nextTick(() => {
-      assert.equal(providers[type], 'TCPWRAP');
-      assert.equal(parentUid, server._handle[uidSymbol],
-                   'server uid doesn\'t match parent uid');
-      assert.equal(parentHandle, server._handle,
-                   'server handle doesn\'t match parent handle');
-      assert.equal(this, client._handle, 'client doesn\'t match context');
+      assert.strictEqual(providers[type], 'TCPWRAP');
+      assert.strictEqual(parentUid, server._handle[uidSymbol],
+                         'server uid doesn\'t match parent uid');
+      assert.strictEqual(parentHandle, server._handle,
+                         'server handle doesn\'t match parent handle');
+      assert.strictEqual(this, client._handle, 'client doesn\'t match context');
     });
   }
 }
@@ -47,5 +47,5 @@ const server = net.createServer(function(c) {
 
 process.on('exit', function() {
   // init should have only been called once with a parent.
-  assert.equal(cntr, 1);
+  assert.strictEqual(cntr, 1);
 });
diff --git a/test/parallel/test-async-wrap-throw-from-callback.js b/test/parallel/test-async-wrap-throw-from-callback.js
@@ -40,8 +40,8 @@ if (typeof process.argv[2] === 'string') {
 } else {
 
   process.on('exit', (code) => {
-    assert.equal(msgCalled, callbacks.length);
-    assert.equal(msgCalled, msgReceived);
+    assert.strictEqual(msgCalled, callbacks.length);
+    assert.strictEqual(msgCalled, msgReceived);
   });
 
   callbacks.forEach((item) => {
@@ -58,7 +58,7 @@ if (typeof process.argv[2] === 'string') {
       if (errstring.includes('Error: ' + item))
         msgReceived++;
 
-      assert.equal(code, 1, `${item} closed with code ${code}`);
+      assert.strictEqual(code, 1, `${item} closed with code ${code}`);
     });
   });
 }
