Motivation
The functional test rpc_signrawtransactionwithkey.py currently creates funding transactions by spending the coinbases of newly mined blocks, which is rather slow and leads to unnecessary complex code (see method send_to_address). It would be simpler to use a MiniWallet instance for that purpose, in combination with the pre-generated test chain which already contains UTXOs for MiniWallet’s default mode ADDRESS_OP_TRUE. That way, the test code is not cluttered with low-level tx details of funding tx creation and more readable. While touching the test, other small improvements could be considered as well (e.g. do we really need spin up two different nodes to achieve our testing goals?).
Possible solution
No response
Useful Skills
- Basic Python 3 skills
- Basic understanding of transactions, difference between output scripts and addresses
- Being familiar with MiniWallet’s capabilities (how to send funds to an arbitrary output script?)
Guidance for new contributors
Want to work on this issue?
For guidance on contributing, please read CONTRIBUTING.md before opening your pull request.
