changelog and test snapshot

hanabi1224 · hanabi1224 · commit d7517f1191e2 · 2025-04-01T00:43:54.000+08:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -32,6 +32,7 @@
 ### Added
 
 - [#5461](https://github.com/ChainSafe/forest/pull/5461) Add `forest-tool shed migrate-state` command.
+- [#5488](https://github.com/ChainSafe/forest/pull/5488) Add paritial support for the `Filecoin.StateCompute` RPC method.
 
 ### Changed
 
diff --git a/src/blocks/tipset.rs b/src/blocks/tipset.rs
@@ -555,7 +555,7 @@ mod lotus_json {
 
     use super::TipsetKey;
 
-    #[derive(Clone, JsonSchema)]
+    #[derive(Debug, PartialEq, Clone, JsonSchema)]
     #[schemars(rename = "Tipset")]
     pub struct TipsetLotusJson(#[schemars(with = "TipsetLotusJsonInner")] Tipset);
 
diff --git a/src/interpreter/vm.rs b/src/interpreter/vm.rs
@@ -357,7 +357,7 @@ where
     ) -> ApplyBlockResult {
         let mut receipts = Vec::new();
         let mut events = Vec::new();
-        let mut processed = HashSet::<Cid>::default();
+        let mut processed = HashSet::default();
 
         for block in messages.iter() {
             let mut penalty = TokenAmount::zero();
diff --git a/src/lotus_json/actor_state.rs b/src/lotus_json/actor_state.rs
@@ -5,7 +5,7 @@ use super::*;
 use crate::shim::{address::Address, econ::TokenAmount, state_tree::ActorState};
 use ::cid::Cid;
 
-#[derive(Clone, Serialize, Deserialize, JsonSchema)]
+#[derive(Debug, PartialEq, Clone, Serialize, Deserialize, JsonSchema)]
 #[serde(rename_all = "PascalCase")]
 #[schemars(rename = "ActorState")]
 pub struct ActorStateLotusJson {
@@ -22,7 +22,6 @@ pub struct ActorStateLotusJson {
     #[schemars(with = "LotusJson<Option<Address>>")]
     #[serde(
         with = "crate::lotus_json",
-        skip_serializing_if = "Option::is_none",
         default
     )]
     delegated_address: Option<Address>,
diff --git a/src/lotus_json/address.rs b/src/lotus_json/address.rs
@@ -4,7 +4,7 @@
 use super::*;
 use crate::shim::address::Address;
 
-#[derive(Clone, Serialize, Deserialize, JsonSchema)]
+#[derive(Debug, PartialEq, Clone, Serialize, Deserialize, JsonSchema)]
 #[schemars(rename = "Address")]
 pub struct AddressLotusJson(
     #[schemars(with = "String")]
diff --git a/src/lotus_json/allocation.rs b/src/lotus_json/allocation.rs
@@ -8,7 +8,7 @@ use fvm_shared4::clock::ChainEpoch;
 use fvm_shared4::piece::PaddedPieceSize;
 use fvm_shared4::ActorID;
 
-#[derive(Clone, Debug, Serialize, Deserialize, JsonSchema)]
+#[derive(Clone, Debug, PartialEq, Serialize, Deserialize, JsonSchema)]
 #[serde(rename_all = "PascalCase")]
 #[schemars(rename = "Allocation")]
 pub struct AllocationLotusJson {
diff --git a/src/lotus_json/beacon_entry.rs b/src/lotus_json/beacon_entry.rs
@@ -5,7 +5,7 @@ use crate::beacon::BeaconEntry;
 
 use super::*;
 
-#[derive(Clone, Serialize, Deserialize, JsonSchema)]
+#[derive(Debug, PartialEq, Clone, Serialize, Deserialize, JsonSchema)]
 #[serde(rename_all = "PascalCase")]
 #[schemars(rename = "BeaconEntry")]
 pub struct BeaconEntryLotusJson {
diff --git a/src/lotus_json/big_int.rs b/src/lotus_json/big_int.rs
@@ -5,7 +5,7 @@ use super::*;
 
 use num::BigInt;
 
-#[derive(Clone, Serialize, Deserialize, JsonSchema)]
+#[derive(Debug, PartialEq, Clone, Serialize, Deserialize, JsonSchema)]
 #[schemars(rename = "BigInt")]
 pub struct BigIntLotusJson(
     #[schemars(with = "String")]
diff --git a/src/lotus_json/bit_field.rs b/src/lotus_json/bit_field.rs
@@ -5,7 +5,7 @@ use super::*;
 
 use fil_actors_shared::fvm_ipld_bitfield::{json::BitFieldJson, BitField};
 
-#[derive(Serialize, Deserialize, JsonSchema)]
+#[derive(Debug, PartialEq, Serialize, Deserialize, JsonSchema)]
 #[schemars(rename = "BitField")]
 pub struct BitFieldLotusJson(#[schemars(with = "Option<Vec<u8>>")] pub BitFieldJson);
 
diff --git a/src/lotus_json/block_header.rs b/src/lotus_json/block_header.rs
@@ -16,7 +16,7 @@ use serde::{Deserialize, Serialize};
 
 use crate::blocks::{CachingBlockHeader, RawBlockHeader};
 
-#[derive(Clone, Serialize, Deserialize, JsonSchema)]
+#[derive(Debug, PartialEq, Clone, Serialize, Deserialize, JsonSchema)]
 #[serde(rename_all = "PascalCase")]
 #[schemars(rename = "BlockHeader")]
 pub struct BlockHeaderLotusJson {
diff --git a/src/lotus_json/cid.rs b/src/lotus_json/cid.rs
@@ -3,7 +3,7 @@
 
 use super::*;
 
-#[derive(Clone, Serialize, Deserialize, JsonSchema)]
+#[derive(Debug, PartialEq, Clone, Serialize, Deserialize, JsonSchema)]
 #[schemars(rename = "Cid")]
 pub struct CidLotusJson {
     #[schemars(with = "String")]
diff --git a/src/lotus_json/key_info.rs b/src/lotus_json/key_info.rs
@@ -4,7 +4,7 @@ use super::*;
 
 use crate::{key_management::KeyInfo, shim::crypto::SignatureType};
 
-#[derive(Clone, Serialize, Deserialize, JsonSchema)]
+#[derive(Debug, PartialEq, Clone, Serialize, Deserialize, JsonSchema)]
 #[serde(rename_all = "PascalCase")]
 #[schemars(rename = "KeyInfo")]
 pub struct KeyInfoLotusJson {
diff --git a/src/lotus_json/message.rs b/src/lotus_json/message.rs
@@ -6,7 +6,7 @@ use super::*;
 use crate::shim::{address::Address, econ::TokenAmount, message::Message};
 use fvm_ipld_encoding::RawBytes;
 
-#[derive(Clone, Serialize, Deserialize, JsonSchema)]
+#[derive(Debug, PartialEq, Clone, Serialize, Deserialize, JsonSchema)]
 #[serde(rename_all = "PascalCase")]
 #[schemars(rename = "Message")]
 pub struct MessageLotusJson {
diff --git a/src/lotus_json/miner_info.rs b/src/lotus_json/miner_info.rs
@@ -12,7 +12,7 @@ use fil_actor_miner_state::v12::{BeneficiaryTerm, PendingBeneficiaryChange};
 use fvm_ipld_encoding::BytesDe;
 use libp2p::PeerId;
 
-#[derive(Clone, Serialize, Deserialize, JsonSchema)]
+#[derive(Debug, PartialEq, Clone, Serialize, Deserialize, JsonSchema)]
 #[serde(rename_all = "PascalCase")]
 #[schemars(rename = "MinerInfo")]
 pub struct MinerInfoLotusJson {
diff --git a/src/lotus_json/miner_power.rs b/src/lotus_json/miner_power.rs
@@ -5,7 +5,7 @@ use super::*;
 use crate::shim::actors::miner::MinerPower;
 use crate::shim::actors::power::Claim;
 
-#[derive(Clone, Serialize, Deserialize, JsonSchema)]
+#[derive(Debug, PartialEq, Clone, Serialize, Deserialize, JsonSchema)]
 #[serde(rename_all = "PascalCase")]
 #[schemars(rename = "MinerPower")]
 pub struct MinerPowerLotusJson {
diff --git a/src/lotus_json/receipt.rs b/src/lotus_json/receipt.rs
@@ -7,7 +7,7 @@ use fvm_ipld_encoding::RawBytes;
 use super::*;
 use crate::shim::executor::Receipt;
 
-#[derive(Clone, Serialize, Deserialize, JsonSchema)]
+#[derive(Debug, PartialEq, Clone, Serialize, Deserialize, JsonSchema)]
 #[serde(rename_all = "PascalCase")]
 #[schemars(rename = "Receipt")]
 pub struct ReceiptLotusJson {
diff --git a/src/lotus_json/signature.rs b/src/lotus_json/signature.rs
@@ -4,7 +4,7 @@
 use super::*;
 use crate::shim::crypto::{Signature, SignatureType};
 
-#[derive(Clone, Serialize, Deserialize, JsonSchema)]
+#[derive(Debug, PartialEq, Clone, Serialize, Deserialize, JsonSchema)]
 #[serde(rename_all = "PascalCase")]
 #[schemars(rename = "Signature")]
 pub struct SignatureLotusJson {
diff --git a/src/lotus_json/signed_message.rs b/src/lotus_json/signed_message.rs
@@ -7,7 +7,7 @@ use ::cid::Cid;
 
 use super::*;
 
-#[derive(Clone, Serialize, Deserialize, JsonSchema)]
+#[derive(Debug, PartialEq, Clone, Serialize, Deserialize, JsonSchema)]
 #[serde(rename_all = "PascalCase")]
 #[schemars(rename = "SignedMessage")]
 pub struct SignedMessageLotusJson {
diff --git a/src/lotus_json/token_amount.rs b/src/lotus_json/token_amount.rs
@@ -5,7 +5,7 @@ use super::*;
 use crate::shim::econ::TokenAmount;
 use num::BigInt;
 
-#[derive(Clone, Serialize, Deserialize, JsonSchema)]
+#[derive(Debug, PartialEq, Clone, Serialize, Deserialize, JsonSchema)]
 #[serde(transparent)] // name the field for clarity
 #[schemars(rename = "TokenAmount")]
 pub struct TokenAmountLotusJson {
diff --git a/src/lotus_json/vec_u8.rs b/src/lotus_json/vec_u8.rs
@@ -7,7 +7,7 @@ use schemars::schema::*;
 // This code looks odd so we can
 // - use #[serde(with = "...")]
 // - de/ser empty vecs as null
-#[derive(Clone, Serialize, Deserialize)]
+#[derive(Debug, PartialEq, Clone, Serialize, Deserialize)]
 pub struct VecU8LotusJson(Option<Inner>);
 
 impl JsonSchema for VecU8LotusJson {
@@ -26,7 +26,7 @@ impl JsonSchema for VecU8LotusJson {
     }
 }
 
-#[derive(Clone, Serialize, Deserialize)]
+#[derive(Debug, PartialEq, Clone, Serialize, Deserialize)]
 struct Inner(#[serde(with = "base64_standard")] Vec<u8>);
 
 impl HasLotusJson for Vec<u8> {
diff --git a/src/rpc/methods/chain.rs b/src/rpc/methods/chain.rs
@@ -750,7 +750,7 @@ fn load_api_messages_from_tipset(
     Ok(messages)
 }
 
-#[derive(Clone, PartialEq, Serialize, Deserialize, JsonSchema)]
+#[derive(Debug, Clone, PartialEq, Serialize, Deserialize, JsonSchema)]
 pub struct BlockMessages {
     #[serde(rename = "BlsMessages", with = "crate::lotus_json")]
     #[schemars(with = "LotusJson<Vec<Message>>")]
diff --git a/src/rpc/methods/chain/types.rs b/src/rpc/methods/chain/types.rs
@@ -12,7 +12,7 @@ pub struct ObjStat {
 }
 lotus_json_with_self!(ObjStat);
 
-#[derive(Serialize, Deserialize, JsonSchema, Clone)]
+#[derive(Debug, Clone, Serialize, Deserialize, JsonSchema, PartialEq)]
 #[serde(rename_all = "PascalCase")]
 pub struct Event {
     /// Actor ID
diff --git a/src/rpc/methods/eth.rs b/src/rpc/methods/eth.rs
@@ -409,13 +409,32 @@ impl ExtBlockNumberOrHash {
     }
 }
 
-#[derive(PartialEq, Debug, Clone, Serialize, Deserialize, JsonSchema)]
+#[derive(Debug, Clone, Serialize, Deserialize, JsonSchema)]
 #[serde(untagged)] // try a Vec<String>, then a Vec<Tx>
 pub enum Transactions {
     Hash(Vec<String>),
     Full(Vec<ApiEthTx>),
 }
 
+impl Transactions {
+    pub fn is_empty(&self) -> bool {
+        match self {
+            Self::Hash(v) => v.is_empty(),
+            Self::Full(v) => v.is_empty(),
+        }
+    }
+}
+
+impl PartialEq for Transactions {
+    fn eq(&self, other: &Self) -> bool {
+        match (self, other) {
+            (Self::Hash(a), Self::Hash(b)) => a == b,
+            (Self::Full(a), Self::Full(b)) => a == b,
+            _ => self.is_empty() && other.is_empty(),
+        }
+    }
+}
+
 impl Default for Transactions {
     fn default() -> Self {
         Self::Hash(vec![])
@@ -516,15 +535,15 @@ impl ApiEthTx {
     }
 }
 
-#[derive(Debug, Clone, Default)]
+#[derive(Debug, Clone, Default, PartialEq, Eq)]
 pub struct EthSyncingResult {
     pub done_sync: bool,
     pub starting_block: i64,
     pub current_block: i64,
     pub highest_block: i64,
 }
 
-#[derive(Debug, Clone, Serialize, Deserialize, JsonSchema)]
+#[derive(Debug, PartialEq, Clone, Serialize, Deserialize, JsonSchema)]
 #[serde(untagged)]
 pub enum EthSyncingResultLotusJson {
     DoneSync(bool),
diff --git a/src/rpc/methods/eth/types.rs b/src/rpc/methods/eth/types.rs
@@ -493,7 +493,7 @@ lotus_json_with_self!(EthFilterSpec);
 /// - A list of block hashes
 /// - A list of transaction hashes
 /// - Or a list of logs
-#[derive(PartialEq, Debug, Clone, Serialize, Deserialize, JsonSchema)]
+#[derive(Debug, Clone, Serialize, Deserialize, JsonSchema)]
 #[serde(untagged)]
 pub enum EthFilterResult {
     Blocks(Vec<EthHash>),
@@ -502,6 +502,27 @@ pub enum EthFilterResult {
 }
 lotus_json_with_self!(EthFilterResult);
 
+impl EthFilterResult {
+    pub fn is_empty(&self) -> bool {
+        match self {
+            Self::Blocks(v) => v.is_empty(),
+            Self::Txs(v) => v.is_empty(),
+            Self::Logs(v) => v.is_empty(),
+        }
+    }
+}
+
+impl PartialEq for EthFilterResult {
+    fn eq(&self, other: &Self) -> bool {
+        match (self, other) {
+            (Self::Blocks(a), Self::Blocks(b)) => a == b,
+            (Self::Txs(a), Self::Txs(b)) => a == b,
+            (Self::Logs(a), Self::Logs(b)) => a == b,
+            _ => self.is_empty() && other.is_empty(),
+        }
+    }
+}
+
 #[derive(PartialEq, Default, Serialize, Deserialize, Debug, Clone, JsonSchema)]
 #[serde(rename_all = "camelCase")]
 pub struct EthCallTraceAction {
diff --git a/src/rpc/methods/misc.rs b/src/rpc/methods/misc.rs
@@ -69,7 +69,7 @@ pub struct ActorEventBlock {
     pub value: LotusJson<Vec<u8>>,
 }
 
-#[derive(PartialEq, Clone, JsonSchema, Serialize, Deserialize)]
+#[derive(Debug, PartialEq, Clone, JsonSchema, Serialize, Deserialize)]
 #[serde(rename_all = "camelCase")]
 pub struct ActorEvent {
     pub entries: Vec<EventEntry>,
diff --git a/src/rpc/methods/net/types.rs b/src/rpc/methods/net/types.rs
@@ -8,7 +8,7 @@ use schemars::JsonSchema;
 use serde::{Deserialize, Serialize};
 
 // Net API
-#[derive(Serialize, Deserialize, Clone, Debug, JsonSchema)]
+#[derive(Serialize, Deserialize, Clone, Debug, PartialEq, JsonSchema)]
 #[serde(rename_all = "PascalCase")]
 pub struct AddrInfo {
     #[serde(rename = "ID")]
@@ -27,7 +27,7 @@ impl AddrInfo {
     }
 }
 
-#[derive(Debug, Default, Serialize, Deserialize, Clone, JsonSchema)]
+#[derive(Debug, Default, Serialize, Deserialize, Clone, JsonSchema, PartialEq)]
 pub struct NetInfoResult {
     pub num_peers: usize,
     pub num_connections: u32,
@@ -52,7 +52,7 @@ impl From<libp2p::swarm::NetworkInfo> for NetInfoResult {
     }
 }
 
-#[derive(Debug, Default, Serialize, Deserialize, Clone, JsonSchema)]
+#[derive(Debug, Default, Serialize, Deserialize, Clone, JsonSchema, PartialEq)]
 #[serde(rename_all = "PascalCase")]
 pub struct NatStatusResult {
     pub reachability: i32,
diff --git a/src/rpc/methods/node.rs b/src/rpc/methods/node.rs
@@ -69,28 +69,28 @@ impl RpcMethod<0> for NodeStatus {
     }
 }
 
-#[derive(Debug, Serialize, Deserialize, Default, Clone, JsonSchema)]
+#[derive(Debug, PartialEq, Eq, Serialize, Deserialize, Default, Clone, JsonSchema)]
 pub struct NodeSyncStatus {
     pub epoch: u64,
     pub behind: u64,
 }
 lotus_json_with_self!(NodeSyncStatus);
 
-#[derive(Debug, Serialize, Deserialize, Default, Clone, JsonSchema)]
+#[derive(Debug, PartialEq, Eq, Serialize, Deserialize, Default, Clone, JsonSchema)]
 pub struct NodePeerStatus {
     pub peers_to_publish_msgs: u32,
     pub peers_to_publish_blocks: u32,
 }
 lotus_json_with_self!(NodePeerStatus);
 
-#[derive(Debug, Serialize, Deserialize, Default, Clone, JsonSchema)]
+#[derive(Debug, PartialEq, Serialize, Deserialize, Default, Clone, JsonSchema)]
 pub struct NodeChainStatus {
     pub blocks_per_tipset_last_100: f64,
     pub blocks_per_tipset_last_finality: f64,
 }
 lotus_json_with_self!(NodeChainStatus);
 
-#[derive(Debug, Deserialize, Default, Serialize, Clone, JsonSchema)]
+#[derive(Debug, Deserialize, Default, Serialize, Clone, JsonSchema, PartialEq)]
 pub struct NodeStatusResult {
     pub sync_status: NodeSyncStatus,
     pub peer_status: NodePeerStatus,
diff --git a/src/rpc/methods/state.rs b/src/rpc/methods/state.rs
diff --git a/src/rpc/methods/state/types.rs b/src/rpc/methods/state/types.rs
diff --git a/src/rpc/methods/sync.rs b/src/rpc/methods/sync.rs
diff --git a/src/rpc/types/mod.rs b/src/rpc/types/mod.rs
diff --git a/src/shim/executor.rs b/src/shim/executor.rs
diff --git a/src/state_manager/mod.rs b/src/state_manager/mod.rs
diff --git a/src/tool/subcommands/api_cmd/api_compare_tests.rs b/src/tool/subcommands/api_cmd/api_compare_tests.rs
diff --git a/src/tool/subcommands/api_cmd/test_snapshot.rs b/src/tool/subcommands/api_cmd/test_snapshot.rs
diff --git a/src/tool/subcommands/api_cmd/test_snapshots.txt b/src/tool/subcommands/api_cmd/test_snapshots.txt
