Fix review comments

massalabs · Oct 25, 2023 · 7d0fd79 · 7d0fd79
1 parent 951317b
commit 7d0fd79
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/massa-pos-exports/src/pos_final_state.rs b/massa-pos-exports/src/pos_final_state.rs
@@ -1572,7 +1572,7 @@ impl PoSFinalState {
     }
 }
 
-/// Helpers for test-exports
+/// Helpers for testing
 #[cfg(feature = "test-exports")]
 impl PoSFinalState {
     /// Queries all the deferred credits in the database

diff --git a/massa-protocol-worker/src/tests/ban_nodes_scenarios.rs b/massa-protocol-worker/src/tests/ban_nodes_scenarios.rs
@@ -147,6 +147,8 @@ fn test_protocol_bans_node_sending_block_header_with_invalid_signature() {
         ))),
     );
     ban_breakpoint.wait();
+
+    // After `unban_everyone_timer` the node should be unbanned
     unban_breakpoint.wait();
 }
 

diff --git a/massa-versioning/src/versioning.rs b/massa-versioning/src/versioning.rs
@@ -1836,7 +1836,7 @@ mod test {
             timeout: MassaTime::from_millis(5),
             activation_delay: MassaTime::from_millis(2),
         };
-        // Another versioning info (from an attacker) for test-exports
+        // Another versioning info (from an attacker) for testing
         let vi_2 = MipInfo {
             name: "MIP-0002".to_string(),
             version: 2,
-Original file line number
+Diff line change
@@ Expand Up @@
             ))),
         );
         ban_breakpoint.wait();
+        // After `unban_everyone_timer` the node should be unbanned
         unban_breakpoint.wait();
     }
@@ Expand Down @@