S3 test (#206)

* S3 test

* clean up

* Update CHANGELOG.md

* review

CHANGELOG.md

@@ -3,6 +3,7 @@
 ## 0.9.0 (Upcoming)
 ### Enhancements
 * NWBZarrIO load_namespaces=True by default. @mavaylon1 [#204](https://github.com/hdmf-dev/hdmf-zarr/pull/204)
+* Added test for opening file with consolidated metadata from DANDI. @mavaylon1 [#206](https://github.com/hdmf-dev/hdmf-zarr/pull/206)
 
 ## 0.8.0 (June 4, 2024)
 ### Bug Fixes

tests/unit/test_fsspec_streaming.py

@@ -2,6 +2,8 @@
 from hdmf_zarr import NWBZarrIO
 from .utils import check_s3fs_ffspec_installed
 
+import zarr
+
 HAVE_FSSPEC = check_s3fs_ffspec_installed()
 
 
@@ -24,3 +26,16 @@ def test_fsspec_streaming(self):
         self.assertEqual(len(nwbfile.electrode_groups), 2)
         self.assertEqual(len(nwbfile.electrodes), 1152)
         self.assertEqual(nwbfile.institution, "AIND")
+
+    @unittest.skipIf(not HAVE_FSSPEC, "fsspec not installed")
+    def test_s3_open_with_consolidated_(self):
+        """
+        The file is a Zarr file with consolidated metadata.
+        """
+        s3_path = "https://dandiarchive.s3.amazonaws.com/zarr/ccefbc9f-30e7-4a4c-b044-5b59d300040b/"
+        with NWBZarrIO(s3_path, mode='r') as read_io:
+            read_io.open()
+            self.assertIsInstance(read_io.file.store, zarr.storage.ConsolidatedMetadataStore)
+        with NWBZarrIO(s3_path, mode='-r') as read_io:
+            read_io.open()
+            self.assertIsInstance(read_io.file.store, zarr.storage.FSStore)

tests/unit/test_zarrio.py

@@ -181,4 +181,3 @@ def test_force_open_without_consolidated_fails(self):
                 read_io._ZarrIO__open_file_consolidated(store=self.store, mode='r')
             except ValueError as e:
                 self.fail("ZarrIO.__open_file_consolidated raised an unexpected ValueError: {}".format(e))
-