From d9d64d2daa53cd1850d8a3d27338ddcc3c7ad0e1 Mon Sep 17 00:00:00 2001 From: verboomp Date: Thu, 29 Jan 2026 15:42:28 +0100 Subject: [PATCH] added unit test --- .../rest/PictureResource.java | 2 +- .../rest/PictureResourceTest.java | 37 ++++++++++++++++++- 2 files changed, 36 insertions(+), 3 deletions(-) diff --git a/hartmann-foto-documentation-app/src/main/java/marketing/heyday/hartmann/fotodocumentation/rest/PictureResource.java b/hartmann-foto-documentation-app/src/main/java/marketing/heyday/hartmann/fotodocumentation/rest/PictureResource.java index a0af887..bff40d7 100644 --- a/hartmann-foto-documentation-app/src/main/java/marketing/heyday/hartmann/fotodocumentation/rest/PictureResource.java +++ b/hartmann-foto-documentation-app/src/main/java/marketing/heyday/hartmann/fotodocumentation/rest/PictureResource.java @@ -47,7 +47,7 @@ public class PictureResource { @Path("evaluation/{id}") @Operation(summary = "Update evaluation for picture data to database") @ApiResponse(responseCode = "200", description = "Task successfully updated") - public Response doUpdateTask(@PathParam("id") Long id, @QueryParam("evaluation") Integer value) { + public Response doUpdateEvaluation(@PathParam("id") Long id, @QueryParam("evaluation") Integer value) { if(value == null || value < 1 || value >3) { return Response.status(Status.BAD_REQUEST).build(); } diff --git a/hartmann-foto-documentation-docker/src/test/java/marketing/heyday/hartmann/fotodocumentation/rest/PictureResourceTest.java b/hartmann-foto-documentation-docker/src/test/java/marketing/heyday/hartmann/fotodocumentation/rest/PictureResourceTest.java index 0b41486..fe27296 100644 --- a/hartmann-foto-documentation-docker/src/test/java/marketing/heyday/hartmann/fotodocumentation/rest/PictureResourceTest.java +++ b/hartmann-foto-documentation-docker/src/test/java/marketing/heyday/hartmann/fotodocumentation/rest/PictureResourceTest.java @@ -34,7 +34,7 @@ public class PictureResourceTest extends AbstractRestTest { } @Test - @Order(1) + @Order(3) public void doDelete() throws IOException { LOG.info("doDelete"); @@ -52,7 +52,7 @@ public class PictureResourceTest extends AbstractRestTest { } @Test - @Order(1) + @Order(2) public void doDeleteNotFound() throws IOException { LOG.info("doDeleteNotFound"); @@ -69,4 +69,37 @@ public class PictureResourceTest extends AbstractRestTest { assertEquals(5, pictureCount()); } + @Test + @Order(1) + public void doEvaluation() throws IOException { + LOG.info("doEvaluation"); + + assertEquals(0, getCount("select count(*) from picture where picture_id = 1 and evaluation = 3")); + + + + String path = deploymentURL + PATH + "/evaluation/1?evaluation=3"; + Request request = Request.Put(path).addHeader("Accept", "application/json; charset=utf-8") + .addHeader("Authorization", getAuthorization()); + + HttpResponse httpResponse = executeRequest(request); + int code = httpResponse.getStatusLine().getStatusCode(); + assertEquals(200, code); + + assertEquals(1, getCount("select count(*) from picture where picture_id = 1 and evaluation = 3")); + } + + @Test + @Order(1) + public void doEvaluationNotFound() throws IOException { + LOG.info("doEvaluationNotFound"); + + String path = deploymentURL + PATH + "/evaluation/6000?evaluation=3"; + Request request = Request.Put(path).addHeader("Accept", "application/json; charset=utf-8") + .addHeader("Authorization", getAuthorization()); + + HttpResponse httpResponse = executeRequest(request); + int code = httpResponse.getStatusLine().getStatusCode(); + assertEquals(404, code); + } }