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] hotfix: 주변 디바이스 토큰 조회시, 500에러 발생하는 버그 수정 #456

Merged
merged 1 commit into from
Aug 18, 2024

Conversation

ehtjsv2
Copy link
Contributor

@ehtjsv2 ehtjsv2 commented Aug 18, 2024

이슈

개발 사항

  • 발자국 저장시 알림보내기 위한 디바이스 토큰 조회하는 부분에서 500에러 발생
  • 모두 다바이스 토큰 등록하는 기능이 있는 최신버전의 어플 사용하면 괜찮은데, 예전 버전 사용하는 사람이 있어 Optional의 value가 없는 경우가 발생
  • 이에따라 코드를 어떤 경우에도 괜찮도록 변경
  • Optional::Stream은 value가 존재하는 것만 stream에 포함하도록 하는 함수

@ehtjsv2 ehtjsv2 added 🐞 bug 버그 수정 🖥 backend backend labels Aug 18, 2024
@ehtjsv2 ehtjsv2 added this to the Sprint4 milestone Aug 18, 2024
@ehtjsv2 ehtjsv2 self-assigned this Aug 18, 2024
@ehtjsv2 ehtjsv2 linked an issue Aug 18, 2024 that may be closed by this pull request
Copy link

Test Results

150 tests  +147   150 ✅ +147   17s ⏱️ +17s
 30 suites + 29     0 💤 ±  0 
 30 files   + 29     0 ❌ ±  0 

Results for commit 34803d4. ± Comparison against base commit 80634f8.

This pull request removes 3 and adds 150 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=
…

Copy link
Contributor

@dpcks0509 dpcks0509 left a comment

Choose a reason for hiding this comment

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

감사합니닷~!

@jinuemong jinuemong self-requested a review August 18, 2024 03:25
Copy link
Contributor

@jinuemong jinuemong left a comment

Choose a reason for hiding this comment

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

hotfix네요

Copy link
Contributor

@takoyakimchi takoyakimchi left a comment

Choose a reason for hiding this comment

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

굿굿~~
Optional.get()은 최대한 피하는 게 좋아보여요.
이전 코드는 IDE에서도 경고 날려주고 있네요!

@ehtjsv2 ehtjsv2 merged commit a23eca6 into develop Aug 18, 2024
3 checks passed
@takoyakimchi takoyakimchi deleted the hotfix/find-device-token branch November 4, 2024 05:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖥 backend backend 🐞 bug 버그 수정
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

주변 토큰 조회시 토큰 없는 경우 예외처리
4 participants