Skip to content

Commit

Permalink
chore: node4 compatible syntax in test
Browse files Browse the repository at this point in the history
  • Loading branch information
Anton Drukh authored and michael-go committed Apr 30, 2018
1 parent 4af5792 commit f5c99b2
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

12 changes: 6 additions & 6 deletions test/proxy.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -76,10 +76,10 @@ test('request respects proxy environment variables', function (t) {
var proxy = http.createServer();
proxy.setTimeout(1000);
proxy.on('connect', (req, cltSocket, head) => {
const {hostname, port} = url.parse(`https://${req.url}`);
t.equal(hostname, url.parse(httpsRequestHost).hostname,
const proxiedUrl = url.parse(`https://${req.url}`);
t.equal(proxiedUrl.hostname, url.parse(httpsRequestHost).hostname,
'https_proxy url hostname ok');
t.equal(port, url.parse(httpsRequestHost).port,
t.equal(proxiedUrl.port, url.parse(httpsRequestHost).port,
'https_proxy url port ok');
cltSocket.write('HTTP/1.1 200 Connection Established\r\n' +
'Proxy-agent: Node.js-Proxy\r\n' +
Expand All @@ -102,10 +102,10 @@ test('request respects proxy environment variables', function (t) {
var proxy = http.createServer();
proxy.setTimeout(1000);
proxy.on('connect', (req, cltSocket, head) => {
const {hostname, port} = url.parse(`https://${req.url}`);
t.equal(hostname, url.parse(httpsRequestHost).hostname,
const proxiedUrl = url.parse(`https://${req.url}`);
t.equal(proxiedUrl.hostname, url.parse(httpsRequestHost).hostname,
'HTTPS_PROXY url hostname ok');
t.equal(port, url.parse(httpsRequestHost).port,
t.equal(proxiedUrl.port, url.parse(httpsRequestHost).port,
'HTTPS_PROXY url port ok');
cltSocket.write('HTTP/1.1 200 Connection Established\r\n' +
'Proxy-agent: Node.js-Proxy\r\n' +
Expand Down

0 comments on commit f5c99b2

Please sign in to comment.