Feat: 스터디룸 채팅 구현 마무리 (#48) #61
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
- REST API에 @AuthenticationPrincipal 적용 - WebSocket에서 Principal로 사용자 정보 추출
- RoomChatMessageRepositoryCustom 인터페이스 구현 - QueryDSL로 동적 쿼리 및 Fetch Join 적용 - before 파라미터 완전 지원 (기존 TODO 해결)
1. 문제 원인 - @SuperBuilder는 필드 초기값(ex. new ArrayList<>())을 무시하여, 빌더로 생성된 객체의 컬렉션 필드가 null이 되는 문제가 있었음. - 이로 인해 테스트 환경에서 NullPointerException 및 DB ConstraintViolationException (NOT NULL 제약 조건 위반)이 연쇄적으로 발생함. - 일부 테스트에서는 엔티티 필드명(userProfile)과 다른 이름(userProfiles)을 리플렉션으로 참조하여 NoSuchFieldException이 발생함. 2. 해결 방법 - @Builder.Default 적용 : Room, User 등 @SuperBuilder를 사용하는 모든 엔티티의 컬렉션 필드 및 초기값을 가진 필드에 @Builder.Default 어노테이션을 추가함. 이를 통해 빌더 사용 시에도 필드가 null이 아닌 기본값으로 안전하게 초기화되도록 보장함. - 테스트 코드 수정 : ChatWebSocketControllerTest에서 User의 userProfile 필드를 잘못된 이름("userProfiles")으로 참조하던 리플렉션 코드를 수정함.
namgigun
approved these changes
Sep 24, 2025
KSH0326
approved these changes
Sep 24, 2025
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.
📌 개요
스터디룸 채팅 기능의 안정성과 효율성을 높이기 위해 JWT 인증을 도입하고, 메시지 조회 로직을 QueryDSL로 개선했습니다.
기능 변경에 따른 테스트 코드와 API 문서를 추가하여 완성도를 높였습니다.
🔨 작업 내용
1. JWT 인증 로직 추가
@AuthenticationPrincipal을 적용하여 인증된 사용자만 API를 호출할 수 있도록 변경.Principal객체에서 사용자 정보를 추출하여, 인증된 사용자의 채팅 메시지만 처리하도록 구현.2. QueryDSL을 이용한 조회 성능 개선
before파라미터를 이용한 이전 메시지 조회(무한 스크롤) 기능 구현. (기존 TODO 해결)3. 기타 개선 사항
Swagger어노테이션을 추가하여 API 명세를 명확하게 정리.🔗 관련 이슈
Closes #48
📝 참고 사항
향후 작업 예정
✅ 체크리스트