Commit ee27917
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: #10725
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Italo A. Casas <me@italoacasas.com>1 parent 9ac2316 commit ee27917
1 file changed
Lines changed: 3 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -32,7 +32,9 @@ r.on('readable', function() { | |||
| 32 | 32 | rs.length); | |
| 33 | 33 | }); | |
| 34 | 34 | ||
| 35 | - r.on('end', common.mustCall(function() {})); | ||
| 35 | + r.on('end', common.mustCall(function() { | ||
| 36 | + assert.strictEqual(pushes, PUSHCOUNT + 1); | ||
| 37 | + })); | ||
| 36 | 38 | ||
| 37 | 39 | var pushes = 0; | |
| 38 | 40 | function push() { | |
@@ -48,7 +50,3 @@ function push() { | |||
| 48 | 50 | if (r.push(new Buffer(PUSHSIZE))) | |
| 49 | 51 | setTimeout(push); | |
| 50 | 52 | } | |
| 51 | - | ||
| 52 | - process.on('exit', function() { | ||
| 53 | - assert.equal(pushes, PUSHCOUNT + 1); | ||
| 54 | - }); | ||
0 commit comments