fs: rename rimraf's maxBusyTries to maxRetries

cjihrig · addaleax · commit fa1f87b1998f · 2019-11-30T03:56:50.000+01:00
This is part of reworking the rimraf retry logic. Refs: #30580 PR-URL: #30644 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/doc/api/fs.md b/doc/api/fs.md
@@ -3220,6 +3220,9 @@ Synchronous rename(2). Returns `undefined`.
 <!-- YAML
 added: v0.0.2
 changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/30644
+    description: The `maxBusyTries` option is renamed to `maxRetries`.
   - version: v12.10.0
     pr-url: https://github.com/nodejs/node/pull/29168
     description: The `recursive`, `maxBusyTries`, and `emfileWait` options are
@@ -3246,7 +3249,7 @@ changes:
   retry the operation with a linear backoff of 1ms longer on each try until the
   timeout duration passes this limit. This option is ignored if the `recursive`
   option is not `true`. **Default:** `1000`.
-  * `maxBusyTries` {integer} If an `EBUSY`, `ENOTEMPTY`, or `EPERM` error is
+  * `maxRetries` {integer} If an `EBUSY`, `ENOTEMPTY`, or `EPERM` error is
   encountered, Node.js will retry the operation with a linear backoff wait of
   100ms longer on each try. This option represents the number of retries. This
   option is ignored if the `recursive` option is not `true`. **Default:** `3`.
@@ -3266,6 +3269,9 @@ Windows and an `ENOTDIR` error on POSIX.
 <!-- YAML
 added: v0.1.21
 changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/30644
+    description: The `maxBusyTries` option is renamed to `maxRetries`.
   - version: v12.10.0
     pr-url: https://github.com/nodejs/node/pull/29168
     description: The `recursive`, `maxBusyTries`, and `emfileWait` options are
@@ -4990,6 +4996,9 @@ upon success.
 <!-- YAML
 added: v10.0.0
 changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/30644
+    description: The `maxBusyTries` option is renamed to `maxRetries`.
   - version: v12.10.0
     pr-url: https://github.com/nodejs/node/pull/29168
     description: The `recursive`, `maxBusyTries`, and `emfileWait` options are
@@ -5004,7 +5013,7 @@ changes:
   retry the operation with a linear backoff of 1ms longer on each try until the
   timeout duration passes this limit. This option is ignored if the `recursive`
   option is not `true`. **Default:** `1000`.
-  * `maxBusyTries` {integer} If an `EBUSY`, `ENOTEMPTY`, or `EPERM` error is
+  * `maxRetries` {integer} If an `EBUSY`, `ENOTEMPTY`, or `EPERM` error is
   encountered, Node.js will retry the operation with a linear backoff wait of
   100ms longer on each try. This option represents the number of retries. This
   option is ignored if the `recursive` option is not `true`. **Default:** `3`.
diff --git a/lib/internal/fs/rimraf.js b/lib/internal/fs/rimraf.js
@@ -35,7 +35,7 @@ function rimraf(path, options, callback) {
   _rimraf(path, options, function CB(err) {
     if (err) {
       if ((err.code === 'EBUSY' || err.code === 'ENOTEMPTY' ||
-           err.code === 'EPERM') && busyTries < options.maxBusyTries) {
+           err.code === 'EPERM') && busyTries < options.maxRetries) {
         busyTries++;
         return setTimeout(_rimraf, busyTries * 100, path, options, CB);
       }
diff --git a/lib/internal/fs/utils.js b/lib/internal/fs/utils.js
@@ -563,7 +563,7 @@ function warnOnNonPortableTemplate(template) {
 
 const defaultRmdirOptions = {
   emfileWait: 1000,
-  maxBusyTries: 3,
+  maxRetries: 3,
   recursive: false,
 };
 
@@ -579,7 +579,7 @@ const validateRmdirOptions = hideStackFrames((options) => {
     throw new ERR_INVALID_ARG_TYPE('recursive', 'boolean', options.recursive);
 
   validateInt32(options.emfileWait, 'emfileWait', 0);
-  validateUint32(options.maxBusyTries, 'maxBusyTries');
+  validateUint32(options.maxRetries, 'maxRetries');
 
   return options;
 });
diff --git a/test/parallel/test-fs-rmdir-recursive.js b/test/parallel/test-fs-rmdir-recursive.js
@@ -156,23 +156,23 @@ function removeAsync(dir) {
 {
   const defaults = {
     emfileWait: 1000,
-    maxBusyTries: 3,
+    maxRetries: 3,
     recursive: false
   };
   const modified = {
     emfileWait: 953,
-    maxBusyTries: 5,
+    maxRetries: 5,
     recursive: true
   };
 
   assert.deepStrictEqual(validateRmdirOptions(), defaults);
   assert.deepStrictEqual(validateRmdirOptions({}), defaults);
   assert.deepStrictEqual(validateRmdirOptions(modified), modified);
   assert.deepStrictEqual(validateRmdirOptions({
-    maxBusyTries: 99
+    maxRetries: 99
   }), {
     emfileWait: 1000,
-    maxBusyTries: 99,
+    maxRetries: 99,
     recursive: false
   });
 
@@ -205,10 +205,10 @@ function removeAsync(dir) {
   });
 
   common.expectsError(() => {
-    validateRmdirOptions({ maxBusyTries: -1 });
+    validateRmdirOptions({ maxRetries: -1 });
   }, {
     code: 'ERR_OUT_OF_RANGE',
     type: RangeError,
-    message: /^The value of "maxBusyTries" is out of range\./
+    message: /^The value of "maxRetries" is out of range\./
   });
 }

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ function rimraf(path, options, callback) {`
`35`	`35`	`_rimraf(path, options, function CB(err) {`
`36`	`36`	`if (err) {`
`37`	`37`	`if ((err.code === 'EBUSY' \|\| err.code === 'ENOTEMPTY' \|\|`
`38`		`- err.code === 'EPERM') && busyTries < options.maxBusyTries) {`
	`38`	`+ err.code === 'EPERM') && busyTries < options.maxRetries) {`
`39`	`39`	`busyTries++;`
`40`	`40`	`return setTimeout(_rimraf, busyTries * 100, path, options, CB);`
`41`	`41`	`}`