From af2f1645465b173d43122c0931b029e65197e9cb Mon Sep 17 00:00:00 2001 From: Ron Rise Date: Mon, 1 Dec 2025 11:16:59 -0500 Subject: [PATCH] feat: implement GenericResponse class and update controllers to use it for consistent JSON responses --- src/Controllers/IndexController.php | 2 +- src/Docs/TokenSecurity.php | 2 +- tests/Controllers/IndexControllerTest.php | 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/Controllers/IndexController.php b/src/Controllers/IndexController.php index 5ccb26e..e9a9059 100644 --- a/src/Controllers/IndexController.php +++ b/src/Controllers/IndexController.php @@ -37,7 +37,7 @@ class IndexController extends Controller #[UnauthorizedResponse] public function get(ServerRequest $request): ResponseInterface { - return JsonResponseFactory::createJsonResponse(['message' => 'Server is running']); + return JsonResponseFactory::createJsonResponse(new GenericResponse('Server is running')); } /** diff --git a/src/Docs/TokenSecurity.php b/src/Docs/TokenSecurity.php index 2998de2..7376ab6 100644 --- a/src/Docs/TokenSecurity.php +++ b/src/Docs/TokenSecurity.php @@ -16,4 +16,4 @@ class TokenSecurity extends OA\SecurityScheme scheme: 'bearer' ); } -} \ No newline at end of file +} diff --git a/tests/Controllers/IndexControllerTest.php b/tests/Controllers/IndexControllerTest.php index 2639a6d..67cff2d 100644 --- a/tests/Controllers/IndexControllerTest.php +++ b/tests/Controllers/IndexControllerTest.php @@ -20,7 +20,7 @@ class IndexControllerTest extends AbstractController $response = $controller->get($this->getMockRequest()); $this->assertEquals(200, $response->getStatusCode()); - $this->assertEquals('{"status_code":200,"message":"Server is running"}', (string)$response->getBody()); + $this->assertEquals('{"message":"Server is running","status_code":200}', (string)$response->getBody()); } /** @@ -35,6 +35,6 @@ class IndexControllerTest extends AbstractController $response = $controller->post($this->getMockRequest()); $this->assertEquals(200, $response->getStatusCode()); - $this->assertEquals('{"status_code":200,"message":"Server is running"}', (string)$response->getBody()); + $this->assertEquals('{"message":"POST request received","status_code":200}', (string)$response->getBody()); } }