From 6785f7a1b06fbc42749a32e80390d9f133c15f7d Mon Sep 17 00:00:00 2001
From: sivaprasanna <sivaprasanna@gmail.com>
Date: Sun, 8 Jan 2017 14:03:09 +0530
Subject: [PATCH 1/2] test: refactor the code of test-keep-alive.js

* use const and let instead of var
* use assert.strictEqual instead of assert.equal
* use arrow functions
---
 test/pummel/test-keep-alive.js | 30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/test/pummel/test-keep-alive.js b/test/pummel/test-keep-alive.js
index d36c542ed8bdd6..50292fd2733276 100644
--- a/test/pummel/test-keep-alive.js
+++ b/test/pummel/test-keep-alive.js
@@ -12,8 +12,8 @@ if (common.isWindows) {
   return;
 }
 
-var body = 'hello world\n';
-var server = http.createServer(function(req, res) {
+const body = 'hello world\n';
+const server = http.createServer(function(req, res) {
   res.writeHead(200, {
     'Content-Length': body.length,
     'Content-Type': 'text/plain'
@@ -22,12 +22,12 @@ var server = http.createServer(function(req, res) {
   res.end();
 });
 
-var keepAliveReqSec = 0;
-var normalReqSec = 0;
+let keepAliveReqSec = 0;
+let normalReqSec = 0;
 
 
 function runAb(opts, callback) {
-  var args = [
+  const args = [
     '-c', opts.concurrent || 100,
     '-t', opts.threads || 2,
     '-d', opts.duration || '10s',
@@ -43,11 +43,11 @@ function runAb(opts, callback) {
 
   //console.log(comm, args.join(' '));
 
-  var child = spawn('wrk', args);
+  const child = spawn('wrk', args);
   child.stderr.pipe(process.stderr);
   child.stdout.setEncoding('utf8');
 
-  var stdout;
+  let stdout;
 
   child.stdout.on('data', function(data) {
     stdout += data;
@@ -60,11 +60,11 @@ function runAb(opts, callback) {
       return;
     }
 
-    var matches = /Requests\/sec:\s*(\d+)\./mi.exec(stdout);
-    var reqSec = parseInt(matches[1]);
+    let matches = /Requests\/sec:\s*(\d+)\./mi.exec(stdout);
+    const reqSec = parseInt(matches[1]);
 
     matches = /Keep-Alive requests:\s*(\d+)/mi.exec(stdout);
-    var keepAliveRequests;
+    let keepAliveRequests;
     if (matches) {
       keepAliveRequests = parseInt(matches[1]);
     } else {
@@ -75,8 +75,8 @@ function runAb(opts, callback) {
   });
 }
 
-server.listen(common.PORT, function() {
-  runAb({ keepalive: true }, function(reqSec) {
+server.listen(common.PORT, () => {
+  runAb({ keepalive: true }, (reqSec) => {
     keepAliveReqSec = reqSec;
     console.log('keep-alive:', keepAliveReqSec, 'req/sec');
 
@@ -89,7 +89,7 @@ server.listen(common.PORT, function() {
 });
 
 process.on('exit', function() {
-  assert.equal(true, normalReqSec > 50);
-  assert.equal(true, keepAliveReqSec > 50);
-  assert.equal(true, normalReqSec < keepAliveReqSec);
+  assert.strictEqual(true, normalReqSec > 50);
+  assert.strictEqual(true, keepAliveReqSec > 50);
+  assert.strictEqual(true, normalReqSec < keepAliveReqSec);
 });

From d0a9f557c286b468e7d22f53f7ec8618240cf058 Mon Sep 17 00:00:00 2001
From: sivaprasanna <sivaprasanna@gmail.com>
Date: Mon, 9 Jan 2017 21:31:48 +0530
Subject: [PATCH 2/2] test: refactor test-keep-alive.js

removed console.log statements
---
 test/pummel/test-keep-alive.js | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/test/pummel/test-keep-alive.js b/test/pummel/test-keep-alive.js
index 50292fd2733276..a9e91e201bb19b 100644
--- a/test/pummel/test-keep-alive.js
+++ b/test/pummel/test-keep-alive.js
@@ -41,8 +41,6 @@ function runAb(opts, callback) {
   args.push(url.format({ hostname: '127.0.0.1',
                          port: common.PORT, protocol: 'http'}));
 
-  //console.log(comm, args.join(' '));
-
   const child = spawn('wrk', args);
   child.stderr.pipe(process.stderr);
   child.stdout.setEncoding('utf8');
@@ -78,11 +76,9 @@ function runAb(opts, callback) {
 server.listen(common.PORT, () => {
   runAb({ keepalive: true }, (reqSec) => {
     keepAliveReqSec = reqSec;
-    console.log('keep-alive:', keepAliveReqSec, 'req/sec');
 
     runAb({ keepalive: false }, function(reqSec) {
       normalReqSec = reqSec;
-      console.log('normal:' + normalReqSec + ' req/sec');
       server.close();
     });
   });