Repository navigation
fix(test): develop androidTest 컴파일 복구 — 배치 머지 API 드리프트 44건 + PR 컴파일 게이트 - #948
Closed
OnlyReadOneBook wants to merge 2 commits into
Closed
OnlyReadOneBook wants to merge 2 commits into
OnlyReadOneBook wants to merge 2 commits into
Conversation
#861/#867 androidTest 배치가 develop 의 도메인 API 교체(#868 등)와 조합 컴파일 검증 없이 머지돼 :app:compileDebugAndroidTestKotlin 이 깨져 있던 것을 복구. ListItem/내비게이션 Long 화, ReceiverHomeViewModel 4-인자 시그니처와 분리 리포지토리 구조 반영, 삭제된 LoadCountResult·withdrawCompleted 참조를 현행 모델(ReceiverMindRecords/ReceivedTimeLetterList·WithdrawUiState)로 교체.
androidTest 를 다루는 워크플로는 android-managed-device.yml(수동+주 2회 스케줄)뿐이라 PR 단계엔 androidTest 컴파일 검증이 전무했다. 실행까진 못 해도 컴파일 깨짐은 PR 에서 잡도록 기존 unit-test 잡의 Gradle 호출에 태스크만 추가.
Collaborator
Author
This was referenced Aug 23, 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.
develop 이
:app:compileDebugAndroidTestKotlin기준으로 깨져 있던 것(에러 44건)을 전수 복구하고, 같은 부류가 다시 미검증으로 착륙하지 못하게 PR unit-test 에 컴파일 게이트를 추가한다.Closes #936
변경
androidTest 복구 (5파일) — 이슈에 분류된 3파일 외에, 순수 develop(6123a68) 전수 컴파일에서 Setting 2파일의
withdrawCompleted참조도 추가 적발돼 함께 복구했다.ListItem.id·홈 내비 콜백 String→LongReceiverHomeViewModel4-인자 시그니처 반영, 삭제된LoadCountResult→ReceiverMindRecords/ReceivedTimeLetterList, 홈 fake 를 현행 분리 리포지토리 구조로 재구성(4-way 홈 시나리오·호출 카운트 배열 보존),ReceivedAfternoteDetailRoute의 필수onNavigateToPlaylist반영·플레이리스트 라우트 Long 화LoadCountResult임포트 정리,onNavigateToPlaylist = {}withdrawCompleted→ 현행WithdrawUiStateCI 게이트 —
unit-test.yml의 Gradle 호출에:app:compileDebugAndroidTestKotlin추가. androidTest 를 다루는 워크플로는 android-managed-device.yml(수동+화·금 스케줄)뿐이라 PR 단계에 androidTest 컴파일 검증이 전무했던 게 #936 의 구조적 원인. androidTest 소스셋은 :app 에만 있어 태스크 1개로 전체가 커버되고 에뮬레이터도 불필요하다. required gate 승격은 #848 의 aggregate gate 작업과 연계해 후속으로.리뷰 포인트 — 기계적 치환이 아닌 곳 2군데
receiver_home_section_count_unavailable(「기록 수를 불러오지 못했습니다.」) 문구라 assertion 을 교체했다.deleteAccount()→실패 시 실패 팝업)으로 바뀌어withdrawFailure_showsFailurePopupAndKeepsSession로 재작성. SettingCompletion 쪽은waitForIdle후 즉시 assert 하던 것을waitUntil(Error/Success)상태 전이 대기로 바꿔 레이스도 제거했다.검증
:app:compileDebugAndroidTestKotlin44건 재현 → 수정 후 동일 커맨드--rerunBUILD SUCCESSFUL (> Task라인으로 태스크 강제 재실행 확인, 증분 stale 마스킹 배제):app:ktlintAndroidTestSourceSetCheck --rerun통과알려둘 점
git merge-tree실측으로 텍스트 충돌은 없음을 확인했으나, 로그아웃·탈퇴 시 로컬 저장소 잔존 수정 — DataStore 수명(scope) 계약 도입 #931 이ReceiverRepository인터페이스를 변경하므로 나중에 머지되는 쪽에서 조합 컴파일 재확인이 필요하다 — 이 PR 의 게이트가 먼저 들어가면 그 확인을 PR CI 가 대신한다.screen.receiver.*) 기준으로 수정했다. 그쪽이 머지되면 수신자 경로 이동에 이 파일들도 함께 이동한다.