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())