fs: handle UV_ENOTDIR in fs.statSync with throwIfNoEntry provided

Fixes: #56993
Signed-off-by: Juan José Arboleda <[email protected]>

Author: juanarbol

src/node_file.cc

@@ -1088,6 +1088,10 @@ constexpr bool is_uv_error_except_no_entry(int result) {
   return result < 0 && result != UV_ENOENT;
 }
 
+constexpr bool is_uv_error_except_no_entry_dir(int result) {
+  return result < 0 && !(result == UV_ENOENT || result == UV_ENOTDIR);
+}
+
 static void Stat(const FunctionCallbackInfo<Value>& args) {
   Realm* realm = Realm::GetCurrent(args);
   BindingData* binding_data = realm->GetBindingData<BindingData>();
@@ -1122,7 +1126,7 @@ static void Stat(const FunctionCallbackInfo<Value>& args) {
     int result;
     if (do_not_throw_if_no_entry) {
       result = SyncCallAndThrowIf(
-          is_uv_error_except_no_entry, env, &req_wrap_sync, uv_fs_stat, *path);
+          is_uv_error_except_no_entry_dir, env, &req_wrap_sync, uv_fs_stat, *path);
     } else {
       result = SyncCallAndThrowOnError(env, &req_wrap_sync, uv_fs_stat, *path);
     }

test/parallel/test-fs-stat.js

@@ -221,3 +221,8 @@ fs.lstat(__filename, undefined, common.mustCall());
     },
   );
 }
+
+{
+  // Test that the throwIfNoEntry option works and returns undefined
+  assert.ok(!(fs.statSync('./wont_exists', { throwIfNoEntry: false })));
+}