Skip to content

Commit 4d15e43

Browse files
committed
update __eq__
1 parent 25676a3 commit 4d15e43

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

src/zarr/storage/_object.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,13 +66,17 @@ class ObjectStore(Store):
6666
"""The underlying obstore instance."""
6767

6868
def __eq__(self, value: object) -> bool:
69+
if not isinstance(value, ObjectStore):
70+
return False
71+
6972
if not isinstance(self.store, type(value.store)):
7073
return False
7174
if not self.read_only == value.read_only:
7275
return False
7376

7477
match value.store:
7578
case AzureStore():
79+
assert isinstance(self.store, AzureStore)
7680
if (
7781
(self.store.config != value.store.config)
7882
or (self.store.client_options != value.store.client_options)
@@ -81,6 +85,7 @@ def __eq__(self, value: object) -> bool:
8185
):
8286
return False
8387
case GCSStore():
88+
assert isinstance(self.store, GCSStore)
8489
if (
8590
(self.store.config != value.store.config)
8691
or (self.store.client_options != value.store.client_options)
@@ -89,6 +94,7 @@ def __eq__(self, value: object) -> bool:
8994
):
9095
return False
9196
case S3Store():
97+
assert isinstance(self.store, S3Store)
9298
if (
9399
(self.store.config != value.store.config)
94100
or (self.store.client_options != value.store.client_options)
@@ -97,13 +103,15 @@ def __eq__(self, value: object) -> bool:
97103
):
98104
return False
99105
case HTTPStore():
106+
assert isinstance(self.store, HTTPStore)
100107
if (
101108
(self.store.url != value.store.url)
102109
or (self.store.client_options != value.store.client_options)
103110
or (self.store.retry_config != value.store.retry_config)
104111
):
105112
return False
106113
case LocalStore():
114+
assert isinstance(self.store, LocalStore)
107115
if self.store.prefix != value.store.prefix:
108116
return False
109117
case MemoryStore():

0 commit comments

Comments
 (0)