Apply suggestions from code review

Co-authored-by: Marek <mail@marek.onl>
ZcashFoundation · Jun 27, 2024 · ce27a78 · ce27a78
1 parent 78fff82
commit ce27a78
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/zebra-chain/src/parameters/network.rs b/zebra-chain/src/parameters/network.rs
@@ -198,7 +198,7 @@ impl Network {
     /// Mandatory checkpoints are a Zebra-specific feature.
     /// If a Zcash consensus rule only applies before the mandatory checkpoint,
     /// Zebra can skip validation of that rule.
-    ///
+    /// This is necessary because Zebra can't fully validate the blocks prior to Canopy.
     // TODO:
     // - Support constructing pre-Canopy coinbase tx and block templates and return `Height::MAX` instead of panicking
     //   when Canopy activation height is `None` (#8434)

diff --git a/zebra-chain/src/parameters/network/tests/prop.rs b/zebra-chain/src/parameters/network/tests/prop.rs
@@ -8,10 +8,8 @@ use crate::{
 
 proptest! {
     /// Check that the mandatory checkpoint is immediately before Canopy activation.
-    ///
-    /// This is necessary because Zebra can't fully validate the blocks prior to Canopy.
     #[test]
-    fn mandatory_checkpoint_is_after_canopy(network in any::<Network>()) {
+    fn mandatory_checkpoint_is_immediately_before_canopy(network in any::<Network>()) {
         let _init_guard = zebra_test::init();
 
         let pre_canopy_activation = NetworkUpgrade::Canopy