use Results in integration tests for more error context

helix-term/tests/integration/commands.rs

@@ -59,7 +59,7 @@ async fn test_buffer_close() -> anyhow::Result<()> {
     .await?;
 
     // verify if writes are queued up, it finishes them before closing the buffer
-    let mut file = tempfile::NamedTempFile::new().unwrap();
+    let mut file = tempfile::NamedTempFile::new()?;
     let mut command = String::new();
     const RANGE: RangeInclusive<i32> = 1..=10;
 

helix-term/tests/integration/helpers.rs

@@ -81,8 +81,10 @@ pub async fn test_key_sequence_with_input_text<T: Into<TestCase>>(
     test_fn: &dyn Fn(&Application),
 ) -> anyhow::Result<()> {
     let test_case = test_case.into();
-    let mut app =
+    let mut app = match app {
-        app.unwrap_or_else(|| Application::new(Args::default(), Config::default()).unwrap());
+        Some(app) => app,
+        None => Application::new(Args::default(), Config::default())?,
+    };
 
     let (view, doc) = helix_view::current!(app.editor);
     let sel = doc.selection(view.id).clone();
@@ -108,7 +110,7 @@ pub async fn test_key_sequence_text_result<T: Into<TestCase>>(
     test_case: T,
 ) -> anyhow::Result<()> {
     let test_case = test_case.into();
-    let app = Application::new(args, config).unwrap();
+    let app = Application::new(args, config)?;
 
     test_key_sequence_with_input_text(Some(app), test_case.clone(), &|app| {
         let doc = doc!(app.editor);
@@ -123,13 +125,16 @@ pub async fn test_key_sequence_text_result<T: Into<TestCase>>(
     .await
 }
 
-pub fn temp_file_with_contents<S: AsRef<str>>(content: S) -> tempfile::NamedTempFile {
+pub fn temp_file_with_contents<S: AsRef<str>>(
-    let mut temp_file = tempfile::NamedTempFile::new().unwrap();
+    content: S,
+) -> anyhow::Result<tempfile::NamedTempFile> {
+    let mut temp_file = tempfile::NamedTempFile::new()?;
+
     temp_file
         .as_file_mut()
-        .write_all(content.as_ref().as_bytes())
+        .write_all(content.as_ref().as_bytes())?;
-        .unwrap();
+
-    temp_file.flush().unwrap();
+    temp_file.flush()?;
-    temp_file.as_file_mut().sync_all().unwrap();
+    temp_file.as_file_mut().sync_all()?;
-    temp_file
+    Ok(temp_file)
 }

helix-term/tests/integration/movement.rs

@@ -109,8 +109,8 @@ async fn insert_to_normal_mode_cursor_position() -> anyhow::Result<()> {
 /// the first grapheme
 #[tokio::test]
 async fn cursor_position_newly_opened_file() -> anyhow::Result<()> {
-    let test = |content: &str, expected_sel: Selection| {
+    let test = |content: &str, expected_sel: Selection| -> anyhow::Result<()> {
-        let file = helpers::temp_file_with_contents(content);
+        let file = helpers::temp_file_with_contents(content)?;
 
         let mut app = Application::new(
             Args {
@@ -118,17 +118,18 @@ async fn cursor_position_newly_opened_file() -> anyhow::Result<()> {
                 ..Default::default()
             },
             Config::default(),
-        )
+        )?;
-        .unwrap();
 
         let (view, doc) = helix_view::current!(app.editor);
         let sel = doc.selection(view.id).clone();
         assert_eq!(expected_sel, sel);
+
+        Ok(())
     };
 
-    test("foo", Selection::single(0, 1));
+    test("foo", Selection::single(0, 1))?;
-    test("👨‍👩‍👧‍👦 foo", Selection::single(0, 7));
+    test("👨‍👩‍👧‍👦 foo", Selection::single(0, 7))?;
-    test("", Selection::single(0, 0));
+    test("", Selection::single(0, 0))?;
 
     Ok(())
 }

helix-term/tests/integration/write.rs

@@ -11,7 +11,7 @@ use super::*;
 
 #[tokio::test]
 async fn test_write() -> anyhow::Result<()> {
-    let mut file = tempfile::NamedTempFile::new().unwrap();
+    let mut file = tempfile::NamedTempFile::new()?;
 
     test_key_sequence(
         &mut Application::new(
@@ -38,7 +38,7 @@ async fn test_write() -> anyhow::Result<()> {
 
 #[tokio::test]
 async fn test_write_concurrent() -> anyhow::Result<()> {
-    let mut file = tempfile::NamedTempFile::new().unwrap();
+    let mut file = tempfile::NamedTempFile::new()?;
     let mut command = String::new();
     const RANGE: RangeInclusive<i32> = 1..=5000;
 
@@ -112,7 +112,6 @@ async fn test_write_fail_mod_flag() -> anyhow::Result<()> {
 }
 
 #[tokio::test]
-#[ignore]
 async fn test_write_fail_new_path() -> anyhow::Result<()> {
     test_key_sequences(
         &mut Application::new(Args::default(), Config::default())?,