Don't early-capture the fs.lchownSync method

isaacs · isaacs · commit 190e311f56eb · 2019-09-24T22:48:49.000-07:00
This is breaking some tests where I clobber the fs method
to add tests or simulate failures.
diff --git a/chownr.js b/chownr.js
@@ -35,7 +35,7 @@ const handleEISDirSync =
       fs.chownSync(path, uid, gid)
     }
   }
-  : fs[LCHOWNSYNC]
+  : (path, uid, gid) => fs[LCHOWNSYNC](path, uid, gid)
 
 // fs.readdir could only accept an options object as of node v6
 const nodeVersion = process.version

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ const handleEISDirSync =`
`35`	`35`	`fs.chownSync(path, uid, gid)`
`36`	`36`	`}`
`37`	`37`	`}`
`38`		`- : fs[LCHOWNSYNC]`
	`38`	`+ : (path, uid, gid) => fs[LCHOWNSYNC](path, uid, gid)`
`39`	`39`
`40`	`40`	`// fs.readdir could only accept an options object as of node v6`
`41`	`41`	`const nodeVersion = process.version`