Forgotten commit for the filecoin-project#650 (filecoin-project#702)

Signed-off-by: Jakub Sztandera <[email protected]> Signed-off-by: Jakub Sztandera <[email protected]>
shamb0 · Jan 31, 2023 · 9a5c840 · 9a5c840
1 parent 07c522f
commit 9a5c840
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 6 deletions.
diff --git a/actors/miner/src/lib.rs b/actors/miner/src/lib.rs
@@ -5192,12 +5192,10 @@ impl ActorCode for Actor {
                 let res = Self::prove_replica_updates(rt, cbor::deserialize_params(params)?)?;
                 Ok(RawBytes::serialize(res)?)
             }
-            #[allow(unreachable_code)]
             Some(Method::PreCommitSectorBatch2) => {
                 Self::pre_commit_sector_batch2(rt, cbor::deserialize_params(params)?)?;
                 Ok(RawBytes::default())
             }
-            #[allow(unreachable_code)]
             Some(Method::ProveReplicaUpdates2) => {
                 let res = Self::prove_replica_updates2(rt, cbor::deserialize_params(params)?)?;
                 Ok(RawBytes::serialize(res)?)

diff --git a/actors/miner/tests/miner_actor_test_precommit_batch.rs b/actors/miner/tests/miner_actor_test_precommit_batch.rs
@@ -314,10 +314,6 @@ mod miner_actor_precommit_batch {
     #[test_case(false; "v1")]
     #[test_case(true; "v2")]
     fn duplicate_sector_rejects_batch(v2: bool) {
-        // This test does not enumerate all the individual conditions that could cause a single precommit
-        // to be rejected. Those are covered in the PreCommitSector tests, and we know that that
-        // method is implemented in terms of a batch of one.
-
         let period_offset = ChainEpoch::from(100);
 
         let h = ActorHarness::new_with_options(HarnessOptions {