Skip to content

Commit e43bd17

Browse files
committed
move data counting to the BufferUsage
Signed-off-by: Vitalii Parfonov <vparfono@redhat.com>
1 parent 4c1d41d commit e43bd17

File tree

1 file changed

+0
-10
lines changed

1 file changed

+0
-10
lines changed

lib/vector-buffers/src/buffer_usage_data.rs

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -469,9 +469,6 @@ mod tests {
469469
.lock()
470470
.unwrap_or_else(std::sync::PoisonError::into_inner);
471471

472-
// Note: In the new architecture, the reporting loop runs in a single thread,
473-
// so we test the mathematical correctness of sequential updates instead
474-
475472
let num_increments = 10000;
476473
let mut updates = Vec::new();
477474

@@ -487,7 +484,6 @@ mod tests {
487484

488485
#[test]
489486
fn test_large_values_stay_within_bounds() {
490-
// Test that our gauge simulation handles large values correctly
491487
let large_value = F64_SAFE_INT_MAX / 2;
492488
let (events, bytes) =
493489
simulate_gauge_updates("test_buffer", 3, &[(large_value, large_value)]);
@@ -498,15 +494,10 @@ mod tests {
498494

499495
#[test]
500496
fn test_gauge_state_behavior_comprehensive() {
501-
// Test comprehensive gauge state behavior that mimics the original tests
502-
503-
// Test basic increment
504497
assert_gauge_simulation("buffer_alpha", 0, &[(100, 2048), (200, 1024)], 300, 3072);
505498

506-
// Test clamping to prevent negatives
507499
assert_gauge_simulation("buffer_beta", 1, &[(100, 1024), (-200, -4096)], 0, 0);
508500

509-
// Test mixed operations
510501
assert_gauge_simulation(
511502
"buffer_gamma",
512503
0,
@@ -515,7 +506,6 @@ mod tests {
515506
1000,
516507
);
517508

518-
// Test multiple buffer IDs work independently (conceptually)
519509
assert_gauge_simulation("buffer_delta", 0, &[(10, 100), (5, 50)], 15, 150);
520510
assert_gauge_simulation("buffer_epsilon", 0, &[(20, 200)], 20, 200);
521511
}

0 commit comments

Comments
 (0)