add test for creation under a parent array

Author: K-Meech

src/zarr/core/metadata/io.py

@@ -87,7 +87,10 @@ async def save_metadata(
         try:
             await asyncio.gather(*ensure_array_awaitables)
         except ContainsArrayError as e:
-            set_awaitables = []  # clear awaitables to avoid printed RuntimeWarning: coroutine was never awaited
+            # clear awaitables to avoid RuntimeWarning: coroutine was never awaited
+            for awaitable in set_awaitables:
+                awaitable.close()
+
             raise ValueError(
                 f"A parent of {store_path} is an array - only groups may have child nodes."
             ) from e

tests/test_group.py

@@ -761,6 +761,24 @@ def test_group_create_array(
     assert np.array_equal(array[:], data)
 
 
+@pytest.mark.parametrize("method", ["create_array", "create_group"])
+def test_create_with_parent_array(store: Store, zarr_format: ZarrFormat, method: str):
+    """Test that groups/arrays cannot be created under a parent array."""
+
+    # create a group with a child array
+    group = Group.from_store(store, zarr_format=zarr_format)
+    group.create_array(name="arr_1", shape=(10, 10), dtype="uint8")
+
+    error_msg = r"A parent of .* is an array - only groups may have child nodes."
+    if method == "create_array":
+        with pytest.raises(ValueError, match=error_msg):
+            group.create_array("arr_1/group_1/group_2/arr_2", shape=(10, 10), dtype="uint8")
+
+    else:
+        with pytest.raises(ValueError, match=error_msg):
+            group.create_group("arr_1/group_1/group_2/group_3")
+
+
 def test_group_array_creation(
     store: Store,
     zarr_format: ZarrFormat,