EPMRPP-97060 upload bugfix (#2144)

reportportal · Jan 15, 2025 · ca6fd32 · ca6fd32
1 parent 109ebf4
commit ca6fd32
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/reportportal-common-api b/reportportal-common-api
diff --git a/src/main/java/com/epam/ta/reportportal/ws/controller/UserController.java b/src/main/java/com/epam/ta/reportportal/ws/controller/UserController.java
@@ -98,10 +98,10 @@ public ResponseEntity<Resource> getUsersUserIdAvatar(Long userId, Boolean thumbn
   @Transactional
   @PreAuthorize(ALLOWED_TO_USER_ITSELF)
   public ResponseEntity<Void> postUsersUserIdAvatar(Long userId,
-      @Parameter(name = "file", description = "")
-      @RequestPart(value = "file", required = false) MultipartFile file) {
+      @Parameter(name = "file")
+      @RequestPart(value = "file") MultipartFile file) {
 
     editUserHandler.uploadPhoto(userId, file);
-    return new ResponseEntity<>(HttpStatus.NO_CONTENT);
+    return new ResponseEntity<>(HttpStatus.CREATED);
   }
 }
diff --git a/src/test/java/com/epam/ta/reportportal/ws/controller/FileStorageControllerTest.java b/src/test/java/com/epam/ta/reportportal/ws/controller/FileStorageControllerTest.java
@@ -60,7 +60,7 @@ void userPhoto() throws Exception {
         .contentType(MediaType.MULTIPART_FORM_DATA);
 
     mockMvc.perform(requestBuilder.with(token(oAuthHelper.getDefaultToken())))
-        .andExpect(status().isNoContent());
+        .andExpect(status().isCreated());
 
     mockMvc.perform(get("/users/2/avatar").with(token(oAuthHelper.getDefaultToken())))
         .andExpect(status().isOk());

diff --git a/src/test/java/com/epam/ta/reportportal/ws/controller/UserControllerTest.java b/src/test/java/com/epam/ta/reportportal/ws/controller/UserControllerTest.java
@@ -492,7 +492,7 @@ void userPhoto() throws Exception {
         .contentType(MediaType.MULTIPART_FORM_DATA);
 
     mockMvc.perform(requestBuilder.with(token(oAuthHelper.getDefaultToken())))
-        .andExpect(status().isNoContent());
+        .andExpect(status().isCreated());
 
     mockMvc.perform(get("/users/2/avatar?thumbnail=false")
             .with(token(oAuthHelper.getDefaultToken())))
+2 −0		api/openapi/apis/users.yaml
+2 −0		api/openapi/reportportal-api.yaml