From 29be4cb7d0fbd29325c995a76ba1b1f47beecca5 Mon Sep 17 00:00:00 2001 From: DanConwayDev Date: Fri, 9 Jan 2026 15:32:02 +0000 Subject: Fix sync tests after Syncing status introduction - Fix relay_connected() helper to check v >= 2 (Syncing/Connected states) - Fix unit test to use status value 3 (Connected) instead of 1 (Connecting) - Fix clippy warning: use .to_vec() instead of .iter().cloned().collect() All 61 sync integration tests now passing. All 238 unit tests passing. Clippy clean. --- tests/common/sync_helpers.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'tests/common/sync_helpers.rs') diff --git a/tests/common/sync_helpers.rs b/tests/common/sync_helpers.rs index 27422e9..d6a6ee4 100644 --- a/tests/common/sync_helpers.rs +++ b/tests/common/sync_helpers.rs @@ -708,7 +708,7 @@ impl ParsedMetrics { /// Check if a specific relay is connected pub fn relay_connected(&self, relay: &str) -> Option { self.gauge("ngit_sync_relay_connected", &[("relay", relay)]) - .map(|v| v == 1) + .map(|v| v >= 2) // Syncing (2), Connected (3), or ConnectedHistoricSyncFailures (4) } /// Get total number of connected relays @@ -1052,7 +1052,7 @@ mod tests { #[test] fn test_parse_metric_with_relay_url_label() { - let text = r#"ngit_sync_relay_connected{relay="ws://127.0.0.1:12345"} 1"#; + let text = r#"ngit_sync_relay_connected{relay="ws://127.0.0.1:12345"} 3"#; let metrics = ParsedMetrics::parse(text); assert_eq!(metrics.relay_connected("ws://127.0.0.1:12345"), Some(true)); } -- cgit v1.2.3