1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: imporove assert messages

Browse Source 
Improve assert messages in test-cluster-worker-disconnect.js.

PR-URL: https://github.com/nodejs/node/pull/16021
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
This commit is contained in:
Hadis-Fard 2017-10-06 12:06:04 -07:00 committed by Rich Trott
parent 65b8d2133d
commit eefee3e9a6
1 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
test/parallel/test-cluster-worker-disconnect.js
View File

@ -58,7 +58,7 @@ if (cluster.isWorker) {
// Disconnect worker when it is ready // Disconnect worker when it is ready
worker.once('listening', common.mustCall(() => { worker.once('listening', common.mustCall(() => {
const w = worker.disconnect(); const w = worker.disconnect();
assert.strictEqual(worker, w, 'did not return a reference'); assert.strictEqual(worker, w, `${worker.id} did not return a reference`);
})); }));
// Check cluster events // Check cluster events
@ -97,10 +97,8 @@ if (cluster.isWorker) {
assert.ok(c.emitExit, 'Exit event did not emit'); assert.ok(c.emitExit, 'Exit event did not emit');
// flags // flags
assert.strictEqual(w.state, 'disconnected', assert.strictEqual(w.state, 'disconnected');
'The state property was not set'); assert.strictEqual(w.voluntaryMode, true);
assert.strictEqual(w.voluntaryMode, true,
'Voluntary exit mode was not set');
// is process alive // is process alive
assert.ok(w.died, 'The worker did not die'); assert.ok(w.died, 'The worker did not die');