Apply suggestions from code review

Co-authored-by: Davis Bennett <[email protected]>

Author: jhamman

src/zarr/api/asynchronous.py

@@ -627,7 +627,6 @@ async def group(
     else:
         mode = "r+"
     store_mode = _handle_store_mode(mode)
-    print(f"store_mode: {store_mode}")
     store_path = await make_store_path(
         store, path=path, mode=store_mode, storage_options=storage_options
     )

src/zarr/storage/common.py

@@ -66,7 +66,13 @@ async def _init(self, mode: AccessModeLiteral) -> None:
         match mode:
             case "w-":
                 if not await self.empty():
-                    raise FileExistsError(f"{self} must be empty. Mode is 'w-'")
+                    msg = (
+                      f"{self} is not empty, but `mode` is set to 'w-'."
+                      "Either remove the existing objects in storage,"
+                      "or set `mode` to a value that handles pre-existing objects"
+                      "in storage, like `a` or `w`."
+                      )
+                    raise FileExistsError(msg)
             case "w":
                 await self.delete_dir()
             case "a" | "r" | "r+":

src/zarr/testing/strategies.py

@@ -138,10 +138,6 @@ def arrays(
     array_path = path + ("/" if not path.endswith("/") else "") + name
     root = zarr.open_group(store, mode="w")
 
-    # try:
-    #     del root[array_path]
-    # except KeyError:
-    #     pass
 
     a = root.create_array(
         array_path,