mirror of
https://github.com/bitcoin/bitcoin.git
synced 2026-05-11 22:43:06 +02:00
test: fix send_batch_request to pass callables when using --usecli
send_batch_request() was building raw dicts without a "jsonrpc" version, which made Core to treat them as version 1.0 requests. This worked in normal mode, but failed with --usecli because TestNodeCLI.batch() expects callables, not dicts. This commit fixes it by using get_request() which is defined in both AuthServiceProxy and TestNodeCLIAttr. The assert is changed because by using get_reques() AuthServiceProxy treats it as "jsonrpc" version 2.0 requests, which don't return "error" keys.
This commit is contained in:
@@ -18,11 +18,11 @@ from typing import List, Any
|
||||
|
||||
def send_batch_request(node: TestNode, method: str, params: List[Any]) -> List[Any]:
|
||||
"""Send batch request and parse all results"""
|
||||
data = [{"method": method, "params": p} for p in params]
|
||||
data = [getattr(node, method).get_request(*p) for p in params]
|
||||
response = node.batch(data)
|
||||
result = []
|
||||
for item in response:
|
||||
assert item["error"] is None, item["error"]
|
||||
assert "error" not in item, item["error"]
|
||||
result.append(item["result"])
|
||||
|
||||
return result
|
||||
|
||||
Reference in New Issue
Block a user