test: remove unused vars from http/https tests

Trott · evanlucas · commit e1aedbf67178 · 2016-07-15T08:09:42.000-05:00
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: #7598 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
diff --git a/test/parallel/test-http-client-timeout-agent.js b/test/parallel/test-http-client-timeout-agent.js
@@ -3,7 +3,6 @@ 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(0, options.host, function(){
diff --git 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 = Buffer.alloc(10240, '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
@@ -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
@@ -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
@@ -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();

-Original file line number
+Diff line change
 varassert=require('assert');
 varhttp=require('http');
 -varrequest_number=0;
 varrequests_sent=0;
 varrequests_done=0;
 varoptions={
 res.write(reqid.toString());
 res.end();
+}
 -request_number+=1;
 });
 server.listen(0,options.host,function(){
-Original file line number
+Diff line change
 functionparent(){
 consthttp=require('http');
 constbigResponse=Buffer.alloc(10240,'x');
 -varrequests=0;
 varconnections=0;
 varbackloggedReqs=0;
 constserver=http.createServer(function(req,res){
 -requests++;
 res.setHeader('content-length',bigResponse.length);
 if(!res.write(bigResponse)){
 if(backloggedReqs===0){
-Original file line number
+Diff line change
 requestCert: true
 };
 -varreqCount=0;
 varCRLF='\r\n';
 varbody='hello world\n';
 varcert;
 varexponent;
 varserver=https.createServer(options,function(req,res){
 -reqCount++;
 console.log('got request');
 cert=req.connection.getPeerCertificate();