[P4ADEV-1828] fixed http status

pagopa · Jan 13, 2025 · 9b119d8 · 9b119d8
1 parent e3d8f7b
commit 9b119d8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/main/java/it/gov/pagopa/pu/fileshare/exception/FileshareExceptionHandler.java b/src/main/java/it/gov/pagopa/pu/fileshare/exception/FileshareExceptionHandler.java
@@ -23,7 +23,7 @@ public class FileshareExceptionHandler {
 
   @ExceptionHandler({InvalidFileException.class})
   public ResponseEntity<FileshareErrorDTO> handleInvalidFileError(RuntimeException ex, HttpServletRequest request){
-    return handleFileshareErrorException(ex, request, HttpStatus.INTERNAL_SERVER_ERROR, CodeEnum.INVALID_FILE);
+    return handleFileshareErrorException(ex, request, HttpStatus.BAD_REQUEST, CodeEnum.INVALID_FILE);
   }
 
   static ResponseEntity<FileshareErrorDTO> handleFileshareErrorException(RuntimeException ex, HttpServletRequest request, HttpStatus httpStatus, FileshareErrorDTO.CodeEnum errorEnum) {

diff --git a/src/test/java/it/gov/pagopa/pu/fileshare/exception/FileshareExceptionHandlerTest.java b/src/test/java/it/gov/pagopa/pu/fileshare/exception/FileshareExceptionHandlerTest.java
@@ -54,7 +54,7 @@ void handleInvalidFileException() throws Exception {
                         .param(DATA, DATA)
                         .contentType(MediaType.APPLICATION_JSON)
                         .accept(MediaType.APPLICATION_JSON))
-                .andExpect(MockMvcResultMatchers.status().isInternalServerError())
+                .andExpect(MockMvcResultMatchers.status().isBadRequest())
                 .andExpect(MockMvcResultMatchers.jsonPath("$.code").value(CodeEnum.INVALID_FILE.toString()))
                 .andExpect(MockMvcResultMatchers.jsonPath("$.message").value("Error"));
     }