refactor: 카테고리, 일시 선택 순서 변경 및 리팩터링 (스타카토 생성/수정) #564 #567
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
변수명 변경 및 public 함수 리팩터링
- initMemoryAndVisitedAt() : memoryId에 따라 일시, 추억을 초기화 - setMemoryCandidateByVisitedAt() : 일시에 따라 추억을 설정
- initMemoryAndVisitedAt() : 스타카토에 따라 일시, 추억을 초기화 - setMemoryCandidateByVisitedAt() : 일시에 따라 추억을 설정
s6m1n
added
android
android/Staccato_AN/app/src/main/java/com/on/staccato/domain/model/MemoryCandidates.kt
Show resolved
Hide resolved
android/Staccato_AN/app/src/main/java/com/on/staccato/domain/model/MemoryCandidates.kt
Show resolved
Hide resolved
...in/java/com/on/staccato/presentation/staccatocreation/viewmodel/StaccatoCreationViewModel.kt
Show resolved
Hide resolved
hxeyexn
approved these changes
Jan 11, 2025
| // 2024년은 윤년이라 2월이 29일까지 있다 | ||
| val expected = (1..29).toList() | ||
| val actual = DateCalendar.of(2024, 2).availableDates | ||
| assertEquals(expected, actual) |
There was a problem hiding this comment.
CallAdapter가 머지된 후에 JUnit5로 마이그레이션해보는 것도 좋을 것 같네용!
Comment on lines
+72
to
+81
| fun `범위보다 작으면 가장 첫번째 월을 반환한다`(targetMonth: Int) { | ||
| val firstMonth = 4 | ||
| val actual = aprilToAugustCalendar.getClosestMonth(targetMonth) | ||
|
|
||
| assertEquals(firstMonth, actual) | ||
| } | ||
|
|
||
| @Test | ||
| @Parameters(method = "monthParametersGraterThan8") | ||
| fun `범위보다 크면 가장 마지막 월을 반환한다`(targetMonth: Int) { |
There was a problem hiding this comment.
어떤 범위인지 좀 더 구체적이면 테스트하고자하는 바가 확실히 전달될 것 같아요!
Junyoung-WON
approved these changes
Jan 11, 2025
Comment on lines
+20
to
+26
| @Test | ||
| fun `기간의 시작과 끝을 정하지 않으면 2025년 2월의 모든 일을 반환한다`() { | ||
| // 2025년은 2월 28일까지 있다 | ||
| val expected = (1..28).toList() | ||
| val actual = DateCalendar.of(2025, 2).availableDates | ||
| assertEquals(expected, actual) | ||
| } |
There was a problem hiding this comment.
윤년인 경우와 아닌 경우도 함께 테스트로 넣어주셨네요! 👍
| DateCalendar.of(2024, 2, periodStart, periodEnd) | ||
| } | ||
|
|
||
| @Test(expected = IllegalArgumentException::class) |
There was a problem hiding this comment.
어노테이션의 파라미터로 예외 발생을 테스트할 수 있군요...? 😮 👍
Ho-Tea
reviewed
Jan 12, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
⭐️ Issue Number
🚩 Summary
스타카토 생성/수정 화면의 카테고리, 일시 관련 로직에 아래의 작업들을 수행했습니다.
리팩터링 내용 정리
생성 화면
fetchMemoryCandidates()로 사용자의 모든 추억을 불러와_memoryCandidates를 업데이트 합니다._memoryCandidates는 최초 한 번 업데이트된 후엔 변하지 않습니다.memoryCandidates.observe람다에서initMemoryAndVisitedAt()를 수행하여 일시, 추억의 초기값을 세팅합니다.LocalDateTime.now()(선택 가능한 일시 범위는 selectedVisitedAt ±10년)updateMemoryCandidateAndVisitedAt()MemoryCandidate의getClosestDateTime()참고)수정 화면
fetchMemoryCandidates())fetchStaccatoBy())🛠️ Technical Concerns
스타카토 생성/수정 ViewModel 테스트
MVVM 구조의 뷰모델 내부 LiveData 값을 테스트하기 위해 아래 Rule을 설정하고 getOrAwaitValue를 추가했습니다.
@get:Rule var instantTaskExecutorRule = InstantTaskExecutorRule() @get:Rule val mainDispatcherRule = MainDispatcherRule()또한 mockK의 coEvery와 coroutines.test의 runTest를 사용했습니다.
🙂 To Reviewer