Skip to content

Commit f2e8ff6

Browse files
author
Max Staack
committed
wip
1 parent 4109ad4 commit f2e8ff6

13 files changed

+68
-65
lines changed

‎composer.json

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,17 +2,16 @@
22
"name": "mstaack/laravel-postgis",
33
"description": "Postgis extensions for laravel. Aims to make it easy to work with geometries from laravel models",
44
"require": {
5-
"php": ">=7.0",
5+
"php": ">=7.1",
66
"illuminate/database": "^6.0|^7.0",
77
"geo-io/wkb-parser": "^1.0",
88
"jmikola/geojson": "^1.0",
99
"bosnadev/database": "0.19.*"
1010
},
1111
"require-dev": {
12-
"phpunit/phpunit": "~4.5",
13-
"mockery/mockery": "0.9.*",
14-
"codeclimate/php-test-reporter": "~0.3",
15-
"illuminate/pagination": "^6.0|^7.0"
12+
"illuminate/pagination": "^6.0|^7.0",
13+
"phpunit/phpunit": "^9.0",
14+
"mockery/mockery": "^1.3"
1615
},
1716
"autoload": {
1817
"psr-4": {
@@ -46,5 +45,8 @@
4645
"MStaack\\LaravelPostgis\\DatabaseServiceProvider"
4746
]
4847
}
48+
},
49+
"scripts": {
50+
"test": "./vendor/bin/phpunit"
4951
}
5052
}

‎phpunit.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
convertNoticesToExceptions="true"
88
convertWarningsToExceptions="true"
99
processIsolation="false"
10-
stopOnFailure="false"
10+
stopOnFailure="true"
1111
syntaxCheck="true"
1212
verbose="true">
1313
<testsuites>

‎tests/BaseTestCase.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,10 @@
11
<?php
22

3-
abstract class BaseTestCase extends PHPUnit_Framework_TestCase
3+
use PHPUnit\Framework\TestCase;
4+
5+
abstract class BaseTestCase extends TestCase
46
{
5-
public function tearDown()
7+
protected function tearDown(): void
68
{
79
Mockery::close();
810
}

‎tests/Eloquent/BuilderTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ class BuilderTest extends BaseTestCase
2020
*/
2121
protected $queryBuilder;
2222

23-
protected function setUp()
23+
protected function setUp(): void
2424
{
2525
$this->queryBuilder = m::mock(QueryBuilder::class);
2626
$this->queryBuilder->makePartial();

‎tests/Eloquent/PostgisTraitTest.php

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -18,13 +18,13 @@ class PostgisTraitTest extends BaseTestCase
1818
*/
1919
protected $queries;
2020

21-
public function setUp()
21+
public function setUp(): void
2222
{
2323
$this->model = new TestModel();
2424
$this->queries = &$this->model->getConnection()->getPdo()->queries;
2525
}
2626

27-
public function tearDown()
27+
public function tearDown(): void
2828
{
2929
$this->model->getConnection()->getPdo()->resetQueries();
3030
}
@@ -34,15 +34,15 @@ public function testInsertPointHasCorrectSql()
3434
$this->model->point = new Point(1, 2);
3535
$this->model->save();
3636

37-
$this->assertContains("public.ST_GeogFromText('POINT(2 1)')", $this->queries[0]);
37+
$this->assertStringContainsString("public.ST_GeogFromText('POINT(2 1)')", $this->queries[0]);
3838
}
3939

4040
public function testInsertPointGeometryHasCorrectSql()
4141
{
4242
$this->model->point2 = new Point(1, 2);
4343
$this->model->save();
4444

45-
$this->assertContains("public.ST_GeomFromText('POINT(2 1)', '27700')", $this->queries[0]);
45+
$this->assertStringContainsString("public.ST_GeomFromText('POINT(2 1)', '27700')", $this->queries[0]);
4646
}
4747

4848
public function testUpdatePointHasCorrectSql()
@@ -51,23 +51,23 @@ public function testUpdatePointHasCorrectSql()
5151
$this->model->point = new Point(2, 4);
5252
$this->model->save();
5353

54-
$this->assertContains("public.ST_GeogFromText('POINT(4 2)')", $this->queries[0]);
54+
$this->assertStringContainsString("public.ST_GeogFromText('POINT(4 2)')", $this->queries[0]);
5555
}
5656

5757
public function testInsertPoint3dHasCorrectSql()
5858
{
5959
$this->model->point = new Point(1, 2, 3);
6060
$this->model->save();
6161

62-
$this->assertContains("public.ST_GeogFromText('POINT Z(2 1 3)')", $this->queries[0]);
62+
$this->assertStringContainsString("public.ST_GeogFromText('POINT Z(2 1 3)')", $this->queries[0]);
6363
}
6464

6565
public function testInsertPoint3dGeometryHasCorrectSql()
6666
{
6767
$this->model->point2 = new Point(1, 2, 3);
6868
$this->model->save();
6969

70-
$this->assertContains("public.ST_GeomFromText('POINT Z(2 1 3)', '27700')", $this->queries[0]);
70+
$this->assertStringContainsString("public.ST_GeomFromText('POINT Z(2 1 3)', '27700')", $this->queries[0]);
7171
}
7272

7373
public function testUpdatePoint3dHasCorrectSql()
@@ -76,7 +76,7 @@ public function testUpdatePoint3dHasCorrectSql()
7676
$this->model->point = new Point(2, 4, 6);
7777
$this->model->save();
7878

79-
$this->assertContains("public.ST_GeogFromText('POINT Z(4 2 6)')", $this->queries[0]);
79+
$this->assertStringContainsString("public.ST_GeogFromText('POINT Z(4 2 6)')", $this->queries[0]);
8080
}
8181
}
8282

‎tests/Geometries/GeometryCollectionTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ class GeometryCollectionTest extends BaseTestCase
1212
private $collection;
1313
private $collection3d;
1414

15-
protected function setUp()
15+
protected function setUp(): void
1616
{
1717
$collection = new LineString(
1818
[

‎tests/Geometries/LineStringTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ class LineStringTest extends BaseTestCase
88
private $points;
99
private $points3d;
1010

11-
protected function setUp()
11+
protected function setUp(): void
1212
{
1313
$this->points = [new Point(1, 1), new Point(2, 2), new Point(3, 3)];
1414
$this->points3d = [new Point(1, 1, 1), new Point(2, 2, 2), new Point(3, 3, 3)];

‎tests/Geometries/MultiPolygonTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ class MultiPolygonTest extends BaseTestCase
1313
private $multiPolygon;
1414
private $multiPolygon3d;
1515

16-
protected function setUp()
16+
protected function setUp(): void
1717
{
1818
$collection1 = new LineString(
1919
[

‎tests/Geometries/PolygonTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ class PolygonTest extends BaseTestCase
99
private $polygon;
1010
private $polygon3d;
1111

12-
protected function setUp()
12+
protected function setUp(): void
1313
{
1414
$collection = new LineString(
1515
[

‎tests/Geometries/UnderLocaleTest.php

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,29 +1,29 @@
11
<?php
22

3-
use MStaack\LaravelPostgis\Geometries\Point;
4-
use MStaack\LaravelPostgis\Geometries\MultiPoint;
3+
use MStaack\LaravelPostgis\Geometries\GeometryCollection;
54
use MStaack\LaravelPostgis\Geometries\LineString;
65
use MStaack\LaravelPostgis\Geometries\MultiLineString;
7-
use MStaack\LaravelPostgis\Geometries\Polygon;
6+
use MStaack\LaravelPostgis\Geometries\MultiPoint;
87
use MStaack\LaravelPostgis\Geometries\MultiPolygon;
9-
use MStaack\LaravelPostgis\Geometries\GeometryCollection;
8+
use MStaack\LaravelPostgis\Geometries\Point;
9+
use MStaack\LaravelPostgis\Geometries\Polygon;
1010

1111
class UnderLocaleTest extends BaseTestCase
1212
{
1313

14-
public static function setUpBeforeClass()
14+
public static function setUpBeforeClass(): void
1515
{
1616
setlocale(LC_NUMERIC, 'fr_FR.utf-8');
1717
}
1818

19-
public static function tearDownAfterClass()
19+
public static function tearDownAfterClass(): void
2020
{
2121
setlocale(LC_NUMERIC, null);
2222
}
2323

24-
public function setUp()
24+
public function setUp(): void
2525
{
26-
if(localeconv()['decimal_point'] == '.') {
26+
if (localeconv()['decimal_point'] == '.') {
2727
$this->markTestSkipped('The locale is not available for testing float output formatting');
2828
}
2929
}

‎tests/PostgisConnectionTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,11 @@
44
use MStaack\LaravelPostgis\Schema\Builder;
55
use Stubs\PDOStub;
66

7-
class PostgisConnectionTest extends PHPUnit_Framework_TestCase
7+
class PostgisConnectionTest extends BaseTestCase
88
{
99
private $postgisConnection;
1010

11-
protected function setUp()
11+
protected function setUp(): void
1212
{
1313
$pgConfig = ['driver' => 'pgsql', 'prefix' => 'prefix', 'database' => 'database', 'name' => 'foo'];
1414
$this->postgisConnection = new PostgisConnection(new PDOStub(), 'database', 'prefix', $pgConfig);

‎tests/Schema/BlueprintTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ class BlueprintTest extends BaseTestCase
88
{
99
protected $blueprint;
1010

11-
public function setUp()
11+
public function setUp(): void
1212
{
1313
parent::setUp();
1414

0 commit comments

Comments
 (0)