Skip to content

Commit 684eb32

Browse files
TrottFishrock123
authored andcommitted
test: remove unused variables from HTTPS tests
PR-URL: #4426 Reviewed-By: Johan Bergström <[email protected]>
1 parent 585c01f commit 684eb32

11 files changed

+3
-13
lines changed

test/parallel/test-https-agent-disable-session-reuse.js

-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ if (!common.hasCrypto) {
1010
const TOTAL_REQS = 2;
1111

1212
const https = require('https');
13-
const crypto = require('crypto');
1413

1514
const fs = require('fs');
1615

test/parallel/test-https-agent-servername.js

-1
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
if (!common.hasCrypto) {
65
console.log('1..0 # Skipped: missing crypto');

test/parallel/test-https-byteswritten.js

-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
var common = require('../common');
33
var assert = require('assert');
44
var fs = require('fs');
5-
var http = require('http');
65

76
if (!common.hasCrypto) {
87
console.log('1..0 # Skipped: missing crypto');

test/parallel/test-https-eof-for-eom.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ var bodyBuffer = '';
5353

5454
server.listen(common.PORT, function() {
5555
console.log('1) Making Request');
56-
var req = https.get({
56+
https.get({
5757
port: common.PORT,
5858
rejectUnauthorized: false
5959
}, function(res) {

test/parallel/test-https-localaddress-bind-error.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ var server = https.createServer(options, function(req, res) {
2828
});
2929

3030
server.listen(common.PORT, '127.0.0.1', function() {
31-
var req = https.request({
31+
https.request({
3232
host: 'localhost',
3333
port: common.PORT,
3434
path: '/',

test/parallel/test-https-socket-options.js

-2
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
if (!common.hasCrypto) {
65
console.log('1..0 # Skipped: missing crypto');
@@ -9,7 +8,6 @@ if (!common.hasCrypto) {
98
var https = require('https');
109

1110
var fs = require('fs');
12-
var exec = require('child_process').exec;
1311

1412
var http = require('http');
1513

test/parallel/test-https-strict.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ function makeReq(path, port, error, host, ca) {
110110
path: path,
111111
ca: ca
112112
};
113-
var whichCa = 0;
113+
114114
if (!ca) {
115115
options.agent = agent0;
116116
} else {

test/parallel/test-https-timeout-server-2.js

-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ if (!common.hasCrypto) {
99
}
1010
var https = require('https');
1111

12-
var net = require('net');
1312
var tls = require('tls');
1413
var fs = require('fs');
1514

test/parallel/test-https-timeout-server.js

-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ if (!common.hasCrypto) {
99
var https = require('https');
1010

1111
var net = require('net');
12-
var tls = require('tls');
1312
var fs = require('fs');
1413

1514
var clientErrors = 0;

test/parallel/test-https-timeout.js

-2
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
if (!common.hasCrypto) {
65
console.log('1..0 # Skipped: missing crypto');
@@ -9,7 +8,6 @@ if (!common.hasCrypto) {
98
var https = require('https');
109

1110
var fs = require('fs');
12-
var exec = require('child_process').exec;
1311

1412
var options = {
1513
key: fs.readFileSync(common.fixturesDir + '/keys/agent1-key.pem'),

test/parallel/test-https-truncate.js

-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ if (!common.hasCrypto) {
99
var https = require('https');
1010

1111
var fs = require('fs');
12-
var path = require('path');
1312

1413
var key = fs.readFileSync(common.fixturesDir + '/keys/agent1-key.pem');
1514
var cert = fs.readFileSync(common.fixturesDir + '/keys/agent1-cert.pem');

0 commit comments

Comments
 (0)