Skip to content

Commit efc32cc

Browse files
authored
refactor: 예외 관련 테스트코드 함수명 수정 (#387)
* refactor: 예외 관련 테스트코드 함수명 수정 - 레이어드 아키텍쳐의 구조에 맞게, 컨트롤러 테스트가 아닌 테스트 코드에서 "응답"이라는 단어를 쓰지 않는다. * refactor: 누락한 용어 변경 반영
1 parent 1ca6126 commit efc32cc

31 files changed

+67
-67
lines changed

README.md

-10 Bytes

컨벤션

⚠️ 주의 !
이 컨벤션은 2024년 7월 지원까지의 코드에 해당합니다.
이후의 규칙은 자유롭게 만들어가주세요.
  • 함수의 이름은 동사구로 통일한다.

  • 어노테이션이 여러개 달리는 경우, 더 핵심이 되는 것을 아래에 위치한다. (개인적으로 어노테이션이 많아지면 코드와 가장 가까운 것에 제일 먼저 눈이 가기 때문)

  • 클래스의 바디가 시작할 때 개행하고, 바디가 끝나고 나서 개행한다. (구글 컨벤션에 의해)

  • 클래스에 있는 함수의 이름은 구체적으로 작성한다.

    • 예를 들어, UniversityController 에 있는 학교 검색 함수의 이름은 search 가 아니라 searchUniversity로 한다.
  • 테스트 코드에서 예외 발생 e2e 테스트는 이름을 "~면_예외_응답을_반환한다"로 통일한다.

  • 테스트 코드에서 예외 발생하는 테스트는 이름을 "~면_예외가_발생한다"로 통일한다.

src/test/java/com/example/solidconnection/admin/dto/validation/RejectedReasonValidatorTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ class GPA_점수_거절_사유_검증 {
5252
}
5353

5454
@Test
55-
void 거절_상태일_때_거절사유가_없으면_예외_응답을_반환한다() {
55+
void 거절_상태일_때_거절사유가_없으면_예외가_발생한다() {
5656
// given
5757
GpaScoreUpdateRequest request = new GpaScoreUpdateRequest(
5858
3.0,
@@ -92,7 +92,7 @@ class 어학_점수_거절_사유_검증 {
9292
}
9393

9494
@Test
95-
void 거절_상태일_때_거절사유가_없으면_예외_응답을_반환한다() {
95+
void 거절_상태일_때_거절사유가_없으면_예외가_발생한다() {
9696
// given
9797
LanguageTestScoreUpdateRequest request = new LanguageTestScoreUpdateRequest(
9898
LanguageTestType.TOEIC,

src/test/java/com/example/solidconnection/admin/service/AdminGpaScoreServiceTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,7 @@ class GPA_점수_검증_및_수정 {
163163
}
164164

165165
@Test
166-
void 존재하지_않는_GPA_수정_시_예외_응답을_반환한다() {
166+
void 존재하지_않는_GPA_수정_시_예외가_발생한다() {
167167
// given
168168
long invalidGpaScoreId = 9999L;
169169
GpaScoreUpdateRequest request = new GpaScoreUpdateRequest(

src/test/java/com/example/solidconnection/admin/service/AdminLanguageTestScoreServiceTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ class 어학점수_검증_및_수정 {
164164
}
165165

166166
@Test
167-
void 존재하지_않는_어학점수_수정_시_예외_응답을_반환한다() {
167+
void 존재하지_않는_어학점수_수정_시_예외가_발생한다() {
168168
// given
169169
long invalidLanguageTestScoreId = 9999L;
170170
LanguageTestScoreUpdateRequest request = new LanguageTestScoreUpdateRequest(

src/test/java/com/example/solidconnection/application/service/ApplicationSubmissionServiceTest.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ void setUp() {
102102
}
103103

104104
@Test
105-
void 미승인된_GPA_성적으로_지원하면_예외_응답을_반환한다() {
105+
void 미승인된_GPA_성적으로_지원하면_예외가_발생한다() {
106106
// given
107107
GpaScore gpaScore = gpaScoreFixture.GPA_점수(VerifyStatus.PENDING, user);
108108
LanguageTestScore languageTestScore = languageTestScoreFixture.어학_점수(VerifyStatus.APPROVED, user);
@@ -122,7 +122,7 @@ void setUp() {
122122
}
123123

124124
@Test
125-
void 미승인된_어학성적으로_지원하면_예외_응답을_반환한다() {
125+
void 미승인된_어학성적으로_지원하면_예외가_발생한다() {
126126
// given
127127
GpaScore gpaScore = gpaScoreFixture.GPA_점수(VerifyStatus.APPROVED, user);
128128
LanguageTestScore languageTestScore = languageTestScoreFixture.어학_점수(VerifyStatus.PENDING, user);
@@ -142,7 +142,7 @@ void setUp() {
142142
}
143143

144144
@Test
145-
void 지원서_수정_횟수를_초과하면_예외_응답을_반환한다() {
145+
void 지원서_수정_횟수를_초과하면_예외가_발생한다() {
146146
// given
147147
GpaScore gpaScore = gpaScoreFixture.GPA_점수(VerifyStatus.APPROVED, user);
148148
LanguageTestScore languageTestScore = languageTestScoreFixture.어학_점수(VerifyStatus.APPROVED, user);

src/test/java/com/example/solidconnection/auth/service/AuthServiceTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ class 토큰을_재발급한다 {
9696
}
9797

9898
@Test
99-
void 요청의_리프레시_토큰이_저장되어있지_않다면_예외_응답을_반환한다() {
99+
void 요청의_리프레시_토큰이_저장되어있지_않다면_예외가_발생한다() {
100100
// given
101101
String invalidRefreshToken = authTokenProvider.generateAccessToken(new Subject("subject")).token();
102102
ReissueRequest reissueRequest = new ReissueRequest(invalidRefreshToken);

src/test/java/com/example/solidconnection/auth/service/EmailSignInServiceTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ class EmailSignInServiceTest {
4848
class 로그인에_실패한다 {
4949

5050
@Test
51-
void 이메일과_일치하는_사용자가_없으면_예외_응답을_반환한다() {
51+
void 이메일과_일치하는_사용자가_없으면_예외가_발생한다() {
5252
// given
5353
EmailSignInRequest signInRequest = new EmailSignInRequest("이메일", "비밀번호");
5454

@@ -59,7 +59,7 @@ class 로그인에_실패한다 {
5959
}
6060

6161
@Test
62-
void 비밀번호가_일치하지_않으면_예외_응답을_반환한다() {
62+
void 비밀번호가_일치하지_않으면_예외가_발생한다() {
6363
// given
6464
String email = "testEmail";
6565
siteUserFixture.사용자(email, "testPassword");

src/test/java/com/example/solidconnection/auth/service/JwtTokenProviderTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ class 토큰으로부터_subject_를_추출한다 {
9090
}
9191

9292
@Test
93-
void 유효하지_않은_토큰의_subject_를_추출하면_예외_응답을_반환한다() {
93+
void 유효하지_않은_토큰의_subject_를_추출하면_예외가_발생한다() {
9494
// given
9595
String subject = "subject123";
9696
String token = createExpiredToken(subject);
@@ -125,7 +125,7 @@ class 토큰으로부터_claim_을_추출한다 {
125125
}
126126

127127
@Test
128-
void 유효하지_않은_토큰의_claim_을_추출하면_예외_응답을_반환한다() {
128+
void 유효하지_않은_토큰의_claim_을_추출하면_예외가_발생한다() {
129129
// given
130130
String subject = "subject";
131131
Claims expectedClaims = Jwts.claims().setSubject(subject);

src/test/java/com/example/solidconnection/auth/service/oauth/OAuthSignUpTokenProviderTest.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ class 주어진_회원가입_토큰을_검증한다 {
8080
}
8181

8282
@Test
83-
void 만료되었으면_예외_응답을_반환한다() {
83+
void 만료되었으면_예외가_발생한다() {
8484
// given
8585
String expiredToken = createExpiredToken();
8686

@@ -91,7 +91,7 @@ class 주어진_회원가입_토큰을_검증한다 {
9191
}
9292

9393
@Test
94-
void 정해진_형식에_맞지_않으면_예외_응답을_반환한다_jwt_가_아닌_토큰() {
94+
void 정해진_형식에_맞지_않으면_예외가_발생한다_jwt_가_아닌_토큰() {
9595
// given
9696
String notJwt = "not jwt";
9797

@@ -102,7 +102,7 @@ class 주어진_회원가입_토큰을_검증한다 {
102102
}
103103

104104
@Test
105-
void 정해진_형식에_맞지_않으면_예외_응답을_반환한다_authType_클래스_불일치() {
105+
void 정해진_형식에_맞지_않으면_예외가_발생한다_authType_클래스_불일치() {
106106
// given
107107
Map<String, Object> wrongClaim = new HashMap<>(Map.of(AUTH_TYPE_CLAIM_KEY, "카카오"));
108108
String wrongAuthType = createBaseJwtBuilder().addClaims(wrongClaim).compact();
@@ -114,7 +114,7 @@ class 주어진_회원가입_토큰을_검증한다 {
114114
}
115115

116116
@Test
117-
void 정해진_형식에_맞지_않으면_예외_응답을_반환한다_subject_누락() {
117+
void 정해진_형식에_맞지_않으면_예외가_발생한다_subject_누락() {
118118
// given
119119
Map<String, Object> claim = new HashMap<>(Map.of(AUTH_TYPE_CLAIM_KEY, AuthType.APPLE));
120120
String noSubject = createBaseJwtBuilder().addClaims(claim).compact();
@@ -126,7 +126,7 @@ class 주어진_회원가입_토큰을_검증한다 {
126126
}
127127

128128
@Test
129-
void 우리_서버에_발급된_토큰이_아니면_예외_응답을_반환한다() {
129+
void 우리_서버에_발급된_토큰이_아니면_예외가_발생한다() {
130130
// given
131131
Map<String, Object> validClaim = new HashMap<>(Map.of(AUTH_TYPE_CLAIM_KEY, AuthType.APPLE));
132132
String signUpToken = createBaseJwtBuilder().addClaims(validClaim).setSubject("email").compact();

src/test/java/com/example/solidconnection/common/exception/CustomAccessDeniedHandlerTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ void setUp() {
3636
}
3737

3838
@Test
39-
void 권한이_없는_사용자_접근시_403_예외_응답을_반환한다() throws IOException {
39+
void 권한이_없는_사용자_접근시_403_예외가_발생한다() throws IOException {
4040
// given
4141
AccessDeniedException accessDeniedException = new AccessDeniedException(ACCESS_DENIED.getMessage());
4242

0 commit comments

Comments
 (0)