fs: make mutating `options` in Promises `readdir()` not affect results · nodejs/node@92bcd52

Original file line number	Diff line number	Diff line change
`@@ -943,6 +943,10 @@ async function readdirRecursive(originalPath, options) {`
`943`	`943`
`944`	`944`	`async function readdir(path, options) {`
`945`	`945`	`options = getOptions(options);`
	`946`	`+`
	`947`	`+// Make shallow copy to prevent mutating options from affecting results`
	`948`	`+options = copyObject(options);`
	`949`	`+`
`946`	`950`	`path = getValidatedPath(path);`
`947`	`951`	`if (options.recursive) {`
`948`	`952`	`return readdirRecursive(path, options);`

Original file line number	Diff line number	Diff line change
`@@ -78,6 +78,14 @@ fs.readdir(readdirDir, {`
`78`	`78`	`assertDirents(dirents);`
`79`	`79`	`})().then(common.mustCall());`
`80`	`80`
	`81`	`+// Check that mutating options doesn't affect results`
	`82`	`+(async () => {`
	`83`	`+const options = { withFileTypes: true };`
	`84`	`+const direntsPromise = fs.promises.readdir(readdirDir, options);`
	`85`	`+options.withFileTypes = false;`
	`86`	`+assertDirents(await direntsPromise);`
	`87`	`+})().then(common.mustCall());`
	`88`	`+`
`81`	`89`	`// Check for correct types when the binding returns unknowns`
`82`	`90`	`const UNKNOWN = constants.UV_DIRENT_UNKNOWN;`
`83`	`91`	`const oldReaddir = binding.readdir;`