From: Arik Sosman <git@arik.io>
Date: Sat, 14 Jan 2023 15:17:12 +0000 (-0800)
Subject: Add error messages to stale gossip cleanup assertions.
X-Git-Tag: v0.0.114-beta~64^2~1
X-Git-Url: http://git.bitcoin.ninja/?a=commitdiff_plain;h=e9d4ae1d3ea3ccfad4ea5ab209d733d54e06ed93;p=rust-lightning

Add error messages to stale gossip cleanup assertions.

Should help debug #1914.
---

diff --git a/lightning/src/routing/gossip.rs b/lightning/src/routing/gossip.rs
index d0682d47d..d90e4c437 100644
--- a/lightning/src/routing/gossip.rs
+++ b/lightning/src/routing/gossip.rs
@@ -2542,13 +2542,13 @@ mod tests {
 			// Should not remove from tracking if insufficient time has passed
 			network_graph.remove_stale_channels_and_tracking_with_time(
 				tracking_time + REMOVED_ENTRIES_TRACKING_AGE_LIMIT_SECS - 1);
-			assert_eq!(network_graph.removed_channels.lock().unwrap().len(), 1);
+			assert_eq!(network_graph.removed_channels.lock().unwrap().len(), 1, "Removed channel count ≠ 1 with tracking_time {}", tracking_time);
 
 			// Provide a later time so that sufficient time has passed
 			network_graph.remove_stale_channels_and_tracking_with_time(
 				tracking_time + REMOVED_ENTRIES_TRACKING_AGE_LIMIT_SECS);
-			assert!(network_graph.removed_channels.lock().unwrap().is_empty());
-			assert!(network_graph.removed_nodes.lock().unwrap().is_empty());
+			assert!(network_graph.removed_channels.lock().unwrap().is_empty(), "Unexpectedly removed channels with tracking_time {}", tracking_time);
+			assert!(network_graph.removed_nodes.lock().unwrap().is_empty(), "Unexpectedly removed nodes with tracking_time {}", tracking_time);
 		}
 
 		#[cfg(not(feature = "std"))]