Skip to content

Commit

Permalink
test: remove unused vars from http/https tests
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Trott authored and Myles Borins committed Jul 12, 2016
1 parent 164b87d commit 4f31f47
Show file tree
Hide file tree
Showing 5 changed files with 0 additions and 12 deletions.
2 changes: 0 additions & 2 deletions test/parallel/test-http-client-timeout-agent.js
Original file line number Diff line number Diff line change
Expand Up @@ -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 = {
Expand All @@ -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() {
Expand Down
2 changes: 0 additions & 2 deletions test/parallel/test-http-pipeline-flood.js
Original file line number Diff line number Diff line change
Expand Up @@ -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) {
Expand Down
3 changes: 0 additions & 3 deletions test/parallel/test-http-upgrade-agent.js
Original file line number Diff line number Diff line change
Expand Up @@ -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');
Expand Down
3 changes: 0 additions & 3 deletions test/parallel/test-http-upgrade-client.js
Original file line number Diff line number Diff line change
Expand Up @@ -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');
Expand Down
2 changes: 0 additions & 2 deletions test/parallel/test-https-foafssl.js
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ var options = {
requestCert: true
};

var reqCount = 0;
var CRLF = '\r\n';
var body = 'hello world\n';
var cert;
Expand All @@ -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();
Expand Down

0 comments on commit 4f31f47

Please sign in to comment.