Skip to content

Commit

Permalink
test: replace closure with arrow function
Browse files Browse the repository at this point in the history
PR-URL: #24489
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
  • Loading branch information
Maya Anilson authored and rvagg committed Nov 28, 2018
1 parent 1cd73a8 commit c28ec86
Showing 1 changed file with 13 additions and 13 deletions.
26 changes: 13 additions & 13 deletions test/pummel/test-net-timeout.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,46 +29,46 @@ let starttime = null;
let timeouttime = null;
const timeout = 1000;

const echo_server = net.createServer(function(socket) {
const echo_server = net.createServer((socket) => {
socket.setTimeout(timeout);

socket.on('timeout', function() {
socket.on('timeout', () => {
console.log('server timeout');
timeouttime = new Date();
console.dir(timeouttime);
socket.destroy();
});

socket.on('error', function(e) {
socket.on('error', (e) => {
throw new Error(
'Server side socket should not get error. We disconnect willingly.');
});

socket.on('data', function(d) {
socket.on('data', (d) => {
console.log(d);
socket.write(d);
});

socket.on('end', function() {
socket.on('end', () => {
socket.end();
});
});

echo_server.listen(common.PORT, function() {
echo_server.listen(common.PORT, () => {
console.log(`server listening at ${common.PORT}`);

const client = net.createConnection(common.PORT);
client.setEncoding('UTF8');
client.setTimeout(0); // disable the timeout for client
client.on('connect', function() {
client.on('connect', () => {
console.log('client connected.');
client.write('hello\r\n');
});

client.on('data', function(chunk) {
client.on('data', (chunk) => {
assert.strictEqual(chunk, 'hello\r\n');
if (exchanges++ < 5) {
setTimeout(function() {
setTimeout(() => {
console.log('client write "hello"');
client.write('hello\r\n');
}, 500);
Expand All @@ -81,22 +81,22 @@ echo_server.listen(common.PORT, function() {
}
});

client.on('timeout', function() {
client.on('timeout', () => {
throw new Error("client timeout - this shouldn't happen");
});

client.on('end', function() {
client.on('end', () => {
console.log('client end');
client.end();
});

client.on('close', function() {
client.on('close', () => {
console.log('client disconnect');
echo_server.close();
});
});

process.on('exit', function() {
process.on('exit', () => {
assert.ok(starttime != null);
assert.ok(timeouttime != null);

Expand Down

0 comments on commit c28ec86

Please sign in to comment.