Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BE] fix: 채팅 알림 기기 토큰 관련 예외 처리 추가, 자신에게는 채팅 알림을 전송하지 않도록 변경 #527

Merged
merged 4 commits into from
Sep 9, 2024

Conversation

takoyakimchi
Copy link
Contributor

@takoyakimchi takoyakimchi commented Aug 22, 2024

이슈

개발 사항

  • 채팅 알림에서 기기 토큰이 비어 있는 경우 예외 발생 구현
  • 자기 자신에게는 채팅 알림을 전송하지 않도록 수정

전달 사항

  • 안드로이드에서 기기 토큰 저장 issue가 해결되었기 때문에, exception은 (정상적인 흐름이라면) 터지지 않을 것 같아요.
  • 크리티컬한 이슈는 아니라서, 바로 머지하지 않아도 괜찮을 것 같네요.

@takoyakimchi takoyakimchi added this to the Sprint4 milestone Aug 22, 2024
@takoyakimchi takoyakimchi self-assigned this Aug 22, 2024
@takoyakimchi takoyakimchi changed the title Feature/be/#524 [BE] refactor: 채팅 알림 기기 토큰 관련 예외 처리 추가, 자신에게는 채팅 알림을 전송하지 않도록 변경 Aug 22, 2024
@takoyakimchi takoyakimchi changed the title [BE] refactor: 채팅 알림 기기 토큰 관련 예외 처리 추가, 자신에게는 채팅 알림을 전송하지 않도록 변경 [BE] fix: 채팅 알림 기기 토큰 관련 예외 처리 추가, 자신에게는 채팅 알림을 전송하지 않도록 변경 Aug 22, 2024
Copy link

github-actions bot commented Aug 22, 2024

Test Results

162 tests  +159   162 ✅ +159   22s ⏱️ +22s
 33 suites + 32     0 💤 ±  0 
 33 files   + 32     0 ❌ ±  0 

Results for commit e0dfaf0. ± Comparison against base commit 1ebfd27.

This pull request removes 3 and adds 162 tests. Note that renamed tests count towards both.
com.happy.friendogly.ExampleUnitTest ‑ addition_isCorrect
com.happy.friendogly.ExampleUnitTest ‑ addition_isCorrect2
com.happy.friendogly.ExampleUnitTest ‑ addition_isCorrect3
com.happy.friendogly.chat.service.ChatRoomCommandServiceTest ‑ 1대1 채팅방 저장 과정에서, 이미 채팅방이 존재하면 기존 채팅방 ID를 반환한다.
com.happy.friendogly.chat.service.ChatRoomCommandServiceTest ‑ 모임에 참여하지 않은 경우, 모임 채팅방에서 나갈 수 없다.
com.happy.friendogly.chat.service.ChatRoomCommandServiceTest ‑ 새로운 1대1 채팅방을 개설할 수 있다.
com.happy.friendogly.chat.service.ChatRoomCommandServiceTest ‑ 채팅방에 참여한 경우, 채팅방에서 나갈 수 있다.
com.happy.friendogly.chat.service.ChatRoomQueryServiceTest ‑ 내가 속해 있는 채팅방을 찾을 수 있다.
com.happy.friendogly.chat.service.ChatRoomQueryServiceTest ‑ 자신이 참여한 채팅방이 아니면 채팅방에서 모임 정보를 받아올 수 없다.
com.happy.friendogly.chat.service.ChatRoomQueryServiceTest ‑ 채팅방 ID로부터 모임 ID, 허용 펫 사이즈, 허용 펫 성별을 조회할 수 있다.
com.happy.friendogly.chat.service.ChatRoomQueryServiceTest ‑ 채팅방 내 멤버 세부정보를 조회할 수 있다.
com.happy.friendogly.club.domain.AddressTest ‑ [1] provinceInput=null
com.happy.friendogly.club.domain.AddressTest ‑ [2] provinceInput=
…

♻️ This comment has been updated with latest results.

Copy link
Contributor

@ehtjsv2 ehtjsv2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good

@takoyakimchi takoyakimchi merged commit 0b2aea6 into develop Sep 9, 2024
3 checks passed
@takoyakimchi takoyakimchi deleted the feature/be/#524 branch September 9, 2024 06:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
4 participants