test: dispose of filehandles in filehandle.read tests

PR-URL: #58543 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: LiviaMedeiros

test/parallel/test-fs-read-empty-buffer.js

@@ -29,7 +29,7 @@ assert.throws(
 );
 
 (async()=>{
-constfilehandle=awaitfsPromises.open(filepath,'r');
+awaitusingfilehandle=awaitfsPromises.open(filepath,'r');
 assert.rejects(
 ()=>filehandle.read(buffer,0,1,0),
 {

test/parallel/test-fs-read-offset-null.js

@@ -35,30 +35,25 @@ fs.open(filepath, 'r', common.mustSucceed((fd) =>{
 }));
 }));
 
-letfilehandle=null;
-
 // Tests for promises api
 (async()=>{
-filehandle=awaitfsPromises.open(filepath,'r');
+awaitusingfilehandle=awaitfsPromises.open(filepath,'r');
 constreadObject=awaitfilehandle.read(buf,{offset: null});
 assert.strictEqual(readObject.buffer[0],120);
 })()
-.finally(()=>filehandle?.close())
 .then(common.mustCall());
 
 // Undocumented: omitted position works the same as position === null
 (async()=>{
-filehandle=awaitfsPromises.open(filepath,'r');
+awaitusingfilehandle=awaitfsPromises.open(filepath,'r');
 constreadObject=awaitfilehandle.read(buf,null,buf.length);
 assert.strictEqual(readObject.buffer[0],120);
 })()
-.finally(()=>filehandle?.close())
 .then(common.mustCall());
 
 (async()=>{
-filehandle=awaitfsPromises.open(filepath,'r');
+awaitusingfilehandle=awaitfsPromises.open(filepath,'r');
 constreadObject=awaitfilehandle.read(buf,null,buf.length,0);
 assert.strictEqual(readObject.buffer[0],120);
 })()
-.finally(()=>filehandle?.close())
 .then(common.mustCall());