diff options
author | stefan judis <stefanjudis@gmail.com> | 2017-01-10 17:38:48 +0100 |
---|---|---|
committer | Italo A. Casas <me@italoacasas.com> | 2017-01-16 17:08:24 -0500 |
commit | c685638986347557b70f9a1d3437508282012ba3 (patch) | |
tree | 56ba82dd519f4b57f224548871273c772045bf57 /test/parallel/test-stream2-large-read-stall.js | |
parent | c963094f3cf3d42af5255f1a340cb97884c641d8 (diff) | |
download | node-new-c685638986347557b70f9a1d3437508282012ba3.tar.gz |
test: improve test-stream2-large-read-stall
* use const instead of var
* use assert.strictEqual instead of assert.equal
* use common.mustCall instead of process.on( 'exit', fn )
PR-URL: https://github.com/nodejs/node/pull/10725
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Italo A. Casas <me@italoacasas.com>
Diffstat (limited to 'test/parallel/test-stream2-large-read-stall.js')
-rw-r--r-- | test/parallel/test-stream2-large-read-stall.js | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/test/parallel/test-stream2-large-read-stall.js b/test/parallel/test-stream2-large-read-stall.js index 36d4e0c6c9..2422747f82 100644 --- a/test/parallel/test-stream2-large-read-stall.js +++ b/test/parallel/test-stream2-large-read-stall.js @@ -33,7 +33,9 @@ r.on('readable', function() { rs.length); }); -r.on('end', common.mustCall(function() {})); +r.on('end', common.mustCall(function() { + assert.strictEqual(pushes, PUSHCOUNT + 1); +})); let pushes = 0; function push() { @@ -49,7 +51,3 @@ function push() { if (r.push(Buffer.allocUnsafe(PUSHSIZE))) setTimeout(push, 1); } - -process.on('exit', function() { - assert.strictEqual(pushes, PUSHCOUNT + 1); -}); |