From be90638487b391cab8a3cee167407c7a30c335f3 Mon Sep 17 00:00:00 2001
From: Daniel Sims <danielsims@quickenloans.com>
Date: Thu, 1 Dec 2016 10:02:15 -0600
Subject: [PATCH] test: refactor test-domain-from-timer
In this change, the setTimeout needed a second argument, so I set that
value to 1. In addition, I changed the assertion to be a strictEquals
instead of equals.
I changed the var declarations to const in this test.
PR-URL: https://github.com/nodejs/node/pull/9889
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
---
test/parallel/test-domain-from-timer.js | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/test/parallel/test-domain-from-timer.js b/test/parallel/test-domain-from-timer.js
index f0115018ec..2ffae758ef 100644
--- a/test/parallel/test-domain-from-timer.js
+++ b/test/parallel/test-domain-from-timer.js
@@ -2,17 +2,17 @@
// Simple tests of most basic domain functionality.
require('../common');
-var assert = require('assert');
+const assert = require('assert');
// timeouts call the callback directly from cc, so need to make sure the
// domain will be used regardless
setTimeout(function() {
- var domain = require('domain');
- var d = domain.create();
+ const domain = require('domain');
+ const d = domain.create();
d.run(function() {
process.nextTick(function() {
console.trace('in nexttick', process.domain === d);
- assert.equal(process.domain, d);
+ assert.strictEqual(process.domain, d);
});
});
-});
+}, 1);