Review Response

filecoin-project · Sep 4, 2023 · 60d12ed · 60d12ed
1 parent 55f3f7a
commit 60d12ed
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/actors/miner/src/testing.rs b/actors/miner/src/testing.rs
@@ -3,7 +3,9 @@ use crate::{
     SectorOnChainInfo, SectorPreCommitOnChainInfo, Sectors, State,
 };
 use fil_actors_runtime::runtime::Policy;
-use fil_actors_runtime::{parse_uint_key, Map, MessageAccumulator, DEFAULT_HAMT_CONFIG};
+use fil_actors_runtime::{
+    parse_uint_key, DealWeight, Map, MessageAccumulator, DEFAULT_HAMT_CONFIG,
+};
 use fvm_ipld_bitfield::BitField;
 use fvm_ipld_blockstore::Blockstore;
 use fvm_ipld_encoding::CborStore;
@@ -80,6 +82,8 @@ pub fn check_state_invariants<BS: Blockstore>(
                         DealSummary {
                             sector_start: sector.activation,
                             sector_expiration: sector.expiration,
+                            deal_weight: sector.deal_weight.clone(),
+                            verified_deal_weight: sector.verified_deal_weight.clone(),
                         },
                     );
                 }
@@ -141,6 +145,8 @@ pub fn check_state_invariants<BS: Blockstore>(
 pub struct DealSummary {
     pub sector_start: ChainEpoch,
     pub sector_expiration: ChainEpoch,
+    pub deal_weight: DealWeight,
+    pub verified_deal_weight: DealWeight,
 }
 
 pub struct StateSummary {

diff --git a/actors/miner/tests/prove_commit.rs b/actors/miner/tests/prove_commit.rs
@@ -85,8 +85,6 @@ fn prove_single_sector() {
 
     assert_eq!(precommit.info.seal_proof, sector.seal_proof);
     assert_eq!(precommit.info.sealed_cid, sector.sealed_cid);
-    assert_eq!(verified_deal_space, sector.verified_deal_weight);
-    assert_eq!(deal_space, sector.deal_weight);
     assert_eq!(*rt.epoch.borrow(), sector.activation);
     assert_eq!(precommit.info.expiration, sector.expiration);