diff --git a/test/simple/test-http-upgrade-client.js b/test/simple/test-http-upgrade-client.js index b09cc58a27..be6c386ef2 100644 --- a/test/simple/test-http-upgrade-client.js +++ b/test/simple/test-http-upgrade-client.js @@ -26,28 +26,31 @@ var srv = net.createServer(function(c) { c.end(); }); }); -srv.listen(common.PORT, '127.0.0.1'); var gotUpgrade = false; -var hc = http.createClient(common.PORT, '127.0.0.1'); -hc.addListener('upgrade', function(res, socket, upgradeHead) { - // XXX: This test isn't fantastic, as it assumes that the entire response - // from the server will arrive in a single data callback - assert.equal(upgradeHead, 'nurtzo'); - - console.log(res.headers); - var expectedHeaders = { "hello": "world" - , "connection": "upgrade" - , "upgrade": "websocket" - }; - assert.deepEqual(expectedHeaders, res.headers); - - socket.end(); - srv.close(); - - gotUpgrade = true; + +srv.listen(common.PORT, '127.0.0.1', function () { + + var hc = http.createClient(common.PORT, '127.0.0.1'); + hc.addListener('upgrade', function(res, socket, upgradeHead) { + // XXX: This test isn't fantastic, as it assumes that the entire response + // from the server will arrive in a single data callback + assert.equal(upgradeHead, 'nurtzo'); + + console.log(res.headers); + var expectedHeaders = { "hello": "world" + , "connection": "upgrade" + , "upgrade": "websocket" + }; + assert.deepEqual(expectedHeaders, res.headers); + + socket.end(); + srv.close(); + + gotUpgrade = true; + }); + hc.request('GET', '/').end(); }); -hc.request('GET', '/').end(); process.addListener('exit', function() { assert.ok(gotUpgrade);