* Add get_network_info endpoint to the base rpc_server class, remove dupe implementations from wallet/full_node
Enables also getting network info from other RPC servers (crawler, etc)
* Add get_network_info test to full_node
* Remove unused var
* Update tests/core/test_full_node_rpc.py
Co-authored-by: Kyle Altendorf <sda@fstab.net>
* Use context manager for client
* Add success: true to the assertion to be complete
* Remove the **
---------
Co-authored-by: Kyle Altendorf <sda@fstab.net>
* Use WalletTestFramework for test_cat_wallet::test_cat_spend
* Fix trade test
* Fix RPC test
* Port tests/wallet/nft_wallet/test_nft_wallet.py::test_nft_wallet_creation_and_transfer to WalletTestFramework
* Get new tx records from add_pending_transactions
* couple more test files
* Remove unused parameter
* Unroll some test cruft
* ('preserve tuple',)
* isort
* update test
* Update the Wallet RPC Api call in is_transaction_confirmed, and remove the no longer used user_wallet_id param.
---------
Co-authored-by: Amine Khaldi <amine.khaldi@reactos.org>
* Don't autopush transactions by default anywhere
* Remove ignore_max_send_amount
* Use TXConfig instead of pending_transactions for coin exclusion in spend_clawback_coins
* Unify transaction pushing into a single method
* Address offline comments by @emlowe
* Rework max send amount to be based on coin quantity rather than amount
* Return offer transactions from create_offer_for_ids
* Return transaction list from CATWallet.create_new_cat_wallet
* Add a marshaller for RPC APIs
* Missed an instance in a newer test
* Missed an instance in a newer test
* Rename bad variable names I guess????
* Fix DL wallet transaction generation
* Add RPCEndpoint comment
* Address comments by @altendky
* Just go back to Streamable for requests
* Just go back to Streamable for requests
* Take suggestion from altendky
Co-authored-by: Kyle Altendorf <sda@fstab.net>
* Terrible suggestion by @altendky
* Inadvertent changes
* Formatting
* Add path for raises in endpoints
* pylint
* We're aiming for WalletRpcApi at the moment and we plan to extend this eventually.
* Import WalletRpcApi only when type checking.
* Sure sure Mr. Pylint..
* Add backwards compatibility code
* Better comment
* variable referenced before assignment
---------
Co-authored-by: Kyle Altendorf <sda@fstab.net>
Co-authored-by: Amine Khaldi <amine.khaldi@reactos.org>
* transition validate_removals() to use the fast function to compute a merkle tree root
* add type annotations to test_merkle_set.py
* add tests for validate_removals() and validate_additions(), just the case of including all items
* Send RewardChainBlockUnfinished to the harvester
- Simplify data pasing in request_signed_values
* Fix instances where the rc_block_unfinished must be sent
* Lint, fix test, generate protocol messages
* Rate Limit Adjustment for RequestSignedValues (#17481)
* Further bump request_signed_values rate limit
---------
Co-authored-by: Amine Khaldi <amine.khaldi@reactos.org>
Co-authored-by: Luna <96448399+DaOneLuna@users.noreply.github.com>
* undo redundant type-casts from slots, SubEpochSummary and SubEpochData
* undo redundant type-casts from proof-of-space, reward chain and foliage types
* undo redundant type-casts from VDFInfo, VDFProof and ClassgroupElement
* remove unnecessary type annotation
* Don't autopush transactions by default anywhere
* Remove ignore_max_send_amount
* Use TXConfig instead of pending_transactions for coin exclusion in spend_clawback_coins
* Unify transaction pushing into a single method
* Address offline comments by @emlowe
* Rework max send amount to be based on coin quantity rather than amount
* Return offer transactions from create_offer_for_ids
* Return transaction list from CATWallet.create_new_cat_wallet
* Missed an instance in a newer test
* Missed an instance in a newer test
* Simplify processing some transaction records.
---------
Co-authored-by: Amine Khaldi <amine.khaldi@reactos.org>