API: Raise AttributeError on closed HDFStore

Previously, we called _check_if_open, which would raise a ClosedFileError
whenever the desired attribute wasn't found. This prevented the check required
for PEP519 to work properly, since hasattr shouldn't raise that error.

Author: TomAugspurger

doc/source/whatsnew/v0.21.0.txt

@@ -36,7 +36,8 @@ Other Enhancements
 Backwards incompatible API changes
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
-
+- Accessing a non-existent attribute on a closed :class:`HDFStore` will now
+raise an ``AttributeError`` rather than a ``ClosedFileError`` (:issue:`16301`)
 
 .. _whatsnew_0210.api:
 

pandas/io/pytables.py

@@ -466,7 +466,6 @@ def __delitem__(self, key):
 
     def __getattr__(self, name):
         """ allow attribute access to get stores """
-        self._check_if_open()
         try:
             return self.get(name)
         except:

pandas/tests/io/test_pytables.py

@@ -4377,7 +4377,7 @@ def f():
             pytest.raises(ClosedFileError, lambda: 'df' in store)
             pytest.raises(ClosedFileError, lambda: len(store))
             pytest.raises(ClosedFileError, lambda: store['df'])
-            pytest.raises(ClosedFileError, lambda: store.df)
+            pytest.raises(AttributeError, lambda: store.df)
             pytest.raises(ClosedFileError, store.select, 'df')
             pytest.raises(ClosedFileError, store.get, 'df')
             pytest.raises(ClosedFileError, store.append, 'df2', df)