fs: throw rm() validation errors

This commit updates validateRmOptions() to throw on input validation failures. This is consistent with how Node handles validation in most places across the codebase. PR-URL: #35602 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Richard Lau <[email protected]>
nodejs · Oct 14, 2020 · 939f8e8 · 939f8e8
1 parent cd0b136
commit 939f8e8
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/lib/internal/fs/utils.js b/lib/internal/fs/utils.js
@@ -673,12 +673,8 @@ const defaultRmdirOptions = {
 };
 
 const validateRmOptions = hideStackFrames((path, options, callback) => {
-  try {
-    options = validateRmdirOptions(options, defaultRmOptions);
-    validateBoolean(options.force, 'options.force');
-  } catch (err) {
-    return callback(err);
-  }
+  options = validateRmdirOptions(options, defaultRmOptions);
+  validateBoolean(options.force, 'options.force');
 
   lazyLoadFs().stat(path, (err, stats) => {
     if (err) {