Skip to content

Commit

Permalink
test: update http test client function signatures
Browse files Browse the repository at this point in the history
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 MylesBorins committed Nov 28, 2017
1 parent 1064258 commit e3e234e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/parallel/test-http-upgrade-client2.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,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}` +
Expand All @@ -26,7 +26,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;

Expand Down

0 comments on commit e3e234e

Please sign in to comment.