a-b-street/abstreet

Design LTN tool - turn restriction editing: testing with `complicated_turn_restictions` required

Open

#1,113 opened on Sep 6, 2023

View on GitHub
 (5 comments) (0 reactions) (0 assignees)Rust (6,779 stars) (256 forks)batch import
good first issueltn

Description

Current behaviour

The LTN turn restriction display and editing features have had limited testing with complicated_turn_restictions. Hence the behaviour may be unspecified and counterintuitive.

Suggesting improvement

Add systematic tests that cover a meaningful range of scenarios involving complicated_turn_restictions.

Contributor guide

Tech stack
rust
Domain
testingbackend
Issue type
test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
mostly clear
Prerequisites
Rust programming languagefamiliarity with abstreet's testing infrastructureunderstanding of turn restrictions in transportation
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60
Research direction
Examine the existing test suite for turn restriction features in the abstreet codebase. Identify the current test coverage and determine where `complicated turn restictions` is handled. Look at the LTN tool implementation for turn restriction editing. Propose test cases covering various scenarios from the issue description. Consult any previous discussions in the issue comments for guidance.
Design LTN tool - turn restriction editing: testing with `complicated_turn_restictions` required · a-b-street/abstreet#1113 | Good First Issue