Skip to content

Commit

Permalink
test: remove unused args and comparison fix
Browse files Browse the repository at this point in the history
Remove unused arguments and change non-strict comparison
to the strict one in test.

PR-URL: #11396
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
  • Loading branch information
sashashakun authored and jasnell committed Mar 7, 2017
1 parent f3546b7 commit 1c2ce91
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/internet/test-dgram-multicast-multi-process.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,14 +21,14 @@ if (common.inFreeBSDJail) {
return;
}

function launchChildProcess(index) {
function launchChildProcess() {
const worker = fork(__filename, ['child']);
workers[worker.pid] = worker;

worker.messagesReceived = [];

// Handle the death of workers.
worker.on('exit', function(code, signal) {
worker.on('exit', function(code) {
// Don't consider this the true death if the worker has finished
// successfully or if the exit code is 0.
if (worker.isDone || code === 0) {
Expand Down Expand Up @@ -188,7 +188,7 @@ if (process.argv[2] === 'child') {

process.send({ message: buf.toString() });

if (receivedMessages.length == messages.length) {
if (receivedMessages.length === messages.length) {
// .dropMembership() not strictly needed but here as a sanity check.
listenSocket.dropMembership(LOCAL_BROADCAST_HOST);
process.nextTick(function() {
Expand Down

0 comments on commit 1c2ce91

Please sign in to comment.