Replaced open with Context Mgrs in Parser Tests

[WillAyd]

Maybe closes #21102 and #19984, though if not should still work as a general cleanup

[alysivji]

Just did a search and there are a surprising number of non-context manager opens in the codebase. Is there a cleanup effort around this? Fix as you see them?

[codecov]

Codecov Report

Merging #21105 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #21105   +/-   ##
=======================================
  Coverage   91.83%   91.83%           
=======================================
  Files         153      153           
  Lines       49497    49497           
=======================================
  Hits        45456    45456           
  Misses       4041     4041

Flag	Coverage Δ
#multiple	90.23% <ø> (ø)	⬆️
#single	41.88% <ø> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d623ffd...e61612a. Read the comment docs.

[WillAyd]

@alysivji are those using IO buffers or file handles? If the former I don't think it's a necessity. The latter should definitely be cleaned up

[jreback]

thanks @WillAyd

if you can see if #21102 and #19984 are fixed would be great.

[WillAyd]

Is that validated by restarting the failed jobs associated with those?

[jreback]

oh those were just transient issues
u can restart older jobs but not much point as you generally need to rebase anyhow

[WillAyd]

OK yea figured that was the case. I'm reading this as "wait and see if this issue pops back up again" so doing nothing for now, but if I'm missing something let me know

[jreback]

yep - nothing to do now