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

Merged
merged 4 commits into from
Nov 12, 2024
Merged

Conversation

jinuemong
Copy link
Contributor

@jinuemong jinuemong commented Nov 6, 2024

이슈

개발 사항

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

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

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

@jinuemong jinuemong self-assigned this Nov 6, 2024
@jinuemong jinuemong added 🔎 test 테스트 🤖 android android labels Nov 6, 2024
Copy link

github-actions bot commented Nov 6, 2024

Test Results

77 tests   77 ✅  1s ⏱️
12 suites   0 💤
12 files     0 ❌

Results for commit ab66a24.

♻️ This comment has been updated with latest results.

Comment on lines 15 to 22
@BeforeEach
fun setUp() {
clubFilterSelector = ClubFilterSelector()
}

@Test
fun `필터를 추가하면 ClubFilterSelector에 해당 필터가 추가되어야 한다`() {
clubFilterSelector = ClubFilterSelector()
Copy link
Member

Choose a reason for hiding this comment

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

이부분은 코드가 중복되는 것 같아요!

@jinuemong jinuemong requested a review from junjange November 11, 2024 10:22
Copy link
Member

@junjange junjange left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~

Copy link
Contributor

@dpcks0509 dpcks0509 left a comment

Choose a reason for hiding this comment

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

테스트 고생하셨습니다~

@jinuemong jinuemong merged commit 6bc242f into develop Nov 12, 2024
2 checks passed
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.

클럽 필터 테스트 로직 작성
3 participants