make MemoryStore allways concurrent

Author: brokkoli71

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

@@ -4,14 +4,11 @@
 import javax.annotation.Nullable;
 import java.nio.ByteBuffer;
 import java.util.*;
+import java.util.concurrent.ConcurrentHashMap;
 import java.util.stream.Stream;
 
 public class MemoryStore implements Store, Store.ListableStore {
-  private final Map<List<String>, byte[]> map = map();
-
-  protected Map<List<String>, byte[]> map(){
-    return new HashMap<>();
-  }
+  private final Map<List<String>, byte[]> map = new ConcurrentHashMap<>();
 
   List<String> resolveKeys(String[] keys) {
     ArrayList<String> resolvedKeys = new ArrayList<>();

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

@@ -83,28 +83,12 @@ public void testHttpStore() throws IOException, ZarrException {
     }
 
     @ParameterizedTest
-    @CsvSource({
-        "false,MemoryStore",
-        "false,ConcurrentMemoryStore",
-        "true,ConcurrentMemoryStore",
-    })
-    public void testMemoryStoreV3(boolean useParallel, String storeType) throws ZarrException, IOException {
+    @CsvSource({"false", "true",})
+    public void testMemoryStoreV3(boolean useParallel) throws ZarrException, IOException {
         int[] testData = new int[1024 * 1024];
         Arrays.setAll(testData, p -> p);
 
-        StoreHandle storeHandle;
-        switch (storeType) {
-            case "ConcurrentMemoryStore":
-                storeHandle = new ConcurrentMemoryStore().resolve();
-                break;
-            case "MemoryStore":
-                storeHandle = new MemoryStore().resolve();
-                break;
-            default:
-                throw new IllegalArgumentException("Unknown store type: " + storeType);
-        }
-
-        Group group = Group.create(storeHandle);
+        Group group = Group.create(new MemoryStore().resolve());
         Array array = group.createArray("array", b -> b
                 .withShape(1024, 1024)
                 .withDataType(DataType.UINT32)
@@ -124,28 +108,12 @@ public void testMemoryStoreV3(boolean useParallel, String storeType) throws Zarr
     }
 
     @ParameterizedTest
-    @CsvSource({
-        "false,MemoryStore",
-        "false,ConcurrentMemoryStore",
-        "true,ConcurrentMemoryStore",
-    })
-    public void testMemoryStoreV2(boolean useParallel, String storeType) throws ZarrException, IOException {
+    @CsvSource({"false", "true",})
+    public void testMemoryStoreV2(boolean useParallel) throws ZarrException, IOException {
         int[] testData = new int[1024 * 1024];
         Arrays.setAll(testData, p -> p);
 
-        StoreHandle storeHandle;
-        switch (storeType) {
-            case "ConcurrentMemoryStore":
-                storeHandle = new ConcurrentMemoryStore().resolve();
-                break;
-            case "MemoryStore":
-                storeHandle = new MemoryStore().resolve();
-                break;
-            default:
-                throw new IllegalArgumentException("Unknown store type: " + storeType);
-        }
-
-        dev.zarr.zarrjava.v2.Group group = dev.zarr.zarrjava.v2.Group.create(storeHandle);
+        dev.zarr.zarrjava.v2.Group group = dev.zarr.zarrjava.v2.Group.create(new MemoryStore().resolve());
         dev.zarr.zarrjava.v2.Array array = group.createArray("array", b -> b
                 .withShape(1024, 1024)
                 .withDataType(dev.zarr.zarrjava.v2.DataType.UINT32)