nordic-dev.net/rust
nordic-dev.net/rust
2
0
Fork 0
mirror of https://github.com/rust-lang/rust.git synced 2025-05-14 02:49:40 +00:00
Code Issues Packages Projects Releases Wiki Activity

internal: Allow explicitly specifying end of fixture annotation

Browse Source
This commit is contained in:
Lukas Wirth 2022-03-18 15:30:58 +01:00
parent 828196be3b
commit 890f98f21f
1 changed files with 28 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
crates/test_utils/src/lib.rs
View File

@ -212,6 +212,13 @@ pub fn add_cursor(text: &str, offset: TextSize) -> String {
/// // ^^^ first line /// // ^^^ first line
/// // | second line /// // | second line
/// ///
/// Trailing whitespace is sometimes desired but usually stripped by the editor
/// if at the end of a line, or incorrectly sized if followed by another
/// annotation. In those cases the annotation can be explicitly ended with the
/// `$` character.
///
/// // ^^^ trailing-ws-wanted $
///
/// Annotations point to the last line that actually was long enough for the /// Annotations point to the last line that actually was long enough for the
/// range, not counting annotations themselves. So overlapping annotations are /// range, not counting annotations themselves. So overlapping annotations are
/// possible: /// possible:
@ -229,9 +236,10 @@ pub fn extract_annotations(text: &str) -> Vec<(TextRange, String)> {
let mut prev_line_annotations: Vec<(TextSize, usize)> = Vec::new(); let mut prev_line_annotations: Vec<(TextSize, usize)> = Vec::new();
for line in text.split_inclusive('\n') { for line in text.split_inclusive('\n') {
let mut this_line_annotations = Vec::new(); let mut this_line_annotations = Vec::new();
let line_length = if let Some(idx) = line.find("//") { let line_length = if let Some((prefix, suffix)) = line.split_once("//") {
let annotation_offset = TextSize::of(&line[..idx + "//".len()]); let ss_len = TextSize::of("//");
for annotation in extract_line_annotations(&line[idx + "//".len()..]) { let annotation_offset = TextSize::of(prefix) + ss_len;
for annotation in extract_line_annotations(suffix.trim_end_matches('\n')) {
match annotation { match annotation {
LineAnnotation::Annotation { mut range, content, file } => { LineAnnotation::Annotation { mut range, content, file } => {
range += annotation_offset; range += annotation_offset;
@ -257,7 +265,7 @@ pub fn extract_annotations(text: &str) -> Vec<(TextRange, String)> {
} }
} }
} }
idx.try_into().unwrap() annotation_offset
} else { } else {
TextSize::of(line) TextSize::of(line)
}; };
@ -294,8 +302,21 @@ fn extract_line_annotations(mut line: &str) -> Vec<LineAnnotation> {
len = 1; len = 1;
} }
let range = TextRange::at(offset, len.try_into().unwrap()); let range = TextRange::at(offset, len.try_into().unwrap());
let next = line[len..].find(marker).map_or(line.len(), |it| it + len); let line_no_caret = &line[len..];
let mut content = &line[len..][..next - len]; let end_marker = line_no_caret.find(|c| c == '$');
let next = line_no_caret.find(marker).map_or(line.len(), |it| it + len);
let mut content = match end_marker {
Some(end_marker)
if end_marker < next
&& line_no_caret[end_marker..]
.strip_prefix(|c: char| c.is_whitespace() || c == '^')
.is_some() =>
{
&line_no_caret[..end_marker]
}
_ => line_no_caret[..next - len].trim_end(),
};
let mut file = false; let mut file = false;
if !continuation && content.starts_with("file") { if !continuation && content.starts_with("file") {
@ -303,7 +324,7 @@ fn extract_line_annotations(mut line: &str) -> Vec<LineAnnotation> {
content = &content["file".len()..]; content = &content["file".len()..];
} }
let content = content.trim().to_string(); let content = content.trim_start().to_string();
let annotation = if continuation { let annotation = if continuation {
LineAnnotation::Continuation { offset: range.end(), content } LineAnnotation::Continuation { offset: range.end(), content }