feature: reload and throw away query string (#178)

closes #177

Author: Greg Bowler

src/Response.php

@@ -29,14 +29,18 @@ public function setExitCallback(callable $callback):void {
 		$this->exitCallback = $callback;
 	}
 
-	public function reload():void {
+	public function reload(bool $keepQuery = true):void {
 		$uri = $this->request?->getUri() ?? new Uri();
-		$this->redirect($uri->withPath("./"));
+		$uri = $uri->withPath("./");
+		if(!$keepQuery) {
+			$uri = $uri->withQuery("");
+		}
+		$this->redirect($uri);
 	}
 
 	public function redirect(string|UriInterface $uri, int $statusCode = 303):void {
 		$this->statusCode = $statusCode;
-		$this->headers->set("Location", $uri);
+		$this->headers->set("Location", (string)$uri);
 		if(isset($this->exitCallback)) {
 			call_user_func($this->exitCallback);
 		}

test/phpunit/ResponseTest.php

@@ -2,7 +2,10 @@
 namespace Gt\Http\Test;
 
 use Gt\Http\Header\ResponseHeaders;
+use Gt\Http\Request;
 use Gt\Http\Response;
+use Gt\Http\StatusCode;
+use Gt\Http\Uri;
 use PHPUnit\Framework\TestCase;
 
 class ResponseTest extends TestCase {
@@ -55,4 +58,53 @@ public function testRedirect() {
 			$sut->getHeaderLine("Location")
 		);
 	}
+
+	public function testReloadKeepsQuery() {
+		$expectedRelativePath = "./?test=123";
+
+		$uri = self::createMock(Uri::class);
+		$uri->expects(self::once())
+			->method("withPath")
+			->with("./")
+			->willReturn($uri);
+		$uri->expects(self::never())
+			->method("withQuery");
+		$uri->method("__toString")
+			->willReturn($expectedRelativePath);
+
+		$request = self::createMock(Request::class);
+		$request->method("getUri")
+			->willReturn($uri);
+
+		$sut = new Response(200, request: $request);
+		self::assertSame(StatusCode::OK, $sut->getStatusCode());
+		$sut->reload();
+		self::assertSame(StatusCode::SEE_OTHER, $sut->getStatusCode());
+		self::assertSame($expectedRelativePath, $sut->getHeaderLine("Location"));
+	}
+
+	public function testReloadWithoutKeepQuery() {
+		$expectedRelativePath = "./";
+
+		$uri = self::createMock(Uri::class);
+		$uri->expects(self::once())
+			->method("withPath")
+			->with("./")
+			->willReturn($uri);
+		$uri->expects(self::once())
+			->method("withQuery")
+			->with("")
+			->willReturn($uri);
+		$uri->method("__toString")
+			->willReturn($expectedRelativePath);
+
+		$request = self::createMock(Request::class);
+		$request->method("getUri")
+			->willReturn($uri);
+
+		$sut = new Response(200, request: $request);
+		$sut->reload(false);
+		self::assertSame(StatusCode::SEE_OTHER, $sut->getStatusCode());
+		self::assertSame($expectedRelativePath, $sut->getHeaderLine("Location"));
+	}
 }