Skip to content

Commit fc179a8

Browse files
committed
fix: issue when changing default in smt
1 parent 7eef408 commit fc179a8

File tree

2 files changed

+7
-4
lines changed

2 files changed

+7
-4
lines changed

tests/test_smt.py

+5-2
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,12 @@
1414
@given(
1515
k=st.binary(min_size=1, max_size=32),
1616
v=st.binary(min_size=1, max_size=32),
17+
default=st.binary(min_size=1, max_size=32),
1718
)
18-
def test_simple_kv(k, v):
19-
smt = SparseMerkleTree(key_size=len(k))
19+
def test_simple_kv(k, v, default):
20+
# default must be different than v
21+
default = BLANK_NODE if default == v else default
22+
smt = SparseMerkleTree(key_size=len(k), default=default)
2023
empty_root = smt.root_hash
2124

2225
# Nothing has been added yet

trie/smt.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -252,7 +252,7 @@ def get(self, key: bytes) -> bytes:
252252
value, _ = self._get(key)
253253

254254
# Ensure that it isn't blank!
255-
if value == BLANK_NODE:
255+
if value == self._default:
256256
raise KeyError("Key does not exist")
257257

258258
return value
@@ -261,7 +261,7 @@ def branch(self, key: bytes) -> Tuple[Hash32]:
261261
value, branch = self._get(key)
262262

263263
# Ensure that it isn't blank!
264-
if value == BLANK_NODE:
264+
if value == self._default:
265265
raise KeyError("Key does not exist")
266266

267267
return branch

0 commit comments

Comments
 (0)