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.

Sign up for GitHub

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

Jump to bottom

[BE] FCM 알림 전송 Topic 구독 방식으로 변경 #771

Merged
merged 5 commits into from
Jan 3, 2025
Merged

[BE] FCM 알림 전송 Topic 구독 방식으로 변경 #771

merged 5 commits into from
Jan 3, 2025

Conversation

ehtjsv2
Copy link
Contributor

@ehtjsv2 ehtjsv2 commented Dec 28, 2024 •
edited
Loading

이슈

개발 사항

전달 사항 (없으면 삭제해 주세요)

  • FCM관련된 내용은 이슈에 더 상세히 적혀있으니 참고바랍니다.
  • 패키지 이동으로 인해, 변경사항이 많이 잡혔으니 커밋단위로 보는게 더 편할 수 있습니다.

@ehtjsv2 ehtjsv2 added 🤹 enhance 성능 개선 🛠️ refactor 리팩터링 🖥 backend backend labels Dec 28, 2024
@ehtjsv2 ehtjsv2 self-assigned this Dec 28, 2024
@ehtjsv2 ehtjsv2 linked an issue Dec 28, 2024 that may be closed by this pull request
알림 서비스 의존 관계 수정 #768
Closed
@ehtjsv2 ehtjsv2 changed the title Perform/be notification [BE] FCM 알림 전송 Topic 구독 방식으로 변경 Dec 28, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 28, 2024 •
edited
Loading

Test Results

227 tests  +150   227 ✅ +150   33s ⏱️ +32s
 47 suites + 35     0 💤 ±  0 
 47 files   + 35     0 ❌ ±  0 

Results for commit de0f183. ± Comparison against base commit 604762f.

This pull request removes 77 and adds 227 tests. Note that renamed tests count towards both. 
com.happy.friendogly.presentation.ui.club.common.model.ClubFilterSelectorTest ‑ Gender 필터를 선택하면 해당 필터들만 반환되어야 한다()
com.happy.friendogly.presentation.ui.club.common.model.ClubFilterSelectorTest ‑ Size 필터를 선택하면 해당 필터들만 반환되어야 한다()
com.happy.friendogly.presentation.ui.club.common.model.ClubFilterSelectorTest ‑ 선택 된 GenderFilter가 없다면, GenderFilter를 확인하는 로직은 false를 반환해야 한다()
com.happy.friendogly.presentation.ui.club.common.model.ClubFilterSelectorTest ‑ 선택 된 GenderFilter가 있다면, GenderFilter를 확인하는 로직은 true를 반환해야 한다()
com.happy.friendogly.presentation.ui.club.common.model.ClubFilterSelectorTest ‑ 선택 된 SizeFilter가 없다면, SizeFilter를 확인하는 로직은 false를 반환해야 한다()
com.happy.friendogly.presentation.ui.club.common.model.ClubFilterSelectorTest ‑ 선택 된 SizeFilter가 있다면, SizeFilter를 확인하는 로직은 true를 반환해야 한다()
com.happy.friendogly.presentation.ui.club.common.model.ClubFilterSelectorTest ‑ 여러 필터를 추가하면 ClubFilterSelector에 모든 필터가 추가되어야 한다()
com.happy.friendogly.presentation.ui.club.common.model.ClubFilterSelectorTest ‑ 이미 추가 된 필터가 있다면 해당 필터는 추가되지 않아야 한다()
com.happy.friendogly.presentation.ui.club.common.model.ClubFilterSelectorTest ‑ 필터 목록을 초기화하면 주어진 필터들로 초기화되어야 한다()
com.happy.friendogly.presentation.ui.club.common.model.ClubFilterSelectorTest ‑ 필터를 제거하면 해당 필터는 ClubFilterSelector에서 삭제되어야 한다()
…

com.happy.friendogly.DataSourceRoutingTest ‑ 읽기 전용 트랜잭션이 아니면, Writer DB 데이터소스가 바운딩된다.
com.happy.friendogly.DataSourceRoutingTest ‑ 읽기전용 트랜잭션이면 reader DB 데이터소스가 바운딩된다.
com.happy.friendogly.chatmessage.controller.ChatMessageControllerTest ‑ 두 시점 사이의 채팅 메시지를 조회할 수 있다.
com.happy.friendogly.chatroom.service.ChatRoomCommandServiceTest ‑ 1대1 채팅방 저장 과정에서, 이미 채팅방이 존재하면 기존 채팅방 ID를 반환한다.
com.happy.friendogly.chatroom.service.ChatRoomCommandServiceTest ‑ 새로운 1대1 채팅방을 개설할 수 있다.
com.happy.friendogly.chatroom.service.ChatRoomCommandServiceTest ‑ 채팅방에서 나간다.
com.happy.friendogly.chatroom.service.ChatRoomCommandServiceTest ‑ 채팅방을 나가고 모임을 나가도 예외가 발생하지 않는다.
com.happy.friendogly.chatroom.service.ChatRoomQueryServiceTest ‑ 내가 속해 있는 채팅방을 찾을 수 있다.
com.happy.friendogly.chatroom.service.ChatRoomQueryServiceTest ‑ 자신이 참여한 채팅방이 아니면 채팅방에서 모임 정보를 받아올 수 없다.
com.happy.friendogly.chatroom.service.ChatRoomQueryServiceTest ‑ 참여중인 1대1 채팅방과 최근 메시지를 조회한다.
…

♻️ This comment has been updated with latest results.

ehtjsv2
ehtjsv2 commented Dec 28, 2024
View reviewed changes
...src/test/java/com/happy/friendogly/playground/service/PlaygroundNotificationServiceTest.java Outdated
// then
Mockito.verify(notificationService, Mockito.times(1)).unsubscribeTopic(any(),any());
}
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

빈줄 추가됐습니다

@ehtjsv2 ehtjsv2 merged commit a7f3c72 into develop Jan 3, 2025
3 checks passed
ehtjsv2 added a commit that referenced this pull request Jan 3, 2025
@ehtjsv2
[BE] FCM 알림 전송 Topic 구독 방식으로 변경 (#771)
935a464
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@takoyakimchi takoyakimchi takoyakimchi approved these changes

@jimi567 jimi567 jimi567 approved these changes

@J-I-H-O J-I-H-O Awaiting requested review from J-I-H-O J-I-H-O is a code owner

Assignees

@ehtjsv2 ehtjsv2

Labels
🖥 backend backend 🤹 enhance 성능 개선 🛠️ refactor 리팩터링
Projects
None yet
Milestone
No milestone
3 participants
@ehtjsv2 @jimi567 @takoyakimchi