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: