Add type hints

Author: enumag

src/Connector/NetteConnector.php

@@ -25,7 +25,7 @@ class NetteConnector extends Client
      */
     protected $containerAccessor;
 
-    public function setContainerAccessor(callable $containerAccessor)
+    public function setContainerAccessor(callable $containerAccessor): void
     {
         $this->containerAccessor = $containerAccessor;
     }
@@ -53,7 +53,7 @@ public function doRequest($request)
             $_POST = $request->getParameters();
         }
 
-        $container = call_user_func($this->containerAccessor);
+        $container = ($this->containerAccessor)();
 
         $httpRequest = $container->getByType(IRequest::class);
         $httpResponse = $container->getByType(IResponse::class);

src/Console/RunTestInput.php

@@ -16,7 +16,7 @@
  */
 class RunTestInput extends ArgvInput
 {
-    public function __construct(InputDefinition $definition = null)
+    public function __construct(?InputDefinition $definition = null)
     {
         $parameters = [$_SERVER['argv'][0], 'run'];
 
@@ -42,7 +42,7 @@ public function __construct(InputDefinition $definition = null)
         parent::__construct($parameters, $definition);
     }
 
-    private function normalizePath($path)
+    private function normalizePath(string $path): string
     {
         return str_replace('\\', '/', $path);
     }

src/Http/Request.php

@@ -32,7 +32,7 @@ public function __construct(RequestFactory $factory)
         $this->reset();
     }
 
-    public function reset()
+    public function reset(): void
     {
         $this->request = $this->factory->createHttpRequest();
     }

src/Http/Response.php

@@ -25,7 +25,7 @@ class Response implements IResponse
      */
     private $headers = [];
 
-    public function reset()
+    public function reset(): void
     {
         $this->code = self::S200_OK;
         $this->headers = [];

src/Module/NetteApplicationModule.php

@@ -67,15 +67,15 @@ public function _after(TestInterface $test)
     /**
      * @param bool $followRedirects
      */
-    public function followRedirects($followRedirects)
+    public function followRedirects(bool $followRedirects): void
     {
         $this->client->followRedirects($followRedirects);
     }
 
     /**
      * @param string $url
      */
-    public function seeRedirectTo($url)
+    public function seeRedirectTo(string $url): void
     {
         if ($this->client->isFollowingRedirects()) {
             $this->fail('Method seeRedirectTo only works when followRedirects option is disabled');
@@ -89,7 +89,7 @@ public function seeRedirectTo($url)
         }
     }
 
-    public function debugContent()
+    public function debugContent(): void
     {
         $this->debugSection('Content', $this->client->getInternalResponse()->getContent());
     }

src/Module/NetteDIModule.php

@@ -78,7 +78,7 @@ public function _after(TestInterface $test)
         }
     }
 
-    public function useConfigFiles(array $configFiles)
+    public function useConfigFiles(array $configFiles): void
     {
         if (!$this->config['newContainerForEachTest']) {
             $this->fail('The useConfigFiles can only be used if the newContainerForEachTest option is set to true.');
@@ -92,7 +92,7 @@ public function useConfigFiles(array $configFiles)
     /**
      * @return Container
      */
-    public function getContainer()
+    public function getContainer(): Container
     {
         if (!$this->container) {
             $this->createContainer();
@@ -106,7 +106,7 @@ public function getContainer()
      *
      * @return object
      */
-    public function grabService($service)
+    public function grabService(string $service)
     {
         try {
             return $this->getContainer()->getByType($service);
@@ -115,7 +115,7 @@ public function grabService($service)
         }
     }
 
-    private function createContainer()
+    private function createContainer(): void
     {
         $configurator = new Configurator();
         if ($this->config['removeDefaultExtensions']) {

tests/functional/src/Fixtures/RouterFactory.php

@@ -17,7 +17,7 @@ class RouterFactory extends Object
     /**
      * @return IRouter
      */
-    public function create()
+    public function create(): IRouter
     {
         $router = new RouteList();
         $router[] = new Route('<presenter>[/<action>[/<id>]]', 'Homepage:default');