[#601] 푸시 알림 설정이 자동으로 upsert되는 현상을 해결한다

[opficdev]

🔗 연관된 이슈

• closed 푸시 알림 설정이 자동으로 upsert되는 현상을 해결한다 #601

🎯 의도

• PushNotificationSettingsView 진입 시 fetch 결과가 binding 액션으로 다시 전달되면서 의도치 않게 푸시 알림 설정 업데이트가 발생하던 문제 해결

📝 작업 내용

📌 요약

• fetchSettings 결과 반영 경로를 binding 액션과 별도 액션으로 분리한 작업
• 화면 진입 중 자동 업데이트가 발생하지 않도록 수정한 작업
• fetch만 수행했을 때 update use case가 다시 호출되지 않는 회귀 테스트 추가 작업

🔍 상세

• PushNotificationSettingsFeature에 applyFetchedSettings 액션 추가
• fetch 결과를 .binding(.set(...)) 대신 applyFetchedSettings로 반영하도록 변경
• 사용자 토글 입력 경로와 서버 응답 반영 경로를 분리하여 binding(\.pushNotificationEnable)에서만 업데이트 effect가 실행되도록 수정
• PushNotificationSettingsFeatureTests에 fetch 결과 반영 중 update use case 재호출이 발생하지 않는 테스트 추가
• 테스트 spy에 update 호출 횟수 추적 로직 추가

📸 영상 / 이미지 (Optional)

[gemini-code-assist]

Code Review

이번 PR에서는 푸시 알림 설정 조회 시 서버 상태를 반영하는 로직을 applyFetchedSettings 액션으로 분리하여 바인딩 루프를 방지하고, 관련 테스트 코드를 추가하였습니다. 또한 의존성 패키지 버전을 .exact로 고정하였습니다. 리뷰에서는 서버 설정을 반영할 때 timePicker가 이미 열려있을 경우 화면과 시트의 시간 불일치가 발생할 수 있으므로 state.timePicker?.time도 함께 업데이트하도록 제안했습니다.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.