Minor rename

This commit is contained in:
Aleksey Kladov 2020-02-06 18:46:11 +01:00
parent 918115c32e
commit 7e73b7a5f8
3 changed files with 11 additions and 9 deletions

View File

@ -8,19 +8,21 @@
use ra_db::FileRange;
use test_utils::{assert_eq_text, extract_range_or_offset};
use crate::resolved_assists;
fn check(assist_id: &str, before: &str, after: &str) {
let (selection, before) = extract_range_or_offset(before);
let (db, file_id) = crate::helpers::with_single_file(&before);
let frange = FileRange { file_id, range: selection.into() };
let assist = crate::assists(&db, frange)
let assist = resolved_assists(&db, frange)
.into_iter()
.find(|assist| assist.label.id.0 == assist_id)
.unwrap_or_else(|| {
panic!(
"\n\nAssist is not applicable: {}\nAvailable assists: {}",
assist_id,
crate::assists(&db, frange)
resolved_assists(&db, frange)
.into_iter()
.map(|assist| assist.label.id.0)
.collect::<Vec<_>>()

View File

@ -59,7 +59,7 @@ pub fn get_first_action(&self) -> AssistAction {
///
/// Assists are returned in the "unresolved" state, that is only labels are
/// returned, without actual edits.
pub fn applicable_assists(db: &RootDatabase, range: FileRange) -> Vec<AssistLabel> {
pub fn unresolved_assists(db: &RootDatabase, range: FileRange) -> Vec<AssistLabel> {
AssistCtx::with_ctx(db, range, false, |ctx| {
assists::all()
.iter()
@ -76,7 +76,7 @@ pub fn applicable_assists(db: &RootDatabase, range: FileRange) -> Vec<AssistLabe
///
/// Assists are returned in the "resolved" state, that is with edit fully
/// computed.
pub fn assists(db: &RootDatabase, range: FileRange) -> Vec<ResolvedAssist> {
pub fn resolved_assists(db: &RootDatabase, range: FileRange) -> Vec<ResolvedAssist> {
AssistCtx::with_ctx(db, range, true, |ctx| {
let mut a = assists::all()
.iter()
@ -301,7 +301,7 @@ mod tests {
use ra_syntax::TextRange;
use test_utils::{extract_offset, extract_range};
use crate::helpers;
use crate::{helpers, resolved_assists};
#[test]
fn assist_order_field_struct() {
@ -310,7 +310,7 @@ fn assist_order_field_struct() {
let (db, file_id) = helpers::with_single_file(&before);
let frange =
FileRange { file_id, range: TextRange::offset_len(before_cursor_pos, 0.into()) };
let assists = super::assists(&db, frange);
let assists = resolved_assists(&db, frange);
let mut assists = assists.iter();
assert_eq!(
@ -333,7 +333,7 @@ pub fn test_some_range(a: int) -> bool {
let (range, before) = extract_range(before);
let (db, file_id) = helpers::with_single_file(&before);
let frange = FileRange { file_id, range };
let assists = super::assists(&db, frange);
let assists = resolved_assists(&db, frange);
let mut assists = assists.iter();
assert_eq!(assists.next().expect("expected assist").label.label, "Extract into variable");

View File

@ -1,7 +1,7 @@
//! FIXME: write short doc here
use either::Either;
use ra_assists::{AssistAction, AssistLabel};
use ra_assists::{resolved_assists, AssistAction, AssistLabel};
use ra_db::{FilePosition, FileRange};
use ra_ide_db::RootDatabase;
@ -17,7 +17,7 @@ pub struct Assist {
}
pub(crate) fn assists(db: &RootDatabase, frange: FileRange) -> Vec<Assist> {
ra_assists::assists(db, frange)
resolved_assists(db, frange)
.into_iter()
.map(|assist| {
let file_id = frange.file_id;