Raise InvalidRequest is the query is not a string

Author: sbscully

lib/opencage/geocoder.rb

@@ -13,6 +13,8 @@ def initialize(default_options = {})
     end
 
     def geocode(location, options = {})
+      raise_error("400 not a valid location: `#{location.inspect}`") unless location.is_a?(String)
+
       request = Request.new(@api_key, location, options)
 
       begin

spec/open_cage/geocoder_spec.rb

@@ -82,6 +82,12 @@ def geo
     it 'empty list when no results are found', :vcr do
       expect(geo.geocode('NOWHERE-INTERESTING')).to eql([])
     end
+
+    it 'throws a useful error when the query is not a string' do
+      expect do
+        geo.geocode({ query: 'NOT-A-STRING' })
+      end.to raise_error(OpenCage::Error::InvalidRequest)
+    end
   end
 
   describe 'user agent' do