Add return types to tests and final|internal|private methods

Author: nicolas-grekas

Tests/Fixtures/CustomRouteCompiler.php

@@ -11,6 +11,7 @@
 
 namespace Symfony\Component\Routing\Tests\Fixtures;
 
+use Symfony\Component\Routing\CompiledRoute;
 use Symfony\Component\Routing\Route;
 use Symfony\Component\Routing\RouteCompiler;
 
@@ -19,7 +20,7 @@ class CustomRouteCompiler extends RouteCompiler
     /**
      * {@inheritdoc}
      */
-    public static function compile(Route $route)
+    public static function compile(Route $route): CompiledRoute
     {
         return new CustomCompiledRoute('', '', [], []);
     }

Tests/Fixtures/CustomXmlFileLoader.php

@@ -19,7 +19,7 @@
  */
 class CustomXmlFileLoader extends XmlFileLoader
 {
-    protected function loadFile($file)
+    protected function loadFile($file): \DOMDocument
     {
         return XmlUtils::loadFile($file, function () { return true; });
     }

Tests/Fixtures/RedirectableUrlMatcher.php

@@ -19,7 +19,7 @@
  */
 class RedirectableUrlMatcher extends UrlMatcher implements RedirectableUrlMatcherInterface
 {
-    public function redirect($path, $route, $scheme = null)
+    public function redirect($path, $route, $scheme = null): array
     {
         return [
             '_controller' => 'Some controller reference...',

Tests/Fixtures/TestObjectRouteLoader.php

@@ -17,6 +17,9 @@ class TestObjectRouteLoader extends ObjectRouteLoader
 {
     public $loaderMap = [];
 
+    /**
+     * @return object
+     */
     protected function getServiceObject($id)
     {
         return $this->loaderMap[$id] ?? null;

Tests/Loader/ObjectLoaderTest.php

@@ -98,11 +98,14 @@ class TestObjectLoader extends ObjectLoader
 {
     public $loaderMap = [];
 
-    public function supports($resource, $type = null)
+    public function supports($resource, $type = null): bool
     {
         return 'service';
     }
 
+    /**
+     * @return object
+     */
     protected function getObject(string $id)
     {
         return $this->loaderMap[$id] ?? null;

Tests/Matcher/CompiledRedirectableUrlMatcherTest.php

@@ -33,7 +33,7 @@ protected function getUrlMatcher(RouteCollection $routes, RequestContext $contex
 
 class TestCompiledRedirectableUrlMatcher extends CompiledUrlMatcher implements RedirectableUrlMatcherInterface
 {
-    public function redirect($path, $route, $scheme = null)
+    public function redirect($path, $route, $scheme = null): array
     {
         return [];
     }

Tests/Matcher/DumpedRedirectableUrlMatcherTest.php

@@ -39,7 +39,7 @@ protected function getUrlMatcher(RouteCollection $routes, RequestContext $contex
 
 class TestDumpedRedirectableUrlMatcher extends UrlMatcher implements RedirectableUrlMatcherInterface
 {
-    public function redirect($path, $route, $scheme = null)
+    public function redirect($path, $route, $scheme = null): array
     {
         return [];
     }

Tests/Matcher/Dumper/CompiledUrlMatcherDumperTest.php

@@ -487,7 +487,7 @@ public function testGenerateDumperMatcherWithObject()
 
 class TestCompiledUrlMatcher extends CompiledUrlMatcher implements RedirectableUrlMatcherInterface
 {
-    public function redirect($path, $route, $scheme = null)
+    public function redirect($path, $route, $scheme = null): array
     {
         return [];
     }

Tests/Matcher/Dumper/PhpMatcherDumperTest.php

@@ -505,7 +505,7 @@ public function testGenerateDumperMatcherWithObject()
 
 abstract class RedirectableUrlMatcherStub extends UrlMatcher implements RedirectableUrlMatcherInterface
 {
-    public function redirect($path, $route, $scheme = null)
+    public function redirect($path, $route, $scheme = null): array
     {
     }
 }