Skip to content

Commit

Permalink
test: remove common.PORT from dgram test
Browse files Browse the repository at this point in the history
Remove common.PORT from
test-dgram-send-callback-buffer-length-empty-address to eliminate
possibility that a dynamic port used in another test will collide
with common.PORT.

PR-URL: #12944
Refs: #12376
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michael Dawson <mhdawson@ibm.com>
  • Loading branch information
vieiraarturg authored and jasnell committed May 23, 2017
1 parent d714861 commit d27c0cf
Showing 1 changed file with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -17,4 +17,6 @@ const onMessage = common.mustCall(function messageSent(err, bytes) {
client.close();
});

client.send(buf, offset, len, common.PORT, onMessage);
client.bind(0, () => client.send(buf, offset, len,
client.address().port,
onMessage));

0 comments on commit d27c0cf

Please sign in to comment.