refator: db와 Redis의 역할 분리, 현재 통합 테스트는 실패

Author: loseminho

src/main/java/com/back/domain/studyroom/dto/RoomMemberResponse.java

@@ -7,22 +7,39 @@
 
 import java.time.LocalDateTime;
 
+/**
+ * 방 멤버 정보 응답 DTO
+ isOnline 필드 제거: 이 DTO로 변환된 멤버는 이미 온라인 상태임을 의미
+ RoomService.getOnlineMembersWithWebSocket()에서 Redis 기반 필터링 후 변환
+ // Redis에서 온라인 사용자만 필터링
+ (Set)Long onlineUserIds = sessionManager.getOnlineUsersInRoom(roomId);
+
+ // 해당 사용자들의 멤버 정보 조회
+ (List)RoomMember onlineMembers = repository.findByRoomIdAndUserIdIn(roomId, onlineUserIds);
+
+ // DTO 변환 (이미 온라인인 멤버들만 변환됨)
+  (List)RoomMemberResponse response = onlineMembers.stream()
+      .map(RoomMemberResponse::from)
+      .collect(Collectors.toList());
+ */
 @Getter
 @Builder
 public class RoomMemberResponse {
     private Long userId;
     private String nickname;
     private RoomRole role;
-    private boolean isOnline;
     private LocalDateTime joinedAt;
     private LocalDateTime lastActiveAt;
 
+    /**
+     * RoomMember 엔티티를 응답 DTO로 변환
+     주의..) 이 메서드로 변환된 멤버는 온라인 상태로 인식되기 때문에 getOnlineMembersWithWebSocket()에서만 사용 해야함!!!!
+     */
     public static RoomMemberResponse from(RoomMember member) {
         return RoomMemberResponse.builder()
                 .userId(member.getUser().getId())
                 .nickname(member.getUser().getNickname())
                 .role(member.getRole())
-                .isOnline(member.isOnline())
                 .joinedAt(member.getJoinedAt())
                 .lastActiveAt(member.getLastActiveAt() != null ? member.getLastActiveAt() : member.getJoinedAt())
                 .build();

src/main/java/com/back/domain/studyroom/repository/RoomMemberRepository.java

@@ -4,12 +4,14 @@
 import org.springframework.data.jpa.repository.JpaRepository;
 import org.springframework.stereotype.Repository;
 
-import java.util.Optional;
-
+/**
+ * 현재.. )
+ findByConnectionId 메서드 제거 (connectionId 필드 제거로 인해)
+ 실시간 상태 관련 쿼리 메서드 제거
+ 커스텀 쿼리는 RoomMemberRepositoryCustom에서 관리
+ */
 @Repository
 public interface RoomMemberRepository extends JpaRepository<RoomMember, Long>, RoomMemberRepositoryCustom {
-    /**
-     * WebSocket 연결 ID로 멤버 조회
-     */
-    Optional<RoomMember> findByConnectionId(String connectionId);
+    // 기본 CRUD는 JpaRepository가 제공
+    // 커스텀 쿼리는 RoomMemberRepositoryCustom에 정의
 }

src/main/java/com/back/domain/studyroom/repository/RoomMemberRepositoryCustom.java

@@ -5,7 +5,14 @@
 
 import java.util.List;
 import java.util.Optional;
-
+import java.util.Set;
+
+/**
+ 멤버십 정보 조회 (영구 데이터)
+ 역할 기반 쿼리
+ 실시간 온라인 상태 (WebSocketSessionManager 사용)
+ 온라인 상태는 Redis(WebSocketSessionManager)에서 관리
+ */
 public interface RoomMemberRepositoryCustom {
 
     /**
@@ -19,19 +26,13 @@ public interface RoomMemberRepositoryCustom {
     List<RoomMember> findByRoomIdOrderByRole(Long roomId);
 
     /**
-     * 방의 온라인 멤버 조회
-     */
-    List<RoomMember> findOnlineMembersByRoomId(Long roomId);
-
-    /**
-     * 방의 활성 멤버 수 조회
-     */
-    int countActiveMembersByRoomId(Long roomId);
-
-    /**
-     * 사용자가 참여 중인 모든 방의 멤버십 조회
+     * 방의 멤버 중 특정 사용자 ID 목록에 해당하는 멤버만 조회
+     WebSocket에서 온라인 사용자 ID 목록을 받아와서 해당 멤버들의 상세 정보 조회
+     * @param roomId 방 ID
+     * @param userIds 조회할 사용자 ID 목록
+     * @return 해당하는 멤버 목록
      */
-    List<RoomMember> findActiveByUserId(Long userId);
+    List<RoomMember> findByRoomIdAndUserIdIn(Long roomId, Set<Long> userIds);
 
     /**
      * 특정 역할의 멤버 조회
@@ -62,14 +63,4 @@ public interface RoomMemberRepositoryCustom {
      * 특정 역할의 멤버 수 조회
      */
     int countByRoomIdAndRole(Long roomId, RoomRole role);
-
-    /**
-     * 방 퇴장 처리 (벌크 업데이트)
-     */
-    void leaveRoom(Long roomId, Long userId);
-
-    /**
-     * 방의 모든 멤버를 오프라인 처리 (방 종료 시)
-     */
-    void disconnectAllMembers(Long roomId);
 }