Skip to content

Commit d488062

Browse files
Merge pull request #29 from TheDragonCode/andrey-helldar-patch-1
Added Laravel 12 support
2 parents 078ff8f + b2fca0b commit d488062

File tree

8 files changed

+28
-72
lines changed

8 files changed

+28
-72
lines changed

.github/workflows/phpunit.yml

+17-2
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,8 @@ jobs:
99
strategy:
1010
fail-fast: true
1111
matrix:
12-
php: [ "7.4", "8.0", "8.1", "8.2", "8.3" ]
13-
laravel: [ "7.0", "8.0", "9.0", "10.0", "11.0" ]
12+
php: [ "7.4", "8.0", "8.1", "8.2", "8.3", "8.4" ]
13+
laravel: [ "7.0", "8.0", "9.0", "10.0", "11.0", "12.0" ]
1414
exclude:
1515
- laravel: "7.0"
1616
php: "8.1"
@@ -21,12 +21,18 @@ jobs:
2121
- laravel: "7.0"
2222
php: "8.3"
2323

24+
- laravel: "7.0"
25+
php: "8.4"
26+
2427
- laravel: "9.0"
2528
php: "7.4"
2629

2730
- laravel: "9.0"
2831
php: "8.3"
2932

33+
- laravel: "9.0"
34+
php: "8.4"
35+
3036
- laravel: "10.0"
3137
php: "7.4"
3238

@@ -42,6 +48,15 @@ jobs:
4248
- laravel: "11.0"
4349
php: "8.1"
4450

51+
- laravel: "12.0"
52+
php: "7.4"
53+
54+
- laravel: "12.0"
55+
php: "8.0"
56+
57+
- laravel: "12.0"
58+
php: "8.1"
59+
4560
name: PHP ${{ matrix.php }}, Laravel ${{ matrix.laravel }}
4661

4762
steps:

composer.json

+4-5
Original file line numberDiff line numberDiff line change
@@ -34,12 +34,11 @@
3434
"require": {
3535
"php": "^7.4 || ^8.0",
3636
"fig/http-message-util": "^1.1",
37-
"illuminate/routing": "^7.0 || ^8.0 || ^9.0 || ^10.0 || ^11.0"
37+
"illuminate/routing": "^7.0 || ^8.0 || ^9.0 || ^10.0 || ^11.0 || ^12.0"
3838
},
3939
"require-dev": {
40-
"dragon-code/extended-routes": "^3.2 || ^4.0",
41-
"orchestra/testbench": "^5.0 || ^6.0 || ^7.0 || ^8.0 || ^9.0",
42-
"phpunit/phpunit": "^9.6 || ^10.0"
40+
"orchestra/testbench": "^5.0 || ^6.0 || ^7.0 || ^8.0 || ^9.0 || ^10.0",
41+
"phpunit/phpunit": "^9.6 || ^10.0 || ^11.0 || ^12.0"
4342
},
4443
"minimum-stability": "stable",
4544
"prefer-stable": true,
@@ -71,4 +70,4 @@
7170
]
7271
}
7372
}
74-
}
73+
}

phpunit.xml

-5
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,6 @@
1717
<include>
1818
<directory suffix=".php">./src</directory>
1919
</include>
20-
<report>
21-
<clover outputFile="build/logs/clover.xml" />
22-
<html outputDirectory="build/logs/coverage" />
23-
<text outputFile="build/logs/coverage.txt" />
24-
</report>
2520
</coverage>
2621
<testsuites>
2722
<testsuite name="Test Suite">

src/Application.php

-18
This file was deleted.

tests/Concerns/Routes.php

-3
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
use Illuminate\Routing\Router;
88
use Tests\Http\Controllers\ApiResourceController;
99
use Tests\Http\Controllers\Controller;
10-
use Tests\Http\Controllers\RestorableResourceController;
1110
use Tests\Http\Controllers\WebResourceController;
1211

1312
trait Routes
@@ -109,8 +108,6 @@ protected function resourceRoutes(Router $router): void
109108
$router->resource('resources/photos', WebResourceController::class);
110109

111110
$router->apiResource('resources/comments', ApiResourceController::class);
112-
113-
$router->apiRestorableResource('resources/pages', RestorableResourceController::class);
114111
}
115112

116113
protected function protectedRoutes(Router $router): void

tests/Http/Controllers/RestorableResourceController.php

-12
This file was deleted.

tests/Routes/ResourcesTest.php

-12
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
namespace Tests\Routes;
66

77
use Tests\Http\Controllers\ApiResourceController;
8-
use Tests\Http\Controllers\RestorableResourceController;
98
use Tests\Http\Controllers\WebResourceController;
109
use Tests\TestCase;
1110

@@ -30,15 +29,4 @@ public function testApi(): void
3029
$this->assertSame('api.resources.comments.update', $this->getRouteName('update', ApiResourceController::class));
3130
$this->assertSame('api.resources.comments.destroy', $this->getRouteName('destroy', ApiResourceController::class));
3231
}
33-
34-
public function testRestorableApi(): void
35-
{
36-
$this->assertSame('api.resources.pages.index', $this->getRouteName('index', RestorableResourceController::class));
37-
$this->assertSame('api.resources.pages.store', $this->getRouteName('store', RestorableResourceController::class));
38-
$this->assertSame('api.resources.pages.trashed', $this->getRouteName('trashed', RestorableResourceController::class));
39-
$this->assertSame('api.resources.pages.show', $this->getRouteName('show', RestorableResourceController::class));
40-
$this->assertSame('api.resources.pages.update', $this->getRouteName('update', RestorableResourceController::class));
41-
$this->assertSame('api.resources.pages.destroy', $this->getRouteName('destroy', RestorableResourceController::class));
42-
$this->assertSame('api.resources.pages.restore', $this->getRouteName('restore', RestorableResourceController::class));
43-
}
4432
}

tests/TestCase.php

+7-15
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
<?php
2+
23
/*
34
* This file is part of the "dragon-code/support" project.
45
*
@@ -16,13 +17,11 @@
1617

1718
namespace Tests;
1819

19-
use DragonCode\ExtendedRoutes\ServiceProvider as ExtendedRoutesServiceProvider;
20-
use DragonCode\LaravelRouteNames\Application;
20+
use DragonCode\LaravelRouteNames\Providers\RoutingServiceProvider;
2121
use DragonCode\LaravelRouteNames\ServiceProvider;
22-
use Illuminate\Foundation\Bootstrap\LoadConfiguration;
22+
use Illuminate\Events\EventServiceProvider;
23+
use Illuminate\Log\LogServiceProvider;
2324
use Illuminate\Support\Facades\Route as RouteFacade;
24-
use Orchestra\Testbench\Bootstrap\LoadConfiguration as OrchestraLoadConfiguration;
25-
use Orchestra\Testbench\Foundation\PackageManifest;
2625
use Orchestra\Testbench\TestCase as BaseTestCase;
2726
use Tests\Concerns\Routes;
2827
use Tests\Http\Controllers\Controller;
@@ -34,20 +33,13 @@ abstract class TestCase extends BaseTestCase
3433
protected function getPackageProviders($app): array
3534
{
3635
return [
37-
ExtendedRoutesServiceProvider::class,
36+
EventServiceProvider::class,
37+
LogServiceProvider::class,
38+
RoutingServiceProvider::class,
3839
ServiceProvider::class,
3940
];
4041
}
4142

42-
protected function resolveApplication()
43-
{
44-
return tap(new Application($this->getBasePath()), function ($app) {
45-
$app->bind(LoadConfiguration::class, OrchestraLoadConfiguration::class);
46-
47-
PackageManifest::swap($app, $this);
48-
});
49-
}
50-
5143
protected function defineRoutes($router)
5244
{
5345
$router

0 commit comments

Comments
 (0)