Page MenuHomePhabricator
Differential D12729

test: use MiniWallet for p2p_filter.py
ClosedPublic
Actions

Authored by Fabien on Dec 2 2022, 13:52.

Details

Reviewers
sdulfari
Group Reviewers
Restricted Project
Commits
rABC8937ba594953: test: use MiniWallet for p2p_filter.py
Summary
This PR enables one more of the non-wallet functional tests (p2p_filter.py) to be run even with the Bitcoin Core wallet disabled by using the MiniWallet instead, as proposed in #20078.
For this purpose, a MiniWallet method send_to is introduced first, which allows to create arbitrary outputs (scriptPubKey/amount). Note that the implementation for this is already present in feature_rbf.py (recently added in PR #22998), i.e. it is simply moved to the MiniWallet interface.

Backport of core#23079.

Also cherry-picked a required change from core#22998 that returns the txid from the wallet sendrawtransaction.

Test Plan
ninja check-all

Diff Detail

Event Timeline

Fabien created this revision.Dec 2 2022, 13:52
Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 2 2022, 13:52
Fabien requested review of this revision.Dec 2 2022, 13:52
teamcity edited the summary of this revision. (Show Details)Dec 2 2022, 13:52
Fabien added a comment.Dec 2 2022, 13:52

Note to reviewers: lint errors are from the mypy issue fixed in D12726

sdulfari accepted this revision.Dec 2 2022, 16:56
This revision is now accepted and ready to land.Dec 2 2022, 16:56
Fabien planned changes to this revision.Dec 2 2022, 17:29
Fabien updated this revision to Diff 36907.Dec 2 2022, 17:42

Account for the miner fund output

This revision is now accepted and ready to land.Dec 2 2022, 17:42
Harbormaster completed remote builds in B21325: Diff 36907.Dec 2 2022, 17:55
Fabien updated this revision to Diff 36911.Dec 2 2022, 18:18

Don't hardcode the index

Harbormaster completed remote builds in B21327: Diff 36911.Dec 2 2022, 18:31
Closed by commit rABC8937ba594953: test: use MiniWallet for p2p_filter.py (authored by Sebastian Falbesoner <sebastian.falbesoner@gmail.com>, committed by Fabien). · Explain WhyDec 2 2022, 19:42
This revision was automatically updated to reflect the committed changes.
Fabien added a commit: rABC8937ba594953: test: use MiniWallet for p2p_filter.py.

Revision Contents
Changeset List

PathSize
test/
functional/
50 lines
test_framework/
45 lines

Diff 36900

View Options

test/functional/p2p_filter.py

Loading...
View Options

test/functional/test_framework/wallet.py

Loading...