modify: 테스트 코드 피드백 반영, 리펙토링 (가독성, 가시성)

Author: Nuung

src/middlewares/__test__/auth.middleware.test.ts

@@ -147,7 +147,7 @@ describe('인증 미들웨어', () => {
       );
     });
 
-    it('사용자를 찾을 수 없으면 next를 호출해야 한다', async () => {
+    it('사용자를 찾을 수 없으면 DBError가 발생해야 한다', async () => {
       // 유효한 토큰 준비
       mockRequest.cookies = {
         'access_token': validToken,
@@ -169,9 +169,15 @@ describe('인증 미들웨어', () => {
       // 검증
       expect(nextFunction).toHaveBeenCalledTimes(1);
       expect(mockRequest.user).toBeUndefined();
+      expect(nextFunction).toHaveBeenCalledWith(
+        expect.objectContaining({
+          name: 'DBError',
+          message: '사용자를 찾을 수 없습니다.'
+        })
+      );
     });
 
-    it('쿠키 대신 요청 본문에서 토큰을 추출해야 한다', async () => {
+    it('쿠키에 토큰이 없으면 헤더에서 토큰을 가져와야 한다', async () => {
       // 요청 본문에 토큰 설정
       mockRequest.body = {
         accessToken: validToken,
@@ -203,38 +209,5 @@ describe('인증 미들웨어', () => {
       expect(nextFunction).not.toHaveBeenCalledWith(expect.any(Error));
       expect(mockRequest.user).toEqual(mockUser);
     });
-
-    it('쿠키와 요청 본문 대신 헤더에서 토큰을 추출해야 한다', async () => {
-      // 헤더에 토큰 설정
-      mockRequest.headers = {
-        'access_token': validToken,
-        'refresh_token': 'refresh-token'
-      };
-
-      // 사용자 정보 mock
-      const mockUser = {
-        id: 1,
-        username: 'testuser',
-        email: 'test@example.com',
-        velog_uuid: 'c7507240-093b-11ea-9aae-a58a86bb0520'
-      };
-
-      // DB 쿼리 결과 모킹
-      (pool.query as jest.Mock).mockResolvedValueOnce({
-        rows: [mockUser]
-      });
-
-      // 미들웨어 실행
-      await authMiddleware.verify(
-        mockRequest as Request,
-        mockResponse as Response,
-        nextFunction
-      );
-
-      // 검증
-      expect(nextFunction).toHaveBeenCalledTimes(1);
-      expect(nextFunction).not.toHaveBeenCalledWith(expect.any(Error));
-      expect(mockRequest.user).toEqual(mockUser);
-    });
   });
 });

src/repositories/__test__/qr.repo.test.ts

@@ -72,6 +72,7 @@ describe('UserRepository - QR Login Token', () => {
     });
 
     it('토큰이 존재하지 않으면 null을 반환해야 한다', async () => {
+      // 참고로 존재하지 않거나, 만료된 토큰 밖에 없거나, 이미 사용된 토큰은 모두 "null 이 되는 것임"
       (mockPool.query as jest.Mock).mockResolvedValueOnce({ rows: [] });
 
       const result = await repo.findQRLoginToken('token');
@@ -102,14 +103,15 @@ describe('UserRepository - QR Login Token', () => {
 
   describe('updateQRLoginTokenToUse', () => {
     it('유저 ID로 토큰을 사용 처리해야 한다', async () => {
+      const targetUserId = 1;
       (mockPool.query as jest.Mock).mockResolvedValueOnce(undefined);
 
-      await expect(repo.updateQRLoginTokenToUse(1)).resolves.not.toThrow();
+      await expect(repo.updateQRLoginTokenToUse(targetUserId)).resolves.not.toThrow();
 
-      expect(mockPool.query).toHaveBeenCalledTimes(1);
+      expect(mockPool.query).toHaveBeenCalledTimes(targetUserId);
       expect(mockPool.query).toHaveBeenCalledWith(
         expect.stringContaining('UPDATE users_qrlogintoken'),
-        [1]
+        [targetUserId]
       );
     });