1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: update http test client function signatures

Browse Source 
remove unused head arguments

PR-URL: https://github.com/nodejs/node/pull/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>
This commit is contained in:
Jakub Mrowiec - Alkagar 2017-10-06 09:25:13 -07:00 committed by Rich Trott
parent b9f90fd0b4
commit b3ce408098
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
test/parallel/test-http-upgrade-client2.js
View File

@ -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;