Correct pre-commit issues

Author: joshmoore

src/zarr/core/common.py

@@ -170,7 +170,7 @@ def parse_bool(data: Any) -> bool:
 def parse_dtype(dtype: Any, zarr_format: ZarrFormat) -> np.dtype[Any]:
     if "datatype" in type(dtype).__name__.lower():
         # Workaround until we have the class in place
-        return dtype.to_numpy()  # TODO: call conversion method
+        return np.dtype(dtype.to_numpy())  # TODO: call conversion method
     if dtype is str or dtype == "str":
         if zarr_format == 2:
             # special case as object

src/zarr/core/metadata/v3.py

@@ -663,7 +663,6 @@ def to_numpy_shortname(self) -> str:
         }
         return data_type_to_numpy[self]
 
-
     def to_numpy(self) -> np.dtypes.StringDType | np.dtypes.ObjectDType | np.dtype[Any]:
         # note: it is not possible to round trip DataType <-> np.dtype
         # due to the fact that DataType.string and DataType.bytes both

src/zarr/registry.py

@@ -19,6 +19,7 @@
     )
     from zarr.core.buffer import Buffer, NDBuffer
     from zarr.core.common import JSON
+    from zarr.core.metadata.v3 import DataType
 
 __all__ = [
     "Registry",

tests/test_array.py

@@ -1447,8 +1447,7 @@ async def test_sharding_coordinate_selection() -> None:
 
 
 @pytest.mark.xfail(reason="zarr.common.core maps any string to the string type")
-async def test_array_v3_ext_uri_by_string(
-) -> None:
+async def test_array_v3_ext_uri_by_string() -> None:
     """
     Test the user impact of the extension mechanism
     """
@@ -1460,11 +1459,10 @@ async def test_array_v3_ext_uri_by_string(
         shape=(12,),
         chunks=(3,),
     )
-    assert arr.data_type == None  # TODO: import the example dtype
+    assert arr.dtype == np.dtype(bool)
 
 
-async def test_array_v3_ext_uri_by_class(
-) -> None:
+async def test_array_v3_ext_uri_by_class() -> None:
     """
     Test the user impact of the extension mechanism
     """

tests/test_metadata/test_v3.py

@@ -412,6 +412,7 @@ def test_dtypes(dtype_str: str) -> None:
         # return type for vlen types may vary depending on numpy version
         assert dt.byte_count is None
 
+
 async def test_ext_uri() -> None:
     metadata_dict = {
         "zarr_format": 3,
@@ -424,5 +425,4 @@ async def test_ext_uri() -> None:
         "fill_value": 0,
     }
     m = ArrayV3Metadata.from_dict(metadata_dict)
-    d = json.loads(m.to_buffer_dict(default_buffer_prototype())["zarr.json"].to_bytes())
     assert m.data_type is DataType.example