🧪 [testing improvement] Add test for SafeStore KeyError exception path
The SafeStore.__getitem__ method (formerly ZarrNotFilledStore) was missing a test for the KeyError exception path where it should raise a FileNotFoundError. This is important for ensuring that zarr propagates the exception rather than filling.
This PR adds:
tests/test_util_safestore.py: A new unit test file for the SafeStore class.
Coverage:
- Happy path for
__getitem__.
KeyError path for __getitem__ (raising FileNotFoundError).__len__, __iter__, __getattr__ (passthrough and __setstate__ restriction).__setitem__ and __delitem__ (raising NotImplementedError).
The improvement ensures better reliability and coverage for this critical data-access utility
🧪 [testing improvement] Add test for SafeStore KeyError exception path
The
SafeStore.__getitem__method (formerlyZarrNotFilledStore) was missing a test for theKeyErrorexception path where it should raise aFileNotFoundError. This is important for ensuring that zarr propagates the exception rather than filling.This PR adds:
tests/test_util_safestore.py: A new unit test file for theSafeStoreclass.Coverage:
__getitem__.KeyErrorpath for__getitem__(raisingFileNotFoundError).__len__,__iter__,__getattr__(passthrough and__setstate__restriction).__setitem__and__delitem__(raisingNotImplementedError).The improvement ensures better reliability and coverage for this critical data-access utility