[prep/test] make wallet_fundrawtransaction's minrelaytxfee assumption explicit
What the developer wrote
Authored by glozow
[prep/test] make wallet_fundrawtransaction's minrelaytxfee assumption explicit
What changed, and why it matters
This commit only changes a single functional test file. It explicitly sets the minimum relay transaction fee in the test setup so the test's assumptions about fees are clear. There is no change to production wallet, consensus, or networking code, and no security issue is present.
No action required. This is a benign test-only change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff adds ‘-minrelaytxfee=0.00001000’ to the extra_args for all nodes in test/functional/wallet_fundrawtransaction.py. This makes the test’s reliance on a specific minimum relay fee explicit rather than depending on the default value. It is a test-only preparation change with no runtime code modifications.
Changed components
test/functional/wallet_fundrawtransaction.pyInspect captured patch +2 / −1
diff --git a/test/functional/wallet_fundrawtransaction.py b/test/functional/wallet_fundrawtransaction.py
index 2d5c5a2d..7cf92876 100755
--- a/test/functional/wallet_fundrawtransaction.py
+++ b/test/functional/wallet_fundrawtransaction.py
@@ -43,7 +43,8 @@ class RawTransactionsTest(BitcoinTestFramework):
def set_test_params(self):
self.num_nodes = 4
self.extra_args = [[
- "-deprecatedrpc=settxfee"
+ "-deprecatedrpc=settxfee",
+ "-minrelaytxfee=0.00001000",
] for i in range(self.num_nodes)]
self.setup_clean_chain = True
# whitelist peers to speed up tx relay / mempool sync
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.