composer.json

@@ -16,11 +16,11 @@
         }
     ],
     "require": {
-        "php": "^8.1",
+        "php": ">=7.4",
-        "laravel/framework": ">=9.19",
+        "laravel/framework": ">=5.8",
-        "spatie/geocoder": "^3.15",
+        "spatie/geocoder": "^3.15"
-        "spatie/laravel-data": "^3.11"
     },
+    "minimum-stability": "stable",
     "extra": {
         "laravel": {
             "providers": [
@@ -29,7 +29,7 @@
         }
     },
     "require-dev": {
-        "orchestra/testbench": "^7.38",
+        "phpunit/phpunit": "^9.6",
-        "phpunit/phpunit": "^9.6"
+        "orchestra/testbench": "^6.41"
     }
 }

phpunit.xml

@@ -1,16 +1,24 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" backupGlobals="false" backupStaticAttributes="false" bootstrap="vendor/autoload.php" colors="true" convertErrorsToExceptions="true" convertNoticesToExceptions="true" convertWarningsToExceptions="true" processIsolation="false" stopOnFailure="false" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.3/phpunit.xsd">
+<phpunit backupGlobals="false"
-  <coverage processUncoveredFiles="true">
+         backupStaticAttributes="false"
-    <include>
+         bootstrap="vendor/autoload.php"
-      <directory suffix=".php">src</directory>
+         colors="true"
-    </include>
+         convertErrorsToExceptions="true"
-  </coverage>
+         convertNoticesToExceptions="true"
-  <testsuites>
+         convertWarningsToExceptions="true"
-    <testsuite name="Unit">
+         processIsolation="false"
-      <directory suffix="Test.php">./tests/Unit</directory>
+         stopOnFailure="false">
-    </testsuite>
+    <testsuites>
-  </testsuites>
+        <testsuite name="Unit">
-  <php>
+            <directory suffix="Test.php">./tests/Unit</directory>
-    <server name="APP_ENV" value="testing"/>
+        </testsuite>
-  </php>
+    </testsuites>
+    <filter>
+        <whitelist processUncoveredFilesFromWhitelist="true">
+            <directory suffix=".php">src</directory>
+        </whitelist>
+    </filter>
+    <php>
+        <server name="APP_ENV" value="testing"/>
+    </php>
 </phpunit>

phpunit.xml.bak

@@ -1,24 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<phpunit backupGlobals="false"
-         backupStaticAttributes="false"
-         bootstrap="vendor/autoload.php"
-         colors="true"
-         convertErrorsToExceptions="true"
-         convertNoticesToExceptions="true"
-         convertWarningsToExceptions="true"
-         processIsolation="false"
-         stopOnFailure="false">
-    <testsuites>
-        <testsuite name="Unit">
-            <directory suffix="Test.php">./tests/Unit</directory>
-        </testsuite>
-    </testsuites>
-    <filter>
-        <whitelist processUncoveredFilesFromWhitelist="true">
-            <directory suffix=".php">src</directory>
-        </whitelist>
-    </filter>
-    <php>
-        <server name="APP_ENV" value="testing"/>
-    </php>
-</phpunit>

src/app/DataTransferObjects/GeocodingResponseData.php

@@ -1,31 +0,0 @@
-<?php
-
-namespace Ae3\LaravelGeoLayer\app\DataTransferObjects;
-
-use Spatie\LaravelData\Data;
-
-class GeocodingResponseData extends Data
-{
-    /**
-     * @param float $lat
-     * @param float $lng
-     * @param string $accuracy
-     * @param string $formatted_address
-     * @param array $viewport
-     * @param array $address_components
-     * @param bool $partial_match
-     * @param string $place_id
-     * @param array $types
-     */
-    public function __construct(
-        public float $lat,
-        public float $lng,
-        public string $accuracy,
-        public string $formatted_address,
-        public array $viewport,
-        public array $address_components,
-        public bool $partial_match,
-        public string $place_id,
-        public array $types
-    ) {}
-}

src/app/Services/Contracts/GeocodingServiceContract.php

@@ -2,26 +2,18 @@
 
 namespace Ae3\LaravelGeoLayer\app\Services\Contracts;
 
-use Ae3\LaravelGeoLayer\app\DataTransferObjects\GeocodingResponseData;
-
 interface GeocodingServiceContract
 {
     /**
-     * Retrieves the coordinates for the given address.
+     * @param string $address
-     *
+     * @return array
-     * @param string $address The address to retrieve coordinates for.
-     *
-     * @return GeocodingResponseData An object containing the latitude and longitude coordinates.
      */
-    public function getCoordinatesForAddress(string $address): GeocodingResponseData;
+    public function getCoordinatesForAddress(string $address): array;
 
     /**
-     * Retrieves the address for the given coordinates.
+     * @param float $latitude
-     *
+     * @param float $longitude
-     * @param float $latitude The latitude of the coordinates.
+     * @return array
-     * @param float $longitude The longitude of the coordinates.
-     *
-     * @return GeocodingResponseData The address information for the given coordinates.
      */
-    public function getAddressForCoordinates(float $latitude, float $longitude): GeocodingResponseData;
+    public function getAddressForCoordinates(float $latitude, float $longitude): array;
 }

src/app/Services/GeocodingService.php

@@ -2,7 +2,6 @@
 
 namespace Ae3\LaravelGeoLayer\app\Services;
 
-use Ae3\LaravelGeoLayer\app\DataTransferObjects\GeocodingResponseData;
 use Ae3\LaravelGeoLayer\app\Services\Contracts\GeocodingServiceContract;
 use GuzzleHttp\Client;
 use Spatie\Geocoder\Geocoder;
@@ -17,10 +16,9 @@ class GeocodingService implements GeocodingServiceContract
     /**
      * Constructs a new instance of the class.
      */
-    public function __construct(
+    public function __construct()
-        protected readonly Client $client
-    )
     {
+        $client = new Client();
         $this->geocoder = new Geocoder($client);
         $this->geocoder->setApiKey(config('laravel-geo-layer.key'));
         $this->geocoder->setRegion(config('laravel-geo-layer.region'));
@@ -33,21 +31,11 @@ class GeocodingService implements GeocodingServiceContract
      *
      * @param string $address The address to retrieve coordinates for.
      *
-     * @return GeocodingResponseData An array containing the latitude and longitude coordinates.
+     * @return array An array containing the latitude and longitude coordinates.
      */
-    public function getCoordinatesForAddress(string $address): GeocodingResponseData
+    public function getCoordinatesForAddress(string $address): array
     {
-        $response = $this->geocoder->getCoordinatesForAddress($address);
+        return $this->geocoder->getCoordinatesForAddress($address);
-
-        $viewport = array_key_exists('viewport', $response)
-            ? get_object_vars($response['viewport'])
-            : [];
-
-        $response['viewport'] = $viewport;
-
-        return GeocodingResponseData::from(
-            $response
-        );
     }
 
     /**
@@ -56,20 +44,10 @@ class GeocodingService implements GeocodingServiceContract
      * @param float $latitude The latitude of the coordinates.
      * @param float $longitude The longitude of the coordinates.
      *
-     * @return GeocodingResponseData The address information for the given coordinates.
+     * @return array The address information for the given coordinates.
      */
-    public function getAddressForCoordinates(float $latitude, float $longitude): GeocodingResponseData
+    public function getAddressForCoordinates(float $latitude, float $longitude): array
     {
-        $response = $this->geocoder->getAddressForCoordinates($latitude, $longitude);
+        return $this->geocoder->getAddressForCoordinates($latitude, $longitude);
-
-        $viewport = array_key_exists('viewport', $response)
-            ? get_object_vars($response['viewport'])
-            : [];
-
-        $response['viewport'] = $viewport;
-
-        return GeocodingResponseData::from(
-            $response
-        );
     }
 }

src/config/config.php

@@ -2,8 +2,8 @@
 
 return [
     /*
-    * The api key used when sending Geocoding requests to Google.
+        * The api key used when sending Geocoding requests to Google.
-    */
+        */
     'key' => env('GOOGLE_MAPS_GEOCODING_API_KEY', ''),
 
     /*
@@ -12,26 +12,26 @@ return [
      * More info: https://developers.google.com/maps/faq#languagesupport
      */
 
-    'language' => env('GOOGLE_MAPS_GEOCODING_LANGUAGE', ''),
+    'language' => '',
 
     /*
      * The region param used to finetune the geocoding process.
      *
      * More info: https://developers.google.com/maps/documentation/geocoding/requests-geocoding#RegionCodes
      */
-    'region' => env('GOOGLE_MAPS_GEOCODING_REGION', 'br'),
+    'region' => 'br',
 
     /*
      * The bounds param used to finetune the geocoding process.
      *
      * More info: https://developers.google.com/maps/documentation/geocoding/requests-geocoding#Viewports
      */
-    'bounds' => env('GOOGLE_MAPS_GEOCODING_BOUNDS', 'Roraima'),
+    'bounds' => 'Roraima',
 
     /*
      * The country param used to limit results to a specific country.
      *
      * More info: https://developers.google.com/maps/documentation/javascript/geocoding#GeocodingRequests
      */
-    'country' => env('GOOGLE_MAPS_GEOCODING_COUNTRY', 'Brazil'),
+    'country' => 'Brazil',
 ];

tests/Unit/GeocodingServiceTest.php

@@ -0,0 +1,67 @@
+<?php
+
+namespace Unit;
+
+use Ae3\LaravelGeoLayer\app\Services\GeocodingService;
+use Ae3\LaravelGeoLayer\Tests\TestCase;
+use Mockery;
+
+class GeocodingServiceTest extends TestCase
+{
+    /**
+     * @var GeocodingService
+     */
+    private GeocodingService $geocodingService;
+
+    private $mockedGeocoder;
+
+    /**
+     * Set up the test
+     */
+    public function setUp(): void
+    {
+        parent::setUp();
+        $this->mockedGeocoder = Mockery::mock('overload:Spatie\Geocoder\Geocoder');
+
+        // Configurando o mock para esperar a chamada de setApiKey
+        $this->mockedGeocoder->shouldReceive('setApiKey')->once()->withAnyArgs();
+        $this->mockedGeocoder->shouldReceive('setRegion')->once()->withAnyArgs();
+        $this->mockedGeocoder->shouldReceive('setCountry')->once()->withAnyArgs();
+        $this->mockedGeocoder->shouldReceive('setBounds')->once()->withAnyArgs();
+
+        // Configurando o mock para esperar a chamada de getCoordinatesForAddress
+        $this->mockedGeocoder->shouldReceive('getCoordinatesForAddress')
+            ->once()
+            ->with('Brussels, Belgium')
+            ->andReturn(['lat' => '50.85045', 'lng' => '4.34878']);
+
+        $this->geocodingService = new GeocodingService();
+    }
+
+
+
+    /**
+     * Test getCoordinatesForAddress function
+     */
+    public function testGetCoordinatesForAddress(): void
+    {
+        // Mocking Geocoder response
+        $this->mockedGeocoder->shouldReceive('getCoordinatesForAddress')
+            ->once()
+            ->with('Brussels, Belgium')
+            ->andReturn(['lat' => '50.85045', 'lng' => '4.34878']);
+
+        // Testing the function
+        $result = $this->geocodingService->getCoordinatesForAddress('Brussels, Belgium');
+        $this->assertEquals(['lat' => '50.85045', 'lng' => '4.34878'], $result);
+    }
+
+    /**
+     * Tear down the test
+     */
+    public function tearDown(): void
+    {
+        Mockery::close();
+        parent::tearDown();
+    }
+}