Update php version to 8.3

Containerfile

@@ -1,4 +1,4 @@
-FROM php:8.1-alpine
+FROM php:8.3-alpine
 
 WORKDIR /code
 COPY --from=composer:2 /usr/bin/composer /usr/bin/composer

composer.json

@@ -14,17 +14,17 @@
         }
     ],
     "require": {
-        "php": "^8.1.0",
-        "symfony/http-client": "^6.1",
-        "symfony/dependency-injection": "^6.1",
-        "symfony/framework-bundle": "^6.1",
-        "symfony/yaml": "^6.1",
+        "php": "^8.3.0",
+        "symfony/http-client": "^7.1",
+        "symfony/dependency-injection": "^7.1",
+        "symfony/framework-bundle": "^7.1",
+        "symfony/yaml": "^7.1",
         "doctrine/orm": "^2.13"
     },
 
     "require-dev": {
         "phpunit/phpunit": "^9.5",
-        "symfony/phpunit-bridge": "^6.1"
+        "symfony/phpunit-bridge": "^7.1"
     },
 
     "autoload": {

src/Model/LatLongModel.php

@@ -23,3 +23,4 @@ class LatLongModel
         return $this->longitude;
     }
 }
+

src/Service/Geocoder.php

@@ -15,17 +15,24 @@ class Geocoder
     public function __construct(private HttpClientInterface $client) {}
 
     /**
-     * Convert a postcode into latitude and longitude
+     * Convert a postcode into latitude and longitude. Returns null if conversion failed.
      *
      * @param string $postcode
-     * @return LatLongModel
+     * @return null|LatLongModel
      */
-    public function geocodePostcode(string $postcode): LatLongModel
+    public function geocodePostcode(string $postcode): ?LatLongModel
     {
         $client = $this->createClient();
         $response = $this->makeApiRequest($client, $postcode);
         $data = $this->getDataFromResponse($response);
-        $this->throwIfDataIsEmpty($data);
+
+        if (empty($data)) {
+            return null;
+        }
+
+        if (array_key_exists('error', $data)) {
+            return null;
+        }
 
         return $this->createLatLongModel($data);
     }
@@ -56,12 +63,6 @@ class Geocoder
         return $response->toArray(false);
     }
 
-    private function throwIfDataIsEmpty(array $data): void
-    {
-        if (empty($data))
-            throw new \Exception("No data was received from API response! Were the arguments valid?");
-    }
-
     private function createLatLongModel(array $data): LatLongModel
     {
         $lat = $this->getLatitudeFromData($data);

tests/Service/GeocodeTest.php

@@ -33,11 +33,10 @@ class GeocodeTest extends KernelTestCase
         $this->assertInstanceOf(Geocoder::class, $this->geocoder);
     }
 
-    public function testGeocodePostcodeThrowsOnInvalidInput(): void
+    public function testGeocodePostcodeReturnsNullOnInvalidInput(): void
     {
         sleep(1);
-        $this->expectException(\Exception::class);
-        $this->geocoder->geocodePostcode('aaaaaaaa');
+        $this->assertNull($this->geocoder->geocodePostcode(''));
     }
 
     public function testGeocodePostcodeReturnsLatLonModel(): void