JaneliaSciComp / pydantic-ome-ngff

Pydantic models for OME-NGFF
BSD 3-Clause "New" or "Revised" License
20 stars 3 forks source link

Don't refer to the `path` attribute for Zarr stores #26

Closed d-v-b closed 7 months ago

d-v-b commented 7 months ago

In main, an exception message refers to the path attribute of a Zarr store, but for zarr.storage.MemoryStore there is no path attribute. Not great that the Zarr store API is ... textured like this. This PR fixes this by changing the error message, and adds tests for the expected behavior. I also took some liberty to tweak type hints, and convert some lists to tuples in the tests.