chore: remove try_from_json method from protocol and arrayv3metadata

We couldn't agree on a name for `try_from_json`, and it's not an essential method, so I'm removing it for now.
If we need this functionality on metadata objects we can always add it later.

Author: d-v-b

src/zarr/abc/serializable.py

@@ -12,14 +12,6 @@ def from_json(cls, obj: T_contra) -> Self:
         """
         ...
 
-    @classmethod
-    def try_from_json(cls, obj: object) -> Self:
-        """
-        Deserialize from an unknown object. Details of any
-        deserialization failure should be conveyed via an exception.
-        """
-        ...
-
     def to_json(self) -> T_co:
         """
         Serialize to JSON.

src/zarr/core/metadata/v3.py

@@ -593,15 +593,6 @@ def from_json(cls, obj: ArrayMetadataJSONLike_V3) -> Self:
             extra_fields=extra_fields or None,  # type: ignore[arg-type]
         )
 
-    @classmethod
-    def try_from_json(cls, obj: object) -> Self:
-        """
-        Construct from an untrusted input (e.g. JSON read from disk).
-        Validates the structure and raises a ``MetadataValidationError``
-        listing all problems found.
-        """
-        return cls.from_json(narrow_array_metadata_json(obj))
-
     def update_shape(self, shape: tuple[int, ...]) -> Self:
         return replace(self, shape=shape)
 

tests/test_metadata/test_v3.py

@@ -22,6 +22,7 @@
     ArrayMetadataJSON_V3,
     ArrayV3Metadata,
     ChunkGridLike,
+    narrow_array_metadata_json,
     parse_codecs,
     parse_dimension_names,
     parse_zarr_format,
@@ -569,8 +570,8 @@ def test_from_json(
     assert result.dimension_names == dimension_names.expected
     assert result.extra_fields == extra_fields.expected
 
-    # try_from_json should produce the same result for the same input
-    assert ArrayV3Metadata.try_from_json(data) == result
+    # narrow + from_json should produce the same result for the same input
+    assert ArrayV3Metadata.from_json(narrow_array_metadata_json(data)) == result
 
 
 _VALID_TRY_FROM_JSON_INPUT: dict[str, object] = {
@@ -599,6 +600,6 @@ def test_from_json(
         ({**_VALID_TRY_FROM_JSON_INPUT, "unknown_field": "value"}, "Disallowed extra fields"),
     ],
 )
-def test_try_from_json_invalid(data: object, error_match: str) -> None:
+def test_narrow_array_metadata_json_invalid(data: object, error_match: str) -> None:
     with pytest.raises(MetadataValidationError, match=error_match):
-        ArrayV3Metadata.try_from_json(data)
+        narrow_array_metadata_json(data)