test: 스터디룸 jwt 테스트 코드 통합

Author: loseminho

README.md

@@ -4,73 +4,10 @@
 <br/>
 <br/>
 
----
-
-# ✅ PR #1: JWT 인증 통합 완료
-
-## 📋 작업 요약
-스터디룸 API에서 하드코딩된 사용자 ID를 제거하고, JWT 인증 시스템을 완전히 통합했습니다.
-
-### 🔄 주요 변경 사항
-
-#### 1. SecurityConfig 수정
-- `/api/rooms/**` 경로의 `permitAll()` 제거
-- 모든 방 API가 이제 JWT 인증 필수
-
-#### 2. RoomController JWT 통합
-- `CurrentUser` 의존성 주입으로 실제 사용자 ID 추출
-- 하드코딩된 `Long currentUserId = 1L;` 제거 (10개 메서드)
-- 불필요한 `@RequestHeader("Authorization")` 파라미터 제거
-
-### 🔐 인증 흐름
-1. **클라이언트 요청**: Authorization 헤더에 "Bearer {token}" 전달
-2. **JwtAuthenticationFilter**: 토큰 추출 및 검증
-3. **Controller**: `CurrentUser.getUserId()`로 사용자 ID 획득
-4. **인증 실패 시**: 401 Unauthorized 자동 응답
-
-### 🧪 테스트 방법
-```bash
-# 1. 로그인하여 JWT 토큰 받기
-POST /api/auth/login
-{
-  "username": "user",
-  "password": "password"
-}
-
-# 2. 토큰으로 방 생성
-POST /api/rooms
-Authorization: Bearer {받은_토큰}
-{
-  "title": "테스트 방",
-  "isPrivate": false
-}
-
-# 3. 토큰 없이 요청 시 401 에러 확인
-POST /api/rooms  # ❌ 401 Unauthorized
-```
-
-### 📊 영향받는 API 엔드포인트
-| 엔드포인트 | 메서드 | 변경 사항 |
-|-----------|--------|----------|
-| `/api/rooms` | POST | JWT 인증 필수 |
-| `/api/rooms/{roomId}/join` | POST | JWT 인증 필수 |
-| `/api/rooms/{roomId}/leave` | POST | JWT 인증 필수 |
-| `/api/rooms` | GET | JWT 인증 필수 |
-| `/api/rooms/{roomId}` | GET | JWT 인증 필수 |
-| `/api/rooms/my` | GET | JWT 인증 필수 |
-| `/api/rooms/{roomId}` | PUT | JWT 인증 필수 |
-| `/api/rooms/{roomId}` | DELETE | JWT 인증 필수 |
-| `/api/rooms/{roomId}/members` | GET | JWT 인증 필수 |
-| `/api/rooms/popular` | GET | JWT 인증 필수 |
-
----
-<br/>
-<br/>
-
 # 개발 및 배포 프로세스 & Git 컨벤션 가이드
 해당 프로젝트는 `dev` 브랜치에서 개발하고, `main`브랜치에서 배포합니다. <br/> <br/>
 아래에 브랜치 전략, 커밋/PR 컨벤션, 워크플로우 전략, 브랜치 보호 규칙, 응답 데이터 및 예외처리 전략을 정리하였습니다. <br/> <br/>
-개발 전에 꼭 읽어봐주세요! 
+개발 전에 꼭 읽어봐주세요!
 <br/>
 <br/>
 
@@ -84,7 +21,7 @@ POST /api/rooms  # ❌ 401 Unauthorized
   - 안정화된 코드를 머지하여 배포
   - `dev` → `main` PR은 관리자 혹은 릴리즈 담당자만 생성 및 승인 가능
   - 직접 push 및 외부 PR 제한
-<br/>
+    <br/>
 
 ## 2. 커밋/PR 컨벤션
 
@@ -216,11 +153,11 @@ public ResponseEntity<RsData<Void>> handleCustomException(
 // 응답 데이터 예시 (json)
 
 {
-  "code": ErrorCode.code,
-  "message": ErrorCode.message,
-  "data": {...} or {null},
-  "success": false
-}
+        "code": ErrorCode.code,
+        "message": ErrorCode.message,
+        "data": {...} or {null},
+        "success": false
+        }
 ```
 
 <br/>

src/test/java/com/back/domain/studyroom/controller/RoomControllerTest.java

@@ -8,6 +8,7 @@
 import com.back.domain.user.entity.UserProfile;
 import com.back.domain.user.entity.UserStatus;
 import com.back.global.common.dto.RsData;
+import com.back.global.security.CurrentUser;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
@@ -32,12 +33,15 @@
 import static org.mockito.Mockito.verify;
 
 @ExtendWith(MockitoExtension.class)
-@DisplayName("RoomController 테스트")
+@DisplayName("RoomController 테스트 - JWT 인증 통합")
 class RoomControllerTest {
 
     @Mock
     private RoomService roomService;
 
+    @Mock
+    private CurrentUser currentUser;
+
     @InjectMocks
     private RoomController roomController;
 
@@ -75,12 +79,16 @@ void setUp() {
 
         // 테스트 멤버 생성
         testMember = RoomMember.createHost(testRoom, testUser);
+
+        // CurrentUser Mock 설정은 각 테스트에서 필요할 때만 설정
     }
 
     @Test
-    @DisplayName("방 생성 API 테스트")
+    @DisplayName("방 생성 API 테스트 - JWT 인증")
     void createRoom() {
         // given
+        given(currentUser.getUserId()).willReturn(1L);
+        
         CreateRoomRequest request = new CreateRoomRequest(
                 "테스트 방",
                 "테스트 설명",
@@ -95,59 +103,66 @@ void createRoom() {
                 anyBoolean(),
                 any(),
                 anyInt(),
-                anyLong()
+                eq(1L)
         )).willReturn(testRoom);
 
         // when
-        ResponseEntity<RsData<RoomResponse>> response = roomController.createRoom(request, "Bearer token");
+        ResponseEntity<RsData<RoomResponse>> response = roomController.createRoom(request);
 
         // then
         assertThat(response.getStatusCode()).isEqualTo(HttpStatus.CREATED);
         assertThat(response.getBody()).isNotNull();
         assertThat(response.getBody().isSuccess()).isTrue();
         assertThat(response.getBody().getData().getTitle()).isEqualTo("테스트 방");
 
+        verify(currentUser, times(1)).getUserId();
         verify(roomService, times(1)).createRoom(
                 anyString(),
                 anyString(),
                 anyBoolean(),
                 any(),
                 anyInt(),
-                anyLong()
+                eq(1L)
         );
     }
 
     @Test
-    @DisplayName("방 입장 API 테스트")
+    @DisplayName("방 입장 API 테스트 - JWT 인증")
     void joinRoom() {
         // given
+        given(currentUser.getUserId()).willReturn(1L);
+        
         JoinRoomRequest request = new JoinRoomRequest(null);
-        given(roomService.joinRoom(anyLong(), any(), anyLong())).willReturn(testMember);
+        given(roomService.joinRoom(eq(1L), any(), eq(1L))).willReturn(testMember);
 
         // when
-        ResponseEntity<RsData<JoinRoomResponse>> response = roomController.joinRoom(1L, request, "Bearer token");
+        ResponseEntity<RsData<JoinRoomResponse>> response = roomController.joinRoom(1L, request);
 
         // then
         assertThat(response.getStatusCode()).isEqualTo(HttpStatus.OK);
         assertThat(response.getBody()).isNotNull();
         assertThat(response.getBody().isSuccess()).isTrue();
 
-        verify(roomService, times(1)).joinRoom(anyLong(), any(), anyLong());
+        verify(currentUser, times(1)).getUserId();
+        verify(roomService, times(1)).joinRoom(eq(1L), any(), eq(1L));
     }
 
     @Test
-    @DisplayName("방 나가기 API 테스트")
+    @DisplayName("방 나가기 API 테스트 - JWT 인증")
     void leaveRoom() {
         // given
+        given(currentUser.getUserId()).willReturn(1L);
+        
         // when
-        ResponseEntity<RsData<Void>> response = roomController.leaveRoom(1L, "Bearer token");
+        ResponseEntity<RsData<Void>> response = roomController.leaveRoom(1L);
 
         // then
         assertThat(response.getStatusCode()).isEqualTo(HttpStatus.OK);
         assertThat(response.getBody()).isNotNull();
         assertThat(response.getBody().isSuccess()).isTrue();
 
-        verify(roomService, times(1)).leaveRoom(anyLong(), anyLong());
+        verify(currentUser, times(1)).getUserId();
+        verify(roomService, times(1)).leaveRoom(eq(1L), eq(1L));
     }
 
     @Test
@@ -174,29 +189,34 @@ void getRooms() {
     }
 
     @Test
-    @DisplayName("방 상세 정보 조회 API 테스트")
+    @DisplayName("방 상세 정보 조회 API 테스트 - JWT 인증")
     void getRoomDetail() {
         // given
-        given(roomService.getRoomDetail(anyLong(), anyLong())).willReturn(testRoom);
-        given(roomService.getRoomMembers(anyLong(), anyLong())).willReturn(Arrays.asList(testMember));
+        given(currentUser.getUserId()).willReturn(1L);
+        
+        given(roomService.getRoomDetail(eq(1L), eq(1L))).willReturn(testRoom);
+        given(roomService.getRoomMembers(eq(1L), eq(1L))).willReturn(Arrays.asList(testMember));
 
         // when
-        ResponseEntity<RsData<RoomDetailResponse>> response = roomController.getRoomDetail(1L, "Bearer token");
+        ResponseEntity<RsData<RoomDetailResponse>> response = roomController.getRoomDetail(1L);
 
         // then
         assertThat(response.getStatusCode()).isEqualTo(HttpStatus.OK);
         assertThat(response.getBody()).isNotNull();
         assertThat(response.getBody().isSuccess()).isTrue();
         assertThat(response.getBody().getData().getTitle()).isEqualTo("테스트 방");
 
-        verify(roomService, times(1)).getRoomDetail(anyLong(), anyLong());
-        verify(roomService, times(1)).getRoomMembers(anyLong(), anyLong());
+        verify(currentUser, times(1)).getUserId();
+        verify(roomService, times(1)).getRoomDetail(eq(1L), eq(1L));
+        verify(roomService, times(1)).getRoomMembers(eq(1L), eq(1L));
     }
 
     @Test
-    @DisplayName("내 참여 방 목록 조회 API 테스트")
+    @DisplayName("내 참여 방 목록 조회 API 테스트 - JWT 인증")
     void getMyRooms() {
         // given
+        given(currentUser.getUserId()).willReturn(1L);
+        
         // Room에 ID 설정 (리플렉션 사용)
         try {
             java.lang.reflect.Field idField = testRoom.getClass().getSuperclass().getDeclaredField("id");
@@ -206,11 +226,11 @@ void getMyRooms() {
             throw new RuntimeException(e);
         }
 
-        given(roomService.getUserRooms(anyLong())).willReturn(Arrays.asList(testRoom));
-        given(roomService.getUserRoomRole(eq(1L), anyLong())).willReturn(RoomRole.HOST);
+        given(roomService.getUserRooms(eq(1L))).willReturn(Arrays.asList(testRoom));
+        given(roomService.getUserRoomRole(eq(1L), eq(1L))).willReturn(RoomRole.HOST);
 
         // when
-        ResponseEntity<RsData<List<MyRoomResponse>>> response = roomController.getMyRooms("Bearer token");
+        ResponseEntity<RsData<List<MyRoomResponse>>> response = roomController.getMyRooms();
 
         // then
         assertThat(response.getStatusCode()).isEqualTo(HttpStatus.OK);
@@ -219,13 +239,16 @@ void getMyRooms() {
         assertThat(response.getBody().getData()).hasSize(1);
         assertThat(response.getBody().getData().get(0).getTitle()).isEqualTo("테스트 방");
 
-        verify(roomService, times(1)).getUserRooms(anyLong());
+        verify(currentUser, times(1)).getUserId();
+        verify(roomService, times(1)).getUserRooms(eq(1L));
     }
 
     @Test
-    @DisplayName("방 설정 수정 API 테스트")
+    @DisplayName("방 설정 수정 API 테스트 - JWT 인증")
     void updateRoom() {
         // given
+        given(currentUser.getUserId()).willReturn(1L);
+        
         UpdateRoomSettingsRequest request = new UpdateRoomSettingsRequest(
                 "변경된 제목",
                 "변경된 설명",
@@ -236,48 +259,54 @@ void updateRoom() {
         );
 
         // when
-        ResponseEntity<RsData<Void>> response = roomController.updateRoom(1L, request, "Bearer token");
+        ResponseEntity<RsData<Void>> response = roomController.updateRoom(1L, request);
 
         // then
         assertThat(response.getStatusCode()).isEqualTo(HttpStatus.OK);
         assertThat(response.getBody()).isNotNull();
         assertThat(response.getBody().isSuccess()).isTrue();
 
+        verify(currentUser, times(1)).getUserId();
         verify(roomService, times(1)).updateRoomSettings(
-                anyLong(),
+                eq(1L),
                 anyString(),
                 anyString(),
                 anyInt(),
                 anyBoolean(),
                 anyBoolean(),
                 anyBoolean(),
-                anyLong()
+                eq(1L)
         );
     }
 
     @Test
-    @DisplayName("방 종료 API 테스트")
+    @DisplayName("방 종료 API 테스트 - JWT 인증")
     void deleteRoom() {
         // given
+        given(currentUser.getUserId()).willReturn(1L);
+        
         // when
-        ResponseEntity<RsData<Void>> response = roomController.deleteRoom(1L, "Bearer token");
+        ResponseEntity<RsData<Void>> response = roomController.deleteRoom(1L);
 
         // then
         assertThat(response.getStatusCode()).isEqualTo(HttpStatus.OK);
         assertThat(response.getBody()).isNotNull();
         assertThat(response.getBody().isSuccess()).isTrue();
 
-        verify(roomService, times(1)).terminateRoom(anyLong(), anyLong());
+        verify(currentUser, times(1)).getUserId();
+        verify(roomService, times(1)).terminateRoom(eq(1L), eq(1L));
     }
 
     @Test
-    @DisplayName("방 멤버 목록 조회 API 테스트")
+    @DisplayName("방 멤버 목록 조회 API 테스트 - JWT 인증")
     void getRoomMembers() {
         // given
-        given(roomService.getRoomMembers(anyLong(), anyLong())).willReturn(Arrays.asList(testMember));
+        given(currentUser.getUserId()).willReturn(1L);
+        
+        given(roomService.getRoomMembers(eq(1L), eq(1L))).willReturn(Arrays.asList(testMember));
 
         // when
-        ResponseEntity<RsData<List<RoomMemberResponse>>> response = roomController.getRoomMembers(1L, "Bearer token");
+        ResponseEntity<RsData<List<RoomMemberResponse>>> response = roomController.getRoomMembers(1L);
 
         // then
         assertThat(response.getStatusCode()).isEqualTo(HttpStatus.OK);
@@ -286,7 +315,8 @@ void getRoomMembers() {
         assertThat(response.getBody().getData()).hasSize(1);
         assertThat(response.getBody().getData().get(0).getNickname()).isEqualTo("테스트유저");
 
-        verify(roomService, times(1)).getRoomMembers(anyLong(), anyLong());
+        verify(currentUser, times(1)).getUserId();
+        verify(roomService, times(1)).getRoomMembers(eq(1L), eq(1L));
     }
 
     @Test