Merge pull request #9 from MetalMatze/feature/point-float

Cast all inputs of point to floats.
Closes #8

Author: phaza

src/Geometries/Point.php

@@ -7,8 +7,8 @@ class Point extends Geometry
 
     public function __construct($lat, $lng)
     {
-        $this->lat = $lat;
-        $this->lng = $lng;
+        $this->lat = (float)$lat;
+        $this->lng = (float)$lng;
     }
 
     public function getLat()
@@ -18,7 +18,7 @@ public function getLat()
 
     public function setLat($lat)
     {
-        $this->lat = $lat;
+        $this->lat = (float)$lat;
     }
 
     public function getLng()
@@ -28,7 +28,7 @@ public function getLng()
 
     public function setLng($lng)
     {
-        $this->lng = $lng;
+        $this->lng = (float)$lng;
     }
 
     public function toPair()
@@ -40,7 +40,7 @@ public static function fromPair($pair)
     {
         list($lng, $lat) = explode(' ', trim($pair));
 
-        return new static($lat, $lng);
+        return new static((float)$lat, (float)$lng);
     }
 
     public function toWKT()

tests/Geometries/PointTest.php

@@ -1,6 +1,5 @@
 <?php
 
-use Phaza\LaravelPostgis\Geometries\Geometry;
 use Phaza\LaravelPostgis\Geometries\Point;
 
 class PointTest extends BaseTestCase
@@ -21,10 +20,36 @@ public function testToWKT()
         $this->assertEquals('POINT(2 1)', $point->toWKT());
     }
 
-    public function testToString()
+    public function testGettersAndSetters()
     {
         $point = new Point(1, 2);
+        $this->assertSame(1.0, $point->getLat());
+        $this->assertSame(2.0, $point->getLng());
+
+        $point->setLat('3');
+        $point->setLng('4');
+
+        $this->assertSame(3.0, $point->getLat());
+        $this->assertSame(4.0, $point->getLng());
+    }
+
+    public function testPair()
+    {
+        $point = Point::fromPair('1.5 2');
+
+        $this->assertSame(1.5, $point->getLng());
+        $this->assertSame(2.0, $point->getLat());
+
+        $this->assertSame('1.5 2', $point->toPair());
+    }
+
+    public function testToString()
+    {
+        $point = Point::fromString('1.3 2');
+
+        $this->assertSame(1.3, $point->getLng());
+        $this->assertSame(2.0, $point->getLat());
 
-        $this->assertEquals('2 1', (string)$point);
+        $this->assertEquals('1.3 2', (string)$point);
     }
 }