Apply ruff/flake8-simplify rule SIM118

SIM118 Use `key in dict` instead of `key in dict.keys()`

Author: DimitriPapadopoulos

src/zarr/core/group.py

@@ -1225,7 +1225,7 @@ def _members_consolidated(
 
         # we kind of just want the top-level keys.
         if consolidated_metadata is not None:
-            for key in consolidated_metadata.metadata.keys():
+            for key in consolidated_metadata.metadata:
                 obj = self._getitem_consolidated(
                     self.store_path, key, prefix=self.name
                 )  # Metadata -> Group/Array

tests/test_api.py

@@ -154,7 +154,7 @@ def test_save(store: Store, n_args: int, n_kwargs: int) -> None:
         assert isinstance(group, Group)
         for array in group.array_values():
             assert_array_equal(array[:], data)
-        for k in kwargs.keys():
+        for k in kwargs:
             assert k in group
         assert group.nmembers() == n_args + n_kwargs
 

tests/test_indexing.py

@@ -1806,7 +1806,10 @@ async def test_accessed_chunks(
             # counts are consistent that this might have happened
             if optype == "__setitem__":
                 assert ("__getitem__", ci) not in delta_counts or delta_counts.pop(
-                    ("__getitem__", ci)
+                    (
+                        "__getitem__",
+                        ci,
+                    )
                 ) == 1
         # Check that no other chunks were accessed
         assert len(delta_counts) == 0
@@ -1893,7 +1896,10 @@ def test_iter_grid(
 
     # generate a numpy array of indices, and index it
     coord_array = np.array(list(itertools.product(*[range(s) for s in grid_shape]))).reshape(
-        (*grid_shape, ndim)
+        (
+            *grid_shape,
+            ndim,
+        )
     )
     coord_array_indexed = coord_array[
         tuple(slice(o, o + s, 1) for o, s in zip(origin, selection_shape, strict=False))