feat: 유저 태그 수정 API

leeedohyun · leeedohyun · commit c40b2432b5c8 · 2025-09-03T23:18:04.000+09:00
diff --git a/capturecat-core/src/main/java/com/capturecat/core/api/user/UserTagController.java b/capturecat-core/src/main/java/com/capturecat/core/api/user/UserTagController.java
@@ -1,13 +1,16 @@
 package com.capturecat.core.api.user;
 
 import org.springframework.security.core.annotation.AuthenticationPrincipal;
+import org.springframework.web.bind.annotation.PatchMapping;
 import org.springframework.web.bind.annotation.PostMapping;
+import org.springframework.web.bind.annotation.RequestBody;
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RequestParam;
 import org.springframework.web.bind.annotation.RestController;
 
 import lombok.RequiredArgsConstructor;
 
+import com.capturecat.core.api.user.dto.TagRenameRequest;
 import com.capturecat.core.service.auth.LoginUser;
 import com.capturecat.core.service.image.TagResponse;
 import com.capturecat.core.service.user.UserTagService;
@@ -26,4 +29,12 @@ public ApiResponse<TagResponse> create(@AuthenticationPrincipal LoginUser loginU
 
 		return ApiResponse.success(tagResponse);
 	}
+
+	@PatchMapping
+	public ApiResponse<TagResponse> update(@AuthenticationPrincipal LoginUser loginUser,
+		@RequestBody TagRenameRequest request) {
+		TagResponse response = userTagService.update(loginUser, request.currentTagId(), request.newTagName());
+
+		return ApiResponse.success(response);
+	}
 }
diff --git a/capturecat-core/src/main/java/com/capturecat/core/api/user/dto/TagRenameRequest.java b/capturecat-core/src/main/java/com/capturecat/core/api/user/dto/TagRenameRequest.java
@@ -0,0 +1,4 @@
+package com.capturecat.core.api.user.dto;
+
+public record TagRenameRequest(Long currentTagId, String newTagName) {
+}
diff --git a/capturecat-core/src/main/java/com/capturecat/core/domain/user/UserTagRepository.java b/capturecat-core/src/main/java/com/capturecat/core/domain/user/UserTagRepository.java
@@ -1,11 +1,15 @@
 package com.capturecat.core.domain.user;
 
+import java.util.Optional;
+
 import org.springframework.data.jpa.repository.JpaRepository;
 
 import com.capturecat.core.domain.tag.Tag;
 
 public interface UserTagRepository extends JpaRepository<UserTag, Long> {
 
+	Optional<UserTag> findByUserAndTag(User user, Tag tag);
+
 	boolean existsByUserAndTag(User user, Tag tag);
 
 	long countByUser(User user);
diff --git a/capturecat-core/src/main/java/com/capturecat/core/service/user/UserTagService.java b/capturecat-core/src/main/java/com/capturecat/core/service/user/UserTagService.java
@@ -9,6 +9,7 @@
 
 import com.capturecat.core.domain.tag.Tag;
 import com.capturecat.core.domain.tag.TagRegister;
+import com.capturecat.core.domain.tag.TagRepository;
 import com.capturecat.core.domain.user.User;
 import com.capturecat.core.domain.user.UserRepository;
 import com.capturecat.core.domain.user.UserTag;
@@ -27,6 +28,7 @@ public class UserTagService {
 
 	private final UserRepository userRepository;
 	private final UserTagRepository userTagRepository;
+	private final TagRepository tagRepository;
 	private final TagRegister tagRegister;
 
 	@Transactional
@@ -46,6 +48,24 @@ public TagResponse create(LoginUser loginUser, String tagName) {
 		}
 	}
 
+	@Transactional
+	public TagResponse update(LoginUser loginUser, Long currentTagId, String newTagName) {
+		User user = userRepository.findByUsername(loginUser.getUsername())
+			.orElseThrow(() -> new CoreException(ErrorType.USER_NOT_FOUND));
+		Tag currentTag = tagRepository.findById(currentTagId)
+			.orElseThrow(() -> new CoreException(ErrorType.TAG_NOT_FOUND));
+		UserTag userTag = userTagRepository.findByUserAndTag(user, currentTag)
+			.orElseThrow(() -> new CoreException(ErrorType.USER_TAG_NOT_FOUND));
+		Tag newTag = tagRegister.registerTagsFor(newTagName);
+
+		validateDuplicateUserTag(user, newTag);
+
+		userTagRepository.delete(userTag);
+		userTagRepository.save(UserTag.create(user, newTag));
+
+		return TagResponse.from(newTag);
+	}
+
 	private void validate(User user, Tag tag) {
 		validateDuplicateUserTag(user, tag);
 		validateUserTagCountLimit(user);
diff --git a/capturecat-core/src/main/java/com/capturecat/core/support/error/ErrorCode.java b/capturecat-core/src/main/java/com/capturecat/core/support/error/ErrorCode.java
@@ -40,7 +40,8 @@ public enum ErrorCode {
 	INTERNAL_SERVER_ERROR("서버에서 오류가 발생했습니다."),
 	INVALID_LOGOUT_AUTH_TOKEN("ACCESS 토큰 또는 REFRESH 토큰이 유효하지 않습니다."),
 	ALREADY_EXISTS_USER_TAG("이미 등록된 유저 태그입니다."),
-	EXCEED_MAX_USER_TAG_COUNT("태그는 한 계정당 최대 30개까지 추가할 수 있어요.");
+	EXCEED_MAX_USER_TAG_COUNT("태그는 한 계정당 최대 30개까지 추가할 수 있어요."),
+	NOT_FOUND_USER_TAG("유저 태그를 찾을 수 없습니다.");
 
 	private final String message;
 
diff --git a/capturecat-core/src/main/java/com/capturecat/core/support/error/ErrorType.java b/capturecat-core/src/main/java/com/capturecat/core/support/error/ErrorType.java
@@ -45,7 +45,8 @@ public enum ErrorType {
 	MISSING_PARAMETER(HttpStatus.BAD_REQUEST, ErrorCode.MISSING_PARAMETER, LogLevel.WARN),
 	INTERNAL_SERVER_ERROR(HttpStatus.INTERNAL_SERVER_ERROR, ErrorCode.INTERNAL_SERVER_ERROR, LogLevel.ERROR),
 	USER_TAG_ALREADY_EXISTS(HttpStatus.BAD_REQUEST, ErrorCode.ALREADY_EXISTS_USER_TAG, LogLevel.WARN),
-	TOO_MANY_USER_TAGS(HttpStatus.BAD_REQUEST, ErrorCode.EXCEED_MAX_USER_TAG_COUNT, LogLevel.WARN);
+	TOO_MANY_USER_TAGS(HttpStatus.BAD_REQUEST, ErrorCode.EXCEED_MAX_USER_TAG_COUNT, LogLevel.WARN),
+	USER_TAG_NOT_FOUND(HttpStatus.NOT_FOUND, ErrorCode.NOT_FOUND_USER_TAG, LogLevel.WARN);
 
 	private final HttpStatus status;
 
diff --git a/capturecat-core/src/test/java/com/capturecat/core/api/user/UserErrorCodeControllerTest.java b/capturecat-core/src/test/java/com/capturecat/core/api/user/UserErrorCodeControllerTest.java
@@ -1,8 +1,10 @@
 package com.capturecat.core.api.user;
 
+import static com.capturecat.core.support.error.ErrorType.TAG_NOT_FOUND;
 import static com.capturecat.core.support.error.ErrorType.TOO_MANY_USER_TAGS;
 import static com.capturecat.core.support.error.ErrorType.USER_NOT_FOUND;
 import static com.capturecat.core.support.error.ErrorType.USER_TAG_ALREADY_EXISTS;
+import static com.capturecat.core.support.error.ErrorType.USER_TAG_NOT_FOUND;
 
 import java.util.List;
 
@@ -19,4 +21,11 @@ class UserErrorCodeControllerTest extends ErrorCodeDocumentTest {
 			TOO_MANY_USER_TAGS, USER_NOT_FOUND);
 		generateErrorDocs("errorCode/createUserTag", errorCodeDescriptors);
 	}
+
+	@Test
+	void 유저_태그_수정_에러_코드_문서화() {
+		List<ErrorCodeDescriptor> errorCodeDescriptors = generateErrorCodeDescriptors(USER_TAG_ALREADY_EXISTS,
+			USER_NOT_FOUND, TAG_NOT_FOUND, USER_TAG_NOT_FOUND);
+		generateErrorDocs("errorCode/updateUserTag", errorCodeDescriptors);
+	}
 }
diff --git a/capturecat-core/src/test/java/com/capturecat/core/api/user/UserTagControllerTest.java b/capturecat-core/src/test/java/com/capturecat/core/api/user/UserTagControllerTest.java
@@ -3,16 +3,20 @@
 import static com.capturecat.test.api.RestDocsUtil.requestPreprocessor;
 import static com.capturecat.test.api.RestDocsUtil.responsePreprocessor;
 import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.ArgumentMatchers.anyLong;
 import static org.mockito.ArgumentMatchers.anyString;
 import static org.mockito.Mockito.mock;
 import static org.springframework.restdocs.headers.HeaderDocumentation.headerWithName;
 import static org.springframework.restdocs.headers.HeaderDocumentation.requestHeaders;
 import static org.springframework.restdocs.mockmvc.MockMvcRestDocumentation.document;
 import static org.springframework.restdocs.payload.PayloadDocumentation.fieldWithPath;
+import static org.springframework.restdocs.payload.PayloadDocumentation.requestFields;
 import static org.springframework.restdocs.payload.PayloadDocumentation.responseFields;
 import static org.springframework.restdocs.request.RequestDocumentation.parameterWithName;
 import static org.springframework.restdocs.request.RequestDocumentation.queryParameters;
 
+import java.util.Map;
+
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.mockito.BDDMockito;
@@ -62,4 +66,29 @@ void setUp() {
 					fieldWithPath("data.id").type(JsonFieldType.NUMBER).description("태그 ID"),
 					fieldWithPath("data.name").type(JsonFieldType.STRING).description("태그 이름"))));
 	}
+
+	@Test
+	void 유저_태그_수정() {
+		// given
+		BDDMockito.given(userTagService.update(any(), anyLong(), anyString())).willReturn(new TagResponse(1L, "java"));
+
+		// when & then
+		given()
+			.header(HttpHeaders.AUTHORIZATION, JwtUtil.BEARER_PREFIX + ACCESS_TOKEN)
+			.contentType(ContentType.JSON)
+			.body(Map.of("currentTagId", 1L, "newTagName", "spring"))
+			.when().patch("/v1/user-tags")
+			.then().status(HttpStatus.OK)
+			.apply(document("updateUserTag", requestPreprocessor(), responsePreprocessor(),
+				requestHeaders(headerWithName(HttpHeaders.AUTHORIZATION).description("유효한 Access 토큰")),
+				requestFields(
+					fieldWithPath("currentTagId").type(JsonFieldType.NUMBER).description("수정할 태그 ID"),
+					fieldWithPath("newTagName").type(JsonFieldType.STRING).description("새로운 태그 이름")
+				),
+				responseFields(
+					fieldWithPath("result").type(JsonFieldType.STRING).description("요청 결과"),
+					fieldWithPath("data").type(JsonFieldType.OBJECT).description("수정된 태그 정보"),
+					fieldWithPath("data.id").type(JsonFieldType.NUMBER).description("태그 ID"),
+					fieldWithPath("data.name").type(JsonFieldType.STRING).description("태그 이름"))));
+	}
 }
diff --git a/capturecat-core/src/test/java/com/capturecat/core/domain/user/UserTagRepositoryTest.java b/capturecat-core/src/test/java/com/capturecat/core/domain/user/UserTagRepositoryTest.java
@@ -0,0 +1,53 @@
+package com.capturecat.core.domain.user;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+import jakarta.persistence.EntityManager;
+
+import org.junit.jupiter.api.Test;
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.boot.test.autoconfigure.orm.jpa.DataJpaTest;
+import org.springframework.context.annotation.Import;
+
+import com.capturecat.core.DummyObject;
+import com.capturecat.core.config.JpaAuditingConfig;
+import com.capturecat.core.config.QueryDslConfig;
+import com.capturecat.core.domain.tag.Tag;
+import com.capturecat.core.domain.tag.TagFixture;
+import com.capturecat.core.domain.tag.TagRepository;
+
+@DataJpaTest
+@Import({QueryDslConfig.class, JpaAuditingConfig.class})
+class UserTagRepositoryTest {
+
+	@Autowired
+	EntityManager entityManager;
+
+	@Autowired
+	UserRepository userRepository;
+
+	@Autowired
+	TagRepository tagRepository;
+
+	@Autowired
+	UserTagRepository userTagRepository;
+
+	@Test
+	void findByUserAndTag() {
+		// given
+		User user = userRepository.save(DummyObject.newUser("test"));
+		Tag tag = tagRepository.save(TagFixture.createTag("A"));
+		UserTag userTag = userTagRepository.save(UserTag.create(user, tag));
+
+		entityManager.flush();
+		entityManager.clear();
+
+		// when
+		UserTag result = userTagRepository.findByUserAndTag(user, tag).orElseThrow();
+
+		// then
+		assertThat(result.getId()).isEqualTo(userTag.getId());
+		assertThat(result.getUser().getId()).isEqualTo(user.getId());
+		assertThat(result.getTag().getId()).isEqualTo(tag.getId());
+	}
+}
diff --git a/capturecat-core/src/test/java/com/capturecat/core/service/user/UserTagServiceTest.java b/capturecat-core/src/test/java/com/capturecat/core/service/user/UserTagServiceTest.java
@@ -3,9 +3,11 @@
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.ArgumentMatchers.anyLong;
 import static org.mockito.ArgumentMatchers.anyString;
 import static org.mockito.ArgumentMatchers.eq;
 import static org.mockito.BDDMockito.given;
+import static org.mockito.BDDMockito.willDoNothing;
 import static org.mockito.Mockito.never;
 import static org.mockito.Mockito.times;
 import static org.mockito.Mockito.verify;
@@ -21,6 +23,7 @@
 import com.capturecat.core.DummyObject;
 import com.capturecat.core.domain.tag.TagFixture;
 import com.capturecat.core.domain.tag.TagRegister;
+import com.capturecat.core.domain.tag.TagRepository;
 import com.capturecat.core.domain.user.UserRepository;
 import com.capturecat.core.domain.user.UserTagFixture;
 import com.capturecat.core.domain.user.UserTagRepository;
@@ -37,6 +40,9 @@ class UserTagServiceTest {
 	@Mock
 	UserTagRepository userTagRepository;
 
+	@Mock
+	TagRepository tagRepository;
+
 	@Mock
 	TagRegister tagRegister;
 
@@ -114,4 +120,79 @@ class UserTagServiceTest {
 
 		verify(userTagRepository, never()).save(any());
 	}
+
+	@Test
+	void 유저_태그를_수정한다() {
+		// given
+		var user = DummyObject.newUser("test");
+		var currentTag = TagFixture.createTag(1L, "java");
+		var newTag = TagFixture.createTag(2L, "spring");
+		var currentUserTag = UserTagFixture.createUserTag(1L, user, currentTag);
+		var newUserTag = UserTagFixture.createUserTag(2L, user, newTag);
+
+		given(userRepository.findByUsername(anyString())).willReturn(Optional.of(user));
+		given(tagRepository.findById(anyLong())).willReturn(Optional.of(currentTag));
+		given(userTagRepository.findByUserAndTag(eq(user), eq(currentTag))).willReturn(Optional.of(currentUserTag));
+		given(tagRegister.registerTagsFor(anyString())).willReturn(newTag);
+		given(userTagRepository.existsByUserAndTag(eq(user), eq(newTag))).willReturn(false);
+		willDoNothing().given(userTagRepository).delete(eq(currentUserTag));
+		given(userTagRepository.save(any())).willReturn(newUserTag);
+
+		// when
+		var response = userTagService.update(new LoginUser(user), 1L, "spring");
+
+		// then
+		assertThat(response.id()).isEqualTo(newUserTag.getId());
+		assertThat(response.name()).isEqualTo(newTag.getName());
+
+		verify(userTagRepository, times(1)).delete(eq(currentUserTag));
+		verify(userTagRepository, times(1)).save(any());
+	}
+
+	@Test
+	void 유저_태그_수정_시_회원이_존재하지_않으면_실패한다() {
+		// given
+		given(userRepository.findByUsername(anyString())).willReturn(Optional.empty());
+
+		// when & then
+		assertThatThrownBy(() -> userTagService.update(new LoginUser(DummyObject.newUser("test")), 1L, "spring"))
+			.isInstanceOf(CoreException.class)
+			.hasMessage(ErrorType.USER_NOT_FOUND.getCode().getMessage());
+
+		verify(userTagRepository, never()).delete(any());
+		verify(userTagRepository, never()).save(any());
+	}
+
+	@Test
+	void 유저_태그_수정_시_기존_태그가_존재하지_않으면_실패한다() {
+		// given
+		var user = DummyObject.newUser("test");
+
+		given(userRepository.findByUsername(anyString())).willReturn(Optional.of(user));
+		given(tagRepository.findById(anyLong())).willReturn(Optional.empty());
+
+		// when & then
+		assertThatThrownBy(() -> userTagService.update(new LoginUser(DummyObject.newUser("test")), 1L, "spring"))
+			.isInstanceOf(CoreException.class)
+			.hasMessage(ErrorType.TAG_NOT_FOUND.getCode().getMessage());
+	}
+
+	@Test
+	void 유저_태그_수정_시_기존_유저_태그가_존재하지_않으면_실패한다() {
+		// given
+		var user = DummyObject.newUser("test");
+		var currentTag = TagFixture.createTag(1L, "java");
+
+		given(userRepository.findByUsername(anyString())).willReturn(Optional.of(user));
+		given(tagRepository.findById(anyLong())).willReturn(Optional.of(currentTag));
+		given(userTagRepository.findByUserAndTag(eq(user), eq(currentTag))).willReturn(Optional.empty());
+
+		// when & then
+		assertThatThrownBy(() -> userTagService.update(new LoginUser(DummyObject.newUser("test")), 1L, "spring"))
+			.isInstanceOf(CoreException.class)
+			.hasMessage(ErrorType.USER_TAG_NOT_FOUND.getCode().getMessage());
+
+		verify(userTagRepository, never()).delete(any());
+		verify(userTagRepository, never()).save(any());
+	}
 }

-Original file line number
+Diff line change
@@ @@ -0,0 +1,4 @@ @@
 +package com.capturecat.core.api.user.dto;
++
 +public record TagRenameRequest(Long currentTagId, String newTagName) {
 +}