diff --git a/backend/security b/backend/security index 2c704c60..902679b0 160000 --- a/backend/security +++ b/backend/security @@ -1 +1 @@ -Subproject commit 2c704c6046a2daab0ba9fdeba9f406c3b88d1009 +Subproject commit 902679b08bd39d4d116d020ae9382537074076fd diff --git a/backend/src/main/java/com/handong/rebon/config/WebConfig.java b/backend/src/main/java/com/handong/rebon/config/WebConfig.java index 3f672043..89736217 100644 --- a/backend/src/main/java/com/handong/rebon/config/WebConfig.java +++ b/backend/src/main/java/com/handong/rebon/config/WebConfig.java @@ -23,7 +23,7 @@ public class WebConfig implements WebMvcConfigurer { public void addCorsMappings(CorsRegistry registry) { registry.addMapping("/**") .allowedMethods("*") - .allowedOrigins("http://rebon.s3-website.ap-northeast-2.amazonaws.com/", "http://localhost:8080"); + .allowedOrigins("http://rebon.s3-website.ap-northeast-2.amazonaws.com/", "http://localhost:3000", "http://localhost:8080"); } @Override diff --git a/backend/src/main/java/com/handong/rebon/member/presentation/ApiMemberController.java b/backend/src/main/java/com/handong/rebon/member/presentation/ApiMemberController.java index 41e02aac..10c8f2fc 100644 --- a/backend/src/main/java/com/handong/rebon/member/presentation/ApiMemberController.java +++ b/backend/src/main/java/com/handong/rebon/member/presentation/ApiMemberController.java @@ -10,6 +10,7 @@ import com.handong.rebon.member.application.dto.response.MemberCreateResponseDto; import com.handong.rebon.member.application.dto.response.MemberReadResponseDto; import com.handong.rebon.member.presentation.dto.MemberAssembler; +import com.handong.rebon.member.presentation.dto.request.DuplicateRequest; import com.handong.rebon.member.presentation.dto.request.MemberCreateRequest; import com.handong.rebon.member.presentation.dto.request.MemberUpdateRequest; import com.handong.rebon.member.presentation.dto.response.MemberCreateResponse; @@ -35,13 +36,6 @@ public ResponseEntity create(@RequestBody MemberCreateRequ .body(MemberCreateResponse.from(response)); } - @PostMapping("/members/nickname/check-duplicate") - public ResponseEntity nicknameDuplicateCheck(@RequestBody String nickname) { - memberService.checkNicknameDuplicate(nickname); - return ResponseEntity.ok() - .build(); - } - @RequiredLogin @PatchMapping("/members") public ResponseEntity update( @@ -55,6 +49,13 @@ public ResponseEntity update( .build(); } + @PostMapping("/members/nickname/check-duplicate") + public ResponseEntity nicknameDuplicateCheck(@RequestBody DuplicateRequest duplicateRequest) { + memberService.checkNicknameDuplicate(duplicateRequest.getNickname()); + return ResponseEntity.ok() + .build(); + } + @RequiredLogin @GetMapping("/members") public ResponseEntity getMemberInfo(@Login LoginMember loginMember) { diff --git a/backend/src/main/java/com/handong/rebon/member/presentation/dto/request/DuplicateRequest.java b/backend/src/main/java/com/handong/rebon/member/presentation/dto/request/DuplicateRequest.java new file mode 100644 index 00000000..f59fef8f --- /dev/null +++ b/backend/src/main/java/com/handong/rebon/member/presentation/dto/request/DuplicateRequest.java @@ -0,0 +1,12 @@ +package com.handong.rebon.member.presentation.dto.request; + +import lombok.AllArgsConstructor; +import lombok.Getter; +import lombok.NoArgsConstructor; + +@NoArgsConstructor +@AllArgsConstructor +@Getter +public class DuplicateRequest { + private String nickname; +} diff --git a/backend/src/test/java/com/handong/rebon/acceptance/member/MemberCreateAcceptanceTest.java b/backend/src/test/java/com/handong/rebon/acceptance/member/MemberCreateAcceptanceTest.java index 6e680a8d..e74f7c0d 100644 --- a/backend/src/test/java/com/handong/rebon/acceptance/member/MemberCreateAcceptanceTest.java +++ b/backend/src/test/java/com/handong/rebon/acceptance/member/MemberCreateAcceptanceTest.java @@ -1,6 +1,7 @@ package com.handong.rebon.acceptance.member; import com.handong.rebon.acceptance.AcceptanceTest; +import com.handong.rebon.member.presentation.dto.request.DuplicateRequest; import com.handong.rebon.member.presentation.dto.request.MemberCreateRequest; import org.springframework.http.HttpStatus; @@ -42,8 +43,9 @@ void saveMember() { void checkDidNotDuplicateNickname() { //given String nickname = "ReBoN"; + DuplicateRequest duplicateRequest = new DuplicateRequest(nickname); //when - ExtractableResponse response = checkDuplicateNickname(nickname); + ExtractableResponse response = checkDuplicateNickname(duplicateRequest); //then assertThat(response.statusCode()).isEqualTo(HttpStatus.OK.value()); @@ -54,16 +56,17 @@ void checkDidNotDuplicateNickname() { void checkDuplicatedNickname() { //given String nickname = "test"; + DuplicateRequest duplicateRequest = new DuplicateRequest(nickname); saveMember(new MemberCreateRequest("test@gmail.com", nickname, "google", true)); //when - ExtractableResponse response = checkDuplicateNickname(nickname); + ExtractableResponse response = checkDuplicateNickname(duplicateRequest); //then assertThat(response.statusCode()).isEqualTo(HttpStatus.BAD_REQUEST.value()); } - private ExtractableResponse checkDuplicateNickname(String nickname) { + private ExtractableResponse checkDuplicateNickname(DuplicateRequest nickname) { return RestAssured.given(getRequestSpecification()) .log().all() .contentType(APPLICATION_JSON_VALUE)