From 880f179c63cc85003fbc64a79db7cdd93fffa320 Mon Sep 17 00:00:00 2001 From: kunsanglee Date: Thu, 26 Sep 2024 14:13:30 +0900 Subject: [PATCH] =?UTF-8?q?refactor:=20API=20=EB=AA=85=EC=84=B8=EC=84=9C?= =?UTF-8?q?=EC=97=90=20=EC=9E=91=EC=84=B1=EB=90=9C=20=EC=98=88=EC=99=B8=20?= =?UTF-8?q?=EC=B5=9C=EC=8B=A0=ED=99=94=20=EB=B0=8F=20=EC=BD=94=EB=93=9C=20?= =?UTF-8?q?=EC=88=98=EC=A0=95?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../server/haengdong/exception/HaengdongErrorCode.java | 7 +++---- .../presentation/request/BillDetailUpdateRequest.java | 1 - .../haengdong/presentation/request/MemberSaveRequest.java | 8 +++++++- .../server/haengdong/application/MemberServiceTest.java | 4 ++-- 4 files changed, 12 insertions(+), 8 deletions(-) diff --git a/server/src/main/java/server/haengdong/exception/HaengdongErrorCode.java b/server/src/main/java/server/haengdong/exception/HaengdongErrorCode.java index 1fcebd225..a59288f4b 100644 --- a/server/src/main/java/server/haengdong/exception/HaengdongErrorCode.java +++ b/server/src/main/java/server/haengdong/exception/HaengdongErrorCode.java @@ -14,8 +14,8 @@ public enum HaengdongErrorCode { BANK_NAME_INVALID("지원하지 않는 은행입니다. 지원하는 은행 목록: %s"), ACCOUNT_LENGTH_INVALID("계좌번호는 %d자 이상 %d자 이하만 입력 가능합니다."), - MEMBER_NAME_LENGTH_INVALID("멤버 이름은 %d자 이상 %d자 이하만 입력 가능합니다."), - MEMBER_NAME_DUPLICATE("중복된 행사 참여 인원 이름이 존재합니다."), + MEMBER_NAME_LENGTH_INVALID("참여자 이름은 %d자 이상 %d자 이하만 입력 가능합니다."), + MEMBER_NAME_DUPLICATE("행사에 중복된 참여자 이름이 존재합니다."), MEMBER_NOT_FOUND("존재하지 않는 참여자입니다."), MEMBER_ALREADY_EXIST("현재 참여하고 있는 인원이 존재합니다."), MEMBER_NAME_CHANGE_DUPLICATE("중복된 참여 인원 이름 변경 요청이 존재합니다."), @@ -27,14 +27,13 @@ public enum HaengdongErrorCode { BILL_DETAIL_NOT_FOUND("존재하지 않는 참여자 지출입니다."), BILL_PRICE_NOT_MATCHED("지출 총액이 일치하지 않습니다."), - DIFFERENT_STEP_MEMBERS("회원 목록이 일치하지 않습니다."), + DIFFERENT_STEP_MEMBERS("참여자 목록이 일치하지 않습니다."), /* Authentication */ PASSWORD_INVALID("비밀번호가 일치하지 않습니다."), TOKEN_NOT_FOUND("토큰이 존재하지 않습니다."), - TOKEN_EXPIRED("만료된 토큰입니다."), TOKEN_INVALID("유효하지 않은 토큰입니다."), FORBIDDEN("접근할 수 없는 행사입니다."), diff --git a/server/src/main/java/server/haengdong/presentation/request/BillDetailUpdateRequest.java b/server/src/main/java/server/haengdong/presentation/request/BillDetailUpdateRequest.java index accaf8e00..de670f28f 100644 --- a/server/src/main/java/server/haengdong/presentation/request/BillDetailUpdateRequest.java +++ b/server/src/main/java/server/haengdong/presentation/request/BillDetailUpdateRequest.java @@ -11,7 +11,6 @@ public record BillDetailUpdateRequest( @NotNull(message = "지출 금액은 공백일 수 없습니다.") Long price, - @NotNull(message = "지출 금액은 공백일 수 없습니다.") boolean isFixed ) { public BillDetailUpdateAppRequest toAppRequest() { diff --git a/server/src/main/java/server/haengdong/presentation/request/MemberSaveRequest.java b/server/src/main/java/server/haengdong/presentation/request/MemberSaveRequest.java index f1d75e1b1..8b763be09 100644 --- a/server/src/main/java/server/haengdong/presentation/request/MemberSaveRequest.java +++ b/server/src/main/java/server/haengdong/presentation/request/MemberSaveRequest.java @@ -1,4 +1,10 @@ package server.haengdong.presentation.request; -public record MemberSaveRequest(String name) { +import jakarta.validation.constraints.NotBlank; + +public record MemberSaveRequest( + + @NotBlank(message = "참여자 이름은 공백일 수 없습니다.") + String name +) { } diff --git a/server/src/test/java/server/haengdong/application/MemberServiceTest.java b/server/src/test/java/server/haengdong/application/MemberServiceTest.java index 767dde440..099326317 100644 --- a/server/src/test/java/server/haengdong/application/MemberServiceTest.java +++ b/server/src/test/java/server/haengdong/application/MemberServiceTest.java @@ -120,7 +120,7 @@ void saveMembersTest2() { assertThatThrownBy(() -> memberService.saveMembers(event.getToken(), request)) .isInstanceOf(HaengdongException.class) - .hasMessageContaining("중복된 행사 참여 인원 이름이 존재합니다."); + .hasMessageContaining("행사에 중복된 참여자 이름이 존재합니다."); } @DisplayName("행사 참여 인원을 삭제한다.") @@ -329,7 +329,7 @@ void updateMembersTest6() { assertThatThrownBy(() -> memberService.updateMembers(event.getToken(), membersUpdateAppRequest)) .isInstanceOf(HaengdongException.class) - .hasMessage("중복된 행사 참여 인원 이름이 존재합니다."); + .hasMessage("행사에 중복된 참여자 이름이 존재합니다."); } @DisplayName("행사에 참여한 전체 인원을 조회한다.")