Skip to content

Commit c30e872

Browse files
committed
[storage] Add test for histogram number-to-bucket and bucket-to-number.
Signed-off-by: Ben Pfaff <blp@feldera.com>
1 parent 653253f commit c30e872

1 file changed

Lines changed: 22 additions & 0 deletions

File tree

crates/storage/src/histogram.rs

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -270,3 +270,25 @@ impl SlidingHistogram {
270270
self.sum -= sample.value;
271271
}
272272
}
273+
274+
#[cfg(test)]
275+
mod test {
276+
use crate::histogram::{bucket_to_range, number_to_bucket, N_BUCKETS};
277+
278+
#[test]
279+
fn buckets() {
280+
let mut base = 1;
281+
while base <= u64::MAX / 10 {
282+
for multiple in 1..10 {
283+
let number = base * multiple;
284+
for number in [number - 1, number, number + 1] {
285+
let bucket = number_to_bucket(number);
286+
assert!((0..N_BUCKETS).contains(&bucket));
287+
let bucket_range = bucket_to_range(bucket);
288+
assert!(bucket_range.contains(&number));
289+
}
290+
}
291+
base *= 10;
292+
}
293+
}
294+
}

0 commit comments

Comments
 (0)