Skip to content

Commit

Permalink
test: fix assert.strictEqual params order
Browse files Browse the repository at this point in the history
PR-URL: #23480
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>
  • Loading branch information
Rock Hu authored and jasnell committed Oct 17, 2018
1 parent ec7375a commit 6936f9c
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/sequential/test-pipe.js
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ const tcp = net.Server(common.mustCall((s) => {
s.on('data', (d) => {
tcpLengthSeen += d.length;
for (let j = 0; j < d.length; j++) {
assert.strictEqual(buffer[i], d[j]);
assert.strictEqual(d[j], buffer[i]);
i++;
}
});
Expand Down Expand Up @@ -103,7 +103,7 @@ function startClient() {
}, common.mustCall((res) => {
res.setEncoding('utf8');
res.on('data', common.mustCall((string) => {
assert.strictEqual('thanks', string);
assert.strictEqual(string, 'thanks');
gotThanks = true;
}));
}));
Expand All @@ -113,5 +113,5 @@ function startClient() {

process.on('exit', () => {
assert.ok(gotThanks);
assert.strictEqual(bufferSize, tcpLengthSeen);
assert.strictEqual(tcpLengthSeen, bufferSize);
});

0 comments on commit 6936f9c

Please sign in to comment.