Add error handling case for HTTP 405

Author: devxiongmao

lib/geocoder/exceptions.rb

@@ -37,4 +37,7 @@ class LookupTimeout < ::Timeout::Error
   class NetworkError < Error
   end
 
+  class MethodNotAllowed < Error
+  end
+
 end

lib/geocoder/lookups/base.rb

@@ -281,6 +281,9 @@ def check_response_for_errors!(response)
         elsif response.code.to_i == 402
           raise_error(Geocoder::OverQueryLimitError) ||
             Geocoder.log(:warn, "Geocoding API error: 402 Payment Required")
+        elsif response.code.to_i == 405
+          raise_error(Geocoder::MethodNotAllowed) ||
+            Geocoder.log(:warn, "Geocoding API error: 405 Method Not Allowed")
         elsif response.code.to_i == 429
           raise_error(Geocoder::OverQueryLimitError) ||
             Geocoder.log(:warn, "Geocoding API error: 429 Too Many Requests")

test/unit/lookups/nominatim_test.rb

@@ -48,4 +48,13 @@ def test_raises_exception_when_over_query_limit
       l.send(:results, Geocoder::Query.new("over limit"))
     end
   end
+
+  def test_raises_exception_when_method_not_allowed
+    Geocoder.configure(:always_raise => [Geocoder::MethodNotAllowed])
+    l = Geocoder::Lookup.get(:nominatim)
+    assert_raises Geocoder::MethodNotAllowed do
+      response = MockHttpResponse.new(code: 405)
+      l.send(:check_response_for_errors!, response)
+    end
+  end
 end