Skip to content

Conversation

@futuremirae
Copy link
Collaborator

📌 관련 이슈 번호 ex) #이슈번호


📘 작업 유형

  • 신규 기능 추가
  • 리팩토링

📙 작업 내역 (구현 내용 및 작업 내역을 기재합니다.)

  • Network모듈의 DefaultNetworkProvider테스트 코드 작성
  • Network모듈의 DefaultOAuthNetworkProvider테스트 코드 작성

📋 체크리스트 (PR을 올리기 전에 스스로 확인해봐요!)

  • PR 제목에 작업 내용을 요약하여 기재했는가?
  • 코딩컨벤션을 준수하는가?
  • 내 코드에 대해 스스로 검토를 했는가?

📝 특이 사항 (Optional)

  • 이번에 테스트 코드를 작성하면서 에러처리를 자세히 하면 좋겠구나라는 것을 깨달았습니다.
  • DefaultOAuthNetworkProvider을 테스트하면서 refreshTokenError을 발생 시키는 테스트 코드를 작성하고 싶었는데 구현하지 못했습니다. accessToken은 nil을 넣으면 에러가 발생하여 테스트 코드 작성에 어려움이 없었습니다.
  • 그러나 refreshToken 은 nil을 넣어도 error 발생이 안되어, 해당 부분을 테스트 하는 방법을 더 생각해봐야 할 것 같습니다.

Comment on lines +12 to +18
final class MockTokenManager: TokenManager {
struct CallCount {
var isAccessTokenValid = 0
var loadAccessToken = 0
var loadRefreshToken = 0
var storeToken = 0
}
Copy link
Member

@Hansangjin98 Hansangjin98 Dec 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CallCount를 통해 호출만 테스트를 하려고 하신 것 같은데 테스트 과정에서 CallCount 체크가 이루어지는 것 같지 않네요!
또한 CallCount만 체크를 했을 때 올바른 테스트가 될지도 고민해봐야 할 것 같습니다.

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.

4 participants