OneBusAway / wayfinder / 28278726324

81%

main: 80%

Pull #518

github

fix(map): address review feedback on route fit & vehicle animations

- Guard SearchPane.handleRouteClick with a routeLoadToken so a superseded
  rapid route switch bails after its awaits instead of fighting the camera,
  re-adding stops, and mismanaging the vehicle polling interval.
- Unify createPolyline failure contract: GoogleMapProvider now returns null
  on decode failure (matching OSM) instead of throwing; correct both doc
  comments to describe the real async-vs-sync behavior; filter null polylines
  in SearchPane and TripPlanModal.
- Wrap RouteMap.loadRouteData in try/catch (with an isolated fit guard so a
  camera failure can't block stop/vehicle rendering) and include tripId in
  the log.
- Make fitToPolylines uniformly async on both providers; clean up the Google
  idle listener on the timeout path; flatten OSM _getRoutePaths with flat(1).
- await createPolyline in TripPlanModal (was fire-and-forget forEach).
- Add an off-route fallback test for animateMarkerTo.

2016 of 2221 branches covered (90.77%)

Branch coverage included in aggregate %.

204 of 381 new or added lines in 9 files covered. (53.54%)

12318 of 15556 relevant lines covered (79.18%)

4.64 hits per line