Repository navigation
androidTest 도메인 API 드리프트 44건+런타임 7건 수정, PR 컴파일 게이트 추가 - #949
Merged
Merged
Conversation
#861 로 들어온 androidTest 직후 배치 머지(#877→#867→#862→#864→#865)가 도메인 API 를 바꿨지만 androidTest 소스셋은 어떤 PR 검증도 컴파일하지 않아 컴파일 에러 44건이 develop 에 미검증 착륙했다(#936). 컴파일 44건 정리: - 수신자 홈 count API 제거(#865) → MindRecordReceiverRepository.getAll() + ReceiverTimeLetterRepository.getReceivedTimeLetters() 구조로 홈 테스트 재작성, 실패 섹션 문구("기록 수를 불러오지 못했습니다") 단언 포함 - author ListItem.id·홈 내비 콜백 String→Long - SettingViewModel.withdrawCompleted(Boolean) → WithdrawUiState 치환 - ReceivedAfternoteDetailRoute 의 onNavigateToPlaylist 필수화 반영 에뮬레이터 실측이 컴파일로는 안 잡히는 런타임 드리프트 7건을 추가 적발해 함께 수정: - SavedStateHandle 에 String id 주입 5건 — ViewModel 이 toRoute 로 Long 을 읽어 런타임 크래시. Long 주입으로 정정 - 마스터 키 입력이 UUID 형식 게이트(#887)에 서버 호출 전 반려 1건 — 유효 UUID 로 교체, 소문자 정규화 검증 겸함 - 탈퇴 확인 1건 — "완료 안내 후 확인 시 호출"이 "문장 일치 시 즉시 호출·결과 팝업" 구조로 바뀌어 시나리오 재작성 수정 6클래스 connectedDebugAndroidTest(Pixel_7_Claude_QA, API 35) 29/29 통과 실측.
Collaborator
Author
|
머지 순서 참고 — 중복으로 닫힌 #948 본문에만 있던 경고를 옮겨 둔다. #931 과 파일 겹침: AppAndReceiverCompletionAndroidTest·ReceiverAdvancedAndroidTest 를 #931 도 수정한다. merge-tree 실측(#948 브랜치 기준, 이 PR 과 파일 세트 동일)으로 텍스트 충돌은 없었지만, #931 이 |
This was referenced Aug 23, 2026
…eature:home 경로에 맞춤 - AppAndReceiverCompletionAndroidTest / ReceiverAdvancedAndroidTest: 충돌한 import 를 develop 의 신규 경로로 채택하고 #936 이 추가한 mindrecord·timeletter import 를 합쳤다. #936 이 사용을 없앤 LoadCountResult import 는 미사용이라 뺐다. - app 모듈 밖으로 옮겨간 문자열 4건은 bare R 로 더 이상 해결되지 않아 alias 로 교정: afternote_detail_delete_failed, receiver_download_all_failed, receiver_download_all_save_failed → AfternoteFeatureR, receiver_home_section_count_unavailable → ReceiverR 검증: :app:compileDebugAndroidTestKotlin, :app:compileDebugUnitTestKotlin, :app:ktlintCheck 통과 (develop 단독 기준선은 androidTest 컴파일 44건 실패).
OnlyReadOneBook
added a commit
that referenced
this pull request
Aug 24, 2026
…경로에서 registry 로 제공 #919 가 IdentityVerification·ReceiverAuthCode DataStore 를 :feature:afternote:data 에서 :feature:receiver:data 로 옮겼고, #912 는 같은 파일들을 삭제하고 LocalStoreRegistry 로 대체했다. - di 모듈 2종: 이관된 신규 패키지에서 registry.store(scope = SESSION) 제공을 유지 - *PreferencesDataStore.kt 2종: #912 의 삭제 유지 (rename/delete 충돌 — 이관본도 삭제) 검증: :app:compileDebugUnitTestKotlin, :core:datastore:ktlintCheck, :feature:receiver:data:ktlintCheck 통과. :app:compileDebugAndroidTestKotlin 은 develop 단독(495ebaf)에서도 44건 실패하는 기존 드리프트로 #949 가 고치는 범위다.
develop 이 unit-test 잡에 --build-cache 를 되살렸고 이 브랜치는 같은 줄에 :app:compileDebugAndroidTestKotlin 게이트를 추가했다. 둘을 합쳤다.
Sadturtleman
approved these changes
Aug 24, 2026
Contributor
OnlyReadOneBook
added a commit
that referenced
this pull request
Aug 24, 2026
#1005(추모 미디어 업로드 경로 통합)가 먼저 develop 에 들어가 사진 전용 MemorialPhotoUpload* 가 MemorialMediaUpload* 로 바뀌었고, 이 브랜치는 AuthorReceiverRepository 를 지우고 UserRepository 로 갈아끼우는 쪽이라 DI 모듈 import 가 양쪽에서 갈렸다. 남는 쪽(MediaUpload)만 두고 지워진 쪽(PhotoUpload·AuthorReceiver) 은 모두 뺀다. androidTest editorViewModel() 헬퍼는 develop 의 itemId(Long) 형태를 취한다 — #949 가 String→Long 드리프트를 고친 결과이고 같은 파일의 다른 호출부도 Long 이다. 수신자 주입만 이 브랜치의 FakeUserRepository 로 남긴다. :app:compileDebugUnitTestKotlin 통과. androidTest 오류 21건은 develop 에서 그대로 물려받은 것으로 오류 집합이 develop 과 동일하다.
This was referenced Aug 24, 2026
Merged
OnlyReadOneBook
added a commit
that referenced
this pull request
Aug 24, 2026
이 PR 이 `DetailReceiver` 에서 `phone` 을 걷어냈는데 `AfternoteAuthorExtendedAndroidTest` 픽스처 한 곳이 남아 `:app:compileDebugAndroidTestKotlin` 이 깨졌다. develop(`af3f4610`) 도 같은 태스크가 21건 깨져 있어(#949 몫) 이 한 건이 그 잡음에 묻혔다. 오류 집합을 develop 과 대조해 증분이 이 한 건임을 확인했고, 제거 후 두 집합이 같아졌다.
OnlyReadOneBook
added a commit
that referenced
this pull request
Aug 24, 2026
develop 병합으로 :app:compileDebugAndroidTestKotlin 게이트(#949)가 들어오면서 옛 통합 Detail·ReceivedAfternoteDetail 형태를 쓰던 픽스처가 19건 깨졌다. - Detail: category·title·type·credentials·processingMethods·memorial 평면 필드를 serviceName + content(DetailContent.SocialNetwork) 로 옮긴다 - ReceivedAfternoteDetail: title 을 serviceName 으로, 사라진 category 인자를 제거 - AfternoteUpdatePayload.category 단언을 type 으로, DetailContentUiModel.Account 를 SocialNetwork 로
This was referenced Aug 24, 2026
Merged
This was referenced Aug 24, 2026
This file contains hidden or 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
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.
Closes #936
변경
1. androidTest 5파일을 현행 도메인 API·화면 계약으로 갱신 (e0df3af)
이슈의 컴파일 에러 44건:
ReceiverHomeViewModel이 현행 3-리포지토리(MindRecordReceiverRepository.getAll()+ReceiverTimeLetterRepository.getReceivedTimeLetters()+ReceiverRepository) 구조라 홈 테스트를 재작성. 부분 실패 시 "기록 수를 불러오지 못했습니다." 중립 문구 단언 포함ListItem.id·홈 내비 콜백 String→LongwithdrawCompleted(Boolean) →WithdrawUiState치환 — 이슈 표의 3파일 외에 같은 44건에 포함돼 있던 Setting 2파일(SettingAccountSecurityAndroidTest·SettingCompletionAndroidTest)도 함께 수정ReceivedAfternoteDetailRoute의onNavigateToPlaylist: (Long) -> Unit필수화 반영추가 적발 — 컴파일로 안 잡히는 런타임 드리프트 7건 (에뮬 1차 실측에서 발견, 같은 커밋에 수정):
toRoute로 Long 읽음 →LongNavType크래시. Long 주입으로 정정"MASTER-KEY-73"입력이 서버 호출 전 반려 → 유효 UUID 로 교체(소문자 정규화 검증 겸함)2. unit-test.yml 에
:app:compileDebugAndroidTestKotlin게이트 추가 (5493498)기존 Gradle 한 방 실행(
--continue --parallel)에 태스크만 추가 — 에뮬레이터 없이 컴파일만이라 러너 비용 소폭. required 승격은 #848 aggregate gate 와 연계 검토(이슈 명시대로 이번 범위 밖).검증
:app:compileDebugAndroidTestKotlin/:app:compileDebugKotlin/:app:compileDebugUnitTestKotlin/:app:ktlintCheckBUILD SUCCESSFUL,actionlint통과connectedDebugAndroidTest— 1차(컴파일 수정만 반영)에서 위 런타임 7건 실패 적발 → 수정 후 2차 29/29 통과(0 failures, 결과 XML 교차 확인). 로컬 QA 증거:.codex/qa-evidence/emulator/54934985….jsonSavedStateHandle주입 전수 grep — String id 패턴 추가 발견 없음(RECEIVER_ID·timeLetterId·draftId모두 Long/null)리뷰어에게