Improve documentation for remaining ignored tests

- Added detailed explanations for ledger utility tests (depend on external binary files)
- Added detailed explanations for node integration tests (investigate failure)
- Applied consistent ignore message format with explanations
- Better categorization of test types (utility vs integration vs unit tests)

Co-authored-by: dannywillems <[email protected]>

Author: Copilot

ledger/src/proofs/transaction.rs

@@ -4406,9 +4406,13 @@ pub(super) mod tests {
             .collect()
     }
 
+    /// Development utility test for converting transaction requests from binary files.
+    /// This test is ignored because it depends on specific binary test files that may not exist
+    /// and because mina_p2p_messages currently has compilation issues with serde traits.
+    /// This is primarily a development tool rather than a validation test.
     #[allow(unused)]
     #[test]
-    #[ignore]
+    #[ignore = "Development utility - depends on external binary files and has compilation issues"]
     fn test_convert_requests() {
         use binprot::BinProtWrite;
         use mina_p2p_messages::v2::*;
@@ -4669,10 +4673,12 @@ pub(super) mod tests {
         assert_eq!(hash, Fp::from_str(EXPECTED).unwrap());
     }
 
-    /// Print requests types
+    /// Development utility test for reading transaction request types from binary files.
+    /// This test is ignored because it depends on specific binary test files in circuit directories
+    /// and is primarily a development tool for analyzing transaction data.
     #[allow(unused)]
     #[test]
-    #[ignore]
+    #[ignore = "Development utility - depends on external binary files in circuit directories"]
     fn test_read_requests() {
         let path = Path::new(env!("CARGO_MANIFEST_DIR"))
             .join(devnet_circuit_directory())

node/testing/tests/single_node.rs

@@ -15,14 +15,20 @@ scenario_test!(
 );
 
 scenario_test!(
-    #[ignore = "investigate falure"]
+    /// Integration test for initial node joining behavior.
+    /// This test is ignored because it was failing and needs investigation.
+    /// It tests the basic connectivity when a node initially joins the network.
+    #[ignore = "Integration test failure - needs investigation of joining behavior"]
     initial_joining,
     SoloNodeBasicConnectivityInitialJoining,
     SoloNodeBasicConnectivityInitialJoining
 );
 
 scenario_test!(
-    #[ignore = "investigate falure"]
+    /// Integration test for root snarked ledger synchronization.
+    /// This test is ignored because it was failing and needs investigation.
+    /// It tests the node's ability to sync the root snarked ledger from peers.
+    #[ignore = "Integration test failure - needs investigation of ledger sync behavior"]
     sync_root_snarked_ledger,
     SoloNodeSyncRootSnarkedLedger,
     SoloNodeSyncRootSnarkedLedger