fs: don't double emit close on error

[ronag]

Readable destroy will emit 'close' on error. No reason to emit it before. Will just cause the error to be emitted twice.

Checklist

• make -j4 test (UNIX), or vcbuild test (Windows) passes
• tests and/or benchmarks are included
• documentation is changed or added
• commit message follows commit guidelines

[ronag]

Not sure how to make a test where fs.close fails?

[ZYSzys]

Logically speaking, we need to emit('close') if err and er are both null right ?

[ronag]

@ZYSzys yes? it will be emitted through the callback to destroy.

[Trott]

@nodejs/fs

[ronag]

fixed through #29048