Style fixes for pre-commit

Author: agriyakhetarpal

zarr/tests/test_core.py

@@ -33,11 +33,19 @@
     from numcodecs.tests.common import greetings
 except ModuleNotFoundError:
     greetings = [
-        '¡Hola mundo!', 'Hej Världen!', 'Servus Woid!', 'Hei maailma!',
-        'Xin chào thế giới', 'Njatjeta Botë!', 'Γεια σου κόσμε!',
-        'こんにちは世界', '世界，你好！', 'Helló, világ!', 'Zdravo svete!',
-        'เฮลโลเวิลด์'
-]
+        "¡Hola mundo!",
+        "Hej Världen!",
+        "Servus Woid!",
+        "Hei maailma!",
+        "Xin chào thế giới",
+        "Njatjeta Botë!",
+        "Γεια σου κόσμε!",
+        "こんにちは世界",
+        "世界，你好！",
+        "Helló, világ!",
+        "Zdravo svete!",
+        "เฮลโลเวิลด์",
+    ]
 
 from numpy.testing import assert_array_almost_equal, assert_array_equal
 
@@ -1066,8 +1074,10 @@ def test_array_0d(self):
         assert 42 == z[()]
         z[()] = 43
         assert 43 == z[()]
+        # Failing here. TODO: Investigate this
         z[()] = z.fill_value
         assert z.fill_value == z[()]
+        # The failure says that z.fill_value is 0, but z[()] is 42...
         with pytest.raises(IndexError):
             z[0] = 42
         with pytest.raises(IndexError):

zarr/tests/test_storage.py

@@ -59,7 +59,14 @@
 )
 from zarr.storage import FSStore, rename, listdir
 from zarr._storage.v3 import KVStoreV3
-from zarr.tests.util import CountingDict, have_fsspec, is_wasm, skip_test_env_var, abs_container, mktemp
+from zarr.tests.util import (
+    CountingDict,
+    have_fsspec,
+    is_wasm,
+    skip_test_env_var,
+    abs_container,
+    mktemp,
+)
 from zarr.util import ConstantMap, json_dumps
 
 

zarr/tests/test_sync.py

@@ -31,6 +31,7 @@ def init_attributes(self, store, read_only=False, cache=True, zarr_version=zarr_
             store, synchronizer=synchronizer, key=key, read_only=read_only, cache=cache
         )
 
+
 @pytest.mark.skipif(is_wasm(), reason="No fcntl support in WASM/Pyodide")
 class TestAttributesProcessSynchronizer(TestAttributes):
     def init_attributes(self, store, read_only=False, cache=True, zarr_version=zarr_version):
@@ -97,6 +98,7 @@ def test_parallel_append(self):
 
         pool.terminate()
 
+
 @pytest.mark.skipif(is_wasm(), reason="No threading support in WASM/Pyodide")
 class TestArrayWithThreadSynchronizer(TestArray, MixinArraySyncTests):
     def create_array(self, read_only=False, **kwargs):

zarr/tests/util.py

@@ -96,7 +96,7 @@ def skip_test_env_var(name):
 
 
 def is_wasm():
-    return (sys.platform == "emscripten" or platform.machine() in ["wasm32", "wasm64"])
+    return sys.platform == "emscripten" or platform.machine() in ["wasm32", "wasm64"]
 
 
 def abs_container():