fix febus format issue

d-chambers · d-chambers · commit 6648cc5079d7 · 2025-02-27T16:48:01.000-08:00
diff --git a/dascore/io/febus/utils.py b/dascore/io/febus/utils.py
@@ -75,7 +75,7 @@ def _get_febus_version_str(hdf_fi) -> str:
             # Hopefully this is the file version...
             version = unbyte(source.attrs.get("Version", version)).split(".")[0]
             is_febus = is_febus and expected_source_attrs.issubset(set(source.attrs))
-    if is_febus:
+    if inst_keys and is_febus:
         return version
     return ""
 
diff --git a/dascore/io/neubrex/core.py b/dascore/io/neubrex/core.py
@@ -11,7 +11,7 @@
 import dascore.io.neubrex.utils_rfs as rfs_utils
 from dascore.constants import SpoolType
 from dascore.io import FiberIO
-from dascore.utils.hdf5 import H5Reader, H5Writer
+from dascore.utils.hdf5 import H5Reader
 
 
 class NeubrexRFSPatchAttrs(dc.PatchAttrs):
@@ -84,31 +84,6 @@ def scan(self, resource: H5Reader, snap=True, **kwargs) -> list[dc.PatchAttrs]:
         attrs["file_version"] = self.version
         return [dc.PatchAttrs(**attrs)]
 
-    def write(
-        self,
-        spool: SpoolType,
-        resource: H5Writer,
-        extra_attrs: dict[str, float | int | str] | None = None,
-        **kwargs,
-    ) -> None:
-        """
-        Write a Neubrex Rayleigh Frequency Shift file.
-
-        Parameters
-        ----------
-        spool
-            The spool to write.
-        resource
-            The H5file to write to.
-        extra_attrs
-            A dict of basic types to write to the Acoustic dataset's
-            attributes.
-        **kwargs
-            Un-used, only here for compat with other write methods.
-        """
-        if len(spool) > 1:
-            raise ValueError("NeubrexRFS only supports writing a single patch.")
-
 
 class NeubrexDASV1(FiberIO):
     """
diff --git a/tests/test_io/test_io_core.py b/tests/test_io/test_io_core.py
@@ -7,6 +7,7 @@
 from pathlib import Path
 from typing import TypeVar
 
+import h5py
 import numpy as np
 import pandas as pd
 import pytest
@@ -260,6 +261,14 @@ def test_implements(self):
 class TestGetFormat:
     """Tests to ensure formats can be retrieved."""
 
+    @pytest.fixture(scope="class")
+    def empty_h5_path(self, tmpdir_factory):
+        """Create an empty HDF5 file."""
+        path = tmpdir_factory.mktemp("empty") / "empty.h5"
+        with h5py.File(path, "w"):
+            pass
+        return path
+
     def test_not_known(self, dummy_text_file):
         """Ensure a non-path/str object raises."""
         with pytest.raises(UnknownFiberFormatError):
@@ -279,6 +288,11 @@ def test_fiberio_directory(self, tmp_path_factory):
         assert fiber_io.name == name
         assert fiber_io.version == version
 
+    def test_empty_hdf5_no_format(self, empty_h5_path):
+        """Ensure the empty hdf5 dorsen't have a format."""
+        with pytest.raises(UnknownFiberFormatError):
+            dc.get_format(empty_h5_path)
+
 
 class TestScan:
     """Tests for scanning fiber files."""
diff --git a/tests/test_io/test_neubrex/test_neubrex_rfs.py b/tests/test_io/test_neubrex/test_neubrex_rfs.py
