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 <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
  • Loading branch information
Alkagar authored and targos committed Oct 18, 2017
1 parent 698f358 commit cbbb229
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 @@ -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}` +
Expand All @@ -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;

Expand Down

0 comments on commit cbbb229

Please sign in to comment.