Refactor/339 기관 프로필 이미지 업데이트 리팩토링 (#348)

7zrv · web-flow · commit b5e2b777d434 · 2025-02-05T14:25:02.000+09:00
* refactor: 선호 물품 컨트롤러 리팩토링

- userId 직렬화 어노테이션 변경
- 테스트및 검증 완료

* feat: 유저 공통속성 엔티티 프로필 url 변경 메서드 구현

- 프로필 이미지 url 필드를 변경하는 메서드 구현
- 테스트및 검증 완료

* feat: 유저 공통속성 프로필 이미지 업데이트 유스케이스 구현

- 프로필 이미지 업데이트 유스케이스및 구현체 구현
- 요청 레코드 생성
- 테스트및 검증 완료

* feat: 기관 프로필 이미지 변경 유스케이스 구

- 프로필 이미지 업데이트 유스케이스및 구현체 구현
- 요청 레코드 생성
- 테스트및 검증 완료

* feat: 기관 프로필 이미지 변경 컨트롤러 구현

- 프로필 이미지 업데이트 엔드 포인트 구현
- 테스트및 검증 완료
diff --git a/src/main/java/com/somemore/center/controller/CenterCommandApiController.java b/src/main/java/com/somemore/center/controller/CenterCommandApiController.java
@@ -0,0 +1,37 @@
+package com.somemore.center.controller;
+
+import com.somemore.center.dto.request.CenterProfileImgUpdateRequestDto;
+import com.somemore.center.usecase.UpdateCenterProfileImgUseCase;
+import com.somemore.global.auth.annotation.UserId;
+import com.somemore.global.common.response.ApiResponse;
+import io.swagger.v3.oas.annotations.Operation;
+import io.swagger.v3.oas.annotations.tags.Tag;
+import lombok.RequiredArgsConstructor;
+import org.springframework.security.access.annotation.Secured;
+import org.springframework.web.bind.annotation.PutMapping;
+import org.springframework.web.bind.annotation.RequestMapping;
+import org.springframework.web.bind.annotation.RestController;
+
+import java.util.UUID;
+
+@Tag(name = "Center Command API", description = "기관 프로필 수정 API")
+@RequiredArgsConstructor
+@RequestMapping("/api/center")
+@RestController
+public class CenterCommandApiController {
+
+    private final UpdateCenterProfileImgUseCase updateCenterProfileImgUseCase;
+
+    @Secured("ROLE_CENTER")
+    @Operation(summary = "기관 프로필 이미지 수정", description = "기관의 프로필 이미지를 수정합니다.")
+    @PutMapping("/profileImgUpdate")
+    public ApiResponse<String> updateCenterProfile(
+            @UserId UUID centerId,
+            CenterProfileImgUpdateRequestDto requestDto
+    ) {
+
+        String presignedUrl = updateCenterProfileImgUseCase.updateCenterProfileImg(centerId, requestDto);
+
+        return ApiResponse.ok(presignedUrl,"센터 프로필 수정 성공");
+    }
+}
diff --git a/src/main/java/com/somemore/center/dto/request/CenterProfileImgUpdateRequestDto.java b/src/main/java/com/somemore/center/dto/request/CenterProfileImgUpdateRequestDto.java
@@ -0,0 +1,9 @@
+package com.somemore.center.dto.request;
+
+import io.swagger.v3.oas.annotations.media.Schema;
+
+public record CenterProfileImgUpdateRequestDto(
+        @Schema(description = "파일 이름", example = "image.png")
+        String fileName
+) {
+}
diff --git a/src/main/java/com/somemore/center/service/UpdateCenterProfileImgService.java b/src/main/java/com/somemore/center/service/UpdateCenterProfileImgService.java
@@ -0,0 +1,46 @@
+package com.somemore.center.service;
+
+import com.somemore.center.domain.NEWCenter;
+import com.somemore.center.dto.request.CenterProfileImgUpdateRequestDto;
+import com.somemore.center.repository.NEWCenterRepository;
+import com.somemore.center.usecase.UpdateCenterProfileImgUseCase;
+import com.somemore.global.exception.BadRequestException;
+import com.somemore.global.imageupload.usecase.ImageUploadUseCase;
+import com.somemore.user.dto.request.UpdateProfileImgUrlRequestDto;
+import com.somemore.user.usecase.UpdateProfileImgUrlUseCase;
+import com.somemore.user.usecase.UserQueryUseCase;
+import lombok.RequiredArgsConstructor;
+import org.springframework.stereotype.Service;
+import org.springframework.transaction.annotation.Transactional;
+
+import java.util.UUID;
+
+import static com.somemore.global.exception.ExceptionMessage.NOT_EXISTS_CENTER;
+
+@RequiredArgsConstructor
+@Service
+@Transactional
+public class UpdateCenterProfileImgService implements UpdateCenterProfileImgUseCase {
+
+    private final ImageUploadUseCase imageUploadUseCase;
+    private final UserQueryUseCase userQueryUseCase;
+    private final UpdateProfileImgUrlUseCase updateProfileImgUrlUseCase;
+    private final NEWCenterRepository centerRepository;
+
+    @Override
+    public String updateCenterProfileImg(UUID centerId, CenterProfileImgUpdateRequestDto requestDto) {
+
+        NEWCenter center = centerRepository.findById(centerId)
+                .orElseThrow(() -> new BadRequestException(NOT_EXISTS_CENTER));
+
+        String presignedUrl = imageUploadUseCase.getPresignedUrl(requestDto.fileName());
+
+        String fileUrl = presignedUrl.split("\\?")[0];
+
+        UpdateProfileImgUrlRequestDto updateProfileImgUrlRequestDto = new UpdateProfileImgUrlRequestDto(center.getUserId(), fileUrl);
+
+        updateProfileImgUrlUseCase.updateProfileImgUrl(updateProfileImgUrlRequestDto);
+
+        return presignedUrl;
+    }
+}
diff --git a/src/main/java/com/somemore/center/usecase/UpdateCenterProfileImgUseCase.java b/src/main/java/com/somemore/center/usecase/UpdateCenterProfileImgUseCase.java
@@ -0,0 +1,9 @@
+package com.somemore.center.usecase;
+
+import com.somemore.center.dto.request.CenterProfileImgUpdateRequestDto;
+
+import java.util.UUID;
+
+public interface UpdateCenterProfileImgUseCase {
+    String updateCenterProfileImg(UUID centerId, CenterProfileImgUpdateRequestDto requestDto);
+}
diff --git a/src/main/java/com/somemore/domains/center/controller/PreferItemCommandApiController.java b/src/main/java/com/somemore/domains/center/controller/PreferItemCommandApiController.java
@@ -4,7 +4,7 @@
 import com.somemore.domains.center.dto.response.PreferItemCreateResponseDto;
 import com.somemore.domains.center.usecase.command.CreatePreferItemUseCase;
 import com.somemore.domains.center.usecase.command.DeletePreferItemUseCase;
-import com.somemore.global.auth.annotation.CurrentUser;
+import com.somemore.global.auth.annotation.UserId;
 import com.somemore.global.common.response.ApiResponse;
 import io.swagger.v3.oas.annotations.Operation;
 import io.swagger.v3.oas.annotations.tags.Tag;
@@ -34,9 +34,9 @@ public class PreferItemCommandApiController {
     @PostMapping()
     public ApiResponse<PreferItemCreateResponseDto> registerPreferItem(
             @Valid @RequestBody PreferItemCreateRequestDto requestDto,
-            @CurrentUser UUID userId) {
+            @UserId UUID centerId) {
 
-        PreferItemCreateResponseDto responseDto = createPreferItemUseCase.createPreferItem(userId,
+        PreferItemCreateResponseDto responseDto = createPreferItemUseCase.createPreferItem(centerId,
                 requestDto);
 
         return ApiResponse.ok(200, responseDto, "관심 기관 등록 성공");
@@ -45,7 +45,7 @@ public ApiResponse<PreferItemCreateResponseDto> registerPreferItem(
     @Secured("ROLE_CENTER")
     @Operation(summary = "기관 선호물품 삭제 API")
     @DeleteMapping("/{preferItemId}")
-    public ApiResponse<String> deletePreferItem(@CurrentUser UUID centerId, @PathVariable Long preferItemId) {
+    public ApiResponse<String> deletePreferItem(@UserId UUID centerId, @PathVariable Long preferItemId) {
 
         deletePreferItemUseCase.deletePreferItem(centerId, preferItemId);
 
diff --git a/src/main/java/com/somemore/user/domain/UserCommonAttribute.java b/src/main/java/com/somemore/user/domain/UserCommonAttribute.java
@@ -53,6 +53,10 @@ public void update(CommonBasicInfoRequestDto commonBasicInfoRequestDto) {
         this.isCustomized = true;
     }
 
+    public void updateImgUrl(String imgUrl) {
+        this.imgUrl = imgUrl;
+    }
+
     public static UserCommonAttribute createDefault(UUID userId, UserRole role) {
         return UserCommonAttribute.builder()
                 .userId(userId)
diff --git a/src/main/java/com/somemore/user/dto/request/UpdateProfileImgUrlRequestDto.java b/src/main/java/com/somemore/user/dto/request/UpdateProfileImgUrlRequestDto.java
@@ -0,0 +1,9 @@
+package com.somemore.user.dto.request;
+
+import java.util.UUID;
+
+public record UpdateProfileImgUrlRequestDto(
+        UUID userId,
+        String profileImgUrl
+) {
+}
diff --git a/src/main/java/com/somemore/user/service/UpdateProfileImgUrlService.java b/src/main/java/com/somemore/user/service/UpdateProfileImgUrlService.java
@@ -0,0 +1,25 @@
+package com.somemore.user.service;
+
+import com.somemore.user.domain.UserCommonAttribute;
+import com.somemore.user.dto.request.UpdateProfileImgUrlRequestDto;
+import com.somemore.user.usecase.UpdateProfileImgUrlUseCase;
+import com.somemore.user.usecase.UserQueryUseCase;
+import lombok.RequiredArgsConstructor;
+import org.springframework.stereotype.Service;
+import org.springframework.transaction.annotation.Transactional;
+
+@RequiredArgsConstructor
+@Service
+@Transactional
+public class UpdateProfileImgUrlService implements UpdateProfileImgUrlUseCase {
+
+    private final UserQueryUseCase userQueryUseCase;
+
+    @Override
+    public void updateProfileImgUrl(UpdateProfileImgUrlRequestDto updateProfileImgUrlRequestDto) {
+        UserCommonAttribute userCommonAttribute =
+                userQueryUseCase.getCommonAttributeByUserId(updateProfileImgUrlRequestDto.userId());
+
+        userCommonAttribute.updateImgUrl(updateProfileImgUrlRequestDto.profileImgUrl());
+    }
+}
diff --git a/src/main/java/com/somemore/user/usecase/UpdateProfileImgUrlUseCase.java b/src/main/java/com/somemore/user/usecase/UpdateProfileImgUrlUseCase.java
@@ -0,0 +1,7 @@
+package com.somemore.user.usecase;
+
+import com.somemore.user.dto.request.UpdateProfileImgUrlRequestDto;
+
+public interface UpdateProfileImgUrlUseCase {
+    void updateProfileImgUrl(UpdateProfileImgUrlRequestDto updateProfileImgUrlRequestDto);
+}
diff --git a/src/test/java/com/somemore/center/controller/CenterCommandApiControllerTest.java b/src/test/java/com/somemore/center/controller/CenterCommandApiControllerTest.java
@@ -0,0 +1,51 @@
+package com.somemore.center.controller;
+
+import com.somemore.center.dto.request.CenterProfileImgUpdateRequestDto;
+import com.somemore.center.usecase.UpdateCenterProfileImgUseCase;
+import com.somemore.support.ControllerTestSupport;
+import static org.mockito.Mockito.times;
+import static org.mockito.Mockito.verify;
+import static org.mockito.Mockito.when;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.DisplayName;
+import org.springframework.boot.test.mock.mockito.MockBean;
+import org.springframework.http.MediaType;
+import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.put;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.jsonPath;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status;
+
+import java.util.UUID;
+
+class CenterCommandApiControllerTest extends ControllerTestSupport {
+
+    @MockBean
+    private UpdateCenterProfileImgUseCase updateCenterProfileImgUseCase;
+
+    @DisplayName("기관 프로필 이미지를 수정할 수 있다. (controller)")
+    @Test
+    void updateCenterProfile() throws Exception {
+
+        // given
+        UUID centerId = UUID.randomUUID();
+        CenterProfileImgUpdateRequestDto requestDto = new CenterProfileImgUpdateRequestDto("test.jpg");
+
+        String expectedPresignedUrl = "https://example.com/presigned-url/test.jpg";
+        when(updateCenterProfileImgUseCase.updateCenterProfileImg(centerId, requestDto))
+                .thenReturn(expectedPresignedUrl);
+
+        // when & then
+        mockMvc.perform(
+                        put("/api/center/profileImgUpdate")
+                                .header("X-User-Id", centerId.toString()) // @UserId를 시뮬레이션
+                                .contentType(MediaType.APPLICATION_JSON)
+                                .content(objectMapper.writeValueAsString(requestDto))
+                )
+                .andExpect(status().isOk())
+                .andExpect(jsonPath("$.code").value("200"))
+                .andExpect(jsonPath("$.message").value("센터 프로필 수정 성공"))
+                .andExpect(jsonPath("$.data").value(expectedPresignedUrl));
+
+        verify(updateCenterProfileImgUseCase, times(1)).updateCenterProfileImg(centerId, requestDto);
+    }
+
+}
diff --git a/src/test/java/com/somemore/center/service/UpdateCenterProfileImgServiceTest.java b/src/test/java/com/somemore/center/service/UpdateCenterProfileImgServiceTest.java
@@ -0,0 +1,67 @@
+package com.somemore.center.service;
+
+import com.somemore.center.domain.NEWCenter;
+import com.somemore.center.dto.request.CenterProfileImgUpdateRequestDto;
+import com.somemore.center.repository.NEWCenterRepository;
+import com.somemore.global.imageupload.usecase.ImageUploadUseCase;
+import com.somemore.support.IntegrationTestSupport;
+import com.somemore.user.domain.UserCommonAttribute;
+import com.somemore.user.repository.usercommonattribute.UserCommonAttributeRepository;
+import org.junit.jupiter.api.DisplayName;
+import org.junit.jupiter.api.Test;
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.transaction.annotation.Transactional;
+
+import java.util.Optional;
+import java.util.UUID;
+
+import static com.somemore.user.domain.UserRole.CENTER;
+import static org.junit.jupiter.api.Assertions.*;
+import static org.assertj.core.api.Assertions.assertThat;
+
+@Transactional
+class UpdateCenterProfileImgServiceTest extends IntegrationTestSupport {
+
+    @Autowired
+    private UpdateCenterProfileImgService updateCenterProfileImgService;
+
+    @Autowired
+    private ImageUploadUseCase imageUploadUseCase;
+
+    @Autowired
+    private NEWCenterRepository centerRepository;
+
+    @Autowired
+    private UserCommonAttributeRepository userCommonAttributeRepository;
+
+    @DisplayName("파일명을 받아 기관 프로필 이미지의 presignedUrl을 발급해준다.")
+    @Test
+    void updateCenterProfileImg() {
+
+        //given
+        UUID userId = UUID.randomUUID();
+
+        NEWCenter center = NEWCenter.createDefault(userId);
+        centerRepository.save(center);
+        UUID centerId = center.getId();
+
+        UserCommonAttribute userCommonAttribute = UserCommonAttribute.createDefault(userId, CENTER);
+        userCommonAttributeRepository.save(userCommonAttribute);
+
+        CenterProfileImgUpdateRequestDto requestDto = new CenterProfileImgUpdateRequestDto("test.png");
+
+        //when
+        String presignedUrl = updateCenterProfileImgService.updateCenterProfileImg(centerId, requestDto);
+
+        //then
+        assertThat(presignedUrl).isNotNull();
+
+        Optional<UserCommonAttribute> updatedUserCommonAttribute = userCommonAttributeRepository.findByUserId(userId);
+        assertThat(updatedUserCommonAttribute)
+                .isNotNull();
+        assertTrue(updatedUserCommonAttribute.get().getImgUrl().contains(requestDto.fileName()));
+
+    }
+
+
+}
diff --git a/src/test/java/com/somemore/domains/center/controller/PreferItemCommandApiControllerTest.java b/src/test/java/com/somemore/domains/center/controller/PreferItemCommandApiControllerTest.java
diff --git a/src/test/java/com/somemore/user/service/UpdateProfileImgUrlServiceTest.java b/src/test/java/com/somemore/user/service/UpdateProfileImgUrlServiceTest.java
