fix ReadOnlyZipStore::list

Author: brokkoli71

src/main/java/dev/zarr/zarrjava/store/ReadOnlyZipStore.java

@@ -146,7 +146,11 @@ public Stream<String[]> list(String[] keys) {
                 if (!entryName.startsWith(prefix) || entryName.equals(prefix)) {
                     continue;
                 }
-                String[] entryKeys = resolveEntryKeys(entryName.substring(prefix.length() + 1));
+                entryName = entryName.substring(prefix.length());
+                if (entryName.startsWith("/")) {
+                    entryName = entryName.substring(1);
+                }
+                String[] entryKeys = resolveEntryKeys(entryName);
                 builder.add(entryKeys);
             }
         } catch (IOException ignored) {

src/test/java/dev/zarr/zarrjava/Utils.java

@@ -21,9 +21,6 @@ public static void zipFile(Path sourceDir, Path targetDir) throws IOException {
     }
 
     static void zipFile(File fileToZip, String fileName, ZipOutputStream zipOut) throws IOException {
-        if (fileToZip.isHidden()) {
-            return;
-        }
         if (fileToZip.isDirectory()) {
             if (fileName.endsWith("/")) {
                 zipOut.putNextEntry(new ZipEntry(fileName));

src/test/java/dev/zarr/zarrjava/store/ReadOnlyZipStoreTest.java

@@ -12,23 +12,38 @@
 import java.util.Arrays;
 import java.util.stream.Collectors;
 
-public class ReadOnlyZipStoreTest extends  StoreTest {
+public class ReadOnlyZipStoreTest extends StoreTest {
 
+    Path storePath = TESTOUTPUT.resolve("readOnlyZipStoreTest.zip");
     StoreHandle storeHandleWithData;
 
     @BeforeAll
     void writeStoreHandleWithData() throws ZarrException, IOException {
-        Path source = TESTDATA.resolve("v2_sample").resolve("bool");
-        Path target = TESTOUTPUT.resolve("readOnlyZipStoreTest.zip");
-        Utils.zipFile(source, target);
-        storeHandleWithData = new ReadOnlyZipStore(target).resolve("0.0.0");
+        Path source = TESTDATA.resolve("v2_sample").resolve("subgroup");
+        Utils.zipFile(source, storePath);
+        storeHandleWithData = new ReadOnlyZipStore(storePath).resolve("array", "0.0.0");
     }
 
     @Override
     StoreHandle storeHandleWithData() {
         return storeHandleWithData;
     }
 
+    @Override
+    @Test
+    void testList() {
+        ReadOnlyZipStore zipStore = new ReadOnlyZipStore(storePath);
+        BufferedZipStore bufferedZipStore = new BufferedZipStore(storePath);
+
+        java.util.Set<String> expectedKeys = bufferedZipStore.resolve().list()
+                .map(node -> String.join("/", node))
+                .collect(Collectors.toSet());
+        java.util.Set<String> actualKeys = zipStore.resolve().list()
+                .map(node -> String.join("/", node))
+                .collect(Collectors.toSet());
+        Assertions.assertEquals(expectedKeys, actualKeys);
+    }
+
     @Test
     public void testOpen() throws ZarrException, IOException {
         Path sourceDir = TESTOUTPUT.resolve("testZipStore");