Geoip2 provider exceptions (#714)

* Create test case and show handling one exception

* Handle credentials and quota exceeded exception transformation, closes #534

* Fix style issues

* Fix style issues - spacing

Author: redthor

src/Provider/GeoIP2/GeoIP2.php

@@ -21,8 +21,12 @@
 use Geocoder\Provider\IpAddressGeocoder;
 use Geocoder\Provider\LocaleAwareGeocoder;
 use Geocoder\Provider\Provider;
-use GeoIp2\Exception\AddressNotFoundException;
 use Geocoder\Exception\UnsupportedOperation;
+use Geocoder\Exception\InvalidCredentials;
+use Geocoder\Exception\QuotaExceeded;
+use GeoIp2\Exception\AddressNotFoundException;
+use GeoIp2\Exception\AuthenticationException;
+use GeoIp2\Exception\OutOfQueriesException;
 
 /**
  * @author Jens Wiese <jens@howtrueisfalse.de>
@@ -114,6 +118,18 @@ private function executeQuery(string $address): string
             $result = $this->adapter->getContent($uri);
         } catch (AddressNotFoundException $e) {
             return '';
+        } catch (AuthenticationException $e) {
+            throw new InvalidCredentials(
+                $e->getMessage(),
+                $e->getCode(),
+                $e
+            );
+        } catch (OutOfQueriesException $e) {
+            throw new QuotaExceeded(
+                $e->getMessage(),
+                $e->getCode(),
+                $e
+            );
         }
 
         return $result;

src/Provider/GeoIP2/Tests/GeoIP2Test.php

@@ -20,6 +20,10 @@
 use Geocoder\Provider\GeoIP2\GeoIP2;
 use Geocoder\Provider\GeoIP2\GeoIP2Adapter;
 use GeoIp2\Database\Reader;
+use GeoIp2\Exception\AuthenticationException;
+use GeoIp2\Exception\OutOfQueriesException;
+use Geocoder\Exception\InvalidCredentials;
+use Geocoder\Exception\QuotaExceeded;
 
 /**
  * @author Jens Wiese <jens@howtrueisfalse.de>
@@ -223,6 +227,32 @@ public function testGeoIp2Encoding()
         $this->assertEquals('Timișoara', $locality);
     }
 
+    /**
+     * @dataProvider provideDataForTestingExceptions
+     *
+     * @param \Exception $original
+     * @param string     $replacementClass
+     */
+    public function testExceptionConversion(\Exception $original, string $replacementClass)
+    {
+        $adapter = $this->getGeoIP2AdapterMock($original);
+        $provider = new GeoIP2($adapter);
+
+        self::expectException($replacementClass);
+        self::expectExceptionMessage($original->getMessage());
+        self::expectExceptionCode($original->getCode());
+
+        $results = $provider->geocodeQuery(GeocodeQuery::create('74.200.247.59'));
+    }
+
+    public static function provideDataForTestingExceptions(): array
+    {
+        return [
+            [new AuthenticationException('Credentials are no good'), InvalidCredentials::class],
+            [new OutOfQueriesException('You ran out'), QuotaExceeded::class],
+        ];
+    }
+
     /**
      * @param mixed $returnValue
      *