distinct crlf and lf tests

helix-editor · Aug 30, 2023 · 84cd92a · 84cd92a
1 parent 2ee32ac
commit 84cd92a
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 17 deletions.
diff --git a/helix-term/tests/integration.rs b/helix-term/tests/integration.rs
@@ -1,4 +1,4 @@
-#[cfg(feature = "integration")]
+// #[cfg(feature = "integration")]
 mod test {
     mod helpers;
 

diff --git a/helix-term/tests/test/movement.rs b/helix-term/tests/test/movement.rs
@@ -520,12 +520,14 @@ async fn previous_occurrence_of_char() -> anyhow::Result<()> {
     use helix_term::config::Config;
     use helix_view::editor::LineEndingConfig;
 
-    let mut config = Config::default();
-    config.editor.default_line_ending = LineEndingConfig::Crlf;
+    let mut config_crlf = Config::default();
+    config_crlf.editor.default_line_ending = LineEndingConfig::Crlf;
+    let mut config_lf = Config::default();
+    config_lf.editor.default_line_ending = LineEndingConfig::LF;
 
     // Move till previous char occurrence should properly handle <ret> when line-ending is Crlf
     test_with_config(
-        AppBuilder::new().with_config(config.clone()),
+        AppBuilder::new().with_config(config_crlf.clone()),
         (
             "text\r\nnext #[l|]#ine",
             "T<ret>",
@@ -534,16 +536,19 @@ async fn previous_occurrence_of_char() -> anyhow::Result<()> {
         ),
     )
     .await?;
-    test((
-        "text\nnext #[l|]#ine",
-        "T<ret>",
-        "text\n#[|next l]#ine",
-        "Move till previous LF",
-    ))
+    test_with_config(
+        AppBuilder::new().with_config(config_lf.clone()),
+        (
+            "text\nnext #[l|]#ine",
+            "T<ret>",
+            "text\n#[|next l]#ine",
+            "Move till previous LF",
+        ),
+    )
     .await?;
 
     test_with_config(
-        AppBuilder::new().with_config(config),
+        AppBuilder::new().with_config(config_crlf),
         (
             "text\r\nnext #[l|]#ine",
             "F<ret>",
@@ -552,12 +557,15 @@ async fn previous_occurrence_of_char() -> anyhow::Result<()> {
         ),
     )
     .await?;
-    test((
-        "text\nnext #[l|]#ine",
-        "F<ret>",
-        "text#[|\nnext l]#ine",
-        "Move to previous LF",
-    ))
+    test_with_config(
+        AppBuilder::new().with_config(config_lf),
+        (
+            "text\nnext #[l|]#ine",
+            "F<ret>",
+            "text#[|\nnext l]#ine",
+            "Move to previous LF",
+        ),
+    )
     .await?;
 
     Ok(())