Skip to content

Commit

Permalink
highway=motorway_link does not imply oneway, fixes #1662
Browse files Browse the repository at this point in the history
  • Loading branch information
daniel-j-h committed May 31, 2016
1 parent 5031372 commit 92337b8
Show file tree
Hide file tree
Showing 5 changed files with 30 additions and 29 deletions.
14 changes: 7 additions & 7 deletions features/bicycle/oneway.feature
Original file line number Diff line number Diff line change
Expand Up @@ -48,13 +48,13 @@ Feature: Bike - Oneway streets

Scenario: Bike - Implied oneways
Then routability should be
| highway | foot | bicycle | junction | forw | backw |
| | no | | | x | x |
| | no | | roundabout | x | |
| motorway | no | yes | | x | |
| motorway_link | no | yes | | x | |
| motorway | no | yes | roundabout | x | |
| motorway_link | no | yes | roundabout | x | |
| highway | foot | bicycle | junction | forw | backw | # |
| | no | | | x | x | |
| | no | | roundabout | x | | |
| motorway | no | yes | | x | | |
| motorway_link | no | yes | | x | x | does not imply oneway |
| motorway | no | yes | roundabout | x | | |
| motorway_link | no | yes | roundabout | x | | |

Scenario: Bike - Overriding implied oneways
Then routability should be
Expand Down
22 changes: 12 additions & 10 deletions features/car/oneway.feature
Original file line number Diff line number Diff line change
Expand Up @@ -17,19 +17,21 @@ Feature: Car - Oneway streets

Scenario: Car - Implied oneways
Then routability should be
| highway | junction | forw | backw |
| motorway | | x | |
| motorway_link | | x | |
| primary | | x | x |
| motorway | roundabout | x | |
| motorway_link | roundabout | x | |
| primary | roundabout | x | |
| highway | junction | forw | backw | # |
| motorway | | x | | |
| motorway_link | | x | x | does not imply oneway |
| primary | | x | x | |
| motorway | roundabout | x | | |
| motorway_link | roundabout | x | | |
| primary | roundabout | x | | |

Scenario: Car - Overrule implied oneway
Then routability should be
| highway | oneway | forw | backw |
| motorway | no | x | x |
| motorway_link | no | x | x |
| highway | oneway | forw | backw | # |
| motorway | no | x | x | |
| motorway_link | no | x | x | |
| motorway_link | yes | x | | |
| motorway_link | | x | x | does not imply onway |

Scenario: Car - Around the Block
Given the node map
Expand Down
20 changes: 10 additions & 10 deletions features/guidance/motorway.feature
Original file line number Diff line number Diff line change
Expand Up @@ -176,11 +176,11 @@ Feature: Motorway Guidance
| chi | motorway_link |

When I route I should get
| waypoints | route | turns |
| a,e | abcde,abcde | depart,arrive |
| f,e | fgc,abcde,abcde | depart,merge slight left,arrive |
| a,i | abcde,chi,chi | depart,off ramp slight right,arrive |
| f,i | fgc,chi,chi | depart,off ramp right,arrive |
| waypoints | route | turns |
| a,e | abcde,abcde | depart,arrive |
| f,e | fgc,abcde,abcde | depart,merge slight left,arrive |
| a,i | abcde,chi,chi | depart,on ramp slight right,arrive |
| f,i | fgc,chi,chi | depart,off ramp right,arrive |

Scenario: On And Off Ramp Left
Given the node map
Expand All @@ -194,11 +194,11 @@ Feature: Motorway Guidance
| chi | motorway_link |

When I route I should get
| waypoints | route | turns |
| a,e | abcde,abcde | depart,arrive |
| f,e | fgc,abcde,abcde | depart,merge slight right,arrive |
| a,i | abcde,chi,chi | depart,off ramp slight left,arrive |
| f,i | fgc,chi,chi | depart,off ramp left,arrive |
| waypoints | route | turns |
| a,e | abcde,abcde | depart,arrive |
| f,e | fgc,abcde,abcde | depart,merge slight right,arrive |
| a,i | abcde,chi,chi | depart,on ramp slight left,arrive |
| f,i | fgc,chi,chi | depart,off ramp left,arrive |

Scenario: Merging Motorways
Given the node map
Expand Down
2 changes: 1 addition & 1 deletion profiles/bicycle.lua
Original file line number Diff line number Diff line change
Expand Up @@ -317,7 +317,7 @@ function way_function (way, result)

-- direction
local impliedOneway = false
if junction == "roundabout" or highway == "motorway_link" or highway == "motorway" then
if junction == "roundabout" or highway == "motorway" then
impliedOneway = true
end

Expand Down
1 change: 0 additions & 1 deletion profiles/car.lua
Original file line number Diff line number Diff line change
Expand Up @@ -390,7 +390,6 @@ function way_function (way, result)
oneway == "1" or
oneway == "true" or
junction == "roundabout" or
(highway == "motorway_link" and oneway ~="no") or
(highway == "motorway" and oneway ~= "no") then
result.backward_mode = mode.inaccessible

Expand Down

0 comments on commit 92337b8

Please sign in to comment.