* Port `nft_mint_nft`
* Fix tests
* More test fixes
* Port `nft_count_nfts`
* Port `nft_list_nfts`
* Fix tests
* Port `nft_set_nft_did`
* Fix tests
* Port `nft_set_did_bulk`
* Port `nft_transfer_bulk`
* Port `nft_get_by_did`
* Port `nft_get_wallet_did`
* fix tests
* Fix one more test
* Okay actually fix the test
* Port `nft_get_wallets_with_dids`
* Port `nft_set_nft_status`
* Port `nft_transfer_nft`
* Use kwargs
* Forgot default raise
* Port `nft_get_info`
* Port `nft_add_uri`
* d
* Port `nft_calculate_royalties`
* Fix tests
* Port `nft_mint_bulk`
* Add test coverage for cmd line offering CAT by asset ID
* Add test coverage for a more complex offer take
* Add test coverage for `NFTCalculateRoyalties.__post_init__()
* test coverage generally
* Actually dont commit puzzle hashes when push=False
* Inadvertent test breakage
* Fix tests
* Fix tests
* Fix test
* Fix hopefully one more test
* Comments by @altendky
* small improvement @altendky
Co-authored-by: Kyle Altendorf <sda@fstab.net>
* small improvement @altendky
Co-authored-by: Kyle Altendorf <sda@fstab.net>
---------
Co-authored-by: Kyle Altendorf <sda@fstab.net>
<!-- Merging Requirements:
- Please give your PR a title that is release-note friendly
- In order to be merged, you must add the most appropriate category
Label (Added, Changed, Fixed) to your PR
-->
<!-- Explain why this is an improvement (Does this add missing
functionality, improve performance, or reduce complexity?) -->
### Purpose:
This PR removes a dependency that is inappropriate and caused a
dependency loop.
<!-- Merging Requirements:
- Please give your PR a title that is release-note friendly
- In order to be merged, you must add the most appropriate category
Label (Added, Changed, Fixed) to your PR
-->
<!-- Explain why this is an improvement (Does this add missing
functionality, improve performance, or reduce complexity?) -->
The modules `chia.protocols` and `chia.server` form a dependency loop.
This change at least removes the direct dependency of `chia.protocols`
on `chia.server`.
### Purpose:
<!-- Does this PR introduce a breaking change? -->
### Current Behavior:
### New Behavior:
<!-- As we aim for complete code coverage, please include details
regarding unit, and regression tests -->
### Testing Notes:
<!-- Attach any visual examples, or supporting evidence (attach any
.gif/video/console output below) -->
* bump chia_rs
* update db_graftroot test
* update SerializedProgram and Program use, to accommodate new chia_rs interface
* review comments
* review comments
---------
Co-authored-by: Matthew Howard <m.o.howard97@gmail.com>
Move `get_unresolved_peer_infos` and `set_peer_info` from
`chia.util.config` to `chia.server.resolve_peer_info.py` so that
`chia.util` no longer depends upon `chia.server`.
* Add some checks when trying to join the same pool already joined
* Add test to test the specific cli case
* Remove line added during debugging
* Add test
* misc updates
* remove cli test
* final tests for joining same pool again
* Update check from code review
* update test framework
* Use click.ClickException for error
* Add check for wallet synced up front as the first thing checked for joining a pool
* Add test for unsynced RPC call
* store merkle, key, and value blobs in files
* update expected columns
* fixup tests
* more
* cleanup disk
* better wrapped path identification
* todos for potential filesystem locking
* suppress not precheck
* handle not-present new path configuration entries
* log exception
* replace hard coded 32
* three layers of directories for bytes paths
* add `chia dev data sync-time` for doing datalayer sync time tests
* add chia stop -d all at the end
* less subprocessing
* remove dead log file code
* add subcommand help messages
* correct for 3.9 support
* allow for days of timing
* format done message
* more info
* fixup
* fixup
* stuff
* more try finally
* add minutes
* add `--working-path`
* add chia dev gh test for launching configured test runs
* empty again
* better
* use the actual authenticated username
* more output
* helpful errors for invalid --only value
* tidy short help
* add at least an invalid --only test case
* prepare a long help
* add gh api runner helper
* use the commit hash for the temporary branch
* and uuid too
* fix up os selection
* add `--remote`
* load a url for a specified `--ref` as well
* both helps
* show defaults in `--help`
* help touchup
* Revert "show defaults in `--help`"
This reverts commit 5d7f4868e0.
* r
* add an actual dispatch test skeleton
* do not conflict check `tmp/**`
* assert a bit more about the resulting run
* oops
* skip the test for now
* pre-commit
* use a class for the chia cli context data
* actually assign to the context properly...
* handle the not-present case
* catch up
* tidy
* more
* stuff
* ...
* more real
* expected address prefix
* Extract TransactionsIn/Out
* Create a helper for coin selection arguments
* Port `chia wallet coins list`
* Unused item
* Add TransactionEndpoint
* Port `chia wallet coins combine`
* Port `chia wallet coins split`
* Delete test_coins.py
* Add test for TXConfig helpers
* Add test for TransactionEndpoint mixin
* Forgot to delete old split command
* test coverage
* coverage ignores
* Link old framework with new
* fix command tests
* fix signer test
* coverage ignores
* pre-commit
* Move tests back to rpc file
* Move coin functions back to their own file
* Move split above combine in rpc tests
* Stop duplicate name definitions
* new puzzle hashes allowed
* Address comments by @altendky
* more comments by @altendky
* Update chia/cmds/cmd_classes.py
Co-authored-by: Kyle Altendorf <sda@fstab.net>
* another comment by @altendky
* this one is the fault of @altendky
---------
Co-authored-by: Kyle Altendorf <sda@fstab.net>
* Port `vc_mint`
* Port `vc_spend`
* Port `vc_revoke`
* Port `vc_get`
* Port `vc_get_list`
* Port `vc_add_proofs`
* Port `vc_get_proofs_for_root`
* address comments by @altendky
Kyle Altendorf
Matt HauffandKyle Altendorf
Almog De Paz
Richard Kiss
Arvid NorbergandMatthew Howard
Earle Lowe
Freddie Coleman
RigidityandArvid Norberg
dependabot[bot]andKyle Altendorf
Amine Khaldi