scriptoriumadmin/fabro
1
0
Fork 0
mirror of https://github.com/fabro-sh/fabro.git synced 2026-08-28 05:27:41 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

fix(test): stabilize file interviewer claim-release timing

Browse source 
The file interviewer tests were assuming a transient claim file would be
observed within a 2ms window, which flaked under full-workspace load.
Make the observation wait explicit so nextest remains reliable.
This commit is contained in:
Bryan Helmkamp 2026-04-05 21:22:11 -04:00
parent a1fd66c8a8
commit 17c65b4c6c
1 changed files with 8 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
lib/crates/fabro-interview/src/file.rs
View file

@ -216,6 +216,11 @@ mod tests {
panic!("{} should exist", path.display());
}
async fn wait_for_claim_observation() {
// Full-workspace test load can delay the poller enough that a 2ms wait is flaky.
time::sleep(Duration::from_millis(25)).await;
}
#[tokio::test]
async fn write_request_poll_response() {
let dir = tempfile::tempdir().unwrap();
@ -287,7 +292,7 @@ mod tests {
std::fs::write(&claim_path, "12345\n").unwrap();
// Let the poll loop see the claim
time::sleep(TEST_POLL_INTERVAL * 2).await;
wait_for_claim_observation().await;
// Simulate attacher departing (deletes claim without writing response)
std::fs::remove_file(&claim_path).unwrap();
@ -323,7 +328,7 @@ mod tests {
// Simulate attacher creating then deleting claim
std::fs::write(&claim_path, "12345\n").unwrap();
time::sleep(TEST_POLL_INTERVAL * 2).await;
wait_for_claim_observation().await;
std::fs::remove_file(&claim_path).unwrap();
let answer = time::timeout(Duration::from_millis(250), ask_handle)
@ -354,7 +359,7 @@ mod tests {
// First attacher creates then releases claim
std::fs::write(&claim_path, "12345\n").unwrap();
time::sleep(TEST_POLL_INTERVAL * 2).await;
wait_for_claim_observation().await;
std::fs::remove_file(&claim_path).unwrap();
// Second attacher picks up and answers before reattach window expires