Skip to content

Commit

Permalink
test: fix assertion argument order
Browse files Browse the repository at this point in the history
PR-URL: nodejs#23545
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
  • Loading branch information
carlrc authored and BridgeAR committed Oct 15, 2018
1 parent f7a50f7 commit 5856a59
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions test/parallel/test-stream-pipe-cleanup.js
Original file line number Diff line number Diff line change
Expand Up @@ -65,8 +65,8 @@ for (i = 0; i < limit; i++) {
r.pipe(w);
r.emit('end');
}
assert.strictEqual(0, r.listeners('end').length);
assert.strictEqual(limit, w.endCalls);
assert.strictEqual(r.listeners('end').length, 0);
assert.strictEqual(w.endCalls, limit);

w.endCalls = 0;

Expand All @@ -75,8 +75,8 @@ for (i = 0; i < limit; i++) {
r.pipe(w);
r.emit('close');
}
assert.strictEqual(0, r.listeners('close').length);
assert.strictEqual(limit, w.endCalls);
assert.strictEqual(r.listeners('close').length, 0);
assert.strictEqual(w.endCalls, limit);

w.endCalls = 0;

Expand All @@ -87,7 +87,7 @@ for (i = 0; i < limit; i++) {
r.pipe(w);
w.emit('close');
}
assert.strictEqual(0, w.listeners('close').length);
assert.strictEqual(w.listeners('close').length, 0);

r = new Readable();
w = new Writable();
Expand Down

0 comments on commit 5856a59

Please sign in to comment.