read_hdf closes HDF5 stores that it didn't open.

[kboone]

• closes read_hdf closes HDF5 stores that it didn't open. #28699
• tests added / passed
• passes black pandas
• passes git diff upstream/master -u -- "*.py" | flake8 --diff
• whatsnew entry

This commit fixes #28699. I added a test for the bug.

[kboone]

I'm not entirely sure why these checks failed. It looks like there is some major reorganization of test_pytables.py happening right now?

[jbrockmendel]

Can you rebase?

I'm not aware of a major reorganization of the file, but there was a recent change that removed an except Exception: that may be causing the merge conflict.

[kboone]

Looks like how paths are created was changed in test_pytables.py yesterday which created some conflicts. I updated my test to the new format and rebased.

[kboone]

Got it sorted out. The previous error was due to me using single quotes somewhere rather than double quotes, and black pandas didn't like that.

[jreback]

lgtm. can you add a whatsnew note in bugfix of 1.0.0 for IO section. note that I am about to merge a refactor of the pytables tests, so will have to rebase on master (but shouldn't be a conflict i think).

[kboone]

I updated the whatsnew for 1.0.0. Let me know if I need to change anything for the refactor of the pytables tests.

[jreback]

can you merge master, we moved some tests around

[kboone]

I merged master and resolved the conflicts.

[WillAyd]

Thanks @kboone !