qa: Remove never used return value of sync_with_ping
This commit is contained in:
parent
fa9de370b1
commit
fafa003970
2 changed files with 6 additions and 4 deletions
|
@ -103,7 +103,8 @@ class AcceptBlockTest(BitcoinTestFramework):
|
||||||
test_node.send_message(msg_block(blocks_h2[0]))
|
test_node.send_message(msg_block(blocks_h2[0]))
|
||||||
white_node.send_message(msg_block(blocks_h2[1]))
|
white_node.send_message(msg_block(blocks_h2[1]))
|
||||||
|
|
||||||
[ x.sync_with_ping() for x in [test_node, white_node] ]
|
for x in [test_node, white_node]:
|
||||||
|
x.sync_with_ping()
|
||||||
assert_equal(self.nodes[0].getblockcount(), 2)
|
assert_equal(self.nodes[0].getblockcount(), 2)
|
||||||
assert_equal(self.nodes[1].getblockcount(), 2)
|
assert_equal(self.nodes[1].getblockcount(), 2)
|
||||||
self.log.info("First height 2 block accepted by both nodes")
|
self.log.info("First height 2 block accepted by both nodes")
|
||||||
|
@ -116,7 +117,8 @@ class AcceptBlockTest(BitcoinTestFramework):
|
||||||
test_node.send_message(msg_block(blocks_h2f[0]))
|
test_node.send_message(msg_block(blocks_h2f[0]))
|
||||||
white_node.send_message(msg_block(blocks_h2f[1]))
|
white_node.send_message(msg_block(blocks_h2f[1]))
|
||||||
|
|
||||||
[ x.sync_with_ping() for x in [test_node, white_node] ]
|
for x in [test_node, white_node]:
|
||||||
|
x.sync_with_ping()
|
||||||
for x in self.nodes[0].getchaintips():
|
for x in self.nodes[0].getchaintips():
|
||||||
if x['hash'] == blocks_h2f[0].hash:
|
if x['hash'] == blocks_h2f[0].hash:
|
||||||
assert_equal(x['status'], "headers-only")
|
assert_equal(x['status'], "headers-only")
|
||||||
|
@ -135,7 +137,8 @@ class AcceptBlockTest(BitcoinTestFramework):
|
||||||
test_node.send_message(msg_block(blocks_h3[0]))
|
test_node.send_message(msg_block(blocks_h3[0]))
|
||||||
white_node.send_message(msg_block(blocks_h3[1]))
|
white_node.send_message(msg_block(blocks_h3[1]))
|
||||||
|
|
||||||
[ x.sync_with_ping() for x in [test_node, white_node] ]
|
for x in [test_node, white_node]:
|
||||||
|
x.sync_with_ping()
|
||||||
# Since the earlier block was not processed by node0, the new block
|
# Since the earlier block was not processed by node0, the new block
|
||||||
# can't be fully validated.
|
# can't be fully validated.
|
||||||
for x in self.nodes[0].getchaintips():
|
for x in self.nodes[0].getchaintips():
|
||||||
|
|
|
@ -1615,7 +1615,6 @@ class NodeConnCB(object):
|
||||||
test_function = lambda: self.last_message.get("pong") and self.last_message["pong"].nonce == self.ping_counter
|
test_function = lambda: self.last_message.get("pong") and self.last_message["pong"].nonce == self.ping_counter
|
||||||
wait_until(test_function, timeout=timeout, lock=mininode_lock)
|
wait_until(test_function, timeout=timeout, lock=mininode_lock)
|
||||||
self.ping_counter += 1
|
self.ping_counter += 1
|
||||||
return True
|
|
||||||
|
|
||||||
# The actual NodeConn class
|
# The actual NodeConn class
|
||||||
# This class provides an interface for a p2p connection to a specified node
|
# This class provides an interface for a p2p connection to a specified node
|
||||||
|
|
Loading…
Reference in a new issue