test: use`mustSucceed` instead of `mustCall` with `assert.ifError`

PR-URL: https://github.com/nodejs/node/pull/42806
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>

1 files changed, 2 insertions, 4 deletions

diff --git a/test/parallel/test-fs-opendir.js b/test/parallel/test-fs-opendir.js
index 1a4aded61dc..dd1724aa3c2 100644
--- a/test/parallel/test-fs-opendir.js
+++ b/test/parallel/test-fs-opendir.js
@@ -67,18 +67,16 @@ const invalidCallbackObj = {
 // Check the opendir async version
 fs.opendir(testDir, common.mustSucceed((dir) => {
   let sync = true;
-  dir.read(common.mustCall((err, dirent) => {
+  dir.read(common.mustSucceed((dirent) => {
     assert(!sync);
-    assert.ifError(err);
 
     // Order is operating / file system dependent
     assert(files.includes(dirent.name), `'files' should include ${dirent}`);
     assertDirent(dirent);
 
     let syncInner = true;
-    dir.read(common.mustCall((err, dirent) => {
+    dir.read(common.mustSucceed((dirent) => {
       assert(!syncInner);
-      assert.ifError(err);
 
       dir.close(common.mustSucceed());
     }));