[🔨 Fix] 푸시알림 서버 QA를 위한 코드 수정 및 요구사항 고도화

[junggyo1020]

⚙️ ISSUE

• closed [Fix] 푸시알림 서버 QA를 위한 코드 수정 및 요구사항 고도화 #68

📄 Work Description

1. 이름 충돌로 인해 /api/v1/notification/create호출 시, messages와 notifications 테이블에 반영이 되지 않는 오류를 발견했습니다. 이를 해결하기 위해 해당 테이블명을 각각 message, notification으로 수정했습니다.



1-1) /api/v1/notification/create 호출 시 추가되는 notification 테이블 내용 (유형 2,3 으로 테스트)





1-2) /api/v1/notification/create 호출 시 추가되는 message 테이블 내용 (유형 2,3 으로 테스트)

• 사용자의 이름이 해당 푸시알림 메세지 > sub에 정상적으로 반영됨을 확인했습니다.

1-3) Postman으로 정상적으로 해당 POST API가 실행됨을 확인했습니다.



2. 전체 대상 푸시알림 발송 시, 정상적으로 발송된 푸시알림의 sent_status가 true로 변경되도록 설정하였습니다.

2-1) 발송 전 상태 (ID 1과 2 모두 sent_status가 false)



2-2) /api/v1/push-notifications/send-all 호출하여 전체 대상 푸시알림 발송 (로그로 2개가 전송됨을 확인)


2-3) 발송 후 상태 (ID 1과 2 모두 sent_status가 true)


2-4) 만약, sent_status가 이미 true라면 중복으로 발송되지 않도록 로직을 구현했습니다. (DDL에서 조회만 하고 푸시알림을 발송하지 않음을 확인)



3. 공통적으로 User의 push_status가 ENABLED인 유저만 푸시알림공고가 create되도록 로직을 추가했습니다.



4. FCM 토큰의 길이가 0이거나, 공백이거나, null 값의 경우 create가 되지 않도록 유효성 검증 로직을 추가했습니다.

4-1) name = “카카오서버QA계정” 의 token값이 빈값이므로 create가 되지 않고, name = “푸시알림QA” 만 create 됨을 확인할 수 있다.




5. 소셜로그인 시, 클라이언트 측에서 fcmToken을 RequestBody로 전달하도록 하고, 해당 fcmToken을 알림서버의 user > token값으로 업데이트 하도록 로직을 추가했습니다!



💬 To Reviewers

• 현재 유효한 fcm토큰을 가지고 있는 유저에게만 전송이 되는 로직이기 떄문에, 다중 유저에 대한 푸시알림 테스트는 진행하지 못한 상태입니다.
• 현재 유형 1에 대한 푸시알림 구현은 수정이 필요한 상태여서 유형 2, 3의 푸시알림 공고에 대한 검증을 위주로 진행했습니다.
• 이후 동기화 관련 로직 검증, 유형 1 로직 검증을 순차적으로 진행할 예정입니다.

✅ PR check list

• Reviewers
• Assignees
• Labels

[JungYoonShin]

수고하셨습니다!!!🍀 구두로 설명 들어서 플로우는 이해되었습니다!!

[JungYoonShin]

요거 아래 코드처럼 filter 다중으로 달아줄 수 있다구 하네요!!
요렇게 사용하면 가독성 좀 더 좋아질 수 있을 것 같아요~!

List<Student> filteredStream = students.stream()
   .filter(s -> s.getMarksAverage() > 50)
   .filter(s -> s.getMarks().size() > 3)
   .filter(not(s -> s.getProfile() == Student.Profile.PHYSICS))

[junggyo1020]

stream API에서 다중 필터가 적용되는지 처음 알았네요..! 좋은 의견 감사합니다 :)