mindRecordNavGraph 의 onSubmitSuccess 2줄에 가드가 없다 #1562

작업 유형

test — 테스트·검증·회귀 방지

주 담당 모듈

mindrecord — 마음의 기록

개요

mindRecordNavGraph 의 매핑 7줄 중 onSubmitSuccess 2줄에 가드가 없다.

DailyQuestionWriteScreen(onSubmitSuccess = actions::popBack, …)
DiaryWriteScreen(onSubmitSuccess = actions::popBack, …)

#1466 이 screenBackButton_returnsToMindRecordHomeFromEveryMindRecordDestination 을 넣어 onBackClick 5줄은 실제로 태우게 됐다. 남은 2줄은 제출 성공 경로라 어느 계측도 지나가지 않는다 — onSubmitSuccess = actions::onNavigateToDraftList 로 오배선해도 세 계측이 전부 초록이고, 콜백 시그니처가 모두 () -> Unit 이라 컴파일도 잡지 못한다(#1466 리뷰에서 실측).

develop 에도 있던 공백이지 #1466 이 만든 것은 아니다. 다만 #1466 전에는 자리마다 이름이 달라(onWriteSubmitSuccess·onWriteBack·onDraftListBack) 이름 자체가 오배선을 드러냈는데, popBack 하나로 합치면서 그 방어가 사라졌다. 남은 2줄은 그만큼 더 노출된다.

필요한 것

제출 성공 흐름을 태우는 계측 한 층. 두 화면 모두 hiltViewModel() 과 서버 제출을 거치므로 #1466 의 뒤로가기 층보다 비용이 크다 — 가짜 제출 성공을 주입할 수 있는지부터 확인이 필요하다. 비용이 값을 넘으면 KDoc 에 «onSubmitSuccess 2줄은 가드 밖» 을 명시하는 선에서 닫아도 된다. 다음 사람이 가드가 있다고 믿지 않게 하는 것이 최소 목표다.

참고

• 선행: refactor(mindrecord): 단순 back 위임 5개를 popBack 하나로 합친다 (#1311) #1466 (refactor(mindrecord): 단순 back 위임 5개를 popBack 하나로 합친다) · 대표 이슈 NavActions 단순 back 위임 5개를 공통 명령으로 통합 #1311
• 실측 근거는 refactor(mindrecord): 단순 back 위임 5개를 popBack 하나로 합친다 (#1311) #1466 의 8/29 리뷰 인라인 코멘트(MindRecordBackStackAndroidTest.kt:42)에 있다.