From 9a5c8402fbf7529c0fbb7ca5b19cc664e4da561b Mon Sep 17 00:00:00 2001 From: Jakub Sztandera Date: Thu, 22 Sep 2022 21:31:53 +0200 Subject: [PATCH] Forgotten commit for the #650 (#702) Signed-off-by: Jakub Sztandera Signed-off-by: Jakub Sztandera --- actors/miner/src/lib.rs | 2 -- actors/miner/tests/miner_actor_test_precommit_batch.rs | 4 ---- 2 files changed, 6 deletions(-) diff --git a/actors/miner/src/lib.rs b/actors/miner/src/lib.rs index 16f9eb388..44c6caef2 100644 --- 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 index 3b30f8d40..710f8dfb2 100644 --- 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 {