add(rpc): note tree sizes to getblock api

[oxarbitrage]

Motivation

We need to add sapling and orchard tree sizes to getblock method.

Close #6952

Solution

We have state methods to get what we need from a block.

This PR is a draft until it passed newly added CI tests.

Review

Reviewer Checklist

• Will the PR name make sense to users?
  • Does it need extra CHANGELOG info? (new features, breaking changes, large changes)
• Are the PR labels correct?
• Does the code do what the ticket and PR says?
  • Does it change concurrent code, unsafe code, or consensus rules?
• How do you know it works? Does it have tests?

Follow Up Work

[arya2]

@Mergifyio update

[mergify]

update

☑️ Nothing to do

• #commits-behind>0 [:pushpin: update requirement]
• -closed [:pushpin: update requirement]

[oxarbitrage]

@Mergifyio update

[mergify]

update

✅ Branch has been successfully updated

[oxarbitrage]

I am still not sure what is going on with the CI here as it seems that it still using the old lightwalletd cache.

I tried running the test of this PR locally and i was able to see the debug log i added here and the test pass:

$ cargo test wallet_grpc_tests --features lightwalletd-grpc-tests -- --ignored --nocapture
...
2023-08-11T23:07:28.892747Z  INFO Some(ChainMetadata { sapling_commitment_tree_size: 1170439, orchard_commitment_tree_size: 2 })
...
test lightwalletd_wallet_grpc_tests ... ok

[oxarbitrage]

./zcash-rpc-diff 6666 getblock 2000000

./zcash-rpc-diff 6666 getblock 2000001

./zcash-rpc-diff 6666 getblock 2000002

./zcash-rpc-diff 6666 getblock 1890007

/zcash-rpc-diff 6666 getblock 1600000

[arya2]

This looks great, thank you!

[arya2]

@Mergifyio refresh

[mergify]

refresh

✅ Pull request refreshed

[gustavovalverde]

@Mergifyio refresh

[mergify]

refresh

✅ Pull request refreshed