Add tests for reader - Githubissues

jni / zarpaint

Paint segmentations directly to on-disk/remote zarr arrays

BSD 3-Clause "New" or "Revised" License

14 stars 8 forks source link

Add tests for reader #24

Closed jamesyan-git closed 2 years ago

jamesyan-git commented 2 years ago

Three tests have been added. One tests opening a file, one tests that the reader returns a valid function, and one does a round trip of creating, writing to, saving and reopening a zarr file. We could do more in-depth testing of the metadata that we are getting/reading, but I will do that via unit tests of the individual functions.

deprecated-napari-hub-preview-bot[bot] commented 2 years ago

Preview page for your plugin is ready here: https://preview.napari-hub.org/jni/zarpaint/24

jni commented 2 years ago

Cool!

This looks ready to go but you couldn't tell from the CI! 😅 The fixes are in jni/affinder#60 and jni/affinder#59. Would you mind copying those changes to this repo, then we can update this PR with those changes?

jni commented 2 years ago

Whoo!