Merge pull request #447 from symfony-cmf/symfony-5

dbu · web-flow · commit e7e6eb3aecab · 2020-04-28T15:08:04.000+02:00
build with symfony 5
diff --git a/.gitignore b/.gitignore
@@ -2,3 +2,4 @@ phpunit.xml
 composer.lock
 vendor
 tests/Fixtures/App/var/
+.phpunit.result.cache 
diff --git a/.travis.yml b/.travis.yml
@@ -16,7 +16,6 @@
 language: php
 
 php:
-  - 7.1
   - 7.2
   - 7.3
 
@@ -29,20 +28,18 @@ cache:
 
 env:
   global:
-    - SYMFONY_PHPUNIT_VERSION=6
+    - SYMFONY_PHPUNIT_VERSION=8
     - COMPOSER_MEMORY_LIMIT=-1
     - SYMFONY_DEPRECATIONS_HELPER="max[self]=0"
 
 matrix:
   include:
-    - php: 7.1
-      env: COMPOSER_FLAGS="--prefer-lowest" SYMFONY_REQUIRE=3.4.* SYMFONY_DEPRECATIONS_HELPER=weak
-    - php: 7.4
-      env: SYMFONY_REQUIRE=3.4.*
-    - php: 7.4
-      env: SYMFONY_REQUIRE=4.3.*
+    - php: 7.2
+      env: COMPOSER_FLAGS="--prefer-lowest" SYMFONY_REQUIRE=4.4.* SYMFONY_DEPRECATIONS_HELPER=weak
     - php: 7.4
       env: SYMFONY_REQUIRE=4.4.*
+    - php: 7.4
+      env: SYMFONY_REQUIRE=5.0.*
   fast_finish: true
   allow_failures:
 
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,13 @@
 Changelog
 =========
 
+2.4.0
+-----
+
+* Minimum PHP version is now 7.2
+* Support Symfony 5. Minimum Symfony version is now 4.4.
+* If you generate routes with the CMF route generator, see the CHANGELOG of the routing component as well!
+
 2.3.0
 -----
 
diff --git a/composer.json b/composer.json
@@ -15,22 +15,29 @@
         }
     ],
     "require": {
-        "php": "^7.1",
-        "symfony-cmf/routing": "^2.1.0",
-        "symfony/framework-bundle": "^3.4 || ^4.3",
+        "php": "^7.2",
+        "symfony-cmf/routing": "dev-master",
+        "symfony/framework-bundle": "^4.4 || ^5.0",
         "twig/twig": "^2.4.4 || ^3.0"
     },
     "require-dev": {
+	"jackalope/jackalope-doctrine-dbal": "^1.3",
         "doctrine/phpcr-odm": "^1.4|^2.0",
-        "symfony/phpunit-bridge": "^4.2.2",
-        "matthiasnoback/symfony-dependency-injection-test": "^2.3.1",
-        "matthiasnoback/symfony-config-test": "^3.1.1",
+        "symfony/phpunit-bridge": "^5.0",
+        "matthiasnoback/symfony-dependency-injection-test": "^4.1.0",
+        "matthiasnoback/symfony-config-test": "^4.1.0",
         "doctrine/orm": "^2.5",
-        "symfony-cmf/testing": "^3@dev",
+        "symfony-cmf/testing": "dev-master",
         "doctrine/data-fixtures": "^1.0.0",
-        "symfony/form": "^3.4 || ^4.3",
-        "symfony/translation": "^3.4 || ^4.3",
-        "symfony/validator": "^3.4 || ^4.3"
+        "symfony/form": "^4.4 || ^5.0",
+        "symfony/translation": "^4.4 || ^5.0",
+        "symfony/validator": "^4.4 || ^5.0",
+        "symfony/security-bundle": "^4.4 || ^5.0",
+        "doctrine/doctrine-bundle": "^2.0",
+        "symfony/twig-bundle": "^4.4 || ^5.0",
+        "symfony/monolog-bundle": "^3.5",
+        "doctrine/phpcr-bundle": "^2.1",
+        "symfony/serializer": "^4.4 || ^5.0"
     },
     "suggest": {
         "doctrine/phpcr-odm": "To enable support for the PHPCR ODM documents (^1.4)",
diff --git a/tests/Fixtures/App/config/config.yml b/tests/Fixtures/App/config/config.yml
@@ -1,3 +1,2 @@
 framework:
     serializer: ~
-    templating: false
diff --git a/tests/Functional/Controller/RedirectControllerTest.php b/tests/Functional/Controller/RedirectControllerTest.php
@@ -26,7 +26,7 @@ class RedirectControllerTest extends BaseTestCase
      */
     protected $controller;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->db('PHPCR')->createTestNode();
diff --git a/tests/Functional/Doctrine/Orm/RedirectRouteTest.php b/tests/Functional/Doctrine/Orm/RedirectRouteTest.php
@@ -22,7 +22,7 @@ class RedirectRouteTest extends OrmTestCase
 
     private $controller;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->clearDb(Route::class);
diff --git a/tests/Functional/Doctrine/Orm/RouteProviderTest.php b/tests/Functional/Doctrine/Orm/RouteProviderTest.php
@@ -18,7 +18,7 @@ class RouteProviderTest extends OrmTestCase
 {
     private $repository;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->clearDb(Route::class);
diff --git a/tests/Functional/Doctrine/Phpcr/RedirectRouteTest.php b/tests/Functional/Doctrine/Phpcr/RedirectRouteTest.php
@@ -21,7 +21,7 @@ class RedirectRouteTest extends BaseTestCase
 {
     const ROUTE_ROOT = '/test/redirectroute';
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->db('PHPCR')->createTestNode();
diff --git a/tests/Functional/Doctrine/Phpcr/RouteProviderTest.php b/tests/Functional/Doctrine/Phpcr/RouteProviderTest.php
@@ -26,7 +26,7 @@ class RouteProviderTest extends BaseTestCase
     /** @var RouteProvider */
     private $repository;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->db('PHPCR')->createTestNode();
diff --git a/tests/Functional/Doctrine/Phpcr/RouteTest.php b/tests/Functional/Doctrine/Phpcr/RouteTest.php
@@ -18,7 +18,7 @@ class RouteTest extends BaseTestCase
 {
     const ROUTE_ROOT = '/test/routing';
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->db('PHPCR')->createTestNode();
diff --git a/tests/Functional/Routing/DynamicRouterTest.php b/tests/Functional/Routing/DynamicRouterTest.php
@@ -41,7 +41,7 @@ class DynamicRouterTest extends BaseTestCase
 
     const ROUTE_ROOT = '/test/routing';
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
 
diff --git a/tests/Unit/DependencyInjection/CmfRoutingExtensionTest.php b/tests/Unit/DependencyInjection/CmfRoutingExtensionTest.php
@@ -19,7 +19,7 @@
 
 class CmfRoutingExtensionTest extends AbstractExtensionTestCase
 {
-    protected function getContainerExtensions()
+    protected function getContainerExtensions(): array
     {
         return [
             new CmfRoutingExtension(),
diff --git a/tests/Unit/DependencyInjection/Compiler/SetRouterPassTest.php b/tests/Unit/DependencyInjection/Compiler/SetRouterPassTest.php
@@ -17,7 +17,7 @@
 
 class SetRouterPassTest extends AbstractCompilerPassTestCase
 {
-    protected function registerCompilerPass(ContainerBuilder $container)
+    protected function registerCompilerPass(ContainerBuilder $container): void
     {
         $container->addCompilerPass(new SetRouterPass());
     }
diff --git a/tests/Unit/DependencyInjection/Compiler/TemplatingValidatorPassTest.php b/tests/Unit/DependencyInjection/Compiler/TemplatingValidatorPassTest.php
@@ -20,7 +20,7 @@
 
 class TemplatingValidatorPassTest extends AbstractCompilerPassTestCase
 {
-    public function setUp()
+    public function setUp(): void
     {
         if (!\class_exists(EngineInterface::class)) {
             $this->markTestSkipped();
@@ -31,7 +31,7 @@ public function setUp()
         $this->registerValidatorService();
     }
 
-    protected function registerCompilerPass(ContainerBuilder $container)
+    protected function registerCompilerPass(ContainerBuilder $container): void
     {
         $container->addCompilerPass(new TemplatingValidatorPass());
     }
diff --git a/tests/Unit/DependencyInjection/Compiler/ValidationPassTest.php b/tests/Unit/DependencyInjection/Compiler/ValidationPassTest.php
@@ -17,7 +17,7 @@
 
 class ValidationPassTest extends AbstractCompilerPassTestCase
 {
-    protected function registerCompilerPass(ContainerBuilder $container)
+    protected function registerCompilerPass(ContainerBuilder $container): void
     {
         $container->addCompilerPass(new ValidationPass());
     }
diff --git a/tests/Unit/DependencyInjection/ConfigurationTest.php b/tests/Unit/DependencyInjection/ConfigurationTest.php
@@ -14,15 +14,17 @@
 use Matthias\SymfonyDependencyInjectionTest\PhpUnit\AbstractExtensionConfigurationTestCase;
 use Symfony\Cmf\Bundle\RoutingBundle\DependencyInjection\CmfRoutingExtension;
 use Symfony\Cmf\Bundle\RoutingBundle\DependencyInjection\Configuration;
+use Symfony\Component\Config\Definition\ConfigurationInterface;
+use Symfony\Component\DependencyInjection\Extension\ExtensionInterface;
 
 class ConfigurationTest extends AbstractExtensionConfigurationTestCase
 {
-    protected function getContainerExtension()
+    protected function getContainerExtension(): ExtensionInterface
     {
         return new CmfRoutingExtension();
     }
 
-    protected function getConfiguration()
+    protected function getConfiguration(): ConfigurationInterface
     {
         return new Configuration();
     }
diff --git a/tests/Unit/DependencyInjection/XmlSchemaTest.php b/tests/Unit/DependencyInjection/XmlSchemaTest.php
@@ -19,7 +19,7 @@ class XmlSchemaTest extends XmlSchemaTestCase
 
     protected $schemaPath;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->fixturesPath = __DIR__.'/../../Fixtures/fixtures/config/';
         $this->schemaPath = __DIR__.'/../../../src/Resources/config/schema/routing-1.0.xsd';
diff --git a/tests/Unit/Doctrine/Orm/ContentRepositoryTest.php b/tests/Unit/Doctrine/Orm/ContentRepositoryTest.php
@@ -27,7 +27,7 @@ class ContentRepositoryTest extends TestCase
 
     private $objectRepository;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->document = new \stdClass();
         $this->objectManager = $this->createMock(ObjectManager::class);
diff --git a/tests/Unit/Doctrine/Orm/RouteProviderTest.php b/tests/Unit/Doctrine/Orm/RouteProviderTest.php
@@ -55,7 +55,7 @@ class RouteProviderTest extends TestCase
      */
     private $candidatesMock;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->routeMock = $this->createMock(Route::class);
         $this->route2Mock = $this->createMock(Route::class);
diff --git a/tests/Unit/Doctrine/Phpcr/ContentRepositoryTest.php b/tests/Unit/Doctrine/Phpcr/ContentRepositoryTest.php
@@ -30,7 +30,7 @@ class ContentRepositoryTest extends TestCase
 
     private $managerRegistry;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->document = new \stdClass();
         $this->document2 = new \stdClass();
diff --git a/tests/Unit/Doctrine/Phpcr/IdPrefixListenerTest.php b/tests/Unit/Doctrine/Phpcr/IdPrefixListenerTest.php
@@ -41,7 +41,7 @@ class IdPrefixListenerTest extends TestCase
      */
     protected $routeMock;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->candidatesMock = $this->createMock(PrefixCandidates::class);
         $this->candidatesMock
diff --git a/tests/Unit/Doctrine/Phpcr/LocaleListenerTest.php b/tests/Unit/Doctrine/Phpcr/LocaleListenerTest.php
@@ -40,7 +40,7 @@ class LocaleListenerTest extends TestCase
      */
     protected $routeMock;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->candidatesMock = $this->createMock(PrefixCandidates::class);
 
diff --git a/tests/Unit/Doctrine/Phpcr/RouteProviderTest.php b/tests/Unit/Doctrine/Phpcr/RouteProviderTest.php
@@ -60,7 +60,7 @@ class RouteProviderTest extends TestCase
      */
     protected $route2Mock;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->routeMock = $this->createMock(Route::class);
         $this->route2Mock = $this->createMock(Route::class);
diff --git a/tests/Unit/Doctrine/Phpcr/RouteTest.php b/tests/Unit/Doctrine/Phpcr/RouteTest.php
@@ -21,7 +21,7 @@ class RouteTest extends TestCase
 
     private $childRoute1;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->route = new Route();
 
diff --git a/tests/Unit/Form/Type/RouteTypeTypeTest.php b/tests/Unit/Form/Type/RouteTypeTypeTest.php
@@ -22,7 +22,7 @@ class RouteTypeTypeTest extends TestCase
      */
     private $type;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->type = new RouteTypeType();
     }
diff --git a/tests/Unit/Routing/DynamicRouterTest.php b/tests/Unit/Routing/DynamicRouterTest.php
@@ -24,6 +24,7 @@
 use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
 use Symfony\Component\Routing\Matcher\UrlMatcherInterface;
 use Symfony\Component\Routing\RequestContext;
+use Symfony\Contracts\EventDispatcher\EventDispatcherInterface as ContractsEventDispatcherInterface;
 
 class DynamicRouterTest extends TestCase
 {
@@ -62,7 +63,7 @@ class DynamicRouterTest extends TestCase
      */
     private $eventDispatcher;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->matcher = $this->createMock(UrlMatcherInterface::class);
         $this->matcher->expects($this->once())
@@ -95,9 +96,15 @@ private function assertRequestAttributes($request)
      */
     public function testMatch()
     {
+        $dispatchParams = [Events::PRE_DYNAMIC_MATCH, $this->equalTo(new RouterMatchEvent())];
+        if ($this->eventDispatcher instanceof ContractsEventDispatcherInterface) {
+            // New Symfony 4.3 EventDispatcher signature
+            $dispatchParams = [$this->equalTo(new RouterMatchEvent()), Events::PRE_DYNAMIC_MATCH];
+        }
+
         $this->eventDispatcher->expects($this->once())
             ->method('dispatch')
-            ->with(Events::PRE_DYNAMIC_MATCH, $this->equalTo(new RouterMatchEvent()))
+            ->with(...$dispatchParams)
         ;
 
         $parameters = $this->router->match('/foo');
@@ -108,9 +115,15 @@ public function testMatch()
 
     public function testMatchRequest()
     {
+        $dispatchParams = [Events::PRE_DYNAMIC_MATCH_REQUEST, $this->equalTo(new RouterMatchEvent($this->request))];
+        if ($this->eventDispatcher instanceof ContractsEventDispatcherInterface) {
+            // New Symfony 4.3 EventDispatcher signature
+            $dispatchParams = [$this->equalTo(new RouterMatchEvent($this->request)), Events::PRE_DYNAMIC_MATCH_REQUEST];
+        }
+
         $this->eventDispatcher->expects($this->once())
             ->method('dispatch')
-            ->with(Events::PRE_DYNAMIC_MATCH_REQUEST, $this->equalTo(new RouterMatchEvent($this->request)))
+            ->with(...$dispatchParams)
         ;
 
         $parameters = $this->router->matchRequest($this->request);
@@ -126,9 +139,15 @@ public function testMatchNoRequest()
     {
         $this->router->setRequestStack(new RequestStack());
 
+        $dispatchParams = [Events::PRE_DYNAMIC_MATCH, $this->equalTo(new RouterMatchEvent())];
+        if ($this->eventDispatcher instanceof ContractsEventDispatcherInterface) {
+            // New Symfony 4.3 EventDispatcher signature
+            $dispatchParams = [$this->equalTo(new RouterMatchEvent()), Events::PRE_DYNAMIC_MATCH];
+        }
+
         $this->eventDispatcher->expects($this->once())
             ->method('dispatch')
-            ->with(Events::PRE_DYNAMIC_MATCH, $this->equalTo(new RouterMatchEvent()))
+            ->with(...$dispatchParams)
         ;
 
         $this->expectException(ResourceNotFoundException::class);
diff --git a/tests/Unit/Validator/Constraints/RouteDefaultsTemplatingValidatorTest.php b/tests/Unit/Validator/Constraints/RouteDefaultsTemplatingValidatorTest.php
@@ -21,7 +21,7 @@ protected function mockEngine()
         return $this->createMock(EngineInterface::class);
     }
 
-    protected function setUp()
+    protected function setUp(): void
     {
         if (!\class_exists(EngineInterface::class)) {
             $this->markTestSkipped();
diff --git a/tests/Unit/Validator/Constraints/RouteDefaultsValidatorTest.php b/tests/Unit/Validator/Constraints/RouteDefaultsValidatorTest.php

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,2 @@`
`1`	`1`	`framework:`
`2`	`2`	`serializer: ~`
`3`		`- templating: false`
Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ class RedirectControllerTest extends BaseTestCase`
`26`	`26`	`*/`
`27`	`27`	`protected $controller;`
`28`	`28`
`29`		`- public function setUp()`
	`29`	`+ public function setUp(): void`
`30`	`30`	`{`
`31`	`31`	`parent::setUp();`
`32`	`32`	`$this->db('PHPCR')->createTestNode();`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ class RedirectRouteTest extends OrmTestCase`
`22`	`22`
`23`	`23`	`private $controller;`
`24`	`24`
`25`		`- public function setUp()`
	`25`	`+ public function setUp(): void`
`26`	`26`	`{`
`27`	`27`	`parent::setUp();`
`28`	`28`	`$this->clearDb(Route::class);`
Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@ class RouteProviderTest extends OrmTestCase`
`18`	`18`	`{`
`19`	`19`	`private $repository;`
`20`	`20`
`21`		`- public function setUp()`
	`21`	`+ public function setUp(): void`
`22`	`22`	`{`
`23`	`23`	`parent::setUp();`
`24`	`24`	`$this->clearDb(Route::class);`
Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ class RedirectRouteTest extends BaseTestCase`
`21`	`21`	`{`
`22`	`22`	`const ROUTE_ROOT = '/test/redirectroute';`
`23`	`23`
`24`		`- public function setUp()`
	`24`	`+ public function setUp(): void`
`25`	`25`	`{`
`26`	`26`	`parent::setUp();`
`27`	`27`	`$this->db('PHPCR')->createTestNode();`
Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ class RouteProviderTest extends BaseTestCase`
`26`	`26`	`/** @var RouteProvider */`
`27`	`27`	`private $repository;`
`28`	`28`
`29`		`- public function setUp()`
	`29`	`+ public function setUp(): void`
`30`	`30`	`{`
`31`	`31`	`parent::setUp();`
`32`	`32`	`$this->db('PHPCR')->createTestNode();`