test: replace callback with arrow functions

PR-URL: #24490 Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Franziska Hinkelmann <[email protected]> Reviewed-By: Sakthipriyan Vairamani <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
nodejs · Nov 28, 2018 · c4f16dd · c4f16dd
1 parent 4da44ad
commit c4f16dd
Showing 1 changed file with 7 additions and 9 deletions.
diff --git a/test/pummel/test-keep-alive.js b/test/pummel/test-keep-alive.js
@@ -32,7 +32,7 @@ const http = require('http');
 const url = require('url');
 
 const body = 'hello world\n';
-const server = http.createServer(function(req, res) {
+const server = http.createServer((req, res) => {
   res.writeHead(200, {
     'Content-Length': body.length,
     'Content-Type': 'text/plain'
@@ -45,7 +45,7 @@ let keepAliveReqSec = 0;
 let normalReqSec = 0;
 
 
-function runAb(opts, callback) {
+const runAb = (opts, callback) => {
   const args = [
     '-c', opts.concurrent || 100,
     '-t', opts.threads || 2,
@@ -66,11 +66,9 @@ function runAb(opts, callback) {
 
   let stdout;
 
-  child.stdout.on('data', function(data) {
-    stdout += data;
-  });
+  child.stdout.on('data', (data) => stdout += data);
 
-  child.on('close', function(code, signal) {
+  child.on('close', (code, signal) => {
     if (code) {
       console.error(code, signal);
       process.exit(code);
@@ -90,20 +88,20 @@ function runAb(opts, callback) {
 
     callback(reqSec, keepAliveRequests);
   });
-}
+};
 
 server.listen(common.PORT, () => {
   runAb({ keepalive: true }, (reqSec) => {
     keepAliveReqSec = reqSec;
 
-    runAb({ keepalive: false }, function(reqSec) {
+    runAb({ keepalive: false }, (reqSec) => {
       normalReqSec = reqSec;
       server.close();
     });
   });
 });
 
-process.on('exit', function() {
+process.on('exit', () => {
   assert.strictEqual(
     normalReqSec > 50,
     true,