test: use MiniWallet to simplify mempool_package_limits.py tests#25379
Merged
maflcko merged 2 commits intoAug 3, 2022
Conversation
Contributor
|
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. ConflictsNo conflicts as of last run. |
maflcko
reviewed
Jun 21, 2022
2b55bd3 to
fab7280
Compare
754874c to
1c9243b
Compare
Contributor
Author
|
Forced-pushed to take advantage of the logic introduced in #25445 |
maflcko
approved these changes
Aug 1, 2022
maflcko
left a comment
Member
There was a problem hiding this comment.
lgtm 1c9243b40db5746575ed55d09774872f0a98b24a 🐐
Show signature
Signature:
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512
lgtm 1c9243b40db5746575ed55d09774872f0a98b24a 🐐
-----BEGIN PGP SIGNATURE-----
iQGzBAEBCgAdFiEE+rVPoUahrI9sLGYTzit1aX5ppUgFAlwqrYAACgkQzit1aX5p
pUjMzgv8Ck6nSSmgAatlxCMG837PArkz0UTGPh4ek3WbN73Mn1dLmSCBsvlzbSVG
4yQlx+vMpDNUrJO07nU7fVszAKK5+BOFUcd1pElpZbIoOWHIudbDdEuOeOIDjuKC
368yHXtcXZUnx4ZUGJvC11LtkH2eZxjrwt70iVWqoNWMz2Wj3dNS01VGOkAN6Bb2
8wUWoNYwgm+iTNJIsQnzPuc9I7qjhILvhcbMTXKX/NQQi4StPq+5nBRFIXSePSuD
ApxcDIGIB2s3cYu/SswObSHtCtRKt/gaG0Xd7OkOMZ8bpNPU0vWiRAFtRn5OhKj7
2csGoyU1prVU52njdCYaC/EAiyWumqBmw/4CtffajgZ4UJluPhvn2ZBjAJDq9DAb
SiFfaITnyqEA327CahBPIkyTXmL15HC0zipUDjs0U5YYLby2z05h6i9htpQv+iCJ
eH1N+nmixN9aA6v1gkXNEs7qVcVWWLhLUuatmxzgaEr5pdhn+7701yNirhqu004l
JrbTuRmo
=dEET
-----END PGP SIGNATURE-----
Member
There was a problem hiding this comment.
Can remove all of these functions now?
Contributor
Author
There was a problem hiding this comment.
create_child_with_parents and make_chain are still used in rpc_packages.py from which I anticipate to remove on a follow-up PR.
8f11c3d to
b43ff6c
Compare
Member
|
Also, should be rebased |
b43ff6c to
48afe03
Compare
kouloumos
commented
Aug 1, 2022
Contributor
Author
There was a problem hiding this comment.
Rebased and addressed @MarcoFalke comments.
The failing CI task is also on master, it seems not to be related with the changes.
Contributor
Author
There was a problem hiding this comment.
create_child_with_parents and make_chain are still used in rpc_packages.py from which I anticipate to remove on a follow-up PR.
maflcko
reviewed
Aug 1, 2022
Moved `bulk_transaction` into MiniWallet class as `_bulk_tx` private helper method to be used when the newly added `target_weight` option is passed to `create_self_transfer*`
With this new method, a chain of transactions can be created. This method is introduced to further simplify the mempool_package_limits.py tests.
48afe03 to
f2f6068
Compare
Member
|
ACK f2f6068 👜 Show signatureSignature: |
Member
glozow
reviewed
Aug 2, 2022
glozow
left a comment
Member
There was a problem hiding this comment.
Concept ACK
I was just thinking it would be nice to get rid of these functions that are only used in package functional tests and use MiniWallet instead, thanks for working on this!
knst
added a commit
to knst/dash
that referenced
this pull request
Jan 4, 2026
Otherwise this error appears:
test_framework.authproxy.JSONRPCException: min relay fee not met, 25500 < 30332 (-26)
knst
added a commit
to knst/dash
that referenced
this pull request
Jan 4, 2026
Otherwise this error appears:
test_framework.authproxy.JSONRPCException: min relay fee not met, 25500 < 30332 (-26)
knst
added a commit
to knst/dash
that referenced
this pull request
Jan 6, 2026
Otherwise this error appears:
test_framework.authproxy.JSONRPCException: min relay fee not met, 25500 < 30332 (-26)
knst
added a commit
to knst/dash
that referenced
this pull request
Jan 7, 2026
Otherwise this error appears:
test_framework.authproxy.JSONRPCException: min relay fee not met, 25500 < 30332 (-26)
knst
added a commit
to knst/dash
that referenced
this pull request
Jan 24, 2026
Otherwise this error appears:
test_framework.authproxy.JSONRPCException: min relay fee not met, 25500 < 30332 (-26)
knst
added a commit
to knst/dash
that referenced
this pull request
Jan 24, 2026
Otherwise this error appears:
test_framework.authproxy.JSONRPCException: min relay fee not met, 25500 < 30332 (-26)
knst
added a commit
to knst/dash
that referenced
this pull request
Feb 6, 2026
Otherwise this error appears:
test_framework.authproxy.JSONRPCException: min relay fee not met, 25500 < 30332 (-26)
knst
added a commit
to knst/dash
that referenced
this pull request
Feb 10, 2026
Otherwise this error appears:
test_framework.authproxy.JSONRPCException: min relay fee not met, 25500 < 30332 (-26)
PastaPastaPasta
added a commit
to dashpay/dash
that referenced
this pull request
Feb 15, 2026
…itcoin#25957, bitcoin#25986, bitcoin#26158, bitcoin#26294, bitcoin#26349, bitcoin#26388, bitcoin#26404, bitcoin#26424, bitcoin#26625, bitcoin#26920, bitcoin#27350 2ca4df5 Merge bitcoin#27350: test: refactor: dedup mempool_package_limits.py subtests via decorator (glozow) f845321 Merge bitcoin#26625: test: Run mempool_packages.py with MiniWallet (fanquake) ce002be Merge bitcoin#25986: test: refactor `RPCPackagesTest` to use `MiniWallet` (glozow) 9cbc631 Merge bitcoin#25773: test: Target exact weight in MiniWallet _bulk_tx (MacroFake) f764261 fix: add missing changes and adjust fee rate to fix bitcoin#25379 (Konstantin Akimov) afb8d47 Merge bitcoin#25379: test: use MiniWallet to simplify mempool_package_limits.py tests (MacroFake) 302ed45 Merge bitcoin#24836: add RPC (-regtest only) for testing package policy (fanquake) f66147b Merge bitcoin#26294: build: move util/url to common/url (fanquake) d7e3cf6 Merge bitcoin#26424: doc: correct deriveaddresses RPC name (MacroFake) 4a15fe7 Merge bitcoin#26404: test: fix intermittent failure in rpc_getblockfrompeer.py (MacroFake) 4401e24 Merge bitcoin#26349: rpc: make `address` field optional `list{transactions, sinceblock}` response (Andrew Chow) 0a269f3 Merge bitcoin#26388: ci: Use `macos-ventura-xcode:14.1` image for "macOS native" task (MacroFake) c2d79d0 fmt: apply clang-format for blsl_dkg.cpp (Konstantin Akimov) b09f75f Merge bitcoin#26158: bench: add "priority level" to the benchmark framework (Andrew Chow) 5d2d43f Merge bitcoin#26920: doc: add release note for bitcoin#25957 (fast wallet rescan) (fanquake) 962e810 Merge bitcoin#25957: wallet: fast rescan with BIP157 block filters for descriptor wallets (Andrew Chow) Pull request description: ## What was done? Regular backports from bitcoin core v25 ## How Has This Been Tested? Run unit & functional tests ## Breaking Changes N/A ## Checklist: - [x] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have added or updated relevant unit/integration/functional/e2e tests - [ ] I have made corresponding changes to the documentation - [x] I have assigned this pull request to a milestone ACKs for top commit: UdjinM6: utACK 2ca4df5 Tree-SHA512: a5153869717e73a38fe8ae0b20cde36fefa84322d8852c21b13548c142836e3013ad4e75858ce1209bf3985f032faad0aabf6406e0710bbaff19d73149faf6fb
Shailendra53
pushed a commit
to CryptoTuraTeam/dash
that referenced
this pull request
Feb 27, 2026
Otherwise this error appears:
test_framework.authproxy.JSONRPCException: min relay fee not met, 25500 < 30332 (-26)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
While
wallet.pyincludes the MiniWallet class and some helper methods, it also includes some methods that have been moved there without having any direct relation with the MiniWallet class. Specificallymake_chain,create_child_with_parentsandcreate_raw_chainmethods that were extracted fromrpc_packages.pyat f8253d6 in order to be used on bothmempool_package_limits.pyandrpc_packages.py.Since that change, due to the introduction of additional methods in MiniWallet, the functionality of those methods can now be replicated with the existing MiniWallet methods and simultaneously simplify those tests by using the MiniWallet.
This PR's goals are
mempool_package_limits.pyfunctional tests with usage of the MiniWallet.make_chain,create_child_with_parentsandcreate_raw_chainmethods ofwallet.py.For the purpose of the aforementioned goals, a helper method
MiniWallet.send_self_transfer_chainis introduced and methodbulk_transactionhas been integrated increate_self_transfer*methods using an optionaltarget_weightoption.