fixup! fs: remove permissive rmdir recursive

nodejs · Mar 20, 2021 · c0bd783 · c0bd783
1 parent 8e1e85e
commit c0bd783
Showing 1 changed file with 6 additions and 7 deletions.
diff --git a/test/parallel/test-fs-rmdir-recursive-throws-on-file.js b/test/parallel/test-fs-rmdir-recursive-throws-on-file.js
@@ -7,24 +7,23 @@ const path = require('path');
 
 tmpdir.refresh();
 
+const code = common.isWindows ? 'ENOENT' : 'ENOTDIR';
+
 {
   const filePath = path.join(tmpdir.path, 'rmdir-recursive.txt');
   fs.writeFileSync(filePath, '');
-  assert.throws(() => fs.rmdirSync(filePath, { recursive: true }), {
-    code: 'ENOTDIR',
-  });
+  assert.throws(() => fs.rmdirSync(filePath, { recursive: true }), { code });
 }
 {
   const filePath = path.join(tmpdir.path, 'rmdir-recursive.txt');
   fs.writeFileSync(filePath, '');
   fs.rmdir(filePath, { recursive: true }, common.mustCall((err) => {
-    assert.strictEqual(err.code, 'ENOTDIR');
+    assert.strictEqual(err.code, code);
   }));
 }
 {
   const filePath = path.join(tmpdir.path, 'rmdir-recursive.txt');
   fs.writeFileSync(filePath, '');
-  assert.rejects(() => fs.promises.rmdir(filePath, { recursive: true }), {
-    code: 'ENOTDIR',
-  }).then(common.mustCall());
+  assert.rejects(() => fs.promises.rmdir(filePath, { recursive: true }),
+                 { code }).then(common.mustCall());
 }