Skip to content

Conversation

@joowojr
Copy link
Contributor

@joowojr joowojr commented Feb 9, 2025

📄 요약(Summary)

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • AuthService 테스트 코드 작성
    • 로그인 성공/ 로그인 실패/ 임시 토큰 발급
  • LoginAttemptService 테스트 코드 작성
    • 로그인 실패 시 실패 기록 저장
    • 기존의 실패 기록 갱신
    • 5회 실패 시에 계정 잠김
    • 5회 실패가 아닐 시에 계정 잠금처리 X
    • 잠긴 계정에 대해 로그인 시도시에 AUTH_012 예외가 발생/ 30분 기한이 지날 시에는 AUTH_012 예외가 발생 X

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

첨부 자료

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

✅ 체크리스트(Checklist)

  • PR 양식에 맞게 작성했습니다
  • 모든 테스트가 통과했습니다
  • 프로그램이 정상적으로 작동합니다
  • 적절한 PR 라벨을 설정했습니다
  • 불필요한 코드를 제거했습니다

🚪 이슈 번호(Issue numbers)

Closes #75

@joowojr joowojr added the 🧪 test 테스트 코드를 추가 및 변경합니다 label Feb 9, 2025
@joowojr joowojr self-assigned this Feb 9, 2025
@joowojr joowojr merged commit 2230a3d into develop Feb 9, 2025
1 check passed
@joowojr joowojr deleted the CLAP-112 branch February 9, 2025 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🧪 test 테스트 코드를 추가 및 변경합니다

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLAP-112 인증 인가 기능에 대한 단위 테스트 코드 추가

2 participants