woowacourse-teams / 2023-festa-go

🎪 페스타고, 대학 축제를 더욱 즐겁게!
71 stars 8 forks source link

[BE] feat: 새로운 FCM 등록 기능 추가 (#998) #1002

Open seokjin8678 opened 3 months ago

seokjin8678 commented 3 months ago

📌 관련 이슈

✨ PR 세부 내용

이슈 내용 그대로 FCM을 등록하는 기능을 새롭게 추가했습니다.

FCM 토큰에 expiredAt 필드가 생겼기 때문에 DB 스키마 또한 변경되었습니다.

이슈에 남겼던 FCM 등록 프로세스인데, MemberFCMCommandServiceTest는 해당 명세에 따라 작성되었습니다.

- 사용자가 등록한 FCM 토큰이 없으면
    - FCM 토큰 등록
- 사용자가 등록한 FCM 토큰이 있으면
    - 요청의 FCM 토큰과 같으면
        - FCM 토큰 등록 기간 갱신
    - 요청의 FCM 토큰과 다르면
        - 기존 토큰 삭제 후 등록

만료 시간과 오래된 토큰의 삭제 정책 같이 변경이 될 수 있는 정책에 대해서는 별도의 클래스로 분리하여 사용하도록 했습니다. (따라서 패키지 위치가 domain 입니다)

그 외 사항에 대해선 코드에 리뷰로 남기겠습니다!

github-actions[bot] commented 3 months ago

Test Results

246 files  246 suites   28s :stopwatch: 795 tests 795 :white_check_mark: 0 :zzz: 0 :x: 813 runs  813 :white_check_mark: 0 :zzz: 0 :x:

Results for commit 83d62b35.

:recycle: This comment has been updated with latest results.