test coverage for h5py compat; fix Group docstring

Author: alimanfoo

docs/api/hierarchy.rst

@@ -8,13 +8,9 @@ Groups (``zarr.hierarchy``)
 .. autoclass:: Group
 
     .. automethod:: __len__
+    .. automethod:: __iter__
     .. automethod:: __contains__
     .. automethod:: __getitem__
-    .. automethod:: __setitem__
-    .. automethod:: __iter__
-    .. automethod:: keys
-    .. automethod:: values
-    .. automethod:: items
     .. automethod:: group_keys
     .. automethod:: groups
     .. automethod:: array_keys

zarr/hierarchy.py

@@ -41,13 +41,9 @@ class Group(Mapping):
     Methods
     -------
     __len__
+    __iter__
     __contains__
     __getitem__
-    __setitem__
-    __iter__
-    keys
-    values
-    items
     group_keys
     groups
     array_keys
@@ -509,7 +505,7 @@ def create_dataset(self, name, data=None, shape=None, chunks=None,
         # N.B., additional kwargs are included in method signature to
         # improve compatibility for users familiar with h5py and adapting
         # code that previously used h5py. These keyword arguments are
-        # ignored here but we raise a warning to let the user know.
+        # ignored here but we issue a warning to let the user know.
         for k in kwargs:
             if k == 'fillvalue':
                 warn("ignoring keyword argument %r; please use 'fill_value' "

zarr/tests/test_hierarchy.py

@@ -285,6 +285,12 @@ def test_create_errors(self):
         with assert_raises(KeyError):
             g.require_dataset('c/d', shape=100, chunks=10)
 
+        # h5py compatibility - accept but ingore some keyword args
+        d = g.create_dataset('x', shape=100, chunks=10, fillvalue=1)
+        assert_is_none(d.fill_value)
+        d = g.create_dataset('y', shape=100, chunks=10, shuffle=True)
+        assert not hasattr(d, 'shuffle')
+
         # read-only
         g = Group(store=store, readonly=True)
         with assert_raises(ReadOnlyError):