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

알림 서비스 강결합 분리 및 비동기 처리 #193

Merged
merged 8 commits into from
Jan 2, 2024
Merged

Conversation

weonest
Copy link
Contributor

@weonest weonest commented Dec 27, 2023

✅ PR 체크리스트

  • 테스트
  • 문서화 작업

💡 어떤 작업을 하셨나요?

Issue Number : close #183

작업 내용

  • 알림 기능의 구조를 변경하였습니다. (프론트와 논의 후 결정하였음)
  • @EventListenr 를 통해 강결합을 제거하였습니다.
  • 비동기 로직을 추가하였습니다.

📝리뷰어에게

  • 신청서가 수락 or 거절되는 등의 상호작용이 발생하면 알림이 생성이 비동기 로직을 통해 생성되게끔 로직을 수정하였는데, 이부분에 대해서 테스트 코드를 작성할 때 어려운 점이 있었습니다. 신청서 생성은 동기로 처리되고 알림 생성이 비동기로 처리되기 때문에 테스트 검증 부분에서 알림 생성을 기다리기 위해서 Thread.sleep()을 사용하였습니다. 혹시, 이보다 더 좋은 방법이 있다면 의견 남겨주시면 감사하겠씁니다.

@weonest weonest added Test 테스트 ✏️ Feature 기능 개발 🧚 Refactor 리팩토링 labels Dec 27, 2023
@weonest weonest self-assigned this Dec 27, 2023
Copy link

github-actions bot commented Dec 27, 2023

Test Results

 27 files   27 suites   8s ⏱️
130 tests 130 ✅ 0 💤 0 ❌
131 runs  131 ✅ 0 💤 0 ❌

Results for commit 73438e4.

♻️ This comment has been updated with latest results.

@weonest weonest merged commit 5020776 into dev Jan 2, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✏️ Feature 기능 개발 🧚 Refactor 리팩토링 size/XL Test 테스트
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[7차 스프린트 - Steady] 알림 기능 강결합 분리 및 비동기 처리
2 participants