Merge pull request #76 from cecileprat/master

Fix route v8 multiple via points

herepy/routing_api.py

@@ -448,7 +448,7 @@ def route_v8(
  via_keys = []
  if via:
  for i, v in enumerate(via):
- key = str.format("{0}{1}", "via", i)
+ key = str.format("{0}{1}_", "via", i)
  via_keys.append(key)
  data[key] = str.format("{0},{1}", v[0], v[1])
  if departure_time:

tests/test_routing_api.py

@@ -1179,3 +1179,25 @@ def test_route_v8_error_access_denied(self):
  truck={"shippedHazardousGoods": ["explosive", "gas"]},
  scooter={"allowHighway": "true"},
  )
+
+ @responses.activate
+ def test_route_v8_multiple_via_points(self):
+ responses.add(
+ responses.GET,
+ "https://router.hereapi.com/v8/routes",
+ "{}",
+ status=200,
+ match=[
+ responses.matchers.query_param_matcher(
+ {"via": ["41.9339,-87.9021"] * 11}, strict_match=False
+ )
+ ],
+ )
+ response = self._api.route_v8(
+ transport_mode=herepy.RoutingTransportMode.car,
+ origin=[41.9798, -87.8801],
+ destination=[41.9043, -87.9216],
+ via=[[41.9339, -87.9021]] * 11,
+ )
+ self.assertTrue(response)
+ self.assertIsInstance(response, herepy.RoutingResponseV8)