fs: use encoding in readFile

Use the encoding parameter passed to fsPromises.readFile if it is
passed. Currently the encoding parameter is ignored in fsPromises.

PR-URL: #19296
Fixes: #19286
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Gus Caplan <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>

lib/fs/promises.js

@@ -157,7 +157,12 @@ async function readFileHandle(filehandle, options) {
       chunks.push(buffer.slice(0, totalRead));
   } while (totalRead === chunkSize);
 
-  return Buffer.concat(chunks);
+  const result = Buffer.concat(chunks);
+  if (options.encoding) {
+    return result.toString(options.encoding);
+  } else {
+    return result;
+  }
 }
 
 // All of the functions are defined as async in order to ensure that errors

test/parallel/test-fs-promises-writefile.js

@@ -35,7 +35,15 @@ async function doRead() {
   assert.deepStrictEqual(buf, data);
 }
 
+async function doReadWithEncoding() {
+  const data = await fsPromises.readFile(dest, 'utf-8');
+  const syncData = fs.readFileSync(dest, 'utf-8');
+  assert.strictEqual(typeof data, 'string');
+  assert.deepStrictEqual(data, syncData);
+}
+
 doWrite()
   .then(doAppend)
   .then(doRead)
+  .then(doReadWithEncoding)
   .then(common.mustCall());