1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: remove unused vars from http/https tests

Browse Source 
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>
This commit is contained in:
Rich Trott 2016-07-07 15:30:50 -07:00
parent 2bd89421d5
commit bc5b89f9ca
5 changed files with 0 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -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
View File

@ -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
View File

@ -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
View File

@ -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
View File

@ -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();