[Feat #26] JWT 관리(저장/재발급) 및 로그아웃 API

[dudxo]

관련 이슈

• close ✨JWT 토큰 관리 및 로그아웃 API #26

📑 작업 상세 내용

• 로그인 시 발급 되는 RefreshToken 관리(Redis 저장)
• 로그아웃 API
  • Redis 저장된 RefreshToken 삭제
  • 발급되어 있는 AccessToken BlackList 지정
• AccessToken 재발급 API
  • 로그아웃 또는 만료된 Access/RefreshToken 검증
  • Access/Refresh Token 재발급
  • Redis Refresh 업데이트

💫 작업 요약

• 로그아웃 API
• 로그아웃 API 테스트
• 토큰 재발급 API
• 토큰 재발급 API 테스트

🔍 중점적으로 리뷰 할 부분

• TokenProvider.class <-> MemberService.class에서 순환참조가 발생해서 임시방편으로 TokenProvider.class에서 MemberService가 아닌 MemberRepository를 의존하도록 변경했습니다.
• 로그아웃/토큰 재발급 API 단위 테스트를 거의 다 Mocking해서 테스트 했는데 이게 맞는가 싶네요,, 다 가짜로 주입하면 이게 성공할 수 밖에 없는 구조아닌가 싶어요,,

[dudxo]

헉 컨트롤러 통합 테스트를 작성 안했네요! 추가 작성해서 커밋하겠습니다,,

[github-actions]

Test Results

43 tests   37 ✅  5s ⏱️
14 suites   6 💤
14 files     0 ❌

Results for commit d40766d.

♻️ This comment has been updated with latest results.

[dudxo]

헉 컨트롤러 통합 테스트를 작성 안했네요! 추가 작성해서 커밋하겠습니다,,

커밋 완료했습니다!

[hyun2371]

고생하셨습니다~

[hyun2371]

로그아웃/토큰 재발급 API 단위 테스트를 거의 다 Mocking해서 테스트 했는데 이게 맞는가 싶네요,, 다 가짜로 주입하면 이게 성공할 수 밖에 없는 구조아닌가 싶어요,,

서비스 내부 로직이 다 호출되는지만 확인하면 되지 않을까요? 통합 테스트에서 확인해도 괜찮을 것 같습니다!!

[dudxo]

로그아웃/토큰 재발급 API 단위 테스트를 거의 다 Mocking해서 테스트 했는데 이게 맞는가 싶네요,, 다 가짜로 주입하면 이게 성공할 수 밖에 없는 구조아닌가 싶어요,,

서비스 내부 로직이 다 호출되는지만 확인하면 되지 않을까요? 통합 테스트에서 확인해도 괜찮을 것 같습니다!!

그렇겠죠? 감사합니다 ㅎㅎ

[hyun2371]

public static Provider fromSocialEmail(String socialEmail){
for (Provider provider : values()) {
  if (socialEmail.contains(provider.getProvider())) {
  return provider;
  }
}
throw new NotFoundException(AuthErrorCode.NOT_FOUND_PROVIDER);
}

enum 안에 함수를 넣어도 괜찮을 것 같아요!

[dudxo]

책임 분배를 생각 못했네요! 반복문 + 조건식보다 스트림이 더 좋겠다고 생각되서 스트림 방식으로 변경했습니다~
추가적으로 Provider Test 코드도 작성했습니당