test: dispatch namespace tests (not dispatcher)

Author: g105b

src/Dispatch/Dispatcher.php

@@ -17,8 +17,8 @@
 use GT\DomTemplate\PlaceholderBinder;
 use GT\DomTemplate\TableBinder;
 use GT\Http\Header\ResponseHeaders;
-use GT\Http\Request;
-use GT\Http\Response;
+use Gt\Http\Request;
+use Gt\Http\Response;
 use Gt\Http\ResponseStatusException\ClientError\HttpNotFound;
 use Gt\Http\ResponseStatusException\ResponseStatusException;
 use Gt\Http\StatusCode;
@@ -129,7 +129,7 @@ public function __construct(
 		$requestInit = $requestInit ?? new RequestInit(
 			$pathNormaliser,
 			$request->getUri(),
-			$config->getBool("app.force_trailing_slash"),
+			$config->getBool("app.force_trailing_slash") ?? true,
 			$this->response->redirect(...),
 			$this->globals["_GET"],
 			$this->globals["_POST"],
@@ -407,7 +407,7 @@ public function processResponse(bool $processingError = false):void {
 			}
 		}
 
-		if($responseWithHeader = $this->headerManager->apply(
+		if($responseWithHeader = $this->headerManager->applyWithHeader(
 			$this->response->getResponseHeaders(),
 			$this->response->withHeader(...)
 		)) {

src/Dispatch/HeaderManager.php

@@ -6,7 +6,7 @@
 use GT\Http\Response;
 
 class HeaderManager {
-	public function apply(ResponseHeaders $responseHeaders, Closure $withHeaderCallback):?Response {
+	public function applyWithHeader(ResponseHeaders $responseHeaders, Closure $withHeaderCallback):?Response {
 		$response = null;
 
 		foreach($responseHeaders->asArray() as $name => $value) {

test/phpunit/Dispatch/DispatcherFactoryTest.php

@@ -0,0 +1,56 @@
+<?php
+namespace GT\WebEngine\Test\Dispatch;
+
+use Gt\Config\Config;
+use Gt\Http\Request;
+use Gt\Http\ResponseStatusException\ClientError\HttpNotFound;
+use Gt\Http\Uri;
+use Gt\Session\FileHandler;
+use GT\WebEngine\Dispatch\DispatcherFactory;
+use PHPUnit\Framework\TestCase;
+
+class DispatcherFactoryTest extends TestCase {
+	public function testCreate():void {
+		$config = self::createMock(Config::class);
+		$config->method("getString")
+			->willReturnMap([
+				["app.namespace", "Example\\NS"],
+				["app.class_dir", "/tmp/phpgt-webengine-test--dispatcher-factory--class"],
+				["router.router_file", "/tmp/phpgt-webengine-test--dispatcher-factory--router"],
+				["router.router_class", "TestRouter"],
+				["router.default_content_type", "unit/test"],
+				["session.name", "GT_Test_Session"],
+				["session.handler", FileHandler::class],
+				["session.path", "/"],
+				["view.component_directory", "/tmp/phpgt-webengine-test--dispatcher-factory--component-dir"],
+				["view.partial_directory", "/tmp/phpgt-webengine-test--dispatcher-factory--partial-dir"],
+			]);
+		$config->method("getInt")
+			->willReturnMap([
+				["router.redirect_response_code", 321],
+			]);
+
+		$requestUri = self::createMock(Uri::class);
+		$requestUri->method("getPath")
+			->willReturn("/test/");
+		$request = self::createMock(Request::class);
+		$request->method("getUri")
+			->willReturn($requestUri);
+		$request->method("getMethod")
+			->willReturn("GET");
+		$globals = [
+			"_GET" => [],
+			"_POST" => [],
+			"_FILES" => [],
+			"_SERVER" => [],
+			"_COOKIE" => [],
+		];
+		$finishCallback = fn() => null;
+		$errorStatus = 123;
+
+		$sut = new DispatcherFactory();
+		$dispatcher = $sut->create($config, $request, $globals, $finishCallback, $errorStatus);
+		self::expectException(HttpNotFound::class);
+		$dispatcher->generateResponse();
+	}
+}

test/phpunit/Dispatch/HeaderManagerTest.php

@@ -0,0 +1,37 @@
+<?php
+namespace GT\WebEngine\Test\Dispatch;
+
+use Gt\Http\Header\ResponseHeaders;
+use GT\WebEngine\Dispatch\HeaderManager;
+use PHPUnit\Framework\TestCase;
+
+class HeaderManagerTest extends TestCase {
+	public function testApplyWithHeader():void {
+		$testHeaderArray = [
+			"X-PHPGT-TEST" => uniqid(),
+			"X-TIMESTAMP" => time(),
+		];
+
+		$callbackParameterCalls = [];
+		$callback = function($name, $value) use(&$callbackParameterCalls) {
+			array_push($callbackParameterCalls, [$name, $value]);
+		};
+
+		$responseHeaders = self::createMock(ResponseHeaders::class);
+		$responseHeaders->expects(self::once())
+			->method("asArray")
+			->willReturn($testHeaderArray);
+
+		$sut = new HeaderManager();
+		$sut->applyWithHeader($responseHeaders, $callback);
+
+		self::assertCount(count($testHeaderArray), $callbackParameterCalls);
+		$index = 0;
+		foreach($testHeaderArray as $key => $value) {
+			$actualCallbackParameters = $callbackParameterCalls[$index];
+			self::assertSame($key, $actualCallbackParameters[0]);
+			self::assertSame($value, $actualCallbackParameters[1]);
+			$index++;
+		}
+	}
+}

test/phpunit/Dispatch/RouterFactoryTest.php

@@ -37,8 +37,8 @@ private function writeRouterFile(string $namespace, string $className, string $f
 		class %CLASS% extends BaseRouter {
 			public ?RouterConfig $receivedConfig = null;
 			public ?Container $receivedContainer = null;
-			public function __construct(RouterConfig $config) {
-				parent::__construct($config);
+			public function __construct(RouterConfig $config, ?int $errorStatus) {
+				parent::__construct($config, errorStatus: $errorStatus);
 				$this->receivedConfig = $config;
 			}
 			public function setContainer(Container $container):void {