BUG: Check complib values, #4582 and #8874

Author: Kevin Sheppard

doc/source/whatsnew/v0.16.2.txt

@@ -164,4 +164,8 @@ Bug Fixes
 - Bug in ``Panel.apply`` when the result has ndim = 0 (:issue:`10332`)
 - Bug in ``read_hdf`` where ``auto_close`` could not be passed (:issue:`9327`).
 - Bug in ``read_hdf`` where open stores could not be used (:issue:`10330`).
+
 - Bug in adding empty ``DataFrame``s, now results in a ``DataFrame`` that ``.equals`` an empty ``DataFrame`` (:issue:`10181`).
+
+
+- Bug in ``to_hdf`` and ``HDFStore`` which did not check that complib choices were valid (:issue:`4582`, :issue:`8874`).

pandas/io/pytables.py

@@ -389,6 +389,10 @@ def __init__(self, path, mode=None, complevel=None, complib=None,
         except ImportError as ex:  # pragma: no cover
             raise ImportError('HDFStore requires PyTables, "{ex}" problem importing'.format(ex=str(ex)))
 
+        if complib not in (None, 'blosc', 'bzip2', 'lzo', 'zlib'):
+            raise ValueError("complib only supports 'blosc', 'bzip2', lzo' "
+                             "or 'zlib' compression.")
+
         self._path = path
         if mode is None:
             mode = 'a'

pandas/io/tests/test_pytables.py

@@ -4718,6 +4718,13 @@ def test_read_hdf_errors(self):
             with open(path, mode='r') as store:
                 self.assertRaises(NotImplementedError, read_hdf, store, 'df')
 
+    def test_invalid_complib(self):
+        df = DataFrame(np.random.rand(4, 5),
+                       index=list('abcd'),
+                       columns=list('ABCDE'))
+        with ensure_clean_path(self.path) as path:
+            self.assertRaises(ValueError, df.to_hdf, path, 'df', complib='blosc:zlib')
+
 def _test_sort(obj):
     if isinstance(obj, DataFrame):
         return obj.reindex(sorted(obj.index))