fix: handle errors returned by Close() in write paths

[wangxiaoxuan273]

What this PR does / why we need it:

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #1516

Please check the following list:

• Does the affected code have corresponding tests, e.g. unit test, E2E test?
• Does this change require a documentation update?
• Does this introduce breaking changes that would require an announcement or bumping the major version?
• Do all new files have an appropriate license header?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.29%. Comparing base (4b55afe) to head (3a60771).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1581      +/-   ##
==========================================
+ Coverage   84.23%   84.29%   +0.06%     
==========================================
  Files         120      120              
  Lines        5392     5401       +9     
==========================================
+ Hits         4542     4553      +11     
+ Misses        604      602       -2     
  Partials      246      246              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[TerryHowe]

/lgtm

Although it would read nicer if there wasn't a swtich from fileCloseErr to closeErr

[shizhMSFT]

LGTM

[qweeah]

LGTM