Merge pull request #78 from James-Burgess/master

herepy/routing_api.py

@@ -987,6 +987,8 @@ def error_from_routing_service_error(json_data):
  # V8 error handling
  if "error" in json_data and json_data["error"] == "Unauthorized":
  return InvalidCredentialsError(json_data["error_description"])
+ elif "error" in json_data and json_data["error"] == "Forbidden":
+ return InvalidCredentialsError(json_data["error_description"])
  elif "status" in json_data:
  error_msg = str.format(
  "Cause: {0}; Action: {1}", json_data["cause"], json_data["action"]

testdata/models/routing_error_forbidden_credentials.json

@@ -0,0 +1,4 @@
+{
+ "error": "Forbidden",
+ "error_description": "These credentials do not authorize access"
+}

tests/test_routing_api.py

@@ -140,6 +140,20 @@ def test_carroute_when_error_invalid_credentials_occurred(self):
  with self.assertRaises(herepy.InvalidCredentialsError):
  api.car_route([11.0, 12.0], [22.0, 23.0])
 
+ @responses.activate
+ def test_carroute_when_error_forbidden_credentials_occurred(self):
+ with open("testdata/models/routing_error_forbidden_credentials.json", "r") as f:
+ expectedResponse = f.read()
+ responses.add(
+ responses.GET,
+ "https://route.ls.hereapi.com/routing/7.2/calculateroute.json",
+ expectedResponse,
+ status=401,
+ )
+ api = herepy.RoutingApi("forbidden_api_key", "forbidden_app_code")
+ with self.assertRaises(herepy.InvalidCredentialsError):
+ api.car_route([11.0, 12.0], [22.0, 23.0])
+
  @responses.activate
  def test_carroute_when_error_no_route_found_occurred(self):
  with open("testdata/models/routing_error_no_route_found.json", "r") as f: