diff options
| author | DanConwayDev <DanConwayDev@protonmail.com> | 2025-12-11 13:19:08 +0000 |
|---|---|---|
| committer | DanConwayDev <DanConwayDev@protonmail.com> | 2025-12-11 13:19:08 +0000 |
| commit | 9d1545b7b13bc6cd3b0a075865918ceaad9e07ff (patch) | |
| tree | 6995cd7824903ad97448dcc7d0e2437f48e0f59d /tests | |
| parent | 7146ee550944c71a4e48018a34820ce9a9d99f95 (diff) | |
fix: sync metrics aggregate relay counts
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/sync/metrics.rs | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/tests/sync/metrics.rs b/tests/sync/metrics.rs index 98ce4f3..26d379d 100644 --- a/tests/sync/metrics.rs +++ b/tests/sync/metrics.rs | |||
| @@ -374,8 +374,6 @@ async fn test_startup_sync_event_count() { | |||
| 374 | /// This test validates that when sync cannot connect to a source relay, | 374 | /// This test validates that when sync cannot connect to a source relay, |
| 375 | /// the connection_attempts_total counter with result="failure" increases. | 375 | /// the connection_attempts_total counter with result="failure" increases. |
| 376 | /// | 376 | /// |
| 377 | /// NOTE: This test may fail until sync metrics recording is fully wired up. | ||
| 378 | /// The test documents the expected behavior. | ||
| 379 | #[tokio::test] | 377 | #[tokio::test] |
| 380 | async fn test_connection_failure_increments_counter() { | 378 | async fn test_connection_failure_increments_counter() { |
| 381 | let mut harness = MetricsTestHarness::with_sources(0).await; // No sources | 379 | let mut harness = MetricsTestHarness::with_sources(0).await; // No sources |
| @@ -489,8 +487,6 @@ async fn test_relay_connected_status() { | |||
| 489 | /// This test validates that `ngit_sync_relay_status` gauge transitions from | 487 | /// This test validates that `ngit_sync_relay_status` gauge transitions from |
| 490 | /// healthy (1) to degraded (2) or dead (3) when a relay cannot be connected to. | 488 | /// healthy (1) to degraded (2) or dead (3) when a relay cannot be connected to. |
| 491 | /// | 489 | /// |
| 492 | /// NOTE: This test may fail until sync metrics recording is fully wired up. | ||
| 493 | /// The test documents the expected behavior. | ||
| 494 | #[tokio::test] | 490 | #[tokio::test] |
| 495 | async fn test_health_state_degrades_on_failure() { | 491 | async fn test_health_state_degrades_on_failure() { |
| 496 | use crate::common::sync_helpers::MetricsTestHarness; | 492 | use crate::common::sync_helpers::MetricsTestHarness; |
| @@ -533,10 +529,7 @@ async fn test_health_state_degrades_on_failure() { | |||
| 533 | /// Note: Current implementation may only support one sync source, so this tests | 529 | /// Note: Current implementation may only support one sync source, so this tests |
| 534 | /// with one source, verifying tracked=1 and connected=1, then connected=0 after stopping. | 530 | /// with one source, verifying tracked=1 and connected=1, then connected=0 after stopping. |
| 535 | /// | 531 | /// |
| 536 | /// NOTE: This test may fail until sync metrics recording is fully wired up. | ||
| 537 | /// The test documents the expected behavior. | ||
| 538 | #[tokio::test] | 532 | #[tokio::test] |
| 539 | #[ignore] // Enable when relay tracking metrics are wired up | ||
| 540 | async fn test_multi_source_aggregate_counts() { | 533 | async fn test_multi_source_aggregate_counts() { |
| 541 | use crate::common::sync_helpers::MetricsTestHarness; | 534 | use crate::common::sync_helpers::MetricsTestHarness; |
| 542 | 535 | ||