Fix LocationManager::get() return type and simplify it.

src/Facades/Location.php

@@ -8,7 +8,7 @@
 
 /**
  * @method static \Stevebauman\Location\Drivers\Driver[] drivers()
- * @method static \Stevebauman\Location\Position|bool get(string $ip = null)
+ * @method static \Stevebauman\Location\Position|false get(string $ip = null)
  * @method static void resolveRequestUsing(callable $callback)
  * @method static void setDriver(\Stevebauman\Location\Drivers\Driver $driver)
  */

src/LocationFake.php

@@ -28,7 +28,7 @@ public function __call(string $method, array $parameters): mixed
     /**
      * Get a fake location instance.
      */
-    public function get(?string $ip = null): Position|bool
+    public function get(?string $ip = null): Position|false
     {
         $ip ??= '127.0.0.1';
 

src/LocationManager.php

@@ -70,13 +70,9 @@ public function setDefaultDriver(): static
     /**
      * Attempt to retrieve the location of the user.
      */
-    public function get(?string $ip = null): Position|bool
+    public function get(?string $ip = null): Position|false
     {
-        if ($location = $this->driver->get($this->request()->setIp($ip))) {
-            return $location;
-        }
-
-        return false;
+        return $this->driver->get($this->request()->setIp($ip));
     }
 
     /**

tests/LocationTest.php

@@ -31,6 +31,20 @@
     $this->assertInstanceOf(Position::class, Location::get());
 });
 
+it('returns false when no driver can resolve a location', function () {
+    $driver = m::mock(Driver::class)
+        ->makePartial()
+        ->shouldAllowMockingProtectedMethods();
+
+    $driver->shouldReceive('process')
+        ->once()
+        ->andReturn(false);
+
+    Location::setDriver($driver);
+
+    expect(Location::get())->toBeFalse();
+});
+
 it('throws an exception when the driver does not exist', function () {
     config(['location.driver' => 'Test']);