Chia-Network / chia-blockchain / 22738296097

91%

Build Type

push

github

Committed by web-flow

Commit Message

Add Remote Wallet and new RPC calls (#20552)

* initial commit

* add init files

* only save info if it changes

* ruff

* persist properly

* ruff and test fix

* enforce only a single RemoteWallet

* initial commit

* add init files

* only save info if it changes

* ruff

* persist properly

* ruff and test fix

* enforce only a single RemoteWallet

* remove name_generation as it's unnecessary as wallet is 1 of 1

* remove dead code

* fix bug with early continue in WSM

* use real RPC client calls instead of fetch

* remove time_out_assert

* no .secret()s

* delete name generation test and move get_remote_wallet to wsm

* use process_pending_states

* use more real context and fewer Mock() calls, and parameterise "trusted"

* use get_existing_remote_wallet in register_remote_coins

* simplify wsm logic around remote wallet selection

* use wallet_id in register_remote_coins()

* catch corner case of wsm processing

* mypy fix

* don't accept irrelevant info for REMOTE_WALLET

* remove redundant code branch and add belt and braces check to if elsewhere

* don't specify trusted

* add test for persistence during restart of WSM

* stop crash on height is None

* remove dead code path in wallet post_processing

* fail on RPC call when wallet already exists

* no fallback for data loading

* simplify post-init processing for extraneuos params

* use hex bytes instead of JSON for RemoteInfo serialisation

* RPC call uses WSM lock

* fix bug with coinrecord updating in remote wallet

* remove uninstantiated CoinInfo for RemoteWallet

* only one uint32 cast

* remove lingering json walletinfo

* don't fetch children for remote_wallet - we don't care!

* move to an SQL database

* ruff

* add comments to test_confirming_txs_not_ours_with_remote_interest_coin to explain what's being tested and why

* remove unused functions

* add state_changed calls to remote wallet code paths in wsm

* store blobs

* add confirmation logic to remote wallet special case
... (continued)

Run Details

10407 of 11585 branches covered (89.83%)

Branch coverage included in aggregate %.

437 of 440 new or added lines in 12 files covered. (99.32%)

50 existing lines in 11 files now uncovered.

105207 of 115294 relevant lines covered (91.25%)

1.83 hits per line