fix sync group constructors (#1652)

Author: jhamman

src/zarr/v3/codecs/bytes.py

@@ -29,7 +29,7 @@ class BytesCodecConfigurationMetadata:
 @frozen
 class BytesCodecMetadata:
     configuration: BytesCodecConfigurationMetadata
-    name: Literal["bytes"] = field(default="bytes", init=False)
+    name: Literal["bytes"] = field(default="bytes", init=True)
 
 
 @frozen

src/zarr/v3/group.py

@@ -12,16 +12,16 @@
 from zarr.v3.common import ZARR_JSON, ZARRAY_JSON, ZATTRS_JSON, ZGROUP_JSON, make_cattr
 from zarr.v3.config import RuntimeConfiguration, SyncConfiguration
 from zarr.v3.store import StoreLike, StorePath, make_store_path
-from zarr.v3.sync import SyncMixin
+from zarr.v3.sync import SyncMixin, sync
 
 logger = logging.getLogger("zarr.group")
 
 
 @frozen
 class GroupMetadata:
     attributes: Dict[str, Any] = field(factory=dict)
-    zarr_format: Literal[2, 3] = 3  # field(default=3, validator=validators.in_([2, 3]))
-    node_type: Literal["group"] = field(default="group", init=False)
+    zarr_format: Literal[2, 3] = 3
+    node_type: Literal["group"] = field(default="group", init=True)
 
     def to_bytes(self) -> Dict[str, bytes]:
         if self.zarr_format == 3:
@@ -52,7 +52,7 @@ async def create(
         *,
         attributes: Optional[Dict[str, Any]] = None,
         exists_ok: bool = False,
-        zarr_format: Literal[2, 3] = 3,  # field(default=3, validator=validators.in_([2, 3])),
+        zarr_format: Literal[2, 3] = 3,
         runtime_configuration: RuntimeConfiguration = RuntimeConfiguration(),
     ) -> AsyncGroup:
         store_path = make_store_path(store)
@@ -305,13 +305,14 @@ def create(
         exists_ok: bool = False,
         runtime_configuration: RuntimeConfiguration = RuntimeConfiguration(),
     ) -> Group:
-        obj = cls._sync(
+        obj = sync(
             AsyncGroup.create(
                 store,
                 attributes=attributes,
                 exists_ok=exists_ok,
                 runtime_configuration=runtime_configuration,
-            )
+            ),
+            loop=runtime_configuration.asyncio_loop,
         )
 
         return cls(obj)
@@ -322,7 +323,9 @@ def open(
         store: StoreLike,
         runtime_configuration: RuntimeConfiguration = RuntimeConfiguration(),
     ) -> Group:
-        obj = cls._sync(AsyncGroup.open(store, runtime_configuration))
+        obj = sync(
+            AsyncGroup.open(store, runtime_configuration), loop=runtime_configuration.asyncio_loop
+        )
         return cls(obj)
 
     def __getitem__(self, path: str) -> Union[Array, Group]:

tests/test_group_v3.py

@@ -54,3 +54,14 @@ def test_group(store_path) -> None:
     # and the attrs were modified in the store
     bar3 = foo["bar"]
     assert dict(bar3.attrs) == {"baz": "qux", "name": "bar"}
+
+
+def test_group_sync_constructor(store_path) -> None:
+
+    group = Group.create(
+        store=store_path,
+        attributes={"title": "test 123"},
+        runtime_configuration=RuntimeConfiguration(),
+    )
+
+    assert group._async_group.metadata.attributes["title"] == "test 123"