Fix Zarr 'number of requests' test (#10492)

Author: ianhi

xarray/tests/conftest.py

@@ -234,6 +234,6 @@ def simple_datatree(create_test_datatree):
     return create_test_datatree()
 
 
-@pytest.fixture(scope="module", params=["s", "ms", "us", "ns"])
+@pytest.fixture(params=["s", "ms", "us", "ns"])
 def time_unit(request):
     return request.param

xarray/tests/test_backends.py

@@ -3545,8 +3545,8 @@ def test_append(self) -> None:
             if has_zarr_v3:
                 # TODO: verify these
                 expected = {
-                    "set": 16,
-                    "get": 8,  # TODO: fixme upstream (should be 4), see https://github.com/zarr-developers/zarr-python/issues/3199
+                    "set": 5,
+                    "get": 4,
                     "list_dir": 2,
                     "list_prefix": 1,
                 }
@@ -3570,8 +3570,8 @@ def test_append(self) -> None:
             # 6057128b: {'iter': 5, 'contains': 2, 'setitem': 5, 'getitem': 10, "listdir": 5, "list_prefix": 0}
             if has_zarr_v3:
                 expected = {
-                    "set": 11,  # TODO: fixme upstream (should be 4), see https://github.com/zarr-developers/zarr-python/issues/3199
-                    "get": 23,  # TODO: fixme upstream (should be 8), see https://github.com/zarr-developers/zarr-python/issues/3199
+                    "set": 4,
+                    "get": 9,  # TODO: fixme upstream (should be 8)
                     "list_dir": 2,  # TODO: fixme upstream (should be 2)
                     "list_prefix": 0,
                 }
@@ -3593,8 +3593,8 @@ def test_append(self) -> None:
 
             if has_zarr_v3:
                 expected = {
-                    "set": 11,  # TODO: fixme upstream (should be 4), see https://github.com/zarr-developers/zarr-python/issues/3199
-                    "get": 23,  # TODO: fixme upstream (should be 8), see https://github.com/zarr-developers/zarr-python/issues/3199
+                    "set": 4,
+                    "get": 9,  # TODO: fixme upstream (should be 8)
                     "list_dir": 2,  # TODO: fixme upstream (should be 2)
                     "list_prefix": 0,
                 }
@@ -3627,8 +3627,8 @@ def test_region_write(self) -> None:
         with self.create_zarr_target() as store:
             if has_zarr_v3:
                 expected = {
-                    "set": 16,
-                    "get": 8,  # TODO: fixme upstream (should be 2), see https://github.com/zarr-developers/zarr-python/issues/3199
+                    "set": 5,
+                    "get": 2,
                     "list_dir": 2,
                     "list_prefix": 4,
                 }