Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Commit b4964b0

Browse filesBrowse files
TrottMyles Borins
Trott
authored and
Myles Borins
committed
test: remove unused variables form http tests
The http tests seem especially prone to including unused variables. This change removes them. PR-URL: #4422 Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
1 parent 0d5a508 commit b4964b0
Copy full SHA for b4964b0
Expand file treeCollapse file tree

29 files changed

+9
-39
lines changed
Open diff view settings
Collapse file

‎test/parallel/test-http-1.0-keep-alive.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-http-1.0-keep-alive.js
-1Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
'use strict';
22
var common = require('../common');
3-
var assert = require('assert');
43
var http = require('http');
54
var net = require('net');
65

Collapse file

‎test/parallel/test-http-304.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-http-304.js
-1Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
'use strict';
22
var common = require('../common');
3-
var assert = require('assert');
43

54
var http = require('http');
65
var childProcess = require('child_process');
Collapse file

‎test/parallel/test-http-abort-client.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-http-abort-client.js
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ var server = http.Server(function(req, res) {
1414
var responseClose = false;
1515

1616
server.listen(common.PORT, function() {
17-
var client = http.get({
17+
http.get({
1818
port: common.PORT,
1919
headers: { connection: 'keep-alive' }
2020

Collapse file

‎test/parallel/test-http-after-connect.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-http-after-connect.js
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ server.listen(common.PORT, function() {
4545
});
4646

4747
function doRequest(i) {
48-
var req = http.get({
48+
http.get({
4949
port: common.PORT,
5050
path: '/request' + i
5151
}, function(res) {
Collapse file

‎test/parallel/test-http-agent-keepalive.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-http-agent-keepalive.js
-1Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ const common = require('../common');
33
const assert = require('assert');
44
const http = require('http');
55
const Agent = require('_http_agent').Agent;
6-
const EventEmitter = require('events').EventEmitter;
76

87
const agent = new Agent({
98
keepAlive: true,
Collapse file

‎test/parallel/test-http-agent-null.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-http-agent-null.js
-1Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
var common = require('../common');
33
var assert = require('assert');
44
var http = require('http');
5-
var net = require('net');
65

76
var request = 0;
87
var response = 0;
Collapse file

‎test/parallel/test-http-byteswritten.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-http-byteswritten.js
-1Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
'use strict';
22
var common = require('../common');
33
var assert = require('assert');
4-
var fs = require('fs');
54
var http = require('http');
65

76
var body = 'hello world\n';
Collapse file

‎test/parallel/test-http-client-abort2.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-http-client-abort2.js
-1Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
'use strict';
22
var common = require('../common');
3-
var assert = require('assert');
43
var http = require('http');
54

65
var server = http.createServer(function(req, res) {
Collapse file

‎test/parallel/test-http-client-encoding.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-http-client-encoding.js
-1Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
'use strict';
22
var common = require('../common');
3-
var assert = require('assert');
43

54
var http = require('http');
65

Collapse file

‎test/parallel/test-http-client-pipe-end.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-http-client-pipe-end.js
-1Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
// see https://github.com/joyent/node/issues/3257
33

44
var common = require('../common');
5-
var assert = require('assert');
65
var http = require('http');
76

87
var server = http.createServer(function(req, res) {

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.