[std/node] add the ability to path argument to be URL type (#5055)

1 files changed, 8 insertions, 4 deletions

diff --git a/std/node/_fs/_fs_chmod.ts b/std/node/_fs/_fs_chmod.ts
index 2adff59ff..b1079e0a1 100644
--- a/std/node/_fs/_fs_chmod.ts
+++ b/std/node/_fs/_fs_chmod.ts
@@ -1,28 +1,32 @@
 // Copyright 2018-2020 the Deno authors. All rights reserved. MIT license.
 
 import { CallbackWithError } from "./_fs_common.ts";
+import { fromFileUrl } from "../path.ts";
 
 const allowedModes = /^[0-7]{3}/;
 
 /**
- * TODO: Also accept 'path' parameter as a Node polyfill Buffer or URL type once these
+ * TODO: Also accept 'path' parameter as a Node polyfill Buffer type once these
  * are implemented. See https://github.com/denoland/deno/issues/3403
  */
 export function chmod(
-  path: string,
+  path: string | URL,
   mode: string | number,
   callback: CallbackWithError
 ): void {
+  path = path instanceof URL ? fromFileUrl(path) : path;
+
   Deno.chmod(path, getResolvedMode(mode))
     .then(() => callback())
     .catch(callback);
 }
 
 /**
- * TODO: Also accept 'path' parameter as a Node polyfill Buffer or URL type once these
+ * TODO: Also accept 'path' parameter as a Node polyfill Buffer type once these
  * are implemented. See https://github.com/denoland/deno/issues/3403
  */
-export function chmodSync(path: string, mode: string | number): void {
+export function chmodSync(path: string | URL, mode: string | number): void {
+  path = path instanceof URL ? fromFileUrl(path) : path;
   Deno.chmodSync(path, getResolvedMode(mode));
 }