Commit e1aedbf
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: #7598
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>1 parent 77afeb2 commit e1aedbf
5 files changed
Lines changed: 0 additions & 12 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -3,7 +3,6 @@ require('../common'); | |||
| 3 | 3 | var assert = require('assert'); | |
| 4 | 4 | var http = require('http'); | |
| 5 | 5 | ||
| 6 | - var request_number = 0; | ||
| 7 | 6 | var requests_sent = 0; | |
| 8 | 7 | var requests_done = 0; | |
| 9 | 8 | var options = { | |
@@ -24,7 +23,6 @@ var server = http.createServer(function(req, res) { | |||
| 24 | 23 | res.write(reqid.toString()); | |
| 25 | 24 | res.end(); | |
| 26 | 25 | } | |
| 27 | - request_number += 1; | ||
| 28 | 26 | }); | |
| 29 | 27 | ||
| 30 | 28 | server.listen(0, options.host, function() { | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -27,12 +27,10 @@ switch (process.argv[2]) { | |||
| 27 | 27 | function parent() { | |
| 28 | 28 | const http = require('http'); | |
| 29 | 29 | const bigResponse = Buffer.alloc(10240, 'x'); | |
| 30 | - var requests = 0; | ||
| 31 | 30 | var connections = 0; | |
| 32 | 31 | var backloggedReqs = 0; | |
| 33 | 32 | ||
| 34 | 33 | const server = http.createServer(function(req, res) { | |
| 35 | - requests++; | ||
| 36 | 34 | res.setHeader('content-length', bigResponse.length); | |
| 37 | 35 | if (!res.write(bigResponse)) { | |
| 38 | 36 | if (backloggedReqs === 0) { | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -11,10 +11,7 @@ var net = require('net'); | |||
| 11 | 11 | ||
| 12 | 12 | // Create a TCP server | |
| 13 | 13 | var srv = net.createServer(function(c) { | |
| 14 | - var data = ''; | ||
| 15 | 14 | c.on('data', function(d) { | |
| 16 | - data += d.toString('utf8'); | ||
| 17 | - | ||
| 18 | 15 | c.write('HTTP/1.1 101\r\n'); | |
| 19 | 16 | c.write('hello: world\r\n'); | |
| 20 | 17 | c.write('connection: upgrade\r\n'); | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -11,10 +11,7 @@ var net = require('net'); | |||
| 11 | 11 | ||
| 12 | 12 | // Create a TCP server | |
| 13 | 13 | var srv = net.createServer(function(c) { | |
| 14 | - var data = ''; | ||
| 15 | 14 | c.on('data', function(d) { | |
| 16 | - data += d.toString('utf8'); | ||
| 17 | - | ||
| 18 | 15 | c.write('HTTP/1.1 101\r\n'); | |
| 19 | 16 | c.write('hello: world\r\n'); | |
| 20 | 17 | c.write('connection: upgrade\r\n'); | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -24,7 +24,6 @@ var options = { | |||
| 24 | 24 | requestCert: true | |
| 25 | 25 | }; | |
| 26 | 26 | ||
| 27 | - var reqCount = 0; | ||
| 28 | 27 | var CRLF = '\r\n'; | |
| 29 | 28 | var body = 'hello world\n'; | |
| 30 | 29 | var cert; | |
@@ -33,7 +32,6 @@ var modulus; | |||
| 33 | 32 | var exponent; | |
| 34 | 33 | ||
| 35 | 34 | var server = https.createServer(options, function(req, res) { | |
| 36 | - reqCount++; | ||
| 37 | 35 | console.log('got request'); | |
| 38 | 36 | ||
| 39 | 37 | cert = req.connection.getPeerCertificate(); | |
0 commit comments