test: check result as early as possible

PR-URL: #2007 Reviewed-By: Rod Vagg <rod@vagg.org>
nodejs · Jun 20, 2015 · 3ba4f71 · 3ba4f71
1 parent 8ac5081
commit 3ba4f71
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/test/parallel/test-http-get-pipeline-problem.js b/test/parallel/test-http-get-pipeline-problem.js
@@ -47,12 +47,10 @@ server.listen(common.PORT, function() {
         var s = fs.createWriteStream(common.tmpDir + '/' + x + '.jpg');
         res.pipe(s);
 
-        // TODO there should be a callback to pipe() that will allow
-        // us to get a callback when the pipe is finished.
-        res.on('end', function() {
+        s.on('finish', function() {
           console.error('done ' + x);
           if (++responses == total) {
-            s.on('close', checkFiles);
+            checkFiles();
           }
         });
       }).on('error', function(e) {