add integration test

Author: kirawi

helix-term/tests/test/commands/write.rs

@@ -933,3 +933,33 @@ async fn test_move_file_when_given_dir_only() -> anyhow::Result<()> {
 
     Ok(())
 }
+
+#[tokio::test(flavor = "multi_thread")]
+#[cfg(unix)]
+async fn test_write_ownership() -> anyhow::Result<()> {
+    // GH CI does not possess CAP_CHOWN
+    if option_env!("GITHUB_ACTIONS").is_some() {
+        return Ok(());
+    }
+    use std::os::unix::fs::MetadataExt;
+
+    let mut file = tempfile::NamedTempFile::new()?;
+    let mut app = helpers::AppBuilder::new()
+        .with_file(file.path(), None)
+        .build()?;
+
+    let nobody_uid = 9999;
+    let nogroup_gid = 9999;
+
+    helix_stdx::faccess::fchown(&file.as_file_mut(), Some(nobody_uid), Some(nogroup_gid))?;
+
+    let old_meta = file.as_file().metadata()?;
+
+    test_key_sequence(&mut app, Some("hello:w<ret>"), None, false).await?;
+    reload_file(&mut file).unwrap();
+
+    let new_meta = file.as_file().metadata()?;
+    assert!(old_meta.uid() == new_meta.uid() && old_meta.gid() == new_meta.gid());
+
+    Ok(())
+}

helix-view/src/document.rs

@@ -1200,7 +1200,7 @@ impl Document {
                             open_opt.mode(mode);
                         }
 
-                        let file = open_opt.open(&path).await?;
+                        let mut file = open_opt.open(&path).await?;
 
                         #[cfg(unix)]
                         {
@@ -1221,6 +1221,7 @@ impl Document {
                             use std::fs::{File, FileTimes};
                             use std::os::macos::fs::FileTimesExt;
 
+                            let file = file.try_clone()?.into_std();
                             let times = FileTimes::new().set_created(meta.created()?);
                             file.set_times(times)?;
                         }