From 9c1fcaca5cb8481b2a46f42e0f43888311b97ef2 Mon Sep 17 00:00:00 2001 From: rkrux Date: Thu, 28 May 2026 19:41:58 +0530 Subject: [PATCH] wallet, test: fix sendall anti-fee-sniping when locktime is not specified This particular test case only needs to ensure that locktime is not specified in the RPC request, it doesn't need to rely on the wallet optin RBF default that causes the test to pass coincidentally. Co-authored-by: maflcko <6399679+maflcko@users.noreply.github.com> --- test/functional/wallet_sendall.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/functional/wallet_sendall.py b/test/functional/wallet_sendall.py index 0524b5c82ed..657cdace2d7 100755 --- a/test/functional/wallet_sendall.py +++ b/test/functional/wallet_sendall.py @@ -59,8 +59,8 @@ class SendallTest(BitcoinTestFramework): return self.wallet.getbalances()["mine"]["trusted"] # Helper schema for success cases - def test_sendall_success(self, sendall_args, remaining_balance = 0): - sendall_tx_receipt = self.wallet.sendall(sendall_args) + def test_sendall_success(self, sendall_args, remaining_balance = 0, *, options=None): + sendall_tx_receipt = self.wallet.sendall(sendall_args, options=options) self.generate(self.nodes[0], 1) # wallet has remaining balance (usually empty) assert_equal(remaining_balance, self.wallet.getbalances()["mine"]["trusted"]) @@ -436,7 +436,7 @@ class SendallTest(BitcoinTestFramework): def sendall_anti_fee_sniping(self): self.log.info("Testing sendall does anti-fee-sniping when locktime is not specified") self.add_utxos([10,11]) - tx_from_wallet = self.test_sendall_success(sendall_args = [self.remainder_target]) + tx_from_wallet = self.test_sendall_success(sendall_args = [self.remainder_target], options={"replaceable":False}) # the locktime should be within 100 blocks of the # block height