test: don't use arguments.callee
authorBen Noordhuis <info@bnoordhuis.nl>
Fri, 19 Jun 2015 18:45:30 +0000 (20:45 +0200)
committerRod Vagg <rod@vagg.org>
Tue, 4 Aug 2015 18:56:14 +0000 (11:56 -0700)
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: https://github.com/nodejs/io.js/pull/2022
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
test/pummel/test-http-many-keep-alive-connections.js

index d90d4d9..293c6ab 100644 (file)
@@ -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: '/',
@@ -30,7 +29,7 @@ server.listen(common.PORT, function() {
   }, function(res) {
     res.on('end', function() {
       if (++responses < expected) {
-        callee();
+        connect();
       } else {
         server.close();
       }