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

[AN] test : 클럽 필터 로직 테스트 #734

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

Conversation

jinuemong
Copy link
Contributor

@jinuemong jinuemong commented Nov 6, 2024

이슈

개발 사항

  • 클럽 필터 로직에 대한 테스트를 작성했습니다.

전달 사항 (없으면 삭제해 주세요)

  • 테스트 중 논리 오류 발견 후 수정이 있는데요, ClubFilterSelector를 initClubFilter()로직과 함께 사용하지 않으면 currentSelectedFilters가 초기화 되지 않아서 null 값으로 저장되고 있었습니다.
  • -> null로 초기화 되는 경우 addFilter()에서 plus(1) 로직이 안먹어서 계속 null로 초기화 되는 문제 발생
  • 다행히 뷰모델에서 항상 initClubFilter() 로직과 함께 사용하여 오류가 없었지만, 테스트 중에 발견했습니다. (라이브 데이터 테스트 시 클럽 필터를 추가해도 null?.plus(newFilter)로 추가되지 않음)
  • 테스트 중 논리 오류 발견해서 수정했습니다.

Copy link

github-actions bot commented Nov 6, 2024

Test Results

71 tests   71 ✅  1s ⏱️
10 suites   0 💤
10 files     0 ❌

Results for commit 279bb28.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖 android android 🔎 test 테스트
Projects
None yet
Development

Successfully merging this pull request may close these issues.

클럽 필터 테스트 로직 작성
1 participant