Skip to content

Commit

Permalink
test: swap the order of arguments
Browse files Browse the repository at this point in the history
Swapped the actual and expected arguments in `assert.strictEqual()`
calls.  Arguments are now in correct order.
Literal value is now the second argument and the value returned by the
function is the first argument.

PR-URL: nodejs#24134
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: George Adams <george.adams@uk.ibm.com>
  • Loading branch information
mhamwala authored and Trott committed Nov 8, 2018
1 parent f884a57 commit 2535aa5
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/parallel/test-fs-write-sync.js
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ tmpdir.refresh();
const fd = fs.openSync(filename, 'w');

let written = fs.writeSync(fd, '');
assert.strictEqual(0, written);
assert.strictEqual(written, 0);

fs.writeSync(fd, 'foo');

Expand All @@ -50,7 +50,7 @@ tmpdir.refresh();
const fd = fs.openSync(filename, 'w');

let written = fs.writeSync(fd, '');
assert.strictEqual(0, written);
assert.strictEqual(written, 0);

fs.writeSync(fd, 'foo');

Expand All @@ -66,7 +66,7 @@ tmpdir.refresh();
const fd = fs.openSync(filename, 'w');

let written = fs.writeSync(fd, '');
assert.strictEqual(0, written);
assert.strictEqual(written, 0);

fs.writeSync(fd, 'foo');

Expand Down

0 comments on commit 2535aa5

Please sign in to comment.