Browse Source

test: remove unused vars from http/https tests

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>
v6.x
Rich Trott 9 years ago
committed by Evan Lucas
parent
commit
e1aedbf671
  1. 2
      test/parallel/test-http-client-timeout-agent.js
  2. 2
      test/parallel/test-http-pipeline-flood.js
  3. 3
      test/parallel/test-http-upgrade-agent.js
  4. 3
      test/parallel/test-http-upgrade-client.js
  5. 2
      test/parallel/test-https-foafssl.js

2
test/parallel/test-http-client-timeout-agent.js

@ -3,7 +3,6 @@ require('../common');
var assert = require('assert'); var assert = require('assert');
var http = require('http'); var http = require('http');
var request_number = 0;
var requests_sent = 0; var requests_sent = 0;
var requests_done = 0; var requests_done = 0;
var options = { var options = {
@ -24,7 +23,6 @@ var server = http.createServer(function(req, res) {
res.write(reqid.toString()); res.write(reqid.toString());
res.end(); res.end();
} }
request_number += 1;
}); });
server.listen(0, options.host, function() { server.listen(0, options.host, function() {

2
test/parallel/test-http-pipeline-flood.js

@ -27,12 +27,10 @@ switch (process.argv[2]) {
function parent() { function parent() {
const http = require('http'); const http = require('http');
const bigResponse = Buffer.alloc(10240, 'x'); const bigResponse = Buffer.alloc(10240, 'x');
var requests = 0;
var connections = 0; var connections = 0;
var backloggedReqs = 0; var backloggedReqs = 0;
const server = http.createServer(function(req, res) { const server = http.createServer(function(req, res) {
requests++;
res.setHeader('content-length', bigResponse.length); res.setHeader('content-length', bigResponse.length);
if (!res.write(bigResponse)) { if (!res.write(bigResponse)) {
if (backloggedReqs === 0) { if (backloggedReqs === 0) {

3
test/parallel/test-http-upgrade-agent.js

@ -11,10 +11,7 @@ var net = require('net');
// Create a TCP server // Create a TCP server
var srv = net.createServer(function(c) { var srv = net.createServer(function(c) {
var data = '';
c.on('data', function(d) { c.on('data', function(d) {
data += d.toString('utf8');
c.write('HTTP/1.1 101\r\n'); c.write('HTTP/1.1 101\r\n');
c.write('hello: world\r\n'); c.write('hello: world\r\n');
c.write('connection: upgrade\r\n'); c.write('connection: upgrade\r\n');

3
test/parallel/test-http-upgrade-client.js

@ -11,10 +11,7 @@ var net = require('net');
// Create a TCP server // Create a TCP server
var srv = net.createServer(function(c) { var srv = net.createServer(function(c) {
var data = '';
c.on('data', function(d) { c.on('data', function(d) {
data += d.toString('utf8');
c.write('HTTP/1.1 101\r\n'); c.write('HTTP/1.1 101\r\n');
c.write('hello: world\r\n'); c.write('hello: world\r\n');
c.write('connection: upgrade\r\n'); c.write('connection: upgrade\r\n');

2
test/parallel/test-https-foafssl.js

@ -24,7 +24,6 @@ var options = {
requestCert: true requestCert: true
}; };
var reqCount = 0;
var CRLF = '\r\n'; var CRLF = '\r\n';
var body = 'hello world\n'; var body = 'hello world\n';
var cert; var cert;
@ -33,7 +32,6 @@ var modulus;
var exponent; var exponent;
var server = https.createServer(options, function(req, res) { var server = https.createServer(options, function(req, res) {
reqCount++;
console.log('got request'); console.log('got request');
cert = req.connection.getPeerCertificate(); cert = req.connection.getPeerCertificate();

Loading…
Cancel
Save