Skip to content

Commit f6c4c62

Browse files
committed
Extend coverage of last sections until end of document
1 parent de090c8 commit f6c4c62

7 files changed

+35
-8
lines changed

crates/ark/src/lsp/folding_range.rs

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@ fn parse_ts_node(
127127
end_node_handler(
128128
document,
129129
folding_ranges,
130-
end.row,
130+
end.row + 1,
131131
&mut child_comment_stack,
132132
&mut child_region_marker,
133133
&mut child_cell_marker,
@@ -479,7 +479,18 @@ f
479479
#+ Chunk section (knitr-style)
480480
g
481481
# + This is not a chunk
482-
h"
482+
h
483+
"
484+
));
485+
}
486+
487+
#[test]
488+
fn test_folding_section_comments_no_trailing_empty_line() {
489+
insta::assert_debug_snapshot!(test_folding_range(
490+
"
491+
# First section ----
492+
a
493+
b"
483494
));
484495
}
485496

crates/ark/src/lsp/snapshots/ark__lsp__folding_range__tests__folding_complex_nested.snap

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ expression: "test_folding_range(\"\n# Complex example ----\nfunction(a, b, c) {\
66
FoldingRange {
77
start_line: 1,
88
start_character: None,
9-
end_line: 18,
9+
end_line: 19,
1010
end_character: None,
1111
kind: Some(
1212
Region,
@@ -62,7 +62,7 @@ expression: "test_folding_range(\"\n# Complex example ----\nfunction(a, b, c) {\
6262
FoldingRange {
6363
start_line: 18,
6464
start_character: None,
65-
end_line: 18,
65+
end_line: 19,
6666
end_character: None,
6767
kind: Some(
6868
Region,

crates/ark/src/lsp/snapshots/ark__lsp__folding_range__tests__folding_empty_sections.snap

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ expression: "test_folding_range(\"\n# Empty section ----\n\n# Another empty sect
2626
FoldingRange {
2727
start_line: 5,
2828
start_character: None,
29-
end_line: 5,
29+
end_line: 6,
3030
end_character: None,
3131
kind: Some(
3232
Region,

crates/ark/src/lsp/snapshots/ark__lsp__folding_range__tests__folding_mixed.snap

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ expression: "test_folding_range(\"\n# First section ----\nfunction() {\n # #reg
5858
FoldingRange {
5959
start_line: 12,
6060
start_character: None,
61-
end_line: 12,
61+
end_line: 13,
6262
end_character: None,
6363
kind: Some(
6464
Region,

crates/ark/src/lsp/snapshots/ark__lsp__folding_range__tests__folding_nested_section_comments.snap

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ expression: "test_folding_range(\"\n# Level 1 ----\na\n\n## Level 2 ----\nb\n\n#
4646
FoldingRange {
4747
start_line: 13,
4848
start_character: None,
49-
end_line: 19,
49+
end_line: 20,
5050
end_character: None,
5151
kind: Some(
5252
Region,

crates/ark/src/lsp/snapshots/ark__lsp__folding_range__tests__folding_section_comments_basic.snap

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ expression: "test_folding_range(\"\n# First section ----\na\nb\n\n# Second secti
1616
FoldingRange {
1717
start_line: 5,
1818
start_character: None,
19-
end_line: 15,
19+
end_line: 16,
2020
end_character: None,
2121
kind: Some(
2222
Region,
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
---
2+
source: crates/ark/src/lsp/folding_range.rs
3+
expression: "test_folding_range(\"\n# First section ----\na\nb\")"
4+
---
5+
[
6+
FoldingRange {
7+
start_line: 1,
8+
start_character: None,
9+
end_line: 3,
10+
end_character: None,
11+
kind: Some(
12+
Region,
13+
),
14+
collapsed_text: None,
15+
},
16+
]

0 commit comments

Comments
 (0)