You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opendal.apache.org by xu...@apache.org on 2026/10/26 07:47:18 UTC
(opendal-yinyang) 02/02: fix(mount): clean up concurrency test compile feedback
This is an automated email from the ASF dual-hosted git repository.
Xuanwo pushed a commit to branch xuanwo/linux-fuse-writable
in repository https://gitbox.apache.org/repos/asf/opendal-yinyang.git
commit e9fdabb9ebb40cd34f25ae4131e448a776c92c9d
Author: Xuanwo <gi...@xuanwo.io>
AuthorDate: Mon Oct 26 15:47:09 2026 +0800
fix(mount): clean up concurrency test compile feedback
The reclaim status probe does not need a mut handle, and failure-path assertions should not require MountFile to be Debug.
Co-authored-by: Chojan Shang <ps...@apache.org>
---
src/mount.rs | 2 +-
tests/mount.rs | 3 ++-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/mount.rs b/src/mount.rs
index 125e448..18e4853 100644
--- a/src/mount.rs
+++ b/src/mount.rs
@@ -252,7 +252,7 @@ impl Mount {
node.clone()
};
{
- let mut handle = node.handle.lock().await;
+ let handle = node.handle.lock().await;
let state = handle.status().await?;
if state.pending || state.frozen || state.error.is_some() {
return Err(Error::State(
diff --git a/tests/mount.rs b/tests/mount.rs
index aeb1c26..6c7a6b6 100644
--- a/tests/mount.rs
+++ b/tests/mount.rs
@@ -421,7 +421,8 @@ async fn failed_first_open_broadcasts_retry_to_all_waiters() {
.await
.expect("failed owner did not finish")
.unwrap()
- .unwrap_err();
+ .err()
+ .expect("failed owner returned a file");
assert_eq!(error.kind(), ErrorKind::Storage);
let mut waiters = waiters.into_iter();
let first = tokio::time::timeout(Duration::from_secs(5), waiters.next().unwrap())