From 1d100f6853a71470c6899a5b87686b2277c14bd9 Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Thu, 7 Jul 2016 15:30:50 -0700
Subject: [PATCH] test: remove unused vars from http/https tests
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Remove handful of variables that are assigned but never used.

(This was missed by the linter in previous versions of ESLint but is
flagged by the current version. Updating the linter is contingent on
this change or some similar remedy landing.)

PR-URL: https://github.com/nodejs/node/pull/7598
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
---
 test/parallel/test-http-client-timeout-agent.js | 2 --
 test/parallel/test-http-pipeline-flood.js       | 2 --
 test/parallel/test-http-upgrade-agent.js        | 3 ---
 test/parallel/test-http-upgrade-client.js       | 3 ---
 test/parallel/test-https-foafssl.js             | 2 --
 5 files changed, 12 deletions(-)

diff --git a/test/parallel/test-http-client-timeout-agent.js b/test/parallel/test-http-client-timeout-agent.js
index 6892b38719..a282eca784 100644
--- a/test/parallel/test-http-client-timeout-agent.js
+++ b/test/parallel/test-http-client-timeout-agent.js
@@ -3,7 +3,6 @@ var common = require('../common');
 var assert = require('assert');
 var http = require('http');
 
-var request_number = 0;
 var requests_sent = 0;
 var requests_done = 0;
 var options = {
@@ -24,7 +23,6 @@ var server = http.createServer(function(req, res) {
     res.write(reqid.toString());
     res.end();
   }
-  request_number += 1;
 });
 
 server.listen(options.port, options.host, function() {
diff --git a/test/parallel/test-http-pipeline-flood.js b/test/parallel/test-http-pipeline-flood.js
index d291ccdb17..382c5d3588 100644
--- a/test/parallel/test-http-pipeline-flood.js
+++ b/test/parallel/test-http-pipeline-flood.js
@@ -27,12 +27,10 @@ switch (process.argv[2]) {
 function parent() {
   const http = require('http');
   const bigResponse = new Buffer(10240).fill('x');
-  var requests = 0;
   var connections = 0;
   var backloggedReqs = 0;
 
   const server = http.createServer(function(req, res) {
-    requests++;
     res.setHeader('content-length', bigResponse.length);
     if (!res.write(bigResponse)) {
       if (backloggedReqs === 0) {
diff --git a/test/parallel/test-http-upgrade-agent.js b/test/parallel/test-http-upgrade-agent.js
index 7590354f50..8c2256db59 100644
--- a/test/parallel/test-http-upgrade-agent.js
+++ b/test/parallel/test-http-upgrade-agent.js
@@ -11,10 +11,7 @@ var net = require('net');
 
 // Create a TCP server
 var srv = net.createServer(function(c) {
-  var data = '';
   c.on('data', function(d) {
-    data += d.toString('utf8');
-
     c.write('HTTP/1.1 101\r\n');
     c.write('hello: world\r\n');
     c.write('connection: upgrade\r\n');
diff --git a/test/parallel/test-http-upgrade-client.js b/test/parallel/test-http-upgrade-client.js
index a45c06b44e..8d90eea116 100644
--- a/test/parallel/test-http-upgrade-client.js
+++ b/test/parallel/test-http-upgrade-client.js
@@ -11,10 +11,7 @@ var net = require('net');
 
 // Create a TCP server
 var srv = net.createServer(function(c) {
-  var data = '';
   c.on('data', function(d) {
-    data += d.toString('utf8');
-
     c.write('HTTP/1.1 101\r\n');
     c.write('hello: world\r\n');
     c.write('connection: upgrade\r\n');
diff --git a/test/parallel/test-https-foafssl.js b/test/parallel/test-https-foafssl.js
index fa6200c845..c91cb0b138 100644
--- a/test/parallel/test-https-foafssl.js
+++ b/test/parallel/test-https-foafssl.js
@@ -24,7 +24,6 @@ var options = {
   requestCert: true
 };
 
-var reqCount = 0;
 var CRLF = '\r\n';
 var body = 'hello world\n';
 var cert;
@@ -33,7 +32,6 @@ var modulus;
 var exponent;
 
 var server = https.createServer(options, function(req, res) {
-  reqCount++;
   console.log('got request');
 
   cert = req.connection.getPeerCertificate();