From 018f826197bfc5b09c2aa2278714aadb7a66879a Mon Sep 17 00:00:00 2001 From: Aleksey Kladov Date: Mon, 9 Nov 2020 14:22:45 +0100 Subject: Support multi-file assist tests --- .../handlers/extract_struct_from_enum_variant.rs | 37 ++++++++++++++++++++++ crates/assists/src/tests.rs | 25 +++++++++++---- 2 files changed, 56 insertions(+), 6 deletions(-) diff --git a/crates/assists/src/handlers/extract_struct_from_enum_variant.rs b/crates/assists/src/handlers/extract_struct_from_enum_variant.rs index 14209b771..58aadcef7 100644 --- a/crates/assists/src/handlers/extract_struct_from_enum_variant.rs +++ b/crates/assists/src/handlers/extract_struct_from_enum_variant.rs @@ -345,6 +345,43 @@ fn another_fn() { ); } + #[test] + fn test_several_files() { + check_assist( + extract_struct_from_enum_variant, + r#" +//- /main.rs +enum E { + <|>V(i32, i32) +} +mod foo; + +//- /foo.rs +use crate::E; +fn f() { + let e = E::V(9, 2); +} +"#, + r#" +//- /main.rs +struct V(pub i32, pub i32); + +enum E { + V(V) +} +mod foo; + +//- /foo.rs +use V; + +use crate::E; +fn f() { + let e = E::V(V(9, 2)); +} +"#, + ) + } + fn check_not_applicable(ra_fixture: &str) { let fixture = format!("//- /main.rs crate:main deps:core\n{}\n{}", ra_fixture, FamousDefs::FIXTURE); diff --git a/crates/assists/src/tests.rs b/crates/assists/src/tests.rs index 849d85e76..709a34d03 100644 --- a/crates/assists/src/tests.rs +++ b/crates/assists/src/tests.rs @@ -7,7 +7,7 @@ use syntax::TextRange; use test_utils::{assert_eq_text, extract_offset, extract_range}; use crate::{handlers::Handler, Assist, AssistConfig, AssistContext, AssistKind, Assists}; -use stdx::trim_indent; +use stdx::{format_to, trim_indent}; pub(crate) fn with_single_file(text: &str) -> (RootDatabase, FileId) { RootDatabase::with_single_file(text) @@ -98,11 +98,24 @@ fn check(handler: Handler, before: &str, expected: ExpectedResult, assist_label: match (assist, expected) { (Some(assist), ExpectedResult::After(after)) => { let mut source_change = assist.source_change; - let change = source_change.source_file_edits.pop().unwrap(); - - let mut actual = db.file_text(change.file_id).as_ref().to_owned(); - change.edit.apply(&mut actual); - assert_eq_text!(after, &actual); + assert!(!source_change.source_file_edits.is_empty()); + let skip_header = source_change.source_file_edits.len() == 1; + source_change.source_file_edits.sort_by_key(|it| it.file_id); + + let mut buf = String::new(); + for source_file_edit in source_change.source_file_edits { + let mut text = db.file_text(source_file_edit.file_id).as_ref().to_owned(); + source_file_edit.edit.apply(&mut text); + if !skip_header { + let sr = db.file_source_root(source_file_edit.file_id); + let sr = db.source_root(sr); + let path = sr.path_for_file(&source_file_edit.file_id).unwrap(); + format_to!(buf, "//- {}\n", path) + } + buf.push_str(&text); + } + + assert_eq_text!(after, &buf); } (Some(assist), ExpectedResult::Target(target)) => { let range = assist.assist.target; -- cgit v1.2.3