From acd2e7f222ffa096a6bcac29449ffc0260a763c5 Mon Sep 17 00:00:00 2001 From: Joonas Koivunen Date: Mon, 21 Aug 2023 09:48:11 +0300 Subject: [PATCH] timeline: fix test after wait_and_evict --- pageserver/src/tenant/timeline.rs | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/pageserver/src/tenant/timeline.rs b/pageserver/src/tenant/timeline.rs index 185acdfb78..a57d3bcc91 100644 --- a/pageserver/src/tenant/timeline.rs +++ b/pageserver/src/tenant/timeline.rs @@ -4286,18 +4286,15 @@ mod tests { let (first, second) = (only_one(first), only_one(second)); - assert_eq!(batch[0].needs_download_blocking().unwrap(), None); - - // both seemingly succeed, but only one will actually evict - match (first, second) { - (Ok(()), Err(EvictionError::NotFound)) | (Err(EvictionError::NotFound), Ok(())) => {} - other => unreachable!("unexpected {:?}", other), - } - batch[0] .needs_download_blocking() .unwrap() .expect("should now have a reason to download"); + + match (first, second) { + (Ok(()), Err(EvictionError::NotFound)) | (Err(EvictionError::NotFound), Ok(())) => {} + other => unreachable!("unexpected {:?}", other), + } } fn any_context() -> crate::context::RequestContext {