Adding missing coverage statements in tests.

Updating return types in Application to match method declaration.
Cleaning up Router::mergeGroupConfigs.

src/Application.php

@@ -151,15 +151,15 @@ public function dispatch(ServerRequestInterface $request): ResponseInterface
 	/**
 	 * Compile a middleware stack.
 	 *
-	 * @param array<MiddlewareInterface>
+	 * @param array<MiddlewareInterface> $middleware
 	 * @param RequestHandlerInterface $kernel
 	 * @return RequestHandlerInterface
 	 */
-	private function compileMiddleware(array $middleware, RequestHandlerInterface $kernel): RequestHandler
+	private function compileMiddleware(array $middleware, RequestHandlerInterface $kernel): RequestHandlerInterface
 	{
 		$middleware = \array_reverse($middleware);
 
-		return \array_reduce($middleware, function(RequestHandlerInterface $handler, MiddlewareInterface $middleware) {
+		return \array_reduce($middleware, function(RequestHandlerInterface $handler, MiddlewareInterface $middleware): RequestHandler {
 
 			return new RequestHandler(function(ServerRequestInterface $request) use ($handler, $middleware): ResponseInterface {
 

src/Exceptions/MethodNotAllowedHttpException.php

@@ -27,6 +27,6 @@ class MethodNotAllowedHttpException extends HttpException
 	public function __construct(array $methodsAllowed, ?string $message = null, ?int $code = null, ?Exception $previous = null)
 	{
 		$this->headers['Allow'] = \implode(", ", $methodsAllowed);
-		parent::__construct($message, $code, $previous);
+		parent::__construct($message ?? "Method not allowed", $code ?? $this->httpStatus, $previous);
 	}
 }

src/Exceptions/ServiceUnavailableHttpException.php

@@ -27,6 +27,6 @@ class ServiceUnavailableHttpException extends HttpException
 	public function __construct(string $retryAfter, ?string $message = null, ?int $code = null, ?Exception $previous = null)
 	{
 		$this->headers["Retry-After"] = $retryAfter;
-		parent::__construct($message, $code, $previous);
+		parent::__construct($message ?? "Service unavailable", $code ?? $this->httpStatus, $previous);
 	}
 }

src/Exceptions/TooManyRequestsHttpException.php

@@ -27,6 +27,6 @@ class TooManyRequestsHttpException extends HttpException
 	public function __construct(string $retryAfter, ?string $message = null, ?int $code = null, ?Exception $previous = null)
 	{
 		$this->headers["Retry-After"] = $retryAfter;
-		parent::__construct($message, $code, $previous);
+		parent::__construct($message ?? "Too many requests", $code ?? $this->httpStatus, $previous);
 	}
 }

src/Exceptions/UnauthorizedHttpException.php

@@ -27,6 +27,6 @@ class UnauthorizedHttpException extends HttpException
 	public function __construct(string $authMethod, ?string $message = null, ?int $code = null, ?Exception $previous = null)
 	{
 		$this->headers["WWW-Authenticate"] = $authMethod;
-		parent::__construct($message, $code, $previous);
+		parent::__construct($message ?? "Unauthorized", $code ?? $this->httpStatus, $previous);
 	}
 }

src/Router/Router.php

@@ -223,7 +223,7 @@ public function options(string $path, $action): Route
     /**
 	 * Group routes together with a set of shared configuration options.
 	 *
-     * @param array $config
+     * @param array $groupConfig
      * @param \closure $callback
      * @return void
      */
@@ -251,15 +251,15 @@ public function group(array $groupConfig, \closure $callback): void
      */
     protected function mergeGroupConfig(array $config, array $groupConfig): array
     {
-		$mergedConfig['scheme'] = $groupConfig['scheme'] ?? $config['scheme'] ?? null;
-        $mergedConfig['hostname'] = $groupConfig['hostname'] ?? $config['hostname'] ?? null;
-        $mergedConfig['prefix'] = $groupConfig['prefix'] ?? $config['pregix'] ?? null;
-		$mergedConfig['namespace'] = $groupConfig['namespace'] ?? $config['namespace'] ?? null;
-		$mergedConfig['middleware'] = \array_merge(
-			$config['middleware'] ?? [],
-			$groupConfig['middleware'] ?? []
-		);
-
-        return $mergedConfig;
+		return [
+			'scheme'=> $groupConfig['scheme'] ?? $config['scheme'] ?? null,
+			'hostname' => $groupConfig['hostname'] ?? $config['hostname'] ?? null,
+			'prefix' => $groupConfig['prefix'] ?? $config['pregix'] ?? null,
+			'namespace' => $groupConfig['namespace'] ?? $config['namespace'] ?? null,
+			'middleware' => \array_merge(
+				$config['middleware'] ?? [],
+				$groupConfig['middleware'] ?? []
+			)
+		];
     }
 }

tests/ApplicationTest.php

@@ -240,6 +240,8 @@ public function test_dispatch_with_resolvable_route()
 		$this->assertEquals("OK", $response->getBody()->getContents());
 	}
 
+
+
 	public function test_send()
 	{
 		$application = new Application(new Router);

tests/RouteTest.php

@@ -11,6 +11,7 @@
 
 /**
  * @covers Limber\Router\Route
+ * @covers Limber\Router\Router
  * @covers ::class_method
  */
 class RouteTest extends TestCase