Geocodio: add limit=1 for exactly_one=True (#468)

geopy/geocoders/geocodio.py

@@ -130,7 +130,8 @@ def geocode(
             raise GeocoderQueryError('If street is provided must also provide city, '
                                      'state, and/or postal_code.')
 
-        api = '%s://%s%s' % (self.scheme, self.domain, self.geocode_path)
+        if exactly_one:
+            limit = 1
 
         params = dict(
             api_key=self.api_key,
@@ -146,6 +147,7 @@ def geocode(
             k: v for k, v in params.items() if v is not None
         }
 
+        api = '%s://%s%s' % (self.scheme, self.domain, self.geocode_path)
         url = "?".join((api, urlencode(params)))
 
         logger.debug("%s.geocode: %s", self.__class__.__name__, url)
@@ -183,11 +185,12 @@ def reverse(
             'q': self._coerce_point_to_string(query),
             'api_key': self.api_key
         }
+        if exactly_one:
+            limit = 1
         if limit is not None:
             params['limit'] = limit
 
         api = '%s://%s%s' % (self.scheme, self.domain, self.reverse_path)
-
         url = "?".join((api, urlencode(params)))
 
         logger.debug("%s.reverse: %s", self.__class__.__name__, url)

test/geocoders/geocodio.py

@@ -69,6 +69,13 @@ async def test_zero_results(self):
         assert str(excinfo.value) == 'Could not geocode address. ' \
                                      'Postal code or city required.'
 
+    async def test_geocode_many_results(self):
+        result = await self.geocode_run(
+            {"query": "Springfield", "exactly_one": False},
+            {}
+        )
+        assert len(result) > 1
+
     async def test_reverse(self):
         location = await self.reverse_run(
             {"query": Point(40.75376406311989, -73.98489005863667)},