fix: process exceptions when trying to open X5

Author: oesteban

nitransforms/io/x5.py

@@ -140,16 +140,21 @@ def to_filename(fname: str | Path, x5_list: List[X5Transform]):
 
 def from_filename(fname: str | Path) -> List[X5Transform]:
     """Read a list of :class:`X5Transform` objects from an X5 HDF5 file."""
-    with h5py.File(str(fname), "r") as in_file:
-        if in_file.attrs.get("Format") != "X5":
-            raise ValueError("Input file is not in X5 format")
-
-        tg = in_file["TransformGroup"]
-        return [
-            _read_x5_group(node)
-            for _, node in sorted(tg.items(), key=lambda kv: int(kv[0]))
-        ]
-
+    try:
+        with h5py.File(str(fname), "r") as in_file:
+            if in_file.attrs.get("Format") != "X5":
+                raise TypeError("Input file is not in X5 format")
+
+            tg = in_file["TransformGroup"]
+            return [
+                _read_x5_group(node)
+                for _, node in sorted(tg.items(), key=lambda kv: int(kv[0]))
+            ]
+    except OSError as err:
+        if "file signature not found" in err.args[0]:
+            raise TypeError(f"Input file is not HDF5.")
+
+        raise
 
 def _read_x5_group(node) -> X5Transform:
     x5 = X5Transform(

nitransforms/tests/test_x5.py

@@ -73,5 +73,5 @@ def test_from_filename_invalid(tmp_path):
     with H5File(fname, "w") as f:
         f.attrs["Format"] = "NOTX5"
 
-    with pytest.raises(ValueError):
+    with pytest.raises(TypeError):
         from_filename(fname)