Skip to content

Commit 7184fb8

Browse files
author
MacroFake
committed
Merge bitcoin/bitcoin#26143: test: wait for the expected basic block filter index in interface_rest
aee29c5 test: wait for the expected basic block filter index in `interface_rest` (brunoerg) Pull request description: Fixes #26098 Wait for the expected 'basic block filter index' to not cause issues when calling `/blockfilterheaders/basic/`, like: https://github.com/bitcoin/bitcoin/blob/9bd842a5928f160c1bc8fca6ca7d8d43f096fd6a/src/rest.cpp#L423-L424 Top commit has no ACKs. Tree-SHA512: 36abe28a76cd01ce7ac1ae9258ce1a9a5473d985c498d915c1130256bc800d0d1207708a195b78bbcb00478ec9c373a2fbfeb26c1fddeb35abe8b253e0308058
2 parents 9e2a2b8 + aee29c5 commit 7184fb8

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

test/functional/interface_rest.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -288,6 +288,10 @@ def run_test(self):
288288

289289
# See if we can get 5 headers in one response
290290
self.generate(self.nodes[1], 5)
291+
expected_filter = {
292+
'basic block filter index': {'synced': True, 'best_block_height': 208},
293+
}
294+
self.wait_until(lambda: self.nodes[0].getindexinfo() == expected_filter)
291295
json_obj = self.test_rest_request(f"/headers/{bb_hash}", query_params={"count": 5})
292296
assert_equal(len(json_obj), 5) # now we should have 5 header objects
293297
json_obj = self.test_rest_request(f"/blockfilterheaders/basic/{bb_hash}", query_params={"count": 5})

0 commit comments

Comments
 (0)