fix(test): de-flake test_duration_collecting on fast CI runners

sassman · sassman · commit 895ef7c4e7f2 · 2026-05-06T12:36:18.000+02:00
Writing 384 bytes (3-byte buffer x 128) can complete in sub-
millisecond time on fast runners, so `as_millis() &gt; 0` flakes. The
intent is to confirm timing was accumulated (not Duration::default()),
which `as_nanos() &gt; 0` checks reliably.
diff --git a/src/utils.rs b/src/utils.rs
@@ -217,6 +217,9 @@ mod tests {
     #[test]
     fn test_duration_collecting() {
         let d = write_once(&[0xff, 0xff, 0xff], 128, &None).unwrap();
-        assert!(d.as_millis() > 0);
+        // Writing 384 bytes can finish in well under a millisecond on fast
+        // runners; assert the timing was accumulated at all rather than
+        // imposing an arbitrary lower bound.
+        assert!(d.as_nanos() > 0);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -217,6 +217,9 @@ mod tests {`
`217`	`217`	`#[test]`
`218`	`218`	`fn test_duration_collecting() {`
`219`	`219`	`let d = write_once(&[0xff, 0xff, 0xff], 128, &None).unwrap();`
`220`		`- assert!(d.as_millis() > 0);`
	`220`	`+ // Writing 384 bytes can finish in well under a millisecond on fast`
	`221`	`+ // runners; assert the timing was accumulated at all rather than`
	`222`	`+ // imposing an arbitrary lower bound.`
	`223`	`+ assert!(d.as_nanos() > 0);`
`221`	`224`	`}`
`222`	`225`	`}`