Skip to content

Commit 8996d3c

Browse files
davis.okoth@kemsa.co.ketargos
authored andcommitted
test: remove third param from assert.strictEqual
Removing third argument in calls to assert.strictEqual() so that the values of the first two arguments are shown instead as this is more useful for debugging. Refs: https://nodejs.org/api/assert.html#assert_assert_strictequal_actual_expected_message PR-URL: #19536 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Franziska Hinkelmann <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
1 parent 5e9f929 commit 8996d3c

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

test/parallel/test-fs-mkdir-rmdir.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,8 @@ fs.mkdir(d, 0o666, common.mustCall(function(err) {
3131
fs.mkdir(d, 0o666, common.mustCall(function(err) {
3232
assert.ok(err, 'got no error');
3333
assert.ok(/^EEXIST/.test(err.message), 'got no EEXIST message');
34-
assert.strictEqual(err.code, 'EEXIST', 'got no EEXIST code');
35-
assert.strictEqual(err.path, d, 'got no proper path for EEXIST');
34+
assert.strictEqual(err.code, 'EEXIST');
35+
assert.strictEqual(err.path, d);
3636

3737
fs.rmdir(d, assert.ifError);
3838
}));

0 commit comments

Comments
 (0)