From e4d51692f275ef3aa9830ea6dd1e68e843025331 Mon Sep 17 00:00:00 2001 From: DanConwayDev Date: Wed, 12 Aug 2026 16:00:16 +0000 Subject: [PATCH] style(test): remove needless URL borrows Apply the remaining Rust 1.96 clippy suggestions in live-sync tests. The metrics helper already accepts string references from TestRelay directly, so borrowing those references again is redundant. This is a mechanical test-only lint repair with no behavior change. Validated by the clean workspace clippy run in the preceding commit. --- tests/sync/live_sync.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/sync/live_sync.rs b/tests/sync/live_sync.rs index df0bbee..c626d25 100644 --- a/tests/sync/live_sync.rs +++ b/tests/sync/live_sync.rs @@ -53,7 +53,7 @@ async fn replacement_announcement_retires_old_relay_only_after_natural_disconnec async move { let deadline = tokio::time::Instant::now() + Duration::from_secs(10); loop { - let metrics = fetch_metrics(&relay_url).await.unwrap_or_default(); + let metrics = fetch_metrics(relay_url).await.unwrap_or_default(); if metrics.contains(&required) && absent.as_ref().is_none_or(|label| !metrics.contains(label)) { @@ -102,7 +102,7 @@ async fn replacement_announcement_retires_old_relay_only_after_natural_disconnec "replacement should establish the new relay" ); assert!( - fetch_metrics(&syncing.url()) + fetch_metrics(syncing.url()) .await .unwrap_or_default() .contains(&old_label), @@ -223,7 +223,7 @@ async fn live_sync_regroups_after_filter_count_refusal() { let refusal_deadline = tokio::time::Instant::now() + Duration::from_secs(10); loop { - let metrics = fetch_metrics(&syncing.url()).await.unwrap_or_default(); + let metrics = fetch_metrics(syncing.url()).await.unwrap_or_default(); if metrics.contains("ngit_sync_policy_refusals_total") && metrics.contains("filter_count") { break; }