update write-quit to wait for saves

helix-term/src/commands/typed.rs

@@ -416,8 +416,12 @@ fn write_quit(
     event: PromptEvent,
 ) -> anyhow::Result<()> {
     write_impl(cx, args.first(), false)?;
-    // TODO: change to use document close
-    helix_lsp::block_on(cx.jobs.finish())?;
+    let doc = doc_mut!(cx.editor);
+
+    tokio::task::block_in_place(|| helix_lsp::block_on(doc.try_flush_saves()))
+        .map(|result| result.map(|_| ()))
+        .unwrap_or(Ok(()))?;
+
     quit(cx, &[], event)
 }
 

helix-term/tests/test/commands.rs

@@ -8,7 +8,7 @@ use helix_term::application::Application;
 
 use super::*;
 
-#[tokio::test]
+#[tokio::test(flavor = "multi_thread")]
 async fn test_write_quit_fail() -> anyhow::Result<()> {
     let file = helpers::new_readonly_tempfile()?;
 
@@ -22,6 +22,11 @@ async fn test_write_quit_fail() -> anyhow::Result<()> {
         )?,
         Some("ihello<esc>:wq<ret>"),
         Some(&|app| {
+            let mut docs: Vec<_> = app.editor.documents().collect();
+            assert_eq!(1, docs.len());
+
+            let doc = docs.pop().unwrap();
+            assert_eq!(Some(file.path()), doc.path().map(PathBuf::as_path));
             assert_eq!(&Severity::Error, app.editor.get_status().unwrap().1);
         }),
         false,

helix-view/src/editor.rs

@@ -787,6 +787,19 @@ impl Editor {
         Ok(id)
     }
 
+    /// Closes every document and view, which will ultimately shut down the app.
+    // pub async fn close(&mut self) -> anyhow::Result<()> {
+    //     let docs: Vec<_> = self.documents().map(|doc| doc.id()).collect();
+
+    //     // closing all documents also closes all views, which will shut
+    //     // down the app
+    //     for doc in docs {
+    //         self.close_document(doc, false).await?;
+    //     }
+
+    //     Ok(())
+    // }
+
     pub fn close(&mut self, id: ViewId) {
         let view = self.tree.get(self.tree.focus);
         // remove selection