From 22a669d27e6ceeea3ee557fad589fe882064ba44 Mon Sep 17 00:00:00 2001 From: Florian Klink Date: Sat, 9 Dec 2023 13:21:07 +0200 Subject: [PATCH] refactor(tvix/castore): address clippy We match to destructure a single pattern. Change-Id: I564a3510b4860e90b3315a9639effc48ee88b483 Reviewed-on: https://cl.tvl.fyi/c/depot/+/10233 Reviewed-by: tazjin Autosubmit: flokli Tested-by: BuildkiteCI --- tvix/castore/src/blobservice/tests.rs | 17 +++++++---------- 1 file changed, 7 insertions(+), 10 deletions(-) diff --git a/tvix/castore/src/blobservice/tests.rs b/tvix/castore/src/blobservice/tests.rs index fe390b537..7480ca808 100644 --- a/tvix/castore/src/blobservice/tests.rs +++ b/tvix/castore/src/blobservice/tests.rs @@ -217,20 +217,17 @@ fn put_seek(blob_service: impl BlobService) { } // seeking past the end… - match r + // should either be ok, but then return 0 bytes. + // this matches the behaviour or a Cursor>. + if let Ok(_pos) = r .seek(io::SeekFrom::Start(fixtures::BLOB_B.len() as u64 + 1)) .await { - // should either be ok, but then return 0 bytes. - // this matches the behaviour or a Cursor>. - Ok(_pos) => { - let mut buf: Vec = Vec::new(); - r.read_to_end(&mut buf).await.expect("must not fail"); - assert!(buf.is_empty(), "expected no more data to be read"); - } - // or not be okay. - Err(_) => {} + let mut buf: Vec = Vec::new(); + r.read_to_end(&mut buf).await.expect("must not fail"); + assert!(buf.is_empty(), "expected no more data to be read"); } + // or not be okay. // TODO: this is only broken for the gRPC version // We expect seeking backwards or relative to the end to fail.