From ea1fda62286615e2d153915f99377684c50f67f4 Mon Sep 17 00:00:00 2001 From: "davis.okoth@kemsa.co.ke" Date: Thu, 22 Mar 2018 22:04:38 +0300 Subject: [PATCH] test: remove third param from assert.strictEqual MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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: https://github.com/nodejs/node/pull/19536 Reviewed-By: Colin Ihrig Reviewed-By: Luigi Pinca Reviewed-By: Rich Trott Reviewed-By: Richard Lau Reviewed-By: James M Snell Reviewed-By: Trivikram Kamat Reviewed-By: Franziska Hinkelmann Reviewed-By: Tobias Nießen --- test/parallel/test-fs-mkdir-rmdir.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/parallel/test-fs-mkdir-rmdir.js b/test/parallel/test-fs-mkdir-rmdir.js index 865a5dba951555..6ab2bfcd866837 100644 --- a/test/parallel/test-fs-mkdir-rmdir.js +++ b/test/parallel/test-fs-mkdir-rmdir.js @@ -31,8 +31,8 @@ fs.mkdir(d, 0o666, common.mustCall(function(err) { fs.mkdir(d, 0o666, common.mustCall(function(err) { assert.ok(err, 'got no error'); assert.ok(/^EEXIST/.test(err.message), 'got no EEXIST message'); - assert.strictEqual(err.code, 'EEXIST', 'got no EEXIST code'); - assert.strictEqual(err.path, d, 'got no proper path for EEXIST'); + assert.strictEqual(err.code, 'EEXIST'); + assert.strictEqual(err.path, d); fs.rmdir(d, assert.ifError); }));