Summary
The snapshots() function is already implemented for all StateDecodeParams types, but it is not being called or used for types outside the Market actor types, such as Miner, Account, etc.
To ensure code quality across all relevant domain types, we need to invoke snapshots() for these types, similar to how we use the test_snapshots macro for Market actor param types.
Additional Links & Resources
https://github.com/ChainSafe/forest/blob/main/src/lotus_json/actors/params/market_params.rs#L1566-L1592
Summary
The
snapshots()function is already implemented for allStateDecodeParamstypes, but it is not being called or used for types outside theMarketactor types, such asMiner,Account, etc.To ensure code quality across all relevant domain types, we need to invoke
snapshots()for these types, similar to how we use thetest_snapshotsmacro forMarketactor param types.Additional Links & Resources
https://github.com/ChainSafe/forest/blob/main/src/lotus_json/actors/params/market_params.rs#L1566-L1592