Skip to content

Commit

Permalink
test: imporove assert messages
Browse files Browse the repository at this point in the history
Improve assert messages in test-cluster-worker-disconnect.js.

PR-URL: #16021
Reviewed-By: Anatoli Papirovski <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
  • Loading branch information
Hadis-Fard authored and MylesBorins committed Nov 28, 2017
1 parent 91f9779 commit 58fe9b4
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions test/parallel/test-cluster-worker-disconnect.js
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ if (cluster.isWorker) {
// Disconnect worker when it is ready
worker.once('listening', common.mustCall(() => {
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
Expand Down Expand Up @@ -80,10 +80,8 @@ if (cluster.isWorker) {
assert.ok(c.emitExit, 'Exit event did not emit');

// flags
assert.strictEqual(w.state, 'disconnected',
'The state property was not set');
assert.strictEqual(w.voluntaryMode, true,
'Voluntary exit mode was not set');
assert.strictEqual(w.state, 'disconnected');
assert.strictEqual(w.voluntaryMode, true);

// is process alive
assert.ok(w.died, 'The worker did not die');
Expand Down

0 comments on commit 58fe9b4

Please sign in to comment.