Changeset View
Changeset View
Standalone View
Standalone View
test/functional/p2p_feefilter.py
Show First 20 Lines • Show All 59 Lines • ▼ Show 20 Lines | |||||
class FeeFilterTest(BitcoinTestFramework): | class FeeFilterTest(BitcoinTestFramework): | ||||
def set_test_params(self): | def set_test_params(self): | ||||
self.num_nodes = 2 | self.num_nodes = 2 | ||||
# We lower the various required feerates for this test | # We lower the various required feerates for this test | ||||
# to catch a corner-case where feefilter used to slightly undercut | # to catch a corner-case where feefilter used to slightly undercut | ||||
# mempool and wallet feerate calculation based on GetFee | # mempool and wallet feerate calculation based on GetFee | ||||
# rounding down 3 places, leading to stranded transactions. | # rounding down 3 places, leading to stranded transactions. | ||||
# See issue #16499 | # See issue #16499 | ||||
self.extra_args = [["-minrelaytxfee=0.00000100", | self.extra_args = [["-minrelaytxfee=1", | ||||
"-mintxfee=0.00000100"]] * self.num_nodes | "-mintxfee=1"]] * self.num_nodes | ||||
def skip_test_if_missing_module(self): | def skip_test_if_missing_module(self): | ||||
self.skip_if_no_wallet() | self.skip_if_no_wallet() | ||||
def run_test(self): | def run_test(self): | ||||
self.test_feefilter_forcerelay() | self.test_feefilter_forcerelay() | ||||
self.test_feefilter() | self.test_feefilter() | ||||
Show All 16 Lines | class FeeFilterTest(BitcoinTestFramework): | ||||
def test_feefilter(self): | def test_feefilter(self): | ||||
node1 = self.nodes[1] | node1 = self.nodes[1] | ||||
node0 = self.nodes[0] | node0 = self.nodes[0] | ||||
conn = self.nodes[0].add_p2p_connection(TestP2PConn()) | conn = self.nodes[0].add_p2p_connection(TestP2PConn()) | ||||
# Test that invs are received by test connection for all txs at | # Test that invs are received by test connection for all txs at | ||||
# feerate of .2 sat/byte | # feerate of .2 sat/byte | ||||
node1.settxfee(Decimal("0.00000200")) | node1.settxfee(Decimal("2")) | ||||
txids = [node1.sendtoaddress(node1.getnewaddress(), 1) | txids = [node1.sendtoaddress(node1.getnewaddress(), 1000000) | ||||
for x in range(3)] | for x in range(3)] | ||||
assert allInvsMatch(txids, conn) | assert allInvsMatch(txids, conn) | ||||
conn.clear_invs() | conn.clear_invs() | ||||
# Set a filter of .15 sat/byte on test connection | # Set a filter of .15 sat/byte on test connection | ||||
conn.send_and_ping(msg_feefilter(150)) | conn.send_and_ping(msg_feefilter(150)) | ||||
# Test that txs are still being received by test connection | # Test that txs are still being received by test connection | ||||
# (paying .15 sat/byte) | # (paying .15 sat/byte) | ||||
node1.settxfee(Decimal("0.00000150")) | node1.settxfee(Decimal("1.5")) | ||||
txids = [node1.sendtoaddress(node1.getnewaddress(), 1) | txids = [node1.sendtoaddress(node1.getnewaddress(), 1000000) | ||||
for x in range(3)] | for x in range(3)] | ||||
assert allInvsMatch(txids, conn) | assert allInvsMatch(txids, conn) | ||||
conn.clear_invs() | conn.clear_invs() | ||||
# Change tx fee rate to .1 sat/byte and test they are no longer received | # Change tx fee rate to .1 sat/byte and test they are no longer received | ||||
# by the test connection | # by the test connection | ||||
node1.settxfee(Decimal("0.00000100")) | node1.settxfee(Decimal("1")) | ||||
[node1.sendtoaddress(node1.getnewaddress(), 1) for x in range(3)] | [node1.sendtoaddress(node1.getnewaddress(), 1000000) for x in range(3)] | ||||
self.sync_mempools() # must be sure node 0 has received all txs | self.sync_mempools() # must be sure node 0 has received all txs | ||||
# Send one transaction from node0 that should be received, so that we | # Send one transaction from node0 that should be received, so that we | ||||
# we can sync the test on receipt (if node1's txs were relayed, they'd | # we can sync the test on receipt (if node1's txs were relayed, they'd | ||||
# be received by the time this node0 tx is received). This is | # be received by the time this node0 tx is received). This is | ||||
# unfortunately reliant on the current relay behavior where we batch up | # unfortunately reliant on the current relay behavior where we batch up | ||||
# to 35 entries in an inv, which means that when this next transaction | # to 35 entries in an inv, which means that when this next transaction | ||||
# is eligible for relay, the prior transactions from node1 are eligible | # is eligible for relay, the prior transactions from node1 are eligible | ||||
# as well. | # as well. | ||||
node0.settxfee(Decimal("0.00020000")) | node0.settxfee(Decimal("200.00")) | ||||
txids = [node0.sendtoaddress(node0.getnewaddress(), 1)] | txids = [node0.sendtoaddress(node0.getnewaddress(), 1000000)] | ||||
assert allInvsMatch(txids, conn) | assert allInvsMatch(txids, conn) | ||||
conn.clear_invs() | conn.clear_invs() | ||||
# Remove fee filter and check that txs are received again | # Remove fee filter and check that txs are received again | ||||
conn.send_and_ping(msg_feefilter(0)) | conn.send_and_ping(msg_feefilter(0)) | ||||
txids = [node1.sendtoaddress(node1.getnewaddress(), 1) | txids = [node1.sendtoaddress(node1.getnewaddress(), 1000000) | ||||
for x in range(3)] | for x in range(3)] | ||||
assert allInvsMatch(txids, conn) | assert allInvsMatch(txids, conn) | ||||
conn.clear_invs() | conn.clear_invs() | ||||
if __name__ == '__main__': | if __name__ == '__main__': | ||||
FeeFilterTest().main() | FeeFilterTest().main() |