diff --git a/api/src/main/java/com/civilwar/boardsignal/user/presentation/UserApiController.java b/api/src/main/java/com/civilwar/boardsignal/user/presentation/UserApiController.java index 6a961edc..448fc599 100644 --- a/api/src/main/java/com/civilwar/boardsignal/user/presentation/UserApiController.java +++ b/api/src/main/java/com/civilwar/boardsignal/user/presentation/UserApiController.java @@ -66,10 +66,10 @@ public ResponseEntity getAnotherUserProfile( @Operation(summary = "닉네임 중복 체크 API") @ApiResponse(useReturnTypeSchema = true) - @PostMapping("/valid") + @GetMapping("/valid") public ResponseEntity validNickname( @Parameter(hidden = true) @AuthenticationPrincipal User loginUser, - @RequestBody ValidNicknameRequest validNicknameRequest + ValidNicknameRequest validNicknameRequest ) { ValidNicknameResponse response = userService.isExistNickname(validNicknameRequest, loginUser); diff --git a/api/src/test/java/com/civilwar/boardsignal/user/presentation/UserApiControllerTest.java b/api/src/test/java/com/civilwar/boardsignal/user/presentation/UserApiControllerTest.java index 9d7ec5f2..814246ac 100644 --- a/api/src/test/java/com/civilwar/boardsignal/user/presentation/UserApiControllerTest.java +++ b/api/src/test/java/com/civilwar/boardsignal/user/presentation/UserApiControllerTest.java @@ -41,6 +41,8 @@ import org.springframework.http.HttpMethod; import org.springframework.http.MediaType; import org.springframework.mock.web.MockMultipartFile; +import org.springframework.util.LinkedMultiValueMap; +import org.springframework.util.MultiValueMap; @DisplayName("[UserController 테스트]") class UserApiControllerTest extends ApiTestSupport { @@ -204,13 +206,15 @@ void getUserProfileTest2() throws Exception { void validNicknameTest1() throws Exception { //given String notExistName = "절대 중복된 이름 아님"; - ValidNicknameRequest validNicknameRequest = new ValidNicknameRequest(notExistName); //then - mockMvc.perform(post("/api/v1/users/valid") + MultiValueMap params = new LinkedMultiValueMap<>(); + params.add("nickname", notExistName); + + mockMvc.perform(get("/api/v1/users/valid") .header(AUTHORIZATION, accessToken) .contentType(MediaType.APPLICATION_JSON) - .content(toJson(validNicknameRequest))) + .params(params)) .andExpect(jsonPath("$.isNotValid").value(true)); } @@ -222,13 +226,15 @@ void validNicknameTest2() throws Exception { loginUser.updateUser(existName, List.of(Category.CUSTOMIZABLE), Gender.MALE, 2000, AgeGroup.TWENTY, "2호선", "사당역", "testURL"); userRepository.save(loginUser); - ValidNicknameRequest validNicknameRequest = new ValidNicknameRequest(existName); //then - mockMvc.perform(post("/api/v1/users/valid") + MultiValueMap params = new LinkedMultiValueMap<>(); + params.add("nickname", existName); + + mockMvc.perform(get("/api/v1/users/valid") .header(AUTHORIZATION, accessToken) .contentType(MediaType.APPLICATION_JSON) - .content(toJson(validNicknameRequest))) + .params(params)) .andExpect(jsonPath("$.isNotValid").value(true)); } @@ -241,13 +247,14 @@ void validNicknameTest3() throws Exception { String existName = anotherUser.getName(); - ValidNicknameRequest validNicknameRequest = new ValidNicknameRequest(existName); - //then - mockMvc.perform(post("/api/v1/users/valid") + MultiValueMap params = new LinkedMultiValueMap<>(); + params.add("nickname", existName); + + mockMvc.perform(get("/api/v1/users/valid") .header(AUTHORIZATION, accessToken) .contentType(MediaType.APPLICATION_JSON) - .content(toJson(validNicknameRequest))) + .params(params)) .andExpect(jsonPath("$.isNotValid").value(true)); } @@ -262,13 +269,13 @@ void validNicknameTest4() throws Exception { AgeGroup.TWENTY, "2호선", "사당역", "testURL"); userRepository.save(loginUser); - ValidNicknameRequest validNicknameRequest = new ValidNicknameRequest(existName); - //then - mockMvc.perform(post("/api/v1/users/valid") + MultiValueMap params = new LinkedMultiValueMap<>(); + params.add("nickname", existName); + mockMvc.perform(get("/api/v1/users/valid") .header(AUTHORIZATION, accessToken) .contentType(MediaType.APPLICATION_JSON) - .content(toJson(validNicknameRequest))) + .params(params)) .andExpect(jsonPath("$.isNotValid").value(true)); } }