Skip to content

Commit

Permalink
test: change anonymous closure function to arrow function
Browse files Browse the repository at this point in the history
PR-URL: #24433
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
  • Loading branch information
nethraravindran authored and rvagg committed Nov 28, 2018
1 parent e8c0fce commit 9bf2659
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions test/parallel/test-net-connect-options-port.js
Original file line number Diff line number Diff line change
Expand Up @@ -76,15 +76,15 @@ const net = require('net');
const expectedConnections = 72;
let serverConnected = 0;

const server = net.createServer(common.mustCall(function(socket) {
const server = net.createServer(common.mustCall((socket) => {
socket.end('ok');
if (++serverConnected === expectedConnections) {
server.close();
}
}, expectedConnections));

server.listen(0, 'localhost', common.mustCall(function() {
const port = this.address().port;
server.listen(0, 'localhost', common.mustCall(() => {
const port = server.address().port;

// Total connections = 3 * 4(canConnect) * 6(doConnect) = 72
canConnect(port);
Expand All @@ -93,7 +93,7 @@ const net = require('net');
}));

// Try connecting to random ports, but do so once the server is closed
server.on('close', function() {
server.on('close', () => {
asyncFailToConnect(0);
asyncFailToConnect(/* undefined */);
});
Expand Down Expand Up @@ -193,7 +193,7 @@ function canConnect(port) {
}

function asyncFailToConnect(port) {
const onError = () => common.mustCall(function(err) {
const onError = () => common.mustCall((err) => {
const regexp = /^Error: connect E\w+.+$/;
assert(regexp.test(String(err)), String(err));
});
Expand Down

0 comments on commit 9bf2659

Please sign in to comment.