Skip to content

Commit

Permalink
test: don't use arguments.callee
Browse files Browse the repository at this point in the history
Fix a strict mode violation that made the test fail to run.  It appears
to be a regression that was introduced in commit f29762f ("test: enable
linting for tests") which I was the sole reviewer of, so mea culpa.

PR-URL: #2022
Reviewed-By: Rod Vagg <[email protected]>
Reviewed-By: Trevor Norris <[email protected]>
  • Loading branch information
bnoordhuis authored and rvagg committed Aug 4, 2015
1 parent d11d40f commit 84cef46
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions test/pummel/test-http-many-keep-alive-connections.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,7 @@ server.once('connection', function(c) {
connection = c;
});

server.listen(common.PORT, function() {
var callee = arguments.callee;
server.listen(common.PORT, function connect() {
var request = http.get({
port: common.PORT,
path: '/',
Expand All @@ -30,7 +29,7 @@ server.listen(common.PORT, function() {
}, function(res) {
res.on('end', function() {
if (++responses < expected) {
callee();
connect();
} else {
server.close();
}
Expand Down

0 comments on commit 84cef46

Please sign in to comment.