Unit Test

Author: ibehnoosh

.phpunit.result.cache

@@ -1 +1 @@
-C:37:"PHPUnit\Runner\DefaultTestResultCache":650:{a:2:{s:7:"defects";a:5:{s:7:"Warning";i:6;s:47:"Tests\Unit\RouterTest::test_it_register_a_route";i:3;s:62:"Tests\Unit\RouterTest::testThereAreNoRoutesWhenRouterIsCreated";i:4;s:45:"Tests\Unit\RouterTest::testItRegisterGetRoute";i:4;s:46:"Tests\Unit\RouterTest::testItRegisterPostRoute";i:4;}s:5:"times";a:6:{s:7:"Warning";d:0.004;s:47:"Tests\Unit\RouterTest::test_it_register_a_route";d:0.009;s:42:"Tests\Unit\RouterTest::testItRegisterRoute";d:0.004;s:45:"Tests\Unit\RouterTest::testItRegisterGetRoute";d:0.039;s:46:"Tests\Unit\RouterTest::testItRegisterPostRoute";d:0;s:62:"Tests\Unit\RouterTest::testThereAreNoRoutesWhenRouterIsCreated";d:0.011;}}}
+C:37:"PHPUnit\Runner\DefaultTestResultCache":995:{a:2:{s:7:"defects";a:5:{s:7:"Warning";i:6;s:47:"Tests\Unit\RouterTest::test_it_register_a_route";i:3;s:62:"Tests\Unit\RouterTest::testThereAreNoRoutesWhenRouterIsCreated";i:4;s:45:"Tests\Unit\RouterTest::testItRegisterGetRoute";i:4;s:46:"Tests\Unit\RouterTest::testItRegisterPostRoute";i:4;}s:5:"times";a:10:{s:7:"Warning";d:0.004;s:47:"Tests\Unit\RouterTest::test_it_register_a_route";d:0.009;s:42:"Tests\Unit\RouterTest::testItRegisterRoute";d:0.004;s:45:"Tests\Unit\RouterTest::testItRegisterGetRoute";d:0.005;s:46:"Tests\Unit\RouterTest::testItRegisterPostRoute";d:0;s:62:"Tests\Unit\RouterTest::testThereAreNoRoutesWhenRouterIsCreated";d:0;s:74:"Tests\Unit\RouterTest::testItThrowsRouteNotFoundException with data set #0";d:0.026;s:74:"Tests\Unit\RouterTest::testItThrowsRouteNotFoundException with data set #1";d:0;s:74:"Tests\Unit\RouterTest::testItThrowsRouteNotFoundException with data set #2";d:0;s:74:"Tests\Unit\RouterTest::testItThrowsRouteNotFoundException with data set #3";d:0;}}}

src/App/Router.php

@@ -21,7 +21,6 @@ public function post(string $route, callable|array $action):self
         $this->register('post',$route,$action);
         return $this;
     }
-
     public function routes(): array
     {
         return $this->routes;

tests/Unit/RouterTest.php

@@ -2,18 +2,30 @@
 
 namespace Tests\Unit;
 
+use App\Exception\RouteNotFoundException;
 use App\Router;
 use PHPUnit\Framework\TestCase;
 
 class RouterTest extends TestCase
 {
     private  Router $router;
-    protected  function  setUp(): void
+
+    public function routeNotFoundCases(): array
+    {
+        return[
+            ['/users','put'],
+            ['/invoices','post'],
+            ['/users','get'],
+            ['/users','post']
+        ];
+    }
+
+    protected function  setUp(): void
     {
         parent::setUp();
         $this->router= new Router();
     }
-    public function  testItRegisterGetRoute():void
+    public function testItRegisterGetRoute():void
     {
         $this->router->get('/users', ['Users' , 'index']);
         $expected=[
@@ -24,9 +36,8 @@ public function  testItRegisterGetRoute():void
         $this->assertEquals($expected, $this->router->routes());
 
     }
-
-    public function  testItRegisterPostRoute():void
-    {//
+    public function testItRegisterPostRoute():void
+    {
         $this->router->post('/users', ['Users' , 'store']);
         $expected=[
             'post'=> [
@@ -38,7 +49,26 @@ public function  testItRegisterPostRoute():void
     }
     public function testThereAreNoRoutesWhenRouterIsCreated()
     {
-        $this->assertEmpty($this->router->routes());
+        $this->assertEmpty((new Router())->routes());
     }
 
+    /**
+     * @test
+     * @dataProvider  routeNotFoundCases
+     */
+    public function testItThrowsRouteNotFoundException(
+        string $requestUri,
+        string $requestMethod
+    ):void
+    {
+        $users= new class(){
+            public function delete():bool{
+                return true;
+            }
+        };
+        $this->router->post('/users',[$users::class,'store']);
+        $this->router->get('/users',['Users','index']);
+        $this->expectException(RouteNotFoundException::class);
+        $this->router->resolve($requestUri,$requestMethod);
+    }
 }