fix tests, clippy lints

helix-core/src/line_ending.rs

@@ -112,7 +112,7 @@ impl LineEnding {
     ///
     /// [`LineEnding::Crlf`] on Windows, otherwise [`LineEnding::LF`].
     #[inline]
-    pub fn native() -> Self {
+    pub const fn native() -> Self {
         #[cfg(target_os = "windows")]
         return LineEnding::Crlf;
         #[cfg(not(target_os = "windows"))]

helix-term/tests/test/auto_pairs.rs

@@ -2,7 +2,7 @@ use helix_core::{auto_pairs::DEFAULT_PAIRS, hashmap};
 
 use super::*;
 
-const LINE_END: &str = helix_core::DEFAULT_LINE_ENDING.as_str();
+const LINE_END: &str = helix_core::LineEnding::native().as_str();
 
 fn differing_pairs() -> impl Iterator<Item = &'static (char, char)> {
     DEFAULT_PAIRS.iter().filter(|(open, close)| open != close)

helix-term/tests/test/helpers.rs

@@ -230,7 +230,7 @@ pub fn temp_file_with_contents<S: AsRef<str>>(
 /// character, and if one doesn't exist already, appends the system's
 /// appropriate line ending to the end of a string.
 pub fn platform_line(input: &str) -> String {
-    let line_end = helix_core::DEFAULT_LINE_ENDING.as_str();
+    let line_end = helix_core::LineEnding::native().as_str();
 
     // we can assume that the source files in this code base will always
     // be LF, so indoc strings will always insert LF

helix-view/src/document.rs

@@ -1391,10 +1391,12 @@ mod test {
 
     #[test]
     fn test_line_ending() {
-        assert_eq!(
-            Document::default().text().to_string(),
-            DEFAULT_LINE_ENDING.as_str()
-        );
+        for line_ending in [LineEnding::LF, LineEnding::Crlf] {
+            assert_eq!(
+                Document::new(line_ending).text().to_string(),
+                line_ending.as_str()
+            );
+        }
     }
 
     macro_rules! decode {