[♻️ refactor] PushToken 네이밍 변경 #37
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
BaseException과 Exception 처리만으로 충분하므로, UnsupportedOperationException, HttpMediaTypeNotSupportedException, SocketTimeoutException에 대한 별도 핸들러를 제거함.
- INVALID_USER_NAME 메시지에 공백 허용 여부를 명확히 표현
- 클래스명 PushToken → FcmToken 으로 명확히 표현 - 생성자, 팩토리 메서드 등 관련 메서드명 일괄 수정 - 예외 코드도 FCM_TOKEN_NOT_NULL 로 변경
- @Embedded 필드 타입을 PushToken → FcmToken으로 수정 - 클래스명 변경에 따른 필드 타입 일관성 유지
- FcmToken 클래스명 변경에 따라 테스트 클래스 및 내부 테스트 대상 수정 - 예외 메시지 검증 시 FCM_TOKEN_NOT_NULL 에 맞춰 assert 문 수정
- PUSH_TOKEN_NOT_NULL → FCM_TOKEN_NOT_NULL 로 수정하여 도메인 명확성 확보
- @DisplayName 어노테이션을 "푸시 토큰 테스트" → "FCM 토큰 테스트"로 변경
junggyo1020
approved these changes
Mar 27, 2025
Contributor
junggyo1020
left a comment
junggyo1020
left a comment
There was a problem hiding this comment.
근데 이전 PR들과 변경사항이 누적되서 나오는 것 같네요!! 피처브랜치에서 develop으로 개별적으로 PR이 올라와서 그런 것 같네요!! 형상관리에 대해 조금 더 같이 이야기를 나눠보면 좋을 것 같아요!
| INVALID_USER_NAME(HttpStatus.BAD_REQUEST, "이름은 한글, 영어, 숫자, 공백으로만 구성할 수 있습니다."), | ||
|
|
||
| PUSH_TOKEN_NOT_NULL(HttpStatus.BAD_REQUEST, "푸시 토큰은 null일 수 없습니다."), | ||
| FCM_TOKEN_NOT_NULL(HttpStatus.BAD_REQUEST, "FCM 토큰은 null일 수 없습니다."), |
Contributor
There was a problem hiding this comment.
조금 더 명확하게 표시된 것 같아서 좋은 것 같아요!
Member
Author
There was a problem hiding this comment.
네! 이번에 네이밍을 정리하면서 에러 메시지도 함께 수정해보았습니다 :) 더 명확하게 전달될 수 있어서 다행이에요.
감사합니다!
Member
Author
좋습니다~! 최대한 충돌 없이 관리하고 싶은데 쉽지 않네요 😅 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📄 Work Description
PushToken클래스명을FcmToken으로 변경User.token) 및 팩토리 메서드, 생성자 명칭 일괄 변경PUSH_TOKEN_NOT_NULL→FCM_TOKEN_NOT_NULL로 명확히 분리💬 To Reviewers
📷 Screenshot
⚙️ ISSUE
✅ PR check list