Fix test

Conrad Irwin created 2 years ago

Change summary

crates/editor/src/display_map.rs  |  2 +-
crates/editor/src/editor_tests.rs | 17 ++++++++---------
crates/editor/src/movement.rs     |  1 -
3 files changed, 9 insertions(+), 11 deletions(-)

Detailed changes

crates/editor/src/display_map.rs

@@ -1007,7 +1007,7 @@ pub mod tests {
     use settings::SettingsStore;
     use smol::stream::StreamExt;
     use std::{env, sync::Arc};
-    use theme::{SyntaxTheme, Theme};
+    use theme::SyntaxTheme;
     use util::test::{marked_text_ranges, sample_text};
     use Bias::*;
 

crates/editor/src/editor_tests.rs

@@ -847,7 +847,6 @@ fn test_move_cursor(cx: &mut TestAppContext) {
 
 #[gpui::test]
 fn test_move_cursor_multibyte(cx: &mut TestAppContext) {
-    todo!();
     init_test(cx, |_| {});
 
     let view = cx
@@ -889,6 +888,11 @@ fn test_move_cursor_multibyte(cx: &mut TestAppContext) {
         );
 
         view.move_down(&MoveDown, cx);
+        assert_eq!(
+            view.selections.display_ranges(cx),
+            &[empty_range(1, "ab⋯e".len())]
+        );
+        view.move_left(&MoveLeft, cx);
         assert_eq!(
             view.selections.display_ranges(cx),
             &[empty_range(1, "ab⋯".len())]
@@ -933,22 +937,17 @@ fn test_move_cursor_multibyte(cx: &mut TestAppContext) {
         view.move_up(&MoveUp, cx);
         assert_eq!(
             view.selections.display_ranges(cx),
-            &[empty_range(0, "ⓐⓑ⋯ⓔ".len())]
-        );
-        view.move_left(&MoveLeft, cx);
-        assert_eq!(
-            view.selections.display_ranges(cx),
-            &[empty_range(0, "ⓐⓑ⋯".len())]
+            &[empty_range(0, "ⓐⓑ".len())]
         );
         view.move_left(&MoveLeft, cx);
         assert_eq!(
             view.selections.display_ranges(cx),
-            &[empty_range(0, "ⓐⓑ".len())]
+            &[empty_range(0, "ⓐ".len())]
         );
         view.move_left(&MoveLeft, cx);
         assert_eq!(
             view.selections.display_ranges(cx),
-            &[empty_range(0, "ⓐ".len())]
+            &[empty_range(0, "".len())]
         );
     });
 }

crates/editor/src/movement.rs

@@ -429,7 +429,6 @@ mod tests {
         test::{editor_test_context::EditorTestContext, marked_display_snapshot},
         Buffer, DisplayMap, ExcerptRange, InlayId, MultiBuffer,
     };
-    use language::language_settings::AllLanguageSettings;
     use project::Project;
     use settings::SettingsStore;
     use util::post_inc;