Skip to content

Commit 2e26b2e

Browse files
doublegateclaude
andcommitted
fix(ci): resolve cross-platform test failures on macOS and Windows
- migration.rs: Increase RTT test tolerance from 200ms to 1s for CI environments - macOS CI can have variable timing, causing test_rtt_measurement to fail - The test sleeps 100ms, so 1s tolerance is generous but reliable for CI - io_uring.rs: Make test_register_buffers Linux-only with #[cfg(target_os = "linux")] - The non-Linux stub doesn't store registered buffers (buffer_count returns 0) - Test only makes sense on Linux where io_uring actually manages buffers 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent e0cf211 commit 2e26b2e

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

crates/wraith-core/src/migration.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -344,8 +344,8 @@ mod tests {
344344

345345
// RTT should be at least the delay
346346
assert!(validated.rtt >= delay);
347-
// RTT should be reasonable (less than 200ms for local test)
348-
assert!(validated.rtt < Duration::from_millis(200));
347+
// RTT should be reasonable (less than 1s for CI environments which can be slow)
348+
assert!(validated.rtt < Duration::from_secs(1));
349349
}
350350

351351
#[test]

crates/wraith-transport/src/io_uring.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -544,6 +544,7 @@ mod tests {
544544
}
545545

546546
#[test]
547+
#[cfg(target_os = "linux")]
547548
fn test_register_buffers() {
548549
let mut ctx = IoUringContext::new(64).unwrap();
549550

0 commit comments

Comments
 (0)