Skip to content

Commit

Permalink
Apply ruff/flake8-simplify rule SIM118
Browse files Browse the repository at this point in the history
SIM118 Use `key in dict` instead of `key in dict.keys()`
  • Loading branch information
DimitriPapadopoulos committed Sep 26, 2024
1 parent 1947917 commit b47ee73
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/zarr/testing/store.py
Original file line number Diff line number Diff line change
Expand Up @@ -226,7 +226,7 @@ async def test_list_prefix(self, store: S) -> None:
for prefix in prefixes:
observed = tuple(sorted(await _collect_aiterator(store.list_prefix(prefix))))
expected: tuple[str, ...] = ()
for key in store_dict.keys():
for key in store_dict:
if key.startswith(prefix):
expected += (key.removeprefix(prefix),)
expected = tuple(sorted(expected))
Expand All @@ -245,7 +245,7 @@ async def test_list_dir(self, store: S) -> None:
await store._set_many(store_dict.items())

keys_observed = await _collect_aiterator(store.list_dir(root))
keys_expected = {k.removeprefix(root + "/").split("/")[0] for k in store_dict.keys()}
keys_expected = {k.removeprefix(root + "/").split("/")[0] for k in store_dict}

assert sorted(keys_observed) == sorted(keys_expected)

Expand Down

0 comments on commit b47ee73

Please sign in to comment.