open update

src/hdmf_zarr/backend.py

@@ -159,7 +159,7 @@ def open(self):
         if self.__file is None:
             # Within zarr, open_consolidated only allows the mode to be 'r' or 'r+'.
             # As a result, when in 'w' mode, the file will not use consolidated metadata.
-            if self.__mode == 'w':
+            if self.__mode not in ['r', 'r+']:
                 self.__file = zarr.open(store=self.path,
                                         mode=self.__mode,
                                         synchronizer=self.__synchronizer,
@@ -716,7 +716,7 @@ def resolve_ref(self, zarr_ref):
         else:
             target_name = ROOT_NAME
 
-        target_zarr_obj = zarr.open(source_file, mode='r', storage_options=self.__storage_options)
+        target_zarr_obj = self.__open_file_consolidated(source_file, mode='r', storage_options=self.__storage_options)
         if object_path is not None:
             try:
                 target_zarr_obj = target_zarr_obj[object_path]

tests/unit/test_zarrio.py

@@ -122,3 +122,20 @@ class TestExportZarrToZarrNestedDirectoryStore(BaseTestExportZarrToZarr):
     def setUp(self):
         super().setUp()
         self.store = [NestedDirectoryStore(p) for p in self.store_path]
+
+
+#########################################
+#  Consolidate Metadata tests
+#########################################
+# class TestConsolidateMetadata(TestCase):
+#     """
+#
+#     """
+#     def test_get_store_path(self):
+#         pass
+#
+#     def test_warning_consolidate_metadata(self):
+#         pass
+#
+#     def test_open_with_zmetadata(self):
+#         pass