increate test coverage

Author: miladrahimi

.gitignore

@@ -1,3 +1,5 @@
 /.idea
 /vendor
 /composer.lock
+/build
+.DS_Store

phpunit.xml

@@ -12,7 +12,7 @@
 	</testsuites>
 	<filter>
 		<whitelist processUncoveredFilesFromWhitelist="true">
-            <directory suffix=".php">./tests</directory>
+            <directory suffix=".php">./src</directory>
 		</whitelist>
 	</filter>
 	<logging>

src/Route.php

@@ -112,14 +112,6 @@ public function getName(): ?string
         return $this->name;
     }
 
-    /**
-     * @param string|null $name
-     */
-    public function setName(?string $name): void
-    {
-        $this->name = $name;
-    }
-
     /**
      * @return string
      */
@@ -128,14 +120,6 @@ public function getUri(): string
         return $this->uri;
     }
 
-    /**
-     * @param string $uri
-     */
-    public function setUri(string $uri): void
-    {
-        $this->uri = $uri;
-    }
-
     /**
      * @return string|null
      */
@@ -144,14 +128,6 @@ public function getMethod(): ?string
         return $this->method;
     }
 
-    /**
-     * @param string|null $method
-     */
-    public function setMethod(?string $method): void
-    {
-        $this->method = $method;
-    }
-
     /**
      * @return callable|string
      */
@@ -160,14 +136,6 @@ public function getController()
         return $this->controller;
     }
 
-    /**
-     * @param callable|string $controller
-     */
-    public function setController($controller): void
-    {
-        $this->controller = $controller;
-    }
-
     /**
      * @return callable[]|string[]
      */
@@ -176,27 +144,11 @@ public function getMiddleware()
         return $this->middleware;
     }
 
-    /**
-     * @param callable[]|string[] $middleware
-     */
-    public function setMiddleware($middleware): void
-    {
-        $this->middleware = $middleware;
-    }
-
     /**
      * @return string|null
      */
     public function getDomain(): ?string
     {
         return $this->domain;
     }
-
-    /**
-     * @param string|null $domain
-     */
-    public function setDomain(?string $domain): void
-    {
-        $this->domain = $domain;
-    }
 }

src/Services/HttpPublisher.php

@@ -9,6 +9,7 @@
  * StreamPublisher publishes the given resp
  *
  * @package MiladRahimi\PhpRouter\Services
+ * @codeCoverageIgnore
  */
 class HttpPublisher implements PublisherInterface
 {
@@ -26,7 +27,7 @@ public function publish($content): void
 
             foreach ($content->getHeaders() as $name => $values) {
                 $value = $content->getHeaderLine($name);
-                header($name.': '.$value);
+                header($name . ': ' . $value);
             }
 
             fwrite($output, $content->getBody());

test.sh

@@ -0,0 +1 @@
+./vendor/bin/phpunit --coverage-text --coverage-html build/html ./tests

tests/Classes/Publisher.php

@@ -9,6 +9,7 @@
  * Class Publisher
  *
  * @package MiladRahimi\PhpRouter\Tests\Classes
+ * @codeCoverageIgnore
  */
 class Publisher implements PublisherInterface
 {

tests/Classes/SampleController.php

@@ -6,6 +6,7 @@
  * Class SampleController
  *
  * @package Classes
+ * @codeCoverageIgnore
  */
 class SampleController
 {

tests/Classes/SampleMiddleware.php

@@ -10,6 +10,7 @@
  * Class SampleMiddleware
  *
  * @package MiladRahimi\PhpRouter\Tests\Classes
+ * @codeCoverageIgnore
  */
 class SampleMiddleware implements Middleware
 {

tests/Classes/StopperMiddleware.php

@@ -11,6 +11,7 @@
  * Class StopperMiddleware
  *
  * @package MiladRahimi\PhpRouter\Tests\Classes
+ * @codeCoverageIgnore
  */
 class StopperMiddleware implements Middleware
 {

tests/RoutingTest.php

@@ -365,6 +365,55 @@ public function test_injection_of_router_by_type()
         $this->assertEquals('home', $this->extract($router));
     }
 
+    /**
+     * @throws Throwable
+     */
+    public function test_injection_of_default_value()
+    {
+        $router = $this->router()
+            ->get('/', function ($default = "Default") {
+                return $default;
+            })
+            ->dispatch();
+
+        $this->assertEquals('Default', $this->extract($router));
+    }
+
+    /**
+     * @throws Throwable
+     */
+    public function test_set_and_get_request()
+    {
+        $router = $this->router();
+
+        $router->get('/', function () use ($router) {
+            $newRequest = $router->getRequest()->withMethod('CUSTOM');
+            $router->setRequest($newRequest);
+
+            return $router->getRequest()->getMethod();
+        })->dispatch();
+
+        $this->assertEquals('CUSTOM', $this->extract($router));
+    }
+
+    /**
+     * @throws Throwable
+     */
+    public function test_default_publisher()
+    {
+        ob_start();
+
+        $router = new Router();
+
+        $router->get('/', function () {
+            return 'home';
+        })->dispatch();
+
+        $this->assertEquals('home', ob_get_contents());
+
+        ob_end_clean();
+    }
+
     /**
      * @throws Throwable
      */
@@ -408,10 +457,33 @@ public function test_not_found_error()
     /**
      * @throws Throwable
      */
-    public function test_with_invalid_controller()
+    public function test_with_class_method_but_invalid_controller_class()
     {
         $this->expectException(InvalidControllerException::class);
 
         $this->router()->get('/', 'UnknownController@method')->dispatch();
     }
+
+    /**
+     * @throws Throwable
+     */
+    public function test_with_invalid_controller_class()
+    {
+        $this->expectException(InvalidControllerException::class);
+
+        $this->router()->get('/', 666)->dispatch();
+    }
+
+    /**
+     * @throws Throwable
+     */
+    public function test_with_invalid_controller_method()
+    {
+        $this->expectException(InvalidControllerException::class);
+
+        $namespace = 'MiladRahimi\PhpRouter\Tests\Classes';
+        $this->router('', $namespace)
+            ->get('/', 'SampleController@invalidMethod')
+            ->dispatch();
+    }
 }