Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Test] Network모듈 테스트 #186

Open
wants to merge 11 commits into
base: develop
Choose a base branch
from
Open

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.

3 participants