test: fix some test case errors.

codeigniter4 · Sep 22, 2023 · a1f2c50 · a1f2c50
1 parent c780a23
commit a1f2c50
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 0 deletions.
diff --git a/tests/system/Router/RouteCollectionTest.php b/tests/system/Router/RouteCollectionTest.php
@@ -1680,6 +1680,8 @@ public function testAutoRoutesControllerNameReturnsFQCN($namespace): void
         $routes->setAutoRoute(true);
         $routes->setDefaultNamespace($namespace);
 
+        $this->expectException(PageNotFoundException::class);
+
         $router = new Router($routes, Services::request());
         $router->handle('/product');
 

diff --git a/tests/system/Router/RouterTest.php b/tests/system/Router/RouterTest.php
@@ -199,6 +199,9 @@ public function testAutoRouteFindsDefaultControllerAndMethod(): void
         $this->collection->setAutoRoute(true);
         $this->collection->setDefaultController('Test');
         $this->collection->setDefaultMethod('test');
+
+        $this->expectException(PageNotFoundException::class);
+
         $router = new Router($this->collection, $this->request);
 
         $router->autoRoute('/');
@@ -210,6 +213,9 @@ public function testAutoRouteFindsDefaultControllerAndMethod(): void
     public function testAutoRouteFindsControllerWithFileAndMethod(): void
     {
         $this->collection->setAutoRoute(true);
+
+        $this->expectException(PageNotFoundException::class);
+
         $router = new Router($this->collection, $this->request);
 
         $router->autoRoute('myController/someMethod');
@@ -221,6 +227,9 @@ public function testAutoRouteFindsControllerWithFileAndMethod(): void
     public function testAutoRouteFindsControllerWithFile(): void
     {
         $this->collection->setAutoRoute(true);
+
+        $this->expectException(PageNotFoundException::class);
+
         $router = new Router($this->collection, $this->request);
 
         $router->autoRoute('myController');
@@ -232,6 +241,7 @@ public function testAutoRouteFindsControllerWithFile(): void
     public function testAutoRouteFindsControllerWithSubfolder(): void
     {
         $this->collection->setAutoRoute(true);
+
         $router = new Router($this->collection, $this->request);
 
         mkdir(APPPATH . 'Controllers/Subfolder');
@@ -734,6 +744,9 @@ public function testTranslateURIDashesForParams(): void
     public function testTranslateURIDashesForAutoRoute(): void
     {
         $this->collection->setAutoRoute(true);
+
+        $this->expectException(PageNotFoundException::class);
+
         $router = new Router($this->collection, $this->request);
         $router->setTranslateURIDashes(true);
 
@@ -749,6 +762,9 @@ public function testTranslateURIDashesForAutoRoute(): void
     public function testAutoRouteMatchesZeroParams(): void
     {
         $this->collection->setAutoRoute(true);
+
+        $this->expectException(PageNotFoundException::class);
+
         $router = new Router($this->collection, $this->request);
 
         $router->autoRoute('myController/someMethod/0/abc');
@@ -769,6 +785,9 @@ public function testAutoRouteMatchesZeroParams(): void
     public function testAutoRouteMethodEmpty(): void
     {
         $this->collection->setAutoRoute(true);
+
+        $this->expectException(PageNotFoundException::class);
+
         $router = new Router($this->collection, $this->request);
         $this->collection->setAutoRoute(true);
 
@@ -820,6 +839,9 @@ public function testRegularExpressionPlaceholderWithUnicode(): void
     public function testRouterPriorDirectory(): void
     {
         $this->collection->setAutoRoute(true);
+
+        $this->expectException(PageNotFoundException::class);
+
         $router = new Router($this->collection, $this->request);
 
         $router->setDirectory('foo/bar/baz', false, true);