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

domain layer 테스트 코드 임시로 작성 #99

Merged
merged 5 commits into from
Dec 16, 2022
Merged

Conversation

soopeach
Copy link
Member

@soopeach soopeach commented Dec 13, 2022

😎 작업 내용

  • domain layer 테스트 코드 임시로 작성
    • GetGroupInfoUseCase()

🧐 변경된 내용

  • 이 PR 자체에서 변경된 내용이 있음.
    • 커밋 내역 중 테스트 코드(임시) 부분은 Fake model, repository를 직접 생성해주었음.
    • 그후 올린 커밋에는 mockito를 사용하여 mocked 객체를 만들어서 활용해보았음.

🥳 동작 화면

  • 동작 화면 없음

🤯 이슈 번호

  • 이슈 없음

🥲 비고

  • 발표자료를 위해 안드로이드 의존성이 없는 domain layer의 useCase 중 하나를 선정하여 임시로 테스트 코드를 작성하였습니다.
    • 안드로이드 의존성을 제거해서 테스트 코드 작성을 용이하게 해준다.
      라는 항목인데... 처음부터 안드로이드 의존성을 제거해서 아직 직접적으로 큰 이점을 느껴지지 않는 것 같습니다.
  • 테스트 코드에서 코루틴을 사용하기 위해 runTest 스코프를 사용하였습니다.
  • Fake Model, Fake Repository를 막 구현해서 추후 공식 가이드라인에 따라 개선하면 좋을 것 같습니다.
  • 테스트 코드에서도 메모리를 신경써줘야 하나 라는 생각이 드는데
    우선은 FakeGroupInfo는 지연 생성되는 싱글턴 객체로 만들었고,
    FakeGroupRepository는 그냥 최상위 변수에 GroupRepository인터페이스 객체를 임시로 구현했습니다.
  • 테스트 코드 관련 질문에 대한 지민님의 답변

Screenshot 2022-12-16 at 11 01 42 AM

  • Domain Layer의 UseCase를 테스트하면서 느낀건데 우리의 UseCase는 복잡한 비즈니스 로직이 아니라서 굳이 테스트할 필요가 없는 것 같음

@soopeach soopeach merged commit 37561b9 into develop Dec 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants