Skip to content

Commit a0106f7

Browse files
ClaudeDennisOSRM
andcommitted
Fix inconsistent cell count in cucumber tables
Co-authored-by: DennisOSRM <1067895+DennisOSRM@users.noreply.github.com>
1 parent 5eb988e commit a0106f7

7 files changed

+54
-54
lines changed

features/guidance/continue.feature

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -169,7 +169,7 @@ Feature: Continue Instructions
169169
| a,f | abcdefb,abcdefb,abcdefb | depart,continue left,arrive | a,a,f |
170170
| a,d | abcdefb,abcdefb,abcdefb | depart,continue right,arrive | a,a,d |
171171
# continuing right here, since the turn to the left is more expensive
172-
| a,e | abcdefb,abcdefb,abcdefb | depart,continue right,arrive |
172+
| a,e | abcdefb,abcdefb,abcdefb | depart,continue right,arrive | a,a,e |
173173

174174
Scenario: End-Of-Road Continue
175175
Given the node map

features/guidance/destination-signs.feature

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -36,14 +36,14 @@ Feature: Destination Signs
3636

3737
When I route I should get
3838
| from | to | route | destinations | ref | # | locations |
39-
| a | b | AB,AB | , | E1,E1 | |
40-
| c | d | CD,CD | Berlin,Berlin | , | |
41-
| e | f | EF,EF | A1: Berlin,A1: Berlin | , | |
42-
| g | h | , | A1: Berlin,A1: Berlin | , | |
43-
| i | j | , | Berlin,Berlin | , | |
44-
| k | l | KL,KL | A1: Berlin,A1: Berlin | E1,E1 | |
45-
| m | n | MN,MN | A1, A2: Berlin, Hamburg,A1, A2: Berlin, Hamburg | , | |
46-
| o | p | OP,OP | , | , | guard against mis-tagging | |
47-
| q | r | QR,QR | A1, A2,A1, A2 | , | |
48-
| s | t | ST,ST | St Street,St Street | , | |
49-
| u | v | UV,UV | Berlin,Berlin | , | city preferred over street | |
39+
| a | b | AB,AB | , | E1,E1 | | a,b |
40+
| c | d | CD,CD | Berlin,Berlin | , | | c,d |
41+
| e | f | EF,EF | A1: Berlin,A1: Berlin | , | | e,f |
42+
| g | h | , | A1: Berlin,A1: Berlin | , | | g,h |
43+
| i | j | , | Berlin,Berlin | , | | i,j |
44+
| k | l | KL,KL | A1: Berlin,A1: Berlin | E1,E1 | | k,l |
45+
| m | n | MN,MN | A1, A2: Berlin, Hamburg,A1, A2: Berlin, Hamburg | , | | m,n |
46+
| o | p | OP,OP | , | , | guard against mis-tagging | o,p |
47+
| q | r | QR,QR | A1, A2,A1, A2 | , | | q,r |
48+
| s | t | ST,ST | St Street,St Street | , | | s,t |
49+
| u | v | UV,UV | Berlin,Berlin | , | city preferred over street | u,v |

features/guidance/directional-destination-signs.feature

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -32,12 +32,12 @@ Feature: Destination Signs
3232

3333
When I route I should get
3434
| from | to | route | destinations | ref | # | locations |
35-
| a | b | AB,AB | Berlin,Berlin | , | |
36-
| c | d | CD,CD | Berlin,Berlin | , | |
37-
| f | e | EF,EF | Hamburg,Hamburg | , | |
38-
| g | h | GH,GH | A1,A1 | , | |
39-
| i | j | IJ,IJ | , | , | guard against mis-tagging | |
40-
| k | l | KL,KL | A1: Berlin,A1: Berlin | , | |
41-
| m | n | MN,MN | A1: Berlin,A1: Berlin | , | |
42-
| p | o | OP,OP | Hamburg,Hamburg | , | |
43-
| r | q | QR,QR | Hamburg,Hamburg | , | |
35+
| a | b | AB,AB | Berlin,Berlin | , | | a,b |
36+
| c | d | CD,CD | Berlin,Berlin | , | | c,d |
37+
| f | e | EF,EF | Hamburg,Hamburg | , | | f,e |
38+
| g | h | GH,GH | A1,A1 | , | | g,h |
39+
| i | j | IJ,IJ | , | , | guard against mis-tagging | i,j |
40+
| k | l | KL,KL | A1: Berlin,A1: Berlin | , | | k,l |
41+
| m | n | MN,MN | A1: Berlin,A1: Berlin | , | | m,n |
42+
| p | o | OP,OP | Hamburg,Hamburg | , | | p,o |
43+
| r | q | QR,QR | Hamburg,Hamburg | , | | r,q |

features/guidance/directional-ref-destination-signs.feature

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -30,10 +30,10 @@ Feature: Destination Signs
3030

3131
When I route I should get
3232
| from | to | route | destinations | ref | # | locations |
33-
| a | b | AB,AB | A1: Berlin,A1: Berlin | , | |
34-
| d | c | CD,CD | A2: Hamburg,A2: Hamburg | , | |
35-
| e | f | EF,EF | A1: Berlin,A1: Berlin | , | |
36-
| h | g | GH,GH | A2: Hamburg,A2: Hamburg | , | |
37-
| i | j | IJ,IJ | A1: Berlin,A1: Berlin | , | |
38-
| l | k | KL,KL | A2: Hamburg,A2: Hamburg | , | |
39-
| m | n | MN,MN | , | , | guard against mis-tagging | |
33+
| a | b | AB,AB | A1: Berlin,A1: Berlin | , | | a,b |
34+
| d | c | CD,CD | A2: Hamburg,A2: Hamburg | , | | d,c |
35+
| e | f | EF,EF | A1: Berlin,A1: Berlin | , | | e,f |
36+
| h | g | GH,GH | A2: Hamburg,A2: Hamburg | , | | h,g |
37+
| i | j | IJ,IJ | A1: Berlin,A1: Berlin | , | | i,j |
38+
| l | k | KL,KL | A2: Hamburg,A2: Hamburg | , | | l,k |
39+
| m | n | MN,MN | , | , | guard against mis-tagging | m,n |

features/guidance/internal-intersections.feature

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -28,19 +28,19 @@ Feature: Internal Intersection Model
2828

2929
When I route I should get
3030
| waypoints | route | turns | # | locations |
31-
| a,k | Broken Land Parkway,Broken Land Parkway | depart,arrive | |
32-
| l,b | Broken Land Parkway,Broken Land Parkway | depart,arrive | |
33-
# | g,j | Patuxent Woods Drive,Snowden River Parkway,Snowden River Parkway | depart,continue,arrive | did not work as expected - might be another issue to handle in post process? |
34-
# | f,c | Snowden River Parkway,Patuxent Woods Drive,Patuxent Woods Drive | depart,continue,arrive | did not work as expected - might be another issue to handle in post process? |
35-
| a,c | Broken Land Parkway,Patuxent Woods Drive,Patuxent Woods Drive | depart,turn right,arrive ||
36-
| g,k | Patuxent Woods Drive,Broken Land Parkway,Broken Land Parkway | depart,turn right,arrive ||
37-
| l,j | Broken Land Parkway,Snowden River Parkway,Snowden River Parkway | depart,turn right,arrive ||
38-
| f,b | Snowden River Parkway,Broken Land Parkway,Broken Land Parkway | depart,turn right,arrive ||
39-
| a,j | Broken Land Parkway,Snowden River Parkway,Snowden River Parkway | depart,turn left,arrive ||
40-
| g,b | Patuxent Woods Drive,Broken Land Parkway,Broken Land Parkway | depart,turn left,arrive ||
41-
| l,c | Broken Land Parkway,Patuxent Woods Drive,Patuxent Woods Drive | depart,turn left,arrive ||
42-
| f,k | Snowden River Parkway,Broken Land Parkway,Broken Land Parkway | depart,turn left,arrive ||
43-
| a,b | Broken Land Parkway,Broken Land Parkway,Broken Land Parkway | depart,continue uturn,arrive ||
44-
| g,c | Patuxent Woods Drive,Patuxent Woods Drive,Patuxent Woods Drive | depart,continue uturn,arrive ||
45-
| l,k | Broken Land Parkway,Broken Land Parkway,Broken Land Parkway | depart,continue uturn,arrive ||
46-
| f,j | Snowden River Parkway,Snowden River Parkway,Snowden River Parkway | depart,continue uturn,arrive ||
31+
| a,k | Broken Land Parkway,Broken Land Parkway | depart,arrive | | a,k |
32+
| l,b | Broken Land Parkway,Broken Land Parkway | depart,arrive | | l,b |
33+
# | g,j | Patuxent Woods Drive,Snowden River Parkway,Snowden River Parkway | depart,continue,arrive | did not work as expected - might be another issue to handle in post process? | |
34+
# | f,c | Snowden River Parkway,Patuxent Woods Drive,Patuxent Woods Drive | depart,continue,arrive | did not work as expected - might be another issue to handle in post process? | |
35+
| a,c | Broken Land Parkway,Patuxent Woods Drive,Patuxent Woods Drive | depart,turn right,arrive | | a,?,c |
36+
| g,k | Patuxent Woods Drive,Broken Land Parkway,Broken Land Parkway | depart,turn right,arrive | | g,?,k |
37+
| l,j | Broken Land Parkway,Snowden River Parkway,Snowden River Parkway | depart,turn right,arrive | | l,?,j |
38+
| f,b | Snowden River Parkway,Broken Land Parkway,Broken Land Parkway | depart,turn right,arrive | | f,?,b |
39+
| a,j | Broken Land Parkway,Snowden River Parkway,Snowden River Parkway | depart,turn left,arrive | | a,?,j |
40+
| g,b | Patuxent Woods Drive,Broken Land Parkway,Broken Land Parkway | depart,turn left,arrive | | g,?,b |
41+
| l,c | Broken Land Parkway,Patuxent Woods Drive,Patuxent Woods Drive | depart,turn left,arrive | | l,?,c |
42+
| f,k | Snowden River Parkway,Broken Land Parkway,Broken Land Parkway | depart,turn left,arrive | | f,?,k |
43+
| a,b | Broken Land Parkway,Broken Land Parkway,Broken Land Parkway | depart,continue uturn,arrive | | a,?,b |
44+
| g,c | Patuxent Woods Drive,Patuxent Woods Drive,Patuxent Woods Drive | depart,continue uturn,arrive | | g,?,c |
45+
| l,k | Broken Land Parkway,Broken Land Parkway,Broken Land Parkway | depart,continue uturn,arrive | | l,?,k |
46+
| f,j | Snowden River Parkway,Snowden River Parkway,Snowden River Parkway | depart,continue uturn,arrive | | f,?,j |

features/guidance/maneuver-tag.feature

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -37,13 +37,13 @@ Feature: Maneuver tag support
3737
When I route I should get
3838
| waypoints | route | turns | locations |
3939
# Testing directly connected from/to
40-
| a,j | A Street,C Street,J Street,J Street | depart,turn sharp right,turn left,arrive |
41-
| b,g | A Street,C Street,C Street | depart,turn sharp right,arrive |
40+
| a,j | A Street,C Street,J Street,J Street | depart,turn sharp right,turn left,arrive | a,?,?,j |
41+
| b,g | A Street,C Street,C Street | depart,turn sharp right,arrive | b,?,g |
4242
# Testing re-awakening suppressed turns
43-
| a,e | A Street,B Street,B Street | depart,turn slight left,arrive |
44-
| e,i | B Street,C Street,C Street | depart,turn straight,arrive |
45-
| i,e | C Street,B Street,B Street | depart,fork slight right,arrive |
46-
| i,a | C Street,A Street,A Street | depart,turn right,arrive |
43+
| a,e | A Street,B Street,B Street | depart,turn slight left,arrive | a,?,e |
44+
| e,i | B Street,C Street,C Street | depart,turn straight,arrive | e,?,i |
45+
| i,e | C Street,B Street,B Street | depart,fork slight right,arrive | i,?,e |
46+
| i,a | C Street,A Street,A Street | depart,turn right,arrive | i,?,a |
4747

4848
Scenario: single via-way
4949
Given the node map

features/guidance/merge-segregated-roads.feature

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -672,10 +672,10 @@ Feature: Merge Segregated Roads
672672
When I route I should get
673673
| waypoints | route | turns | # | locations |
674674
| a,c | otto,otto | depart,arrive | | a,c |
675-
| a,f | otto,otto,otto | depart,continue uturn,arrive | |
676-
| a,1 | otto,otto,otto | depart,continue left,arrive | |
677-
| a,d | otto,neu,neu | depart,turn left,arrive | |
678-
| c,1 | otto,otto | depart,arrive | |
675+
| a,f | otto,otto,otto | depart,continue uturn,arrive | | a,?,f |
676+
| a,1 | otto,otto,otto | depart,continue left,arrive | | a,?,1 |
677+
| a,d | otto,neu,neu | depart,turn left,arrive | | a,?,d |
678+
| c,1 | otto,otto | depart,arrive | | c,1 |
679679
| c,f | otto,otto,otto | depart,continue left,arrive | Ideally, this would be depart,arrive, but the obvious discovery making the turn onto `1` from `c` obvious interferes here | c,o,f |
680680

681681
# https://www.openstreetmap.org/#map=18/50.94608/7.02030

0 commit comments

Comments
 (0)