Small edits

Author: sourabhmarathe

lightning/src/ln/functional_tests.rs

@@ -3907,6 +3907,7 @@ fn test_no_txn_manager_serialize_deserialize() {
 
 #[test]
 fn test_manager_serialize_deserialize_events() {
+    // This test makes sure the events field in ChannelManager survives de/serialization
 	let chanmon_cfgs = create_chanmon_cfgs(2);
 	let node_cfgs = create_node_cfgs(2, &chanmon_cfgs);
 	let node_chanmgrs = create_node_chanmgrs(2, &node_cfgs, &[None, None]);
@@ -3916,7 +3917,7 @@ fn test_manager_serialize_deserialize_events() {
 	let nodes_0_deserialized: ChannelManager<EnforcingChannelKeys, &test_utils::TestChannelMonitor, &test_utils::TestBroadcaster, &test_utils::TestKeysInterface, &test_utils::TestFeeEstimator>;
 	let mut nodes = create_network(2, &node_cfgs, &node_chanmgrs);
 
-	// Start creating the channel, but stop right before broadcasting the event message FundingBroadcastSafe
+	// Start creating a channel, but stop right before broadcasting the event message FundingBroadcastSafe
 	let channel_value = 100000;
 	let push_msat = 10001;
 	let a_flags = InitFeatures::known();
@@ -3947,6 +3948,7 @@ fn test_manager_serialize_deserialize_events() {
 		assert_eq!(added_monitors[0].0, funding_output);
 		added_monitors.clear();
 	}
+    // Normally, this is where node_a would check for a FundingBroadcastSafe event, but the test de/serializes first instead
 
 	nodes.push(node_a);
 	nodes.push(node_b);
@@ -3998,7 +4000,7 @@ fn test_manager_serialize_deserialize_events() {
 		_ => panic!("Unexpected event"),
 	};
 
-	// Make sure the channel is functioning as thought the de/serialization never happened
+	// Make sure the channel is functioning as though the de/serialization never happened
 	nodes[0].block_notifier.register_listener(nodes[0].node);
 	assert_eq!(nodes[0].node.list_channels().len(), 1);
 	check_added_monitors!(nodes[0], 1);