Permalink
Browse files

test: update http test client function signatures

remove unused head arguments

PR-URL: #15807
Reviewed-By: Ryan Graham <r.m.graham@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information...
Alkagar authored and targos committed Oct 6, 2017
1 parent 698f358 commit cbbb229c81df823145046b1855129ff9706667d3
Showing with 2 additions and 2 deletions.
  1. +2 −2 test/parallel/test-http-upgrade-client2.js
@@ -26,7 +26,7 @@ const http = require('http');
const CRLF = '\r\n';
const server = http.createServer();
server.on('upgrade', function(req, socket, head) {
server.on('upgrade', function(req, socket) {
socket.write(`HTTP/1.1 101 Ok${CRLF}` +
`Connection: Upgrade${CRLF}` +
`Upgrade: Test${CRLF}${CRLF}` +
@@ -47,7 +47,7 @@ server.listen(0, common.mustCall(function() {
});
let wasUpgrade = false;
function onUpgrade(res, socket, head) {
function onUpgrade(res, socket) {
console.log('client upgraded');
wasUpgrade = true;

0 comments on commit cbbb229

Please sign in to comment.