Skip to content

Commit 08b911b

Browse files
committed
Fix tests for trailing and leading delimiter for multiple arguments
1 parent bd59837 commit 08b911b

File tree

2 files changed

+29
-21
lines changed

2 files changed

+29
-21
lines changed

data/fixtures/scopes/r/argument.actual.scope

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -6,12 +6,12 @@ func(x, y)
66
>-<
77
0| func(x, y)
88

9-
[#1 Removal] = 0:5-0:7
10-
>--<
9+
[#1 Removal] = 0:5-0:8
10+
>---<
1111
0| func(x, y)
1212

13-
[#1 Trailing delimiter] = 0:6-0:7
14-
>-<
13+
[#1 Trailing delimiter] = 0:6-0:8
14+
>--<
1515
0| func(x, y)
1616

1717
[#1 Insertion delimiter] = ", "
@@ -22,12 +22,12 @@ func(x, y)
2222
>-<
2323
0| func(x, y)
2424

25-
[#2 Removal] = 0:7-0:9
26-
>--<
25+
[#2 Removal] = 0:6-0:9
26+
>---<
2727
0| func(x, y)
2828

29-
[#2 Leading delimiter] = 0:7-0:8
30-
>-<
29+
[#2 Leading delimiter] = 0:6-0:8
30+
>--<
3131
0| func(x, y)
3232

3333
[#2 Insertion delimiter] = ", "

queries/r.scm

Lines changed: 21 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -27,10 +27,6 @@
2727
;; alternative: (braced_expression)? @branch.end
2828
;; ) @branch.iteration
2929

30-
;;!! foo(x)
31-
;;! ^^^^^^
32-
(call) @functionCall
33-
3430
;;!! function(x){ }
3531
;;! ^^^^^^^^^^^^^^
3632
(function_definition) @anonymousFunction
@@ -39,19 +35,20 @@
3935
;;! ^^^^^
4036
(_
4137
arguments: (arguments
42-
(_)? @_.leading.endOf
43-
.
44-
(argument) @argumentOrParameter
45-
.
46-
(_)? @_.trailing.endOf
47-
)
48-
@_dummy
38+
;; (
39+
(_)? @_.leading.endOf
40+
.
41+
(argument) @argumentOrParameter
42+
.
43+
(_)? @_.trailing.startOf
44+
;; ) @_.domain
45+
) @_dummy
4946
(#not-type? @argumentOrParameter "comment")
5047
(#single-or-multi-line-delimiter! @argumentOrParameter @_dummy ", " ",\n")
5148
)
5249

53-
;;!! function(bar){ }
54-
;;! ^^^
50+
;;!! function(a, b){ }
51+
;;! ^^^^
5552
(_
5653
parameters: (parameters
5754
open: "(" @argumentList.start.endOf @argumentOrParameter.iteration.start.endOf
@@ -61,6 +58,17 @@
6158
(#empty-single-multi-delimiter! @argumentList.start.endOf @_dummy "" ", " ",\n")
6259
) @argumentList.domain @argumentOrParameter.iteration.domain
6360

61+
;;!! foo(a, b)
62+
;;! ^^^^
63+
(_
64+
arguments: (arguments
65+
open: "(" @argumentList.start.endOf @argumentOrParameter.iteration.start.endOf
66+
;; (parameter) @argumentOrParameter
67+
close: ")" @argumentList.end.startOf @argumentOrParameter.iteration.end.startOf
68+
) @_dummy
69+
(#empty-single-multi-delimiter! @argumentList.start.endOf @_dummy "" ", " ",\n")
70+
) @argumentList.domain @argumentOrParameter.iteration.domain
71+
6472
(arguments
6573
"(" @argumentOrParameter.iteration.start.endOf
6674
")" @argumentOrParameter.iteration.end.startOf

0 commit comments

Comments
 (0)