Merge pull request #129 from mstaack/rework-tests

cleanup and namespaces

Author: mstaack

.github/workflows/run-tests.yml

@@ -1,6 +1,6 @@
 name: Test Suite
 
-on: [push, pull_request]
+on: [push]
 
 jobs:
   tests:

README.md

@@ -1,24 +1,36 @@
-Laravel postgis extension
-=========================
+Laravel Wrapper for PostgreSQL's Geo-Extension Postgis
+======================================================
 
 ![Build Status](https://github.com/mstaack/laravel-postgis/workflows/Test%20Suite/badge.svg)
 
 ## Features
 
- * Work with geometry classes instead of arrays. (`$myModel->myPoint = new Point(1,2)`)
- * Adds helpers in migrations. (`$table->polygon('myColumn')`)
+ * Work with geometry classes instead of arrays.
+```php
+$model->myPoint = new Point(1,2);  //lat, long
+```
+
+* Adds helpers in migrations.
+```php
+$table->polygon('myColumn');
+```
 
 ## Versions
 - Use 4.* for Laravel 5
 - Use 5.* for Laravel 6/7
 
 ## Warning
 This Package has been moved to a new owner and aims for Laravel 6/7  and PHP 7 support only soon!
+
+Replace all your references to the new namespace: `MStaack\LaravelPostgis`
+
 Thanks to :
 - https://github.com/njbarrett
 - https://github.com/phaza
 - https://github.com/mirzap
 
+Fluent in Laravel Packages and Postgres/Postgis? Consider contributing! We are looking for anyone that wants to help out!
+
 ## Installation
 
 ```bash

composer.json

@@ -19,10 +19,9 @@
     }
   },
   "autoload-dev": {
-    "classmap": [
-      "tests/BaseTestCase.php",
-      "tests/Stubs/"
-    ]
+    "psr-4": {
+      "MStaack\\LaravelPostgis\\Tests\\": "tests/"
+    }
   },
   "license": "MIT",
   "authors": [

config/postgis.php

@@ -1,7 +1,5 @@
 <?php
 
 return [
-
-  'schema' => 'public'  // Schema for the Postgis extension
-
-];
+    'schema' => 'public'  // Schema for the Postgis extension
+];

src/DatabaseServiceProvider.php

@@ -1,20 +1,19 @@
-<?php namespace MStaack\LaravelPostgis;
+<?php
 
+namespace MStaack\LaravelPostgis;
+
+use Bosnadev\Database\DatabaseServiceProvider as PostgresDatabaseServiceProvider;
 use Illuminate\Database\DatabaseManager;
 use MStaack\LaravelPostgis\Connectors\ConnectionFactory;
-use Bosnadev\Database\DatabaseServiceProvider as PostgresDatabaseServiceProvider;
 
-/**
- * Class DatabaseServiceProvider
- * @package MStaack\LaravelPostgis
- */
 class DatabaseServiceProvider extends PostgresDatabaseServiceProvider
 {
-    public function boot() {
-      // Load the config
-      $config_path = __DIR__ . '/../config/postgis.php';
-      $this->publishes([$config_path => config_path('postgis.php')], 'postgis');
-      $this->mergeConfigFrom($config_path, 'postgis');
+    public function boot()
+    {
+        // Load the config
+        $config_path = __DIR__ . '/../config/postgis.php';
+        $this->publishes([$config_path => config_path('postgis.php')], 'postgis');
+        $this->mergeConfigFrom($config_path, 'postgis');
     }
 
     /**

src/Geometries/Factory.php

@@ -1,44 +1,46 @@
-<?php namespace MStaack\LaravelPostgis\Geometries;
+<?php
 
-class Factory implements \GeoIO\Factory {
-    public function createPoint( $dimension, array $coordinates, $srid = null )
+namespace MStaack\LaravelPostgis\Geometries;
+
+class Factory implements \GeoIO\Factory
+{
+    public function createPoint($dimension, array $coordinates, $srid = null)
     {
-        return new Point( $coordinates['y'], $coordinates['x'], isset($coordinates['z']) ? $coordinates['z'] : null );
+        return new Point($coordinates['y'], $coordinates['x'], $coordinates['z'] ?? null);
     }
 
-    public function createLineString( $dimension, array $points, $srid = null )
+    public function createLineString($dimension, array $points, $srid = null)
     {
-        return new LineString( $points );
+        return new LineString($points);
     }
 
-    public function createLinearRing( $dimension, array $points, $srid = null )
+    public function createLinearRing($dimension, array $points, $srid = null)
     {
-        return new LineString( $points );
+        return new LineString($points);
     }
 
-    public function createPolygon( $dimension, array $lineStrings, $srid = null )
+    public function createPolygon($dimension, array $lineStrings, $srid = null)
     {
-        return new Polygon( $lineStrings );
+        return new Polygon($lineStrings);
     }
 
-    public function createMultiPoint( $dimension, array $points, $srid = null )
+    public function createMultiPoint($dimension, array $points, $srid = null)
     {
-        return new MultiPoint( $points );
+        return new MultiPoint($points);
     }
 
-    public function createMultiLineString( $dimension, array $lineStrings, $srid = null )
+    public function createMultiLineString($dimension, array $lineStrings, $srid = null)
     {
-        return new MultiLineString( $lineStrings );
+        return new MultiLineString($lineStrings);
     }
 
-    public function createMultiPolygon( $dimension, array $polygons, $srid = null )
+    public function createMultiPolygon($dimension, array $polygons, $srid = null)
     {
-        return new MultiPolygon( $polygons );
+        return new MultiPolygon($polygons);
     }
 
-    public function createGeometryCollection( $dimension, array $geometries, $srid = null )
+    public function createGeometryCollection($dimension, array $geometries, $srid = null)
     {
-        return new GeometryCollection( $geometries );
+        return new GeometryCollection($geometries);
     }
-
 }

src/Geometries/Geometry.php

@@ -1,4 +1,6 @@
-<?php namespace MStaack\LaravelPostgis\Geometries;
+<?php
+
+namespace MStaack\LaravelPostgis\Geometries;
 
 use GeoIO\WKB\Parser\Parser;
 use MStaack\LaravelPostgis\Exceptions\UnknownWKTTypeException;

src/Geometries/GeometryCollection.php

@@ -1,7 +1,8 @@
-<?php namespace MStaack\LaravelPostgis\Geometries;
+<?php
+
+namespace MStaack\LaravelPostgis\Geometries;
 
 use Countable;
-use GeoJson\GeoJson;
 use InvalidArgumentException;
 
 class GeometryCollection extends Geometry implements Countable

src/Geometries/GeometryInterface.php

@@ -1,4 +1,6 @@
-<?php namespace MStaack\LaravelPostgis\Geometries;
+<?php
+
+namespace MStaack\LaravelPostgis\Geometries;
 
 interface GeometryInterface
 {

src/Geometries/LineString.php

@@ -1,17 +1,19 @@
-<?php namespace MStaack\LaravelPostgis\Geometries;
+<?php
+
+namespace MStaack\LaravelPostgis\Geometries;
 
 class LineString extends PointCollection implements GeometryInterface
 {
     public function is3d()
     {
-        if(count($this->points) === 0) return false;
+        if (count($this->points) === 0) return false;
         return $this->points[0]->is3d();
     }
 
     public function toWKT()
     {
         $wktType = 'LINESTRING';
-        if($this->is3d()) $wktType .= ' Z';
+        if ($this->is3d()) $wktType .= ' Z';
         return sprintf('%s(%s)', $wktType, $this->toPairList());
     }