Prevent route groups with middlewares to affect the request in the next routes

Author: IngeniozIT

src/Router.php

@@ -79,9 +79,12 @@ private function handleRoutes(ServerRequestInterface $request): ResponseInterfac
                 $newRouter = new Router(
                     $route,
                     $this->container,
-                    $this->handle(...),
                 );
-                return $newRouter->handle($request);
+                try {
+                    return $newRouter->handle($request);
+                } catch (EmptyRouteStack) {
+                    continue;
+                }
             }
 
             $matchedParams = $route->match($request);

tests/MiddlewaresTest.php

@@ -119,4 +119,26 @@ public static function providerInvalidMiddlewares(): array
             ],
         ];
     }
+
+    public function testRouteGroupWithMiddlewareDoesNotAffectTheNextRoutes(): void
+    {
+        $routeGroup = new RouteGroup([
+            new RouteGroup(
+                routes: [
+                    Route::get(path: '/no', callback: static fn(): ResponseInterface => self::response('NO')),
+                ],
+                middlewares: [
+                    fn($request, $handler) => self::response(
+                        strrev((string)$handler->handle($request)->getBody())
+                    )
+                ],
+            ),
+            Route::get(path: '/', callback: static fn(): ResponseInterface => self::response('TEST')),
+        ]);
+        $request = self::serverRequest('GET', '/');
+
+        $response = $this->router($routeGroup)->handle($request);
+
+        self::assertEquals('TEST', (string)$response->getBody());
+    }
 }