handles unreachable network exception (#1030)

* handles unreachable network exception

* adds network error class

* tests unreachable host exception handling

lib/geocoder/exceptions.rb

@@ -34,4 +34,7 @@ module Geocoder
   class LookupTimeout < ::Timeout::Error
   end
 
+  class NetworkError < Error
+  end
+
 end

lib/geocoder/lookups/base.rb

@@ -286,6 +286,8 @@ module Geocoder
         end
       rescue Timeout::Error
         raise Geocoder::LookupTimeout
+      rescue Errno::EHOSTUNREACH, Errno::ETIMEDOUT, Errno::ENETUNREACH
+        raise Geocoder::NetworkError
       end
 
       def use_ssl?

test/test_helper.rb

@@ -129,6 +129,7 @@ module Geocoder
         raise Timeout::Error if query.text == "timeout"
         raise SocketError if query.text == "socket_error"
         raise Errno::ECONNREFUSED if query.text == "connection_refused"
+        raise Errno::EHOSTUNREACH if query.text == "host_unreachable"
         if query.text == "invalid_json"
           return MockHttpResponse.new(:body => 'invalid json', :code => 200)
         end

test/unit/error_handling_test.rb

@@ -75,4 +75,16 @@ class ErrorHandlingTest < GeocoderTestCase
       end
     end
   end
+
+  def test_always_raise_host_unreachable_error
+    Geocoder.configure(:always_raise => [Errno::EHOSTUNREACH])
+    Geocoder::Lookup.all_services_except_test.each do |l|
+      next if l == :maxmind_local || l == :geoip2 # local, does not use cache
+      lookup = Geocoder::Lookup.get(l)
+      set_api_key!(l)
+      assert_raises Errno::EHOSTUNREACH do
+        lookup.send(:results, Geocoder::Query.new("host_unreachable"))
+      end
+    end
+  end
 end